Problem z parzystością, c++

Question

Witam! Mam problem z parzystością w podanym kodzie(zadanie jest w srodku kodu) : http://ideone.com/BelFqL
Podczas podawania liczby nieparzystej program wykonuje działanie nie zważając na else if'a o instrukcji dla nieparzystej tablicy.

Comments

U mnie działa prawidłowo.

Zgubiles return 0; na koncu.

Answer 1

Usunąć system("pause").

Comments

Nie słyszałem, ani nie widziałem w innych kodach dynamicznej alokacji tablic, więc u siebie tego nie stosowałem. A co do system("pause") to dodałem sobie to tylko i wyłącznie dla picu. W Windowsie usunięcie system("pause") nie skutkuje wyeliminowaniem problemu.

Answer 2

Używasz typu int w swojej tablicy. Pamiętaj, że jeśli wpiszesz do niej liczbę ujemną np. -3 to -3 % 2 może zwrócić albo -1 albo 1, zależnie od kompilatora/systemu itp. Wtedy żaden z warunków if(warunek) nie zostanie spełniony. Lepiej zamiast drugiego warunku else if(warunek) napisać po prostu else. Dodatkowy plus, że program działa wtedy minimalnie szybciej, bo nie jest sprawdzane czy spełniony jest ten warunek.

Answer 3

W jaki sposób to w ogóle kompilujesz skoro wielkością tablicy statycznej może być tylko wartość stała (const).

Np:

const int n = 5;
int tab[n];

Jeżeli chcesz sam ustalić wielkość spróbuj w ten sposób:

cin >> n;
int *tab = new int[n];

 

Comments

W codeblocksie to działało, ale lepiej tego nie używać.

---

Właśnie doczytałem, że niektóre kompilatory przyjmują taki zapis, ale z tego co wiem nie jest to dobra praktyka.