Jak to zrobić lepiej c++

Question

Cześć,

Przerabiam ostatnio książkę z zadaniami dla języka python, ale w c++. Aktualnie jestem na zadaniu nr 5, które 

brzmi następująco: 

„Masz takie wyrażenie: (((a1+a2)*a3)-a4)/a5 (elementy od a1 do a5 są typu zmiennoprzecinkowego: float). Pobierz z klawiatury każdą ze zmiennych a1 do a5, oblicz wartość wyrażenia i wyświetl wynik. Zwróć uwagę na wartość zmiennej a5. W przypadku, gdy jej wartość będzie równa 0, nie wykonuj działania, tylko poinformuj o tym.”

Fragmenty z książki: Tomasz Jaśniewski. „Python. Zbiór zadań z rozwiązaniami”. 

#include <iostream>

using namespace std;

float a [5];
float suma;

int main()
{
for(int i = 0; i<4;i++)
	{

		cout<<"Podaj liczbę a"<<i+1<<": ";
		cin>> a[i];a

	}
	cout<<"Podaj liczbę a5: ";
	cin>> a[4];
if(a[4]<=0)
	{
		while(a[4]==0)
		{
			cout<<"Podaj liczbę większą niż 0";
			cin>> a[4];
		}
	}
suma = (((a[0]+a[1])*a[2])-a[3])/a[4];	
cout<<suma;
}

Problemy które rozwiązałem na około to jest pobranie liczby nr 5 poza pętlą, żeby móc zrobić while który sprawdzi czy liczba jest większa od zera. Wygląda to dla mnie koślawo i myślę, że można by to zapisać lepiej, no ale nie wiem jak. Każde rady na wagę złota.

Answer 1

Nie używaj zmiennych globalnych bez potrzeby, using namespace std; też nie jest zalecane, choć przy prostych problemach nie robi różnicy. Co do twojego pytania, to najbardziej eleganckie jest wczytanie wszystkich zmiennych w pętli, a następnie warunkiem sprawdzić, czy a5 jest równe 0. Warto też podzielić kod na funkcje i by funkcja main zwracała wartość. Ogólnie w C/C++ obowiązuje konwencja, że w przypadku zakończenia programu sukcesem zwracamy 0, a w przeciwnym wypadku kod błędu.