KalkulatorV2 (C++) [ankieta]

pytanie zadane 15 kwietnia 2016 w Nasze projekty przez TheFeniks Gaduła (4,690 p.)
edycja 15 kwietnia 2016 przez TheFeniks

Witam !

Właśnie ukończyłem mój pierwszy obiektowy a zarazem najnowszy projekt: KalkulatorV2 .

Czemu V2 ? No cóz jest to remake mojego pierwszego kalkulatora napisanego procedularnie.

Chciałem podziękować użytkownikom: badbullpl , Radfler , draghan. Za pomoc w rozwiązywaniu problemów! :)

Proszę o uwagi dotyczące kodu jaki samego programu. I napisz co ci się podoba a co nie. :)

Paczka z programem i z kodem : http://sendfile.es/pokaz/772123---gqfm.html

main.cpp: http://wklej.org/id/2285868/

list.h: http://wklej.org/id/2285880/

methods.cpp: http://wklej.org/id/2285882/

Pozdrawiam ! :)

Możliwe odpowiedzi:

Fajny (1 głos, 33%)

Średni (2 głosów, 67%)

Słaby (0 głosów)

6 odpowiedzi

odpowiedź 15 kwietnia 2016 przez Eryk Andrzejewski Mędrzec (164,260 p.)

Panie, nie lepiej wstawić kod na githuba? Tak jak ktoś chce sobie obejrzeć kod, to musi pobierać całą paczkę, rozpakowywać itd. A tak to podasz link i już widać kod i po całym projekcie można się pięknie poruszać.

komentarz 15 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

Poprawione :)

odpowiedź 15 kwietnia 2016 przez niezalogowany

Ja bym zamiast


         if(choice=='1')//Dodawanie
         {
            Addition add1;
            add1.AD_input_M();
            add1.AD_calculate();
         }

         if(choice=='2')//Odejmowanie
         {
            Subtraction subt1;
            subt1.SU_input_M();
            subt1.SU_calculate();
         }

         if(choice=='3')//Mnozenie
         {
            Multiplication mul1;
            mul1.MU_input_M();
            mul1.MU_calculate();
         }

         if(choice=='4')//Dzielenie
         {
            Division div1;
            div1.DI_input_M();
            div1.DI_calculate();
         }

Dał switche, a tak to nawet przystępny kalkulatorek ;)

ps: a i jeszcze hermatyzacja danych:

public:

powinno być pod zmiennymi w klasach.

komentarz 15 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

W sumie nie najgorszy pomysł :)

Ale jednak wole if'y. :P

komentarz 15 kwietnia 2016 przez niezalogowany

z tym public zobacz czy na pewno nic nie popsuje, patrzyłem pobieżnie

komentarz 15 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

A to nie jest przypadkiem tak że wszystko co jest pod: public,private itp.

Jest prywatne/publiczne.

Że nie trzeba pod zmiennymi drugi raz pisać ?

komentarz 15 kwietnia 2016 przez niezalogowany

zmienna1
zmienna2
public:
metoda1
metoda2

zmienna1 i zmienna2 -> prywatne

metoda1 i metoda2 -> publiczne

komentarz 15 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

Tak oto mi chodziło.

A gdyby było tak:

public:
zmienna1
zmienna2
metoda1
metoda2

Wszystkie są publiczne :)

komentarz 15 kwietnia 2016 przez niezalogowany

no teraz tak masz, zrób tak byś miał hermetyzacje danych ;)

komentarz 16 kwietnia 2016 przez Aisekai Nałogowiec (42,190 p.)

Jeśli wolisz ify ... Wiesz jak w ogóle zastosować if? Jeśli sprawdzasz np czy ktoś chce dodawać,odejmować, mnożyć czy dzielić to nie stosujesz 4x if. Tylko if, i 3x else-if

odpowiedź 16 kwietnia 2016 przez maly Nałogowiec (37,190 p.)

if((!(counter))||(counter<0))

nie prościej tak?

if(counter <= 0)

Po co te prefixy i postfixy?
Co się stanie kiedy zamiast liczby wpiszesz literę?
Co się stanie jesli ilość liczb do wpisania będzie większa od rozmiaru tablicy input?
Każda klasa to praktycznie ten sam kod, poczytaj sobie o dziedziczeniu.

komentarz 16 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

Dzięki za uwage. :)

Kiedy wpiszemy litery to wykona się ten if.

A obliczałeś kiedyś na kalkulatorze 100 liczb na raz ? :P

odpowiedź 16 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

odpowiedź 16 kwietnia 2016 przez Muminek Dyskutant (8,650 p.)

Przy dodawaniu, odejmowaniu, operacji na wartości 0 wyskakuję - nieprawidłowe wejście. Dlaczego? Świat ulega samozagładzie tylko przy dzieleniu przez 0 :)

Ogólnie kalkulator jak kalkulator, doby sposób na nauke :)