Kalkulator

[IchHeibe]

Witam czy możecie powiedzieć mi czy kod jest poprawny i jakie zmiany muszę wprowadzić

 

daje like

 

 

 

#include <iostream>

#include <conio.h>

#include <cstdio>

using namespace std;

 

int main()

{

int l[100000]; //Wiem że dużo

int wynik[100000]; //Wiem że dużo

cout<<"Kalkulator Wpisz 1 (Dodawanie) Wpisz 2 (Odejmowanie) 3 (mnozenie) 4 (Dzielenie)"<<endl;

cin>>l[1];

switch ( l[1] )

{

case 1:

cout<<"Wpisz 1 liczbe"<<endl;

cin>>l[2];

cout<<"Wpisz 2 liczbe"<<endl;

cin>>l[3];

wynik[1] = l[2] + l[3];

cout<<"Wynik dodawania to "<<wynik[1]<<endl;

break;

 

case 2:

cout<<"Wpisz 1 liczbe"<<endl;

cin>>l[4];

cout<<"Wpisz 2 liczbe"<<endl;

cin>>l[5];

wynik[2] = l[4] - l[5];

cout<<"Wynik odejmowania to "<<wynik[2]<<endl;

break;

 

case 3:

cout<<"Wpisz 1 liczbe"<<endl;

cin>>l[6];

cout<<"Wpisz 2 liczbe"<<endl;

cin>>l[7];

wynik[3] = l[6] * l[7];

cout<<"Wynik mnozenia to "<<wynik[3]<<endl;

break;

 

case 4:

cout<<"Wpisz 1 liczbe"<<endl;

cin>>l[8];

cout<<"Wpisz 2 liczbe"<<endl;

cin>>l[9];

wynik[4] = l[8] / l[9];

 

cout<<"Wynik dzielenia to "<<wynik[4]<<endl;

break;

 

default :

cout<<"Nie ma takiej opcji wybierz inna\n\n\n"<<endl;

main();

 

 

 

}

 

 

 

getch();

}

 

 

[AbyssGod]

Kod jest poprawny, jednak powinieneś dopisać do kodu informacje o dzieleniu przez zero, nie wolno dzielić przez zero, jeżeli u Cb dzielę liczbe 1 przez liczbe 2 przy czym liczba 2 jest równa 0 to kalkulator przestaje pracować

 

Kod powinien wyglądać następująco

#include <iostream>
#include <conio.h>
#include <cstdio>
using namespace std;
int main()
{
int l[100000];  //Wiem że dużo 
int wynik[100000];  //Wiem że dużo 
cout<<"Kalkulator Wpisz 1 (Dodawanie) Wpisz 2 (Odejmowanie) 3 (mnozenie) 4 (Dzielenie)"<<endl;
cin>>l[1];
switch ( l[1] )
{
case 1:
cout<<"Wpisz 1 liczbe"<<endl;
cin>>l[2];
cout<<"Wpisz 2 liczbe"<<endl;
cin>>l[3];
wynik[1] = l[2] + l[3];
cout<<"Wynik dodawania to "<<wynik[1]<<endl;
break;
case 2:
cout<<"Wpisz 1 liczbe"<<endl;
cin>>l[4];
cout<<"Wpisz 2 liczbe"<<endl;
cin>>l[5];
wynik[2] = l[4] - l[5];
cout<<"Wynik odejmowania to "<<wynik[2]<<endl;
break;
case 3:
cout<<"Wpisz 1 liczbe"<<endl;
cin>>l[6];
cout<<"Wpisz 2 liczbe"<<endl;
cin>>l[7];
wynik[3] = l[6] * l[7];
cout<<"Wynik mnozenia to "<<wynik[3]<<endl;
break;
case 4:
cout<<"Wpisz 1 liczbe"<<endl;
cin>>l[8];
cout<<"Wpisz 2 liczbe"<<endl;
cin>>l[9];
if  ( l[9] != 0 )
wynik[4] = l[8] / l[9];
else
		    cout << "Nie dziel przez zero!" << endl << endl;
cout<<"Wynik dzielenia to "<<wynik[4]<<endl;
break;
default :
cout<<"Nie ma takiej opcji wybierz inna\n\n\n"<<endl;
main();

:

[Xau]

Trochę bezsensu te rozwiązanie z tablicami.

wystarczą tylko 3 zmienne: dwie na liczby i jedna na wynik.

Należy uważnie operować pamięcią, a nie deklarować ile wlezie.

 

No i te conio....

zamiast getch użyj cin.get();

 

A i cstdio na pewno jest tu potrzebne?

[IchHeibe]

cin.get(); co to ?

 

tamta biblioteka mi sie automatycznie pojawia bo ja używam dev c++

[iLikeGood]

1. Zmień deva na coś innego (cb, visual).

2. cin.get() "zatrzymuje" Twój program czyli coś jak getch. Ładnie opisane masz tutaj.

3. Poczytaj jeszcze troszkę.

[mjut]

Ale duże tablice

[IchHeibe]

Wiem dopiero sie nauczylem (b prsote 0-0 )