Gra w c++ (podsumowanie pierwszych 6 odcinków)

Question

Cześć!

Obejrzałem i przerobiłem 6 pierwszych odcinków Mirosława Zelenta i  postanowiłem napisać jakąś ez game na "rozgrzewke". W sumie dużo nie zrobiłem, ale prosiłbym o poprawienie błędów i pomoc w jednym problemie.

Kod here -> http://wklej.to/hygOy

Jak zalogowano zwraca wartość true to to menu ciągle mi się powtarza (po zalogowaniu) (zajrzyjcie w kod to bd wiedzieć o co mi chodzi).

Z góry dzięki za odpowiedzi, pozdrawiam

Answer 1

Witam serdecznie,

dam Ci parę wskazówek. Nie pisz zmiennych globalnych gdyż to zła praktyka i powinieneś dążyc do ukrycia danych(hermetyzacja danych). Przekonasz się w większych projektach. Staraj się czyścić buffor po użyciu jakiejś kluczowej zmiennej używając metody cin.sync(); . . Program powtarza się nieskończoną ilość razy gdyż wykorzystałeś pętle for(;;). ,która powtarza się nieskończoność ilość razy(nie podałeś warunków krańcowych). Jeśli chcerz wyjść z jakiejś pętli użyj definicji break; np. po społnieniu jakiegoś warunku.

Comments

Czyli zmienne umieszczać wewnątrz funkcji?

Inne funkcje będą miałe wtedy do nich dostęp? (z tego co pamiętam to możnabyło ustawiać public czy coś)

---

Tak, gdyż w ten sposób ukrywasz zmienne.

Postaram ci się to wyjaśnić obrazowo:

typ_zmiennej a1;

int main()

{

// w tym miejscu na całym obszarze 'main' jest widoczna zmienna a1

typ_zmiennaj a2 ;

for(;;;) lub if() lub jakakolwiek inna funkcja np while() , twojafunkaca()

{

//wedle tego obszaru pamięci zmienna a2 jest zmienna globalną i możesz się do niej //odwoływać w ciele powyższej funkcji lub w obrębie funcji main()

typ_zmiennej a3 ;

for(;;;) lub if() lub jakakolwiek inna funkcja np while() , twojafunkaca()

{

//tutaj możesz bez problemu operować na zmiennej a3 gdyż wedle tego obszaru pamięci

//zmienna a3 jest globalna

}

}

//tu nie możesz się odwołać do zmiennej do zmiennej a3 np a3 = 3 ; gdyż obszar pamięci

//tej zmiennej jest schowany w wnętrzu funkcii for(;;;) czy innych ;

a1= 1 ;

a2 = 2;

//powyższe dwa zapisy są możliwe gdyż zmienna a1 jest globalna względem 'main'

//natomiast a2 jest 'zamkniete' klamrami w funkcji main

}

// unikaj pisania zmiennych globalnych gdyż sercem programu jest funkcja main i na jej

//na jej zasięgu powinny być zamknięte zmienne

//muwi się, że im lepszy kod im mniej zmiennych globalnych więc używa się i w ostaczetności

Mam nadzieje że troszkę rozjaśniłem wątpliwość,

Pozdrawiam

---

Widać, że sam nie wiesz dlaczego nie powinno się używać zmiennych globalnych... I bynajmniej nie chodzi tu o to, że funkcja main "jest sercem programu" :)

Answer 2

Taki jest kod - powtarza tylko zalogowanie, ponieważ warunek jest spełniony a petla obraca się zawsze, program działa ok. Musisz dokleić częśc kodu z else{} :)

Comments

Czyli jak mogę złamać tą pętle? ;d

---

Jeżeli warunek będzie niespełniony, czyli na przykład  

for(bool test=true; test!=zalogowany;1+2)

osobiście polecam jendak pętle do{}while(warunek), która wykona się zawsze jeden raz - tutaj zapis będzie

do

{

}while(zalogowany!=true);

czyli czynność będzie się powtarzała dopóki nie będziesz zalogowany. Już nie jestem pewny, tego co piszę - jest późno, w razie problemów napisz je, postaram się na nie odpowiedzieć rano ;)