May 2009
13 / 38
Jan 2017
Jan 2024
Szubiniov
Jul '15

Przy równych wymiarach i gdy n jest większe od m program działa. Jednak gdy m jest większe od n występuje błąd ochrony pamięci. Nie wiem czemu tak jest, przecież cała tablica jest już zainicjowana przed wpisywaniem wartości.

edit. AC, usunąłem kod.

6 months later
timmyyy
Jan '16
1

Wali NZEC'iem jakiś pomysł?
Program w C#
EDIT:
jeszcze wpadł mi do głowy pomysł, że to przez tego stringa w tablicach ale zmieniłem na inty i dalej to samo frowning

using System;

namespace 1042.Transponowaniemacierzy___TRN
{
class Program
{
static void Main(string[] args)
{
string line;
int columns=0, rows=0;

        line = Console.ReadLine();

        string[] split = line.Split(' ');

        rows = Convert.ToInt32(split[0]);
        columns = Convert.ToInt32(split[1]);
        string[,] macierz = new string[rows, columns];
        string[,] transponowana = new string[columns, rows];
        for(int i =0; i < rows; i++)
        {
            line = Console.ReadLine();
            string[] split1 = line.Split(' ');
            for(int j = 0; j < split1.Length; j++)
            {
                macierz[i, j] = split1[j];
            }
        }

        for(int i =0; i < columns; i++)
        {
            for(int j =0; j < rows; j++)
            {
                transponowana[i, j] = macierz[j, i];
            }
        }

        for(int i =0; i < columns; i++)
        {
            for(int j =0; j < rows; j++)
            {

                if (j != rows - 1)
                    Console.Write(transponowana[i, j] + " ");
                else if (i == columns - 1)
                    Console.Write(transponowana[i, j]);
                else
                    Console.WriteLine(transponowana[i, j]);

            }
        }
    }
}

}

4 months later
silence96
May '16

Cześć :smiley: nie mam pojęcia dlaczego kod nie przechodzi :confused: u mnie działa poprawnie. Pisane w javie:

`import java.util.*;
import java.lang.*;

class Main
{
public static void main (String[] args) throws java.lang.Exception
{
int m,n;
Scanner scan= new Scanner(System.in);
m=scan.nextInt();
n=scan.nextInt();

	int tab[][]=new int [m][n];
	for(int i=0; i<m; i++)
		for(int j=0; j<n; j++){
			tab[i][j]=scan.nextInt();
		}

	int tab2 [][]=new int [n][m];
	for(int j=0; j<n; j++)
		for(int i=0; i<m; i++){
			tab2[j][i]=tab[i][j];
		}
	for(int j=0; j<n; j++){
		for(int i=0; i<m; i++){
			System.out.print(tab2[j][i]+ " ");
		}
		System.out.println();
	}
}

}`

7 months later
konrad15
Jan '17
#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 ...

lordarab
Jan '17

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...

1 month later
bartol_alert
Feb '17
1

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)

etheral6661
Feb '17

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.

bartol_alert
Feb '17

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.

1 month later
jumper_5
Apr '17
1

Cześć!
Co się nie zgadza w tym kodzie?
...tu był kod...
Dla różnych wartości jakie wpiszę program się wykonuje, mimo tego sędzia pokazuje mi "NZEC". Spacje czy entery są uwzględnione. Z czego wynika problem?

Już nieaktualne :slight_smile:

1 month later
efielo
May '17

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();
        }`
mariusz193
May '17

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++ :slight_smile:

1 year later
kbaraniecki
Jun '18
1

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;
}
yula
Jun '18

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ć

j4rooo
Jun '18
kbaraniecki

Cześć,
zawsze jak piszesz posta to możesz zaznaczyć kod i zobaczyć co robią te guziczki u góry, np </> sformatuje Ci kod.

To:

int tab[m + 1][n+1];

nie jest prawidłowa deklaracja tablicy o wielkości nie znanej podczas kompilacji. Powinieneś uzyć new[] lub vektora. Ale w gcc zazwyczaj działa mimo, że ta składnia nie jest częścią jezyka

Za to jak wykonasz delete na czymś czego nie tworzyłeś za pomocą słowa new to wtedy zazwyczaj dostaniesz bład i to właśnie prawdopodobnie SIGABRT.

A tak w ogóle to delete Ci się raczej na spoju nie przyda, ale w bardzo małym odsetku zadań. A niech sobie sędzia sam z pamięcią radzi :wink:

tarpauwatratar
Jun '18
1

  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 :wink: ), 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 :slight_smile:

  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ć :wink: Użycie --pedantic od razu daje dodatkowe ostrzeżenia :wink:

    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ć :wink:

kbaraniecki
Jun '18

Udało się :slight_smile: 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ą :slight_smile:

8 months later
pagazix
Feb '19

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;
}