C++ | FUNKCJE, PROSTY KALKULATOR : Gdzie jest błąd ?

Question

Witam,

Chciałem napisać prosty kalkulator który opiera się na funkcjach.

Kod wydaje się być poprawny, ale nie działa jak należy..

Niestety funkcje nie zwracają wyników.

Oto mój kod:

#include <iostream>

using namespace std;

void podaj(){
	int a, b;
	cout <<"Podaj 1 liczbe: "; cin >> a;
	cout <<"Podaj 2 liczbe: "; cin >> b;
}

float dodaj(float a, float b)
{
		return a+b;
}

float odejmuj(float a, float b)
{
		return a-b;
}

float mnoz(float a, float b)
{
		return a*b;
}

float dziel(float a, float b)
{
		if(b==0)
		{	cout << "Nie dziel przez 0 cholero ! \n";		}
		return a/b;
}




int main(int argc, char** argv) {
int wybor, a, b;	
do{
cout << "Kalkulator v.1 Archeon.exe \n";
cout << "__________M E N U__________ \n";
cout << "[1]. Dodawanie \n";	
cout << "[2]. Odejmowanie \n";	
cout << "[3]. Mnozenie \n";	
cout << "[4]. Dzielenie \n";	
cout << "[5]. Wyjscie \n";
cout << "NACISNIJ ENTERE ABY ROZPOCZAC PROGRAM: \n";	
	
cin.clear();
cin.ignore();
cout << "Wybierz cyfre: "; cin >> wybor;	
}while(cin.fail());

switch (wybor)
{
case 1: cout << "[1]. Dodawanie \n";
{
	podaj();
	dodaj(a,b);	
	break;
}
case 2: cout << "[2]. Odejmowanie \n";
{
	podaj();
	odejmuj(a,b);	
	break;
}
case 3: cout << "[3]. Mnozenie \n";
{
	podaj();
	mnoz(a,b);	
	break;
}
case 4: cout << "[4]. Dzielenie \n";
{
	podaj();
	dziel(a,b);	
	break;
}
case 5: cout << "WYJSCIE \n";
{
	system("EXIT");	
	break;
}

}	
	
	return 0;
}

 

Answer 1

    dodaj(a,b); 

zwracają tylko ich nigdzie nie przypisujesz. Zrób tak:

double wynik = dodaj(a,b);

lub od razu wypisz wynik:

cout<<dodaj(a,b)<<endl;

I tutaj masz problem:

void podaj(){
    int a, b;
    cout <<"Podaj 1 liczbe: "; cin >> a;
    cout <<"Podaj 2 liczbe: "; cin >> b;
}

a i b są zmiennymi lokalnymi, które przestają istnieć po wyjściu z funkcji. Jeśli chcesz zwrócić ich wartości poza funkcję to masz kilka opcji

• przekazać je do funkcji przez wskaźnik (poczytaj na necie, jest milion tutoriali w każym języku o tym)
• zwrócić returnem, aczkolwiek masz dwie wartości więc musiałbyś je złączyć definiujac strukture wynik ( j.w. - struktury C++)
• zdefiniować je poza mainem jako zmienne globalne, wtedy będą widoczne wszędzie (wiem, że brzmi kusząco, ale to niezalecana praktyka)

int wybor, a, b; 

te zmienne zdefiniowane w mainie nie mają żadnego powiązania z tymi, które zdeklarowałeś w funkcji.

Comments

wywaliłem całkowicie linijkę z system("exit") i nadal tak samo :/

---

bo w ifie wewnatrz funkcji dziel, zwracasz tylko jesli b nie jest 0. jesli jest 0 nic nie zwracasz i dlatego program glupieje...

---

Ale przecież mam else .. Nie rozumiem. To jak to powinno być ?

---

zamiast system exit wrzuc w tego ifa jakiegos returna "domyslnego" typu return 0 albo cos. mozesz tez przemyslec czy nie zrobic inaczej calej funkcji (dzielenie bywa zdradliwe)

---

@RafalS,

exit(1) załatwił sprawę :D

 

Dziękuję wszystkim za pomoc. Temat do zamknięcia.