Witam
od niedawna siedzę w C++ (jakieś... 2 dni xD ) i stworzyłem 2 programy, chciałbym poznać waszą opinię na temat tego jak mi idzie nauka.
Przeliczanie liczb arabskich na rzymskie (praca domowa xD ) :
http://www.wrzuc.to/XgFYZtV2gT.wt
Prosty kalkulator:
http://www.wrzuc.to/h4Ho1ycZf.wt
W razie krytyki proszę wziąć pod uwagę fakt, że dopiero co startuję z C++ ;)
Wrzuć tu kod. Dlaczego mielibyśmy pobierać cokolwiek z nieznanego źródła? Tam mogą być wirusy i inne świństwa(a tak już się zdarzało), a jak pokażesz kod, to każdy będzie mógł go skompilować i ocenić.
Korzystasz z Dev-C++?
Jeżeli tak, to przestań i pobierz coś innego.
Moje uwagi:
-nie korzystaj z system("pause") oraz system("cls");. Żadne z tych komend nie działają na Linuxie.
-Liczby rzymskie:
-po wpisaniu dużej liczby zaczyna cały czas wyświetlać M
-po wpisaniu dużej liczby ujemnej wyświetla tekst Podaj liczbę dodatnią!, po czym nic nie można wpisać
-jest także problem z liczbami zmiennoprzecinkowymi
-Kalkulator:
-"Liczba pierwsza:" kojarzy mi się z czymś innym, niżeli składnikiem dodawania
-po wpisaniu nie liczby program wyświetla No to na razie!, po czym wychodzi.
-po wpisaniu nie liczby zamiast dzielnika program wyświetla Przy dzieleniu nie używamy liczby '0', po czym się zawiesza i nic nie można zrobić.
-po wpisaniu nie liczby zamiast odjemnej program nie pyta się o drugą liczbę i wykonuje odejmowanie z jakąś nieznaną liczbą, po czym zawiesza się.
A teraz uwagi cd.kodu:
Kalkulator:
if (opcja==1)
(...)
if (opcja==2)
(...)
if (opcja==3)
(...)
if (opcja==4)
1.Zamiast if'ów użyj case.
2.Porozdzielaj program na funkcje; nie trzymaj wszystkiego w bloku main
3.W pewnych momentach jest dosyć dziwne formatowanie kodu.
Liczby rzymskie:
1.Formatowanie kodu jest dosyć dziwne w pewnych miejscach.
Ale ogólnie nie jest źle; spodziewałem się czegoś gorszego.
Biorąc pod uwagę, że jesteś początkujący, niektóre błędy można wybaczyć.
Lecz na początek pozbądź się Dev'a ;)