[C++] Problem z wypisywaniem

transatlantyk a · Post autor: **transatlantyk a** » 12 gru 2015, o 02:47

Witam,
Pisze gre w statki i potrzebuje do tego napisać tablicę z liczbami losowymi
zapis tablicy wygląda u mnie tak

void los()
{
	srand(time(NULL));
	int const N = 10;
	int los[N][N];
	for (int i = 1; i < N; i++)
	{
		for (int k = 1; k < N; k++)
			los[k][i] = ((rand() % 2) + 0);
		cout << los;
		
	}

}

nie rozumiem, jednak, dlaczego przy wypisywaniu tablicy, otrzymuje owszem liczby losowe, ale właściwie z całego ASCII a chodziło mi tylko o 0 i 1

bartek118 · Post autor: **bartek118** » 12 gru 2015, o 09:10

Bo wypisujesz adresy komórek w pamięci, a nie ich zawartość.

musialmi · Post autor: **musialmi** » 12 gru 2015, o 09:39

Innymi słowy, cout << los; nie wypisze ci tablicy w taki sposób, w jaki byś chciał. Bo skąd C++ wie jak byś chciał? Musisz ręcznie wypisywać element po elemencie.

transatlantyk a · Post autor: **transatlantyk a** » 13 gru 2015, o 20:55

w takim razie to powinno załatwić sprawę?
tylko teraz wyświetla mi same 0

Kod: Zaznacz cały

srand(time(NULL));

	int const N = 10;
	int los[N][N] = { ((rand() % 2) + 0) };
	for (int k = 0; k <N; k++)
	{
		for (int i = 0; i < N; i++)
		{
			cout << los[k][i] << " ";
			if (i == 9) cout << " 
";
		}
	}

bartek118 · Post autor: **bartek118** » 13 gru 2015, o 21:07

Bo teraz nie losujesz nic w tablicy, jest cała zerowa.

athame · Post autor: **athame** » 13 gru 2015, o 21:13

Losowanie robisz tylko dla pozycji 10:10. Pozostałe komórki są domyślnie inicjowane na 0, więc rezultat jest taki a nie inny.

Poprawne rozwiązanie to np.

Kod: Zaznacz cały

    srand(time(0));
    
    int const N = 10;
    int los[N][N];
    for (int k = 0; k < N; k++)
    {
        for (int i = 0; i < N; i++)
        {
            los[k][i] = rand()%2;
            cout << los[k][i] << " ";
            if (i == 9) cout << " 
";
        }
    }

jarzabek89 · Post autor: **jarzabek89** » 13 gru 2015, o 21:17

athame pisze:Losowanie robisz tylko dla pozycji 10:10.

No nie bardzo, numery indeksów tej tablicy kończą się na 9.

Kod: Zaznacz cały

 if (i == 9) cout << " 
";

to brzydko wygląda. Wystarczy samo

Kod: Zaznacz cały

 cout << " 
";

po zakończeniu drugiej pętli for.

transatlantyk a · Post autor: **transatlantyk a** » 13 gru 2015, o 21:25

ale wtedy tablica nie wyświetla mi się w ogóle..

jarzabek89 · Post autor: **jarzabek89** » 13 gru 2015, o 21:26

Pokaż kod dla którego Ci się nic nie wyświetla.

transatlantyk a · Post autor: **transatlantyk a** » 13 gru 2015, o 21:44

Kod: Zaznacz cały

srand(time(0));
    
    int const N = 10;
    int los[N][N];
    for (int k = 0; k < N; k++)
    {
        for (int i = 0; i < N; i++)
        {
            los[k][i] = rand()%2;
            cout << los[k][i] << " ";
            if (i == 9) cout << " 
";
        }
    }

athame · Post autor: **athame** » 13 gru 2015, o 21:54

Bez żartów. Pokaż nagłówki pliku źródłowego.

transatlantyk a · Post autor: **transatlantyk a** » 13 gru 2015, o 21:55

Kod: Zaznacz cały

#include<iostream>
#include<ctime>
#include<string>
#include<cstdlib>

tę poprzednią mi wyświetlało, tylko właśnie z samymi zerami

athame · Post autor: **athame** » 13 gru 2015, o 22:01

Jaki kompilator/IDE? Ten kod działa bez problemów i zgodnie z oczekiwaniami pod g++.

transatlantyk a · Post autor: **transatlantyk a** » 13 gru 2015, o 22:07

o juezu, jak sprawdzić jaki mam kompilator? i jeszcze dostałam wiadomość, że cin i cout są dwuznaczne...

athame · Post autor: **athame** » 13 gru 2015, o 22:27

W jakim środowisku piszesz?

W tej wersji z pierwszego posta zamiast cout << los wpisz cout << los[i][k] i zamknij to w klamrze razem z poprzednią instrukcją.

PS. Jaką wiadomość? Z kompilatora? Jeśli tak to zamiast cout używaj std::cout.