Refaktoryzacja kodu - bankomat w C++

Question

Witam, postanowiłem stworzyć program symulujący działanie bankomatu. Użytkownik ma 1+3 próby wpisania PINu. Zauważyłem, że dwa razy powtarzam if'a z prawidłowym PIN'em. Mam jeszcze wątpliwości co do użycia pętli do ... while. Czy mógłby ktoś nakierować mnie na uproszczenie kodu? Zamieszczam kod źródłowy poniżej.

#include <iostream>

using namespace std;

string PIN;
int prob=4;

int main() {
    cout << "Witaj w naszym banku!" << endl;
    cout << "Podaj kod PIN: ";
    cin >> PIN;

    if (PIN=="1729")
    {
        cout << "Twoj kod PIN jest poprawny!";
    } else
    do
    {
        prob--;
        cout << "Bledny kod PIN! Pozostalo " << prob <<" prob. Sprobuj ponownie ";
        cin >> PIN;

        if (PIN=="1729")
        {
            cout << "Twoj kod PIN jest poprawny!";
            return 0;
        }

    } while (prob>=2);

    if (prob==1)
    {
        cout << "Twoje konto zostalo zablokowane!";
    }

    return 0;
}

 

Answer 1

Jeśli i tak używasz do...while, to dlaczego wszystkiego nie wrzucisz w tą pętle?
 

do{
  //wprowadzanie pinu
}while(PIN != "<poprawny pin>" || prob < max_prob);

 

Comments

Jeśli zalezy ci na tym komunikacie o błędnym pinie to stwórz sobie np. pomocniczego boola który bedzie ustawiany na true po pierwszym obrocie pętli.
if(pomocniczy_bool) wyswietl kominikat

Answer 2

Może coś w tym stylu:

bool stan=0;
	do
	{
	   cout << "Podaj pin: ";
	   cin >> pin; 
	   if(pin==prawidlowy_pin)
	   		stan = 1;
	   else 
	      cout << "Bledny pin! Pozostało" << --prob << "prob.";
	}
	while(stan==0&&prob>0);
	if(stan)
	   cout << "Twoj PIN jest poprawny!";
	else 
	   cout << "Konto zablokowane!";