1042. Transponowanie macierzy [TRN]

#include <iostream>

int n, m;

using namespace std;

int main()
{
    cin >> m >> n;
    int t[m][n];
    for (int i=1;i<=m;i++)
    {
        for (int j=1;j<=n;j++)
        {
            cin >> t[i][j];
        }
    }
        for (int i=1;i<=n;i++)
    {
        for (int j=1;j<m;j++)
        {
            cout << t[j][i] << " ";
        }
        cout << t[m][i];
        cout << endl;
    }

    return 0;
}

bardzo prosze o pomoc xd wyniki dzialaja, spacje sa opanowane ale cos nie smiga ...

nie smiga bo dostajesz (SIGSEGV) ...
jezeli robisz sobie dla przykładu tablice: int t[2]; to masz w niej dwa elementy o indeksach 0 oraz 1...
wiec zawolanie np std::cin >> t[2]; konczy sie SIGSEGV...

Niby najprostsze zadanie, a nie mogę pogodzić się z sędzią: "Błędna odpowiedź".
Zestawy testowe działają.

Czy jest ktoś w stanie wskazać błąd w tym kodzie?

(kod usunięty)

Poprawny out:

1 4 3 8
2 3 4 7
5 3 9 7

Twój out:

5 x \n
1438
2347
5397

Poza tym wygląda ok.

Out się zgadzał. Błąd polegał na tym, że "endl" był w złym miejscu. Trochę niedokładnie sprecyzowane polecenie moim zdaniem, ale pokombinowałem i dostałem AC. Dzięki za pomoc.
Usuwam kod źródłowy.

Poniższy kod generuje werdykt przekroczony limit czasu. Czy można przyśpieszyć poniższy kod, czy konieczne będzie inne rozwiązanie?
`Scanner in = new Scanner(System.in);
int w = in.nextInt();
int k = in.nextInt();
int[] values = new int[w*k];

        for(int i=0;i<w*k;i++){
            values[i]=in.nextInt();
        }

        for(int i=0;i<k;i++){
            for(int j=0;j<w;j++){
                System.out.print(values[i+j*k] + " ");
            }
            System.out.println();
        }`

Możesz spróbować wyprowadzić dane poprzez StringBuilder, ale podejrzewam, że sama zmiana wyjścia nie wystarczy. Na wejściu można spróbować wczytać wszystkie dane do stringa i samemu potem go pociąć.

Zdecydowanie łatwiej przejść na C/C++

Przerobiłem na c++ i acc...

Hej, nie mogę znaleźć błędu. SPOJ informuje mnie o (SIGABRT) ale nie wiem gdzie to mogę to poprawić.

KOD c++:

#include <iostream>

using namespace std;
int n, m, liczba;
int main()
{
    cin >> m >> n;
    int tab[m + 1][n+1];
    for(int j=1; j<=m; j++)
    {
        for(int i=1; i<=n; i++)
        {
            cin>>liczba;
            tab[i][j]=liczba;
        }
    }
    for(int j=1; j<=n; j++)
    {
 
        for(int i=1; i<=m; i++)
        {
            if(i<m)
            {
                cout<<tab[j][i]<<" ";
            }
            else
            {
                cout<<tab[j][i];
            }
 
        }
        if(j<n) cout<<endl;
    }
 
    for (int i = 0; i<m; i++)
    {
        delete[] tab[i];
    }
    delete[] tab;
 
 
    return 0;
}

A pamiętasz że n może być różne od m? Wydaje mi się że wychodzisz poza tablicę. Może się mylę, jest późno i jestem na tel, nie mogę sprawdzić

1. Czytamy komunikaty błędów i warningi. Drugie można zignorować (podobnie jak errory - w końcu jesteś wolnym człowiekiem i nie musisz ograniczać się do kompilujących, działających kodów ), ale na ogół sugerują jakiś błąd, który wyjdzie przy wykonywaniu kodu.

   main.cpp: In function ‘int main()’:
   main.cpp:37:23: warning: deleting array 'tab[i]'
   delete[] tab[i];
   ^
   main.cpp:39:14: warning: deleting array 'tab’
   delete[] tab;
   ^

2. Co do:

   Nie, nie powinieneś użyć vector - najpierw naucz się tablic a potem vector

