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

No przecież !

Działa jak należy. Póki co zadeklarowałem je jako zmienne globalne.

Jeszcze jedno pytanko : Po wybraniu opcji dzielnia : przykł. (10/2) wyświetla mi cout'a a za nim napis "inf". Dlaczego tak jest i jak się tego pozbyć ?

---

co dokładnie zmieniłeś w kodzie?

---

Zadeklarowałem zmienne globalnie, czyli nad mainem :

#include <iostream>

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

I wywaliłem je też z maina:

int main(int argc, char** argv) {
int wybor;	
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";

Reszta bez zmian.

---

A wynik wypisujesz?

---

Wypisuję :D Tak to wygląda : 

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

}	

 

---

Wydaje mi się, że problem jest taki, że wysyłasz do funkcji inty, a funkcja przyjmuje floaty... dawno sie nie zajmowałem cpp więc może mówie coś źle...

---

Niejawna konwersja.Tutaj wszystko jest ok. Wrzuć jeszcze raz cały kod, bo u mnie to działało

---

#include <iostream>
#include <cstdlib>
#include <conio.h>

using namespace std;
int a, b;
void podaj(){
do{
	system("cls");
	cin.clear();
	cin.ignore();
	cout <<"Podaj 1 liczbe: "; cin >> a;
 }while(cin.fail());
do{
	system("cls");
	cin.clear();
	cin.ignore();
	cout <<"Podaj 2 liczbe: "; cin >> b;
 }while(cin.fail());
}

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 "; 
		system("EXIT");				
		}
		else 
		return a/b;
}




int main(int argc, char** argv) {
int wybor;	
do{
cout << "Kalkulator v.1 Archeon.exe \n";
cout << "__________M E N U__________ \n \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();
	cout << dodaj(a,b);	
	break;
}
case 2: cout << "[2]. Odejmowanie \n";
{
	podaj();
	cout << odejmuj(a,b);	
	break;
}
case 3: cout << "[3]. Mnozenie \n";
{
	podaj();
	cout << mnoz(a,b);	
	break;
}
case 4: cout << "[4]. Dzielenie \n";
{
	podaj();
	cout << dziel(a,b);	
	break;
}
case 5: cout << "WYJSCIE \n";
{
	system("EXIT");	
	break;
}

}	
	
	return 0;
}

 

---

Wpisuje dokładnie: enter 4 enter 10 enter 2 enter.

Pieknie wypisuje 5. Nie wiem co Ci nie działa

---

Chodzi o to że jak wybierzemy dzielnie i za 2 liczbę wpiszemy 0 to wypisuje komunikat tak jak powinien, ale jeszcze niżej wypisywane jest "0". O to właśnie chodzi. Dlaczego tak jest?

---

Aaa. To zmienia postać rzeczy. W przypadku dzielenia przez 0 robisz bardzo brzydką rzecz: Nie zwracasz nic z funkcji, która powinna zwrócić float tylko wywołujesz system("exit"), które generalnie powinno zamknąć konsole, czego nie robi. Koniec końców zostają zwrócone jakieś śmieci, bo program kontynuuje działanie. System("exit") to bardzo kiepski sposób na przerwanie programu. Po pierwsze nie przerywaj programu w środku funkcji, a jesli już koniecznie musisz to użyj exit(1).

---

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.