problem z programem w C++ (Kalkulator)

[matys11 0]

oto kod programu:

 

#include<iostram>
#include<conio.h>
#include<math.h>
using namespace std;
double a = 0.0;
double b = 0.0;
double w = 0.0;
int dzialania;
int main(int argc, char "argv[])

{
system("color 20");
cout << "\t\t\t\tKALKULATOR" << endl;
cout << "\t\t\t\t=============" << endl;
{

	 int sw = 1;
	 while(sw == 1)
	 {
			  cout << "Wybirz typ działania:\n" << endl;
			  cout << "  Dodawanie-1	odejmowanie-2 Mnorzenie-3\tDzielenie-4"
			  cout<< "\t\t\t\tZakoncz-o" << endl << endl;
			  cin >> dzialania: system("cls");
			  switch(dzialania)

			  {
							   case 1:
									cout << "Podaj pierwsza liczbe: ";
									cin >> a;
									cout << "Podaj druga liczbe: ";
									cin >> b;
									w = (a + b);
									break:

							   case 2:
									cout << "Podaj pierwsza liczbe: ";
									cin >> a;
									cout << "Podaj druga liczbe: ";
									cin >> b;
									w = (a - b);
									break:

							   case 3:										  
									cout << "Podaj pierwsza liczbe: ";
									cin >> a;
									cout << "Podaj druga liczbe: ";
									cin >> b;
									w = (a * b);
									break:

							   case 4:
									cout << "Podaj pierwsza liczbe: ";
									cin >> a;
									cout << "Podaj druga liczbe: ";
									cin >> b;
									w = (a / b);
									break:

							   case 5:
									cout << "Podaj pierwsza liczbe: ";
									cin >> a;
									cout << "Podaj druga liczbe: ";
									cin >> b;
									w = pow(a,  b);
									break:  
							   case 0:
									cout << "\t\t Dzienkuje za skorzystanie z programu!";
									gretch():
									return 0;
									break:

									 default:
											 cout << "\t\tniepopzawny typ dzelenia. Dowidzenia!" << endl;
											 gretch():
											 return 0;

											 }
											 cout << "\nWynik to: " << w << endl;
											 cout << "\nJeszzce raz? [1-TAK, 0-NIE]\n"
						  cin >> sw: system("cls"):

							  }

										 cout << "\n\t\t Dzienkuje za skorzystanie \n" endl;
										 getch():
										 return 0:
}
										 }

 

 

 

możecie powiedzieć co jest źle w tym kodzie bo jak chcę skomplikować ten problem to co chwile mi błąd wyskakuje

[januzi 24]

bledne linie:

iostram

cin >> dzialania: system("cls");

cin >> sw: system("cls"):

gretch():

break:

return 0:

cout << "\nJeszzce raz? [1-TAK, 0-NIE]\n"

 

i pewnie jeszcze pare

[marcin2213 2]

int main(int argc, char "argv[])

cout << " Dodawanie-1 odejmowanie-2 Mnorzenie-3\tDzielenie-4"

cout << "\n\t\t Dzienkuje za skorzystanie \n" endl;

getch():

 

Czyli ogólnie, zapominasz średników albo za miast nich dajesz :.

Pisz tak żeby wszystko nie było w jednej lini tzn. unikaj czekoś takiego:

while(sw == 1)
{
cout << "Wybirz typ działania:\n" << endl;
cout << " Dodawanie-1 odejmowanie-2 Mnorzenie-3\tDzielenie-4"

lepiej rób

while(sw == 1)
{
cout << "Wybirz typ działania:\n" << endl;
cout << " Dodawanie-1 odejmowanie-2 Mnorzenie-3\tDzielenie-4"

Będzie Ci wtedy łatwiej ogarnąć wzrokiem cały kod.

[matys11 0]

mam prośbę do was cy mogli byście mi prawic ten kod?

[W2K 0]

mam prośbę do was cy mogli byście mi prawic ten kod?

Nie, bo każdy programista sam poprawia swój kod, tym bardziej jeśli są to tak błahe błędy jak te... Zresztą jakkbyś podał błędy któe ci wyskakują byłoby łatwiej cokolwiek stwierdzić...

[matys11 0]

88 C:\Documents and Settings\Mateusz\Pulpit\wszystko\BezNazwy1.cpp expected `,' or `;' at end of input

88 C:\Documents and Settings\Mateusz\Pulpit\wszystko\BezNazwy1.cpp expected init-declarator at end of input

88 C:\Documents and Settings\Mateusz\Pulpit\wszystko\BezNazwy1.cpp expected `)' at end of input

11 C:\Documents and Settings\Mateusz\Pulpit\wszystko\BezNazwy1.cpp expected `,' or `...' before '{' token

9 C:\Documents and Settings\Mateusz\Pulpit\wszystko\BezNazwy1.cpp missing terminating " character

 

oto błędy jakie mi wyskakują

[Bono[UG] 1920]

Więc je czytasz i wiesz gdzie mniej więcej są i co jest nie tak.

[matys11 0]

poprawiłem już kilka błędów i zostały mi teraz te i nie wiem co dalej

9 C:\Documents and Settings\Mateusz\Pulpit\wszystko\kaukulator.cpp expected `,' or `...' before string constant

11 C:\Documents and Settings\Mateusz\Pulpit\wszystko\kaukulator.cpp expected unqualified-id before '{' token

11 C:\Documents and Settings\Mateusz\Pulpit\wszystko\kaukulator.cpp expected `,' or `;' before '{' token

[staffer 12]

1. Ortografia. Literówki i przestawienia mogą się każdemu zdarzyć przy pisaniu "na szybko", ale jak widzę "Dzienkuje" albo "Mnorzenie" to mi się nóż w kieszeni otwiera. Chociaż jak zobaczyłem "Kaukulator" to się uśmiałem, przez wiadome skojarzenie.

 

2. Po break powinien być średnik, nie ":". http://swiatinformatyki.ovh.org/kurscplus5.php Tutaj masz prosty opis działania funkcji switch.

 

3. Czy to na pewno jest Twój program? . Menu główne nie wykorzystuje wszystkich funkcji kodu (dla 5 też masz przecież podnoszenie do potęgi). Ponadto litera "o", a cyfra "0" to nie to samo i mało kto, mając taki opis w menu, wyjdzie z programu za pomocą proponowanej przez Ciebie metody, a nie przez default

 

4. Masz niepotrzebnie 1 klamrę.

int main(int argc, char "argv[])

{
system("color 20");
cout << "\t\t\t\tKALKULATOR" << endl;
cout << "\t\t\t\t=============" << endl;
{ <--- tutaj

Wg mnie nie jest ona potrzebna do niczego, a musisz później dodatkowo o niej pamiętać, no chyba, że określa ona zakres dla funkcji stąd: sw: system("cls"), ale w ten sposób się nie bawiłem dotychczas, więc po prostu nie wiem.

 

5. Żeby mieć pewność, że wszystkie błędy znaczą dokładnie to, co jest napisane, wpierw usuń wszystkie błędy związane z interpunkcją kodu, a następnie zacznij czytać i analizować, co który oznacza. Z resztą 2 z nich wyraźnie mówią o braku ";" w odpowiednim miejscu.

[matys11 0]

mój kod wygląda teraz tak:

 

#include<iostream>
#include<conio.h>
#include<math.h>
using namespace std;
double a = 0.0;
double b = 0.0;
double w = 0.0;
int dzialania;
int main(int argc, char "argv[]");

{
system("color 20");
cout << "\t\t\t\tKALKULATOR" << endl;
cout << "\t\t\t\t=============" << endl;


int sw = 1;
while(sw == 1) 
switch(dzialania)
{
cout << "Wybirz typ działania:\n" << endl;
cout << " Dodawanie-1 odejmowanie-2 Mnorzenie-3\tDzielenie-4"
cout<< "\t\t\t\tZakoncz-0" << endl << endl;
cin >> dzialania; system("cls");


{
case 1:
cout << "Podaj pierwsza liczbe: ";
cin >> a;
cout << "Podaj druga liczbe: ";
cin >> b;
w = (a + b);
break;

case 2:
cout << "Podaj pierwsza liczbe: ";
cin >> a;
cout << "Podaj druga liczbe: ";
cin >> b;
w = (a - b);
break;

case 3:
cout << "Podaj pierwsza liczbe: ";
cin >> a;
cout << "Podaj druga liczbe: ";
cin >> b;
w = (a * b);
break;

case 4:
cout << "Podaj pierwsza liczbe: ";
cin >> a;
cout << "Podaj druga liczbe: ";
cin >> b;
w = (a / b);
break;

case 5:
cout << "Podaj pierwsza liczbe: ";
cin >> a;
cout << "Podaj druga liczbe: ";
cin >> b;
w = pow(a, b);
break;
case 0:
cout << "\t\t Dzienkuje za skorzystanie z programu!";
gretch():
return 0:
break;

default:
cout << "\t\tniepopzawny typ dzelenia. Dowidzenia!" << endl;
gretch();
return 0:

}
cout << "\nWynik to: " << w << endl;
cout << "\nJeszzce raz? [1-TAK, 0-NIE]\n"
cin >> sw: system("cls"):



cout << "\n\t\t Dzienkuje za skorzystanie \n" endl;
getch():
return 0:
}

[staffer 12]

1. Umieszczaj kod programu w znacznikach

(code)(/code), tyle że w nawiasach kwadratowych

2. Instrukcje kończysz średnikiem, nie dwukropkiem, tak że po returnach powinieneś poprawić.

3. Nie kojarzę instrukcji gretch(), google też nie, no chyba że chodzi o getch().

4. Jak możesz to popraw ortografię, bo można cierpieć czytając to.

5. Wyjście z programu powinieneś opatrzyć jakąś pętlą albo czymś, bo ktoś może Tobie wysypać program wciskając "Z" zamiast 1 lub 0. Nie powinno się tak tego zostawiać.

[matys11 0]

powiedz mi co dokładnie jest źle w linijce 9i11. proszę

[januzi 24]

po main nie daje sie srednika

[gates22 0]

wlasnie, po main nie daje sie srednika, a po drugie ma byc nie char "argv[]" tylko ma być char *argv[]

[matys11 0]

Dziękuje wam wszystkim już działa poprawnie ten kalkulator.