Optymalizowanie kodu wykorzystujące dziedziczenie (++)

Question

Witam!

Program który napisałem nie jest niczym wielkim ogólnie rzecz biorąc było to ćwiczenie dziedziczenia (jednak nie ma go tutaj za wiele).

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

class Bank
{
private:
	std::string name;
	int age;
	int PIN;
	double value = 22422.44;
public:
	Bank(const std::string, int);
	void check_pin();
};

class Person : public Bank
{
public:
	Person(std::string, int);
};

Bank::Bank(const std::string nm, int ag) : name(nm), age(ag)
{
	std::cout << "Podaj imie i nazwisko: "; std::getline(std::cin, name);
	std::cout << "Podaj swoj wiek: " << ag << std::endl;
}

void Bank::check_pin()
{
	std::cout << "Podaj kod PIN: ";
	std::cin >> PIN;
	if (PIN == 7913)
	{
		std::cout << "Kod prawidlowy!\n";
		std::cout << "Saldo: " << value << std::endl;
	}
	else
		std::cout << "Kod nieprawidlowy!\n";
	std::cout << "Dziekujemy za przybycie!\n";
}

Person::Person(std::string nm, int ag) : Bank(nm, ag) { }

int main()
{
	std::string nm;
	Person klient(nm, 23);
	klient.check_pin();
	Sleep(5000);
	return 0;
}

Mam pytania:

1. W jaki sposób mogę stworzyć warunek pętli opierając się na wytycznych (program zakończy się, wyświetlając prawidłowe dane)

• PIN == 7913, PIN > 999, PIN < 9999,
• Jeżeli PIN jest różny od 7913, może się wykonać 3 razy (właśnie w tym miejscu mam problem, ponieważ wielokrotnie chciałem połączyć w waruneku pętli powyższe dane i wyświetlenie 3 krotne pętli.

Chodzi dokładnie o tą część kodu:

std::cout << "Podaj kod PIN: "; std::cin >> PIN; if (PIN == 7913) { std::cout << "Kod prawidlowy!\n"; std::cout << "Saldo: " << value << std::endl; } else std::cout << "Kod nieprawidlowy!\n";

 

2. Korzystając z dziedziczenia oraz różnych metod w jaki sposób mogę zoptymalizować ten kod ?

Answer 1

Po pierwsze uzywaj plikow naglowkowych. Po drugie pamietaj ze dziedziczenie jest odwzorowaniem rzeczywistosci i wyjasnij jaki sens ma ze czlowiek dziedziczy z banku? Pamietaj ze nie wszystko da sie napisac uzywajac dziedziczenia. Tutaj chcesz tylko zrobic symulacje banku a to kwestia kilku linijek w mainie. Chyba ze chcesz uzyc obiektowki to podziel to sobie na dwie odrebne klasy Bank, klient i stworz wspolprace miedzy obiektami tych klas. Dziedziczenie jak najbardziej nie ma tu sensu

Comments

wyjasnij jaki sens ma ze czlowiek dziedziczy z banku

Podobnie, dlaczego PIN i wiek są cechami banku?

---

Bank posiada klientów, to drugie to po prostu start z dziedziczeniem jeszcze nie wiem do końca jak to działa.

Tak lepiej ?:

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

class Bank
{
private:
	int PIN;
	double value = 22422.44;
public:
	Bank() {};
	void check_pin();
};

class Person : public Bank
{
private:
	std::string name;
	int age;
public:
	Person(std::string, int);
};

void Bank::check_pin() // dodać opcje 3 krotnej próby
{
	std::cout << "Podaj kod PIN: ";
	std::cin >> PIN;
	if (PIN == 7913)
	{
		std::cout << "Kod prawidlowy!\n";
		std::cout << "Saldo: " << value << std::endl;
	}
	else if (PIN < 999 || PIN > 9999)
		std::cout << "Podanych liczb jest za duzo lub za malo!\n";
	else
		std::cout << "Kod nieprawidlowy!\n";
	std::cout << "Dziekujemy za przybycie!\n";
}

Person::Person(std::string nm, int ag) : name(nm), age(ag)  {
	std::cout << "Podaj imie i nazwisko: "; std::getline(std::cin, name);
	std::cout << "Podaj swoj wiek: " << ag << std::endl;
}

int main()
{
	std::string nm;
	Person klient(nm, 23);
	klient.check_pin();
	Sleep(5000);
	return 0;
}

Jaki warunek pętli muszę teraz dodać, aby program sprawdzał PIN (check_pin()) 3 krotnie (w przypadku błędu) lub jeżeli kod jest odpowiedni program się zamyka ?

---

Dalej masz dziedziczenie ale jak chcesz.  w czym problem tworzysz petle xd

for(int i=0;i<3;i++)
///wczytuj 
/// if poprawny pin opusc petle
///else komunikat podaj pin jescze raz

 

---

Osoba wciąż nie powinna być bankiem.

---

Jednak break, zadziałało dziękuję Ci.

Tak lepiej ?

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

class Bank_account
{
private:
	int PIN;
	double value = 22422.44;
public:
	Bank_account() {};
	void check_pin();
};

class Person : public Bank_account
{
private:
	std::string name;
	int age;
public:
	Person(std::string, int);
};

void Bank_account::check_pin() // dodać opcje 3 krotnej próby
{
	std::cout << "Podaj kod PIN: ";
	std::cin >> PIN;
	int i = 0;
	for(int i=0; i<3;i++) {
		if (PIN == 7913)
		{
			std::cout << "Kod prawidlowy!\n";
			std::cout << "Saldo: " << value << std::endl;
			break;
		}
		else if (PIN < 999 || PIN > 9999)
			std::cout << "Podanych liczb jest za duzo lub za malo!\n";
		else
			std::cout << "Kod nieprawidlowy!\n";
		i++;
		std::cout << "Podaj kod PIN: ";
		std::cin >> PIN;
	}
	std::cout << "Dziekujemy za przybycie!\n";
}

Person::Person(std::string nm, int ag) : name(nm), age(ag)  {
	std::cout << "Podaj imie i nazwisko: "; std::getline(std::cin, name);
	std::cout << "Podaj swoj wiek: " << ag << std::endl;
}

int main()
{
	std::string nm;
	Person klient(nm, 23);
	klient.check_pin();
	Sleep(5000);
	return 0;
}

 

---

Osoba nie jest też kontem bankowym. Co najwyżej ma konto bankowe.