Ocena programów

0

Witam, programowania w Pascalu uczę się już rok, c++ z miesiąc. Chciał bym prosić o ocenę moich programów napisanych w C++, jeden w Pascalu, oraz podanie nowych propozycji programów. Planuje zacząć programowanie obiektowe, jednak nie wiem czy mam wystarczające umiejętności.

Pascal: Program Inwentaryzacja
http://ideone.com/cUSis

C++: Ciąg Fibonacciego
http://ideone.com/EeESP

C++: Silnia
http://ideone.com/ywLhd

C++: NWD
http://ideone.com/KG3E6

C++: Program liczący równanie kwadratowe
http://ideone.com/eZqwW

1

http://ideone.com/eZqwW - niespecjalne (hint - popatrz na wynik)...

Przede wszystkim - rób wcięcia. I nie pisz w Dev-ie, to krzywdzi. Nie używaj conio.h. I jeszcze rób wcięcia.

Poza tym - ujdzie, bo nie ma tu wiele do oceniania :>

0

Wcięcia staram się robić, mam tak kod uporządkowany że wiem gdzie co jest. Co do tego jednego programu, w kompilatorze na kompie działa normalnie, Ideone coś tam nie obsługuje chyba wszystkiego. Co do <conio.h>, czemu mam go nie używać, skoro np. getch() korzysta z tej biblioteki.
PS: Korzystam z codeblocks.

0

Silnia zwraca niepoprawny wynik dla każdego ujemnego wejścia.
OT. Równania się rozwiązuje, a nie oblicza.

2

Wcięcia staram się robić, mam tak kod uporządkowany że wiem gdzie co jest. Co do tego jednego programu, w kompilatorze na kompie działa normalnie, Ideone coś tam nie obsługuje chyba wszystkiego. Co do <conio.h>, czemu mam go nie używać, skoro np. getch() korzysta z tej biblioteki.

conio.h jest niezgodne ze standardem, działa tylko na jednym kompilatorze i powoduje problemy - jak widać np. na przykładzie ideone (a to wierzchołek góry lodowej).

PS: Korzystam z codeblocks.

Jak widać wszędzie da się robić kreatywne wcięcia ;)

0
  1. http://ideone.com/cUSis - zamień if'y na else if lub przynajmniej na case, użyj funkcji UpCase aby reagowało tak samo na 'w' jak na 'W' itp.
  2. http://ideone.com/EeESP - a co będzie jak ktoś poda 0? a jak poda 2? licznik = 0; while(licznik != pobrana-2) czy nie lepiej licznik=2; while(licznik<pobrana)?
  3. http://ideone.com/ywLhd - nie może działać pom = pom--; zmniejszasz pom ale stary wynik zapisujesz do pom czyli nić się nie dzieje. while(pom-->1) pobrana*=pom;
  4. http://ideone.com/KG3E6 - strasznie nakombinowano, wystarczy while(grupa2) { int pom=grupa1%grupa2; grupa1=grupa2; grupa2=pom; }
  5. http://ideone.com/eZqwW - kompletny bezsens: cout << rownanie(a,b,delta(a,b,c)); rownanie - nic nie zwraca zaś wypluwa coś na cout.

1 użytkowników online, w tym zalogowanych: 0, gości: 1