Mógłby ktoś ocenić?

[Will]

Napisałem pierwszy program w c++ sam bez niczyjej pomocy.. i chciałbym żebyście go ocenili ^^.

Tzn co w nim można by było poprawić.. żeby lepiej działał P.S C++ uczę się od 2 dni..

 

#include<iostream>
#include<conio.h>
using namespace std;
int main()
{
    double liczba1;
    double liczba2;
    double wynik;
    bool chcesz;
int wybor;
    cout<<"witam w moim kalkulatorze:";
        

    while (chcesz)
    {
        int wybor;
        cout<<"wybierz odpowiednia liczbe: "<<endl

        <<"  1. nacisnij 1. aby dodawac " <<endl
        <<" 2. nacisnij 2. aby mnozyc " <<endl
        <<"3. nacisnij 3. aby dzielic " <<endl
        << " 4. nacisnij 4. aby odejmowac " <<endl
        << "wybrales: ";
        cin>>wybor;
        
        

        switch (wybor)
        {
        case 1:


        {
            cout<<"podaj 1 liczbe";
            cin>>liczba1;
                cout<<"podaj 2 liczbe: ";
            cin>>liczba2;
            cout<<liczba1<<"+"<<liczba2<<"="<<endl
                <<liczba1+liczba2;
            break;
        }
        case 2:
        {
            cout<<"podaj 1 liczbe";
            cin>>liczba1;
                cout<<"podaj 2 liczbe: ";
            cin>>liczba2;
            cout<<liczba1<<"*"<<liczba2<<"="<<endl
                <<liczba1 * liczba2;
            break;
        }
        case 3:
            {
                cout<<"podaj 1 liczbe";
            cin>>liczba1;
                cout<<"podaj 2 liczbe: ";
                cin>>liczba2;
cout<<liczba1<<"/"<<liczba2<<"="<<endl
<<liczba1 / liczba2;
break;
    }
        case 4:
            {
                cout<<"podaj 1 liczbe";
            cin>>liczba1;
                cout<<"podaj 2 liczbe: ";
                cin>>liczba2;
                cout<<liczba1<<"-"<<liczba2<<endl
                    <<liczba1 - liczba2;
            }
        default: cout<<"nie znam tego";
        }

cout<<" jesli chcesz dalej pracowac nacisnij 1 a jesli nei daj 0: ";
cin>>chcesz;
}
getch();
}

[kryniak]

No, no. Jestem pod wrażeniem :P.

[spyro]

Nie lepiej było od razu skompilować ? :3

 

Bo wiesz, sam kod to może i robi wrażenie, ale wygląda jak jakis pokaz umiejętności kopiowania kodu, a sam program robiłby wrażenie większe :P

 

Zresztą programik prosty, acz jak na newbiowski rozum i Twój debiut, to całkiem całkiem :D Poza tym, jesli się dwa dni C++ uczysz, to tym bardziej gratz ! ^^

[Will]

program kompilowałem u siebie i działał. Uczę się jednocześnie z Grębosza "od zera do gier kodera" i z neta..

To chyba najlepszy sposób.. ^^ a program jest na moim uploadzie: o to link https://gmclan.org/up2096_4_1d04b99f47.html

 

I dziękuje bardzo za dobre komentarze to tylko motywuje do dalszej pracy. :)

[blackmaul]

Hm, zastosowanie switch, while, cout i cin. Jak na 2 dni nauki to wyśmienicie.

[123el]

To taka jakby rozbudowana wersja kalkulatorka z tutorialu "Od zera...", budzi podziw zastosowanie switch'a i while'a, na razie ich trochę nie kapuję, ale przyjdzie czas...

[Vardi]

Wszystko ok. ;) Dodaj do niego opcję liczenia średniej arytmetycznej (można zrobić z zastosowaniem tablicy i pętli for).

[Luksor]

Jest trochę błędów , ale jak na drugi dzień nauki to super ci poszło! :]

[Ranmus]

Jest trochę błędów

Dobra maniera nakazuje by w takim przypadku również je wskazać. :)

[blackmaul]

Jest trochę błędów , ale jak na drugi dzień nauki to super ci poszło! :]

Dobra maniera nakazuje by w takim przypadku również je wskazać. :)

Nie mając pojęcia o C++ każdy próbuje zaistnieć. =)

[Micja]

JA powiem, że program można zrobić o wiele bardziej czytelnie i zmniejszyć ilość linijek, używajac funkcji.

 

Pozdro.

 

JA powiem, że program można zrobić o wiele bardziej czytelnie i zmniejszyć ilość linijek, używajac funkcji.

 

Pozdro.

[123el]

Kompilator zazwyczaj sam zajmuje się układem linijek...;)

 

 

Zresztą tak jest bardzo przejrzyście...:)