Jak wyżej... jeśli można go uprościć to można prosić o taki kod?
Dziękuję z góry! :)
Pozdrawiam.
0
2
tak, ponad polowa kodu jest zbędna.
5
Zobacz:
#include<iostream>
#include<thread>
#include<string>
using namespace std;
int main() {
cout << "Za dwie sekundy pojawi sie panel logowania...\n";
this_thread::sleep_for(chrono::milliseconds(2000));
string password = "", correctPassword = "ryciak";
string login = "", correctLogin = "hubert";
do {
cout << "Podaj login: "; cin >> login;
cout << "Podaj haslo: "; cin >> password;
} while (login != correctLogin || password != correctPassword);
cout << "Zalogowany\n";
return 0;
}
- Korzystaj z
#include <thread>
żeby na chwilę wstrzymać proces; - Unikaj
magic string'ów
, bo później żeby zmienić wartość jakiegoś łańcucha w programie będziesz musiał latać po całym kodzie zamiast zmienić wartość jednego obiektu i mieć spokój. Tutaj zastąpiłem takowe obiektamicorrectLogin
icorrectPassword
. Prawda, że lepiej teraz dokonywać zmian?; - Skorzystaj z pętli
do { } while(warunek)
, która wykona się raz przed sprawdzeniem warunku. Nie będziesz musiał się powtarzać zif'ami
, bo pętlawhile(warunek) { }
sprawdza najpierw warunek w nawiasie i jeżeli nie jest spełniony nie wykona iteracji; - Nie używaj
getchar();getchar();
albosystem("pause");
IDE ma opcje zatrzymania konsoli. W VS2015 możesz to zrobić uruchamiając program spod skróruCTRL-F5
.