[C++] Liczby pierwsze

Witam!
W szkole dostałem zadanie napisania programu do sprawdzenia czy liczba jest liczbą pierwszą, ale mój nauczyciel powiedział, że muszę poprawić program bo jest źle napisany i mam mu przynieść do oceny jak poprawie. Więc od razu mówię,że jestem kompletnym amatorem, program napisałem w 5min i myślałem, że tak powinno być, więc proszę o ewentualnie wskazówki co powinienem poprawić ( nauczyciel mówi co jest źle dopiero po ocenie :p). Z góry dziękuje za pomoc.

1. Spróbuj lepiej formatować kod, przede wszystkim dawać więcej enterów wtedy wygodniej się to czyta.
2. Spróbuj inaczej nazywać zmienne, bo jak widzę  (co nawiasem mówiąc powinno wyglądać tak: ) to muszę się domyślać i sprawdzać co to oznacza.
3. Nie używaj float tylko double, no chyba, że musisz.
4. W ogóle zmienna wynik nie jest potrzebna wystarczy .
5. W pętli  zamienił bym na
Ciężko coś tutaj namieszać bo to jest dość prosty algorytm, i wydaje mi się, że twój nauczyciel miał właśnie zastrzeżenia dotyczące formy i ogólnych praktyk(czyli to co wypunktowałem).

Pomijając dziwny kod, masz błędy merytoryczne.

1. Nie słuchaj mojego poprzednika w kwestii typu, bo liczby pierwsze są wszystkie jak jedna naturalne a więc całkowite dodatnie. Zrezygnuj z float.


2. Liczba "1" jest tzw. niezmiennikiem mnożenia. Nie jest liczbą pierwszą i jako taka powinna być odrzucana wcześniej. Ogolnie przydałaby się instrukcja w stylu


3.

Bo nie możesz wymieniać wsrod dzielnikow jedynki i liczby badanej. Kazda liczba dzieli się przez 1 i sama siebie.

Hmm szczerze mówiąc to nie wiem o co chodzi z tym "Nie słuchaj mojego poprzednika w kwestii typu". bombatom69 jak byś mógł wyjaśnić.

Hmm szczerze mówiąc to nie wiem o co chodzi z tym "Nie słuchaj mojego poprzednika w kwestii typu". bombatom69 jak byś mógł wyjaśnić.

Nie ma tam w programie wskazania do używania liczb ułamkowych. Zbiór liczb pierwszych jest podzbiorem naturalnych. Reszta z dzielenia również jest całkowitą wartością. Więc i z punktu widzenia zarówno matematyki jak i C/C++, typ zmiennoprzecinkowy nie powinien się tam znaleźć żaden.

No tak mi tylko chodziło o to, żeby programując nie używał float tylko double, ponieważ double jest bardziej dokładne.

Wybacz, ale to co piszesz jest jakąś nowoczesną paplaniną.

1. Gdyby float było z natury złe(jak można wywnioskować z Twojej wypowiedzi), byłoby z całą pewnością już dawno wycięte.
2. Zapoznaj się z opisem formatu, wówczas może dowiesz się jakie warunki musiałbyś spełnić aby float stało się niedokładne. Sprawdź na swojej maszynie rozmiar float i zobacz parametry tego formatu dla takiego rozmiaru.
3. Nakłanianie ludzi aby korzystali z double zamiast float, jest zupełnie dla mnie niezrozumiałe. Przyjdzie Ci kiedyś fantazja namówić siebie na rezygnację z double na rzecz long double, pomimo skromnych potrzeb. Może się zdarzy, że będziesz potrzebował zaalokować sporą tablicę. Wówczas zaczniesz zachodzić w głowę, dlaczego właściwie stało się tak, że program wyssał Ci tyle pamięci.

Tutaj pytający nie potrzebuje w ogóle ułamków a Ty zachęcasz go do zwiększenia zakresu. Gdzie w tym sens?

Może inaczej, ja mam złe doświadczenia z float, dlatego dla liczb zmiennoprzecinkowych stosuję double.