3. Jak słusznie zauważył @j4rooo:

   Skoro już wiesz, że gcc/g++ to jedno, a C/C++ to drugie, możesz się zastanawiać, jak to ze sobą połączyć Użycie --pedantic od razu daje dodatkowe ostrzeżenia

   main.cpp:8:23: warning: ISO C++ forbids variable length array ‘tab’ [-Wvla]
   int tab[m + 1][n+1];
   ^
   main.cpp:8:23: warning: ISO C++ forbids variable length array ‘tab’ [-Wvla]
   main.cpp:37:23: warning: deleting array 'tab[i]'
   delete[] tab[i];
   ^
   main.cpp:39:14: warning: deleting array 'tab’
   delete[] tab;
   ^

   Oczywiście w praktyce cwane sztuczki z użyciem komplikatorów życia mogą mieć uzasadnienie, natomiast na SPOJu radzę traktować kodzenie jak sztukę - pisać kody zgodne z powszechnie obowiązującymi standardami.

4. A na koniec:

   Prawda, dotyczy to także destruktorów, natomiast na razie niech Ci się to przydaje - jak będziesz umiał tego użyć to potem świadomie, w pełni rozumiejąc konsekwencje swoich poczynań, możesz kombinować

Udało się dziękuję.

Zastosowałem:

int **tab= new int *[n];
for (int i=0; i<n; i++)
{
tab[i]= new int[m];
}

i parę drobnych zmian.

Jeszcze dużo nauki przede mną

Niby odpowiedzi wychodzą git, ale sędzia mówi, że naruszenie pamięci. Ktoś ma pomysł gdzie to może być?

#include <iostream>
#include <vector>

using namespace std;

void wpisz(vector<vector<int> > & tab)
{
    for(int n=0 ; n<tab.size() ; n++)
    {
        for(int m=0 ; m<tab[n].size() ; m++)
        {
            int a;
            cin>>a;
            tab[m][n]=a;
        }
        ;
    }
}
void pokaz(vector<vector<int> >& tab)
{
    for(int m=0 ; m<tab[m].size() ; m++)
    {
        for(int n=0 ; n<tab.size() ; n++)
        {
            cout<<tab[m][n];
        }
        cout<<endl;
    }

}
int main()
{
    int m,n;
    cin>>m>>n;
    vector<vector<int> >tab (n,vector<int>(m));
    wpisz(tab);
    pokaz(tab);

    return 0;
}

tab[m][n]=a;

Co zrobi ta linijka, gdy wymiary macierzy będą wynosiły np. 5 na 8 (czyli n=5, m=8)?

Ta zmienna a chyba nawet nie jest potrzebna. Chodzi mi tam tylko o wypełnienie tablicy.

Nie chodzi o zmienną a. Pomyśl: ilość wektorów to 5, a rozmiar pojedynczego wektora to 8. Czy w takim wypadku odwołanie się do 5 wartości w 8 wektorze jest prawidłowe?

Zrobiłem od nowa ten kodzik i fajnie, bo nie ma błędu pamięci. W zamian błąd odpowiedzi. Na ideone.com5 odpowiedź się zgadza. Może problem w tych spacjach albo endlach? W sumie sam już nw.

  #include <iostream>
#include <vector>

using namespace std;

void wpisz(vector<vector<int> > & tab)
{
    for(int m=0 ; m<tab.size() ; m++)
    {
        for(int n=0 ; n<tab[m].size() ; n++)
        {

            cin>>tab[m][n];
        }
        cout<<endl;
    }
}

void poka(vector<vector<int> >& tab)
{
    for(int m=0 ; m<tab[m].size() ; m++)
    {
        for(int n=0; n<tab.size(); n++)
        {
            cout<<tab[n][m]<<" ";
        }
        cout<<endl;
    }

}
int main()
{
    int m,n;
    cin>>m>>n;
    cout<<endl;
    vector<vector<int> >tab (m,vector<int>(n));
    if(m>0 and n>0 and m<201 and n<201)
    {
        wpisz(tab);
        poka(tab);
    }
    return 0;
}

2 5
1 2 3 4 5
6 7 8 9 10

Wskazówka: błąd jest w warunku zakończenia jednej z pętli.