I have some trouble with my program called “Kalkulator ceny paliwa”.
I have this code
#include <iostream>
#include <math.h>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
using namespace std;
double benzyna(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double diesel(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double lpg(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double cng(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double wodor(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
int main(int argc, char** argv) {
int wyborPaliwa = 0;
double dystans, avgSpalanie, cenaPaliwa, kosztPaliwa;
cout << "Podaj rodzaj paliwa, na ktorym jezdzi Twoj samochod (1 - benzyna, 2 - olej napedowy, 3 - LPG, 4 - CNG, 5 - wodór): " << endl;
switch(wyborPaliwa) {
case 1:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynoszą: " << benzyna(a,b,c) << "zl" << endl;
break;
case 2:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynoszą: " << diesel(a,b,c) << "zl" << endl;
break;
case 3:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynoszą: " << lpg(a,b,c) << "zl" << endl;
break;
case 4:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynoszą: " << cng(a,b,c) << "zl" << endl;
break;
case 5:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynoszą: " << wodor(a,b,c) << "zl" << endl;
break;
}
return 0;
}
And i’m getting these errors:
44 62 C:UsersAntoniDocumentsProjekty C++KalkulatorZuzyciaPaliwamain.cpp [Error] 'a' was not declared in this scope
44 64 C:UsersAntoniDocumentsProjekty C++KalkulatorZuzyciaPaliwamain.cpp [Error] 'b' was not declared in this scope
44 66 C:UsersAntoniDocumentsProjekty C++KalkulatorZuzyciaPaliwamain.cpp [Error] 'c' was not declared in this scope
Where i’ve done the mistakes?
I’m beginner, so please do not hit me.
I hope that i will receive help
I want these errors, that i’ve written before gone.
gryc3k is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.
10
- Add the cin command to let user put in the value for wyborPaliwa right
before Switch statement, that would be used by Switch statment to call its
corresponding case. Without it the case 0 will be run that acutally
doesn’t exist. - Change the a, b, c to dystans, avgSpalanie, cenaPaliwa respectively.
- Pass there addresses that would be stored in the receiving pointers.
i.e. &dystans. - Pass the kosztPaliwa variable as 4th argument without using ‘&’ as the
receiver is not the pointer. - Add the default case at the very last of Switch bracket.
Note: Working with pointers would be done with extra precautions as the real memory is at the stake. If anything bad happens, the actual data would be courrputed or lost.
#include <iostream>
#include <math.h>
using namespace std;
double benzyna(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double diesel(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double lpg(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double cng(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
double wodor(double* dystans, double* avgSpalanie, double* cenaPaliwa, double kosztPaliwa) {
return kosztPaliwa = ((*dystans)/100) * (*avgSpalanie) * (*cenaPaliwa);
}
int main(int argc, char** argv) {
int wyborPaliwa = 0;
double dystans, avgSpalanie, cenaPaliwa, kosztPaliwa;
cout << "Podaj rodzaj paliwa, na ktorym jezdzi Twoj samochod (1 - benzyna, 2 - olej napedowy, 3 - LPG, 4 - CNG, 5 - wodór): " << endl;
cin>> wyborPaliwa;
switch(wyborPaliwa) {
case 1:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynosza: " << benzyna(&dystans,&avgSpalanie,&cenaPaliwa, kosztPaliwa) << "zl" << endl;
break;
case 2:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynosza: " << diesel(&dystans,&avgSpalanie,&cenaPaliwa, kosztPaliwa) << "zl" << endl;
break;
case 3:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynosza: " << lpg(&dystans,&avgSpalanie,&cenaPaliwa, kosztPaliwa) << "zl" << endl;
break;
case 4:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynosza: " << cng(&dystans,&avgSpalanie,&cenaPaliwa, kosztPaliwa) << "zl" << endl;
break;
case 5:
cout << "Podaj dystans do przejechania: ";
cin >> dystans;
cout << "Podaj srednie spalanie twojego samochodu: ";
cin >> avgSpalanie;
cout << "Podaj cene benzyny za litr na Twojej stacji: ";
cin >> cenaPaliwa;
cout << "Koszty przejechania trasy wynosza: " << wodor(&dystans,&avgSpalanie,&cenaPaliwa, kosztPaliwa) << "zl" << endl;
break;
default:
cout << "Invalid choice. Please select a valid option." << endl;
return 1;
}
return 0;
}
2