Skopiowany program C++ nie działa w nowym programie

Question

Witam,

Moim zadaniem jest napisać program, który ma imitować 6 rzutów kostką sześcienną. Będzie on losował liczby od przedziału 1-6 a na koniec podsumowywał ilość punktów. 

Zrobiłam to zadanie poprawnie na moim komputerze. Jednakże gdy próbowałam przesłać program na mojego maila, cały czas wyskakiwało mi zero. Nie wiem, jak sobie z tym poradzić.

Tutaj zamieszczam, jak wygląda program.

#include <iostream>
#include <windows.h>
#include <time.h>


using namespace std;

#include <cstdio>

int liczba,a,b,c,d,e,f, suma;
int main()
{
   
    cout<<"Za chwile nastapi 6 rzutow kostka!";
    Sleep(3000);
    cout<<endl;
   
    srand(time(NULL));
   
   for(int i = 1; liczba==6; i++)
         
    
   
	cout<<a<<"\a"<<endl;
	 		
   	cout<<b<<"\a"<<endl;
   
	cout<<c<<"\a"<<endl;
	 	
	cout<<d<<"\a"<<endl;
    
	cout<<e<<"\a"<<endl;
	
	cout<<f<<"\a"<<endl;
	
	liczba=a+b+c+d+e+f;
	;suma=liczba
;cout<<"Wylosowales w sumie "<<suma<<" punktow";

    getchar();
    return 0;
}

 

Tutaj zamieszczam też filmik z wizualizacją problemu dla lepszego zrozumienia https://youtu.be/q_Hf2XvazC8

Z góry czekam na Wasze sugestie dot. rozwiązania problemu.

Pozdrawiam,

Lucifiery

 

Comments

Proszę umieścić kod w odpowiednim bloczku.

 

Answer 1

Prawie wszystko źle.

Po co tyle zmiennych - masz gdzieś pamiętać te wynik? Nie wystaczy wpisywać bezpośrednio po wylosowaniu i tylko uaktualniać sumę?

Pętla for (linia 20) nie wykona się ani razu. Nie powinno tam być < albo <=? Poza tym pętla (brak {}) obejmuje tylko wypisanie zmiennej a.

I jak już autorzy poprzednich postów zauważyli brakuje samego losowania. Funkcja srand inicjuje generator liczb losowych, ale właściwego losowania brakuje. Wypisujesz po prostu wartości zmiennych z którymi wcześniej nic nie robisz.

Answer 2

Twój program odczytuje losowe dane z pamięci (niezainicjalizowane zmienne). Użyj funkcji rand do losowania liczb.

Comments

Zmienne globalne standardowo są zerowane przed uruchomieniem main().