Помогите с задачей на с++

Собственно задача такая. - С клавиатуры вводятся целые числа. Признак конца ввода – 0. Определить количество введенных нечетных чисел.
Я написал программу, но она работает некорректно, а я в упор не вижу, где тут ошибка...

#include <cstdlib>
#include <iostream>

using namespace std;

int main(int argc, char *argv[])
{
int m=0, n=0, i=0, j;
int a[100];
cout<<"\n Vvodite chisla. Esli hotite zakonchit', vvedite 0\n";
while (i<100)
{cin>>a[i];
i++;
if (a[i]==0) break;

}
for (j=1;j<=i;j++)
{
if ((a[i] % 2) == 1) m++;
}
cout<<"Kolichestvo nechetnyh chisel = "<<m;
system("PAUSE");
return EXIT_SUCCESS;
}

Помогите, пожалуйста.

как на счет поменять местами эти два оператора:

i++;
if (a[i]==0) break;

а то получается, что Вы увеличиваете индекс на единицу...и только тогда обращаетесь к массиву, т.е. Вы проверяете не текущее значение (введённое с клавиатуры) на ноль, а следующее в массиве...т.е. наступит момент, когда Вы обратитесь за границы массива..а это уже не желательно


for (j=1;j<=i;j++)

здесь j нужно инициализировать нулём, а не единицей...а то получается, что нулевой эл.-т массива не участвует в проверке


и ещё...хоть убейте, но не понимаю зачем здесь нужна переменная n=0

[QUOTE=wolverston;20563370]...
Я написал программу, но она работает некорректно, а я в упор не вижу, где тут ошибка.......[/QUOTE]
а желание воспользоваться дебагером случайно не возникало?
и пошагово просмотреть что же делает это мега программа....
:rolleyes:

Спасибо за помощь. С первым советом действительно косяк идиотский, а вот подсчет отрицательных во втором цикле не удался никак, пришлось вносить счетчик в первый же цикл, даже симпатичнее получилось ))

[QUOTE=wolverston;20573093]Спасибо за помощь. С первым советом действительно косяк идиотский, а вот подсчет отрицательных во втором цикле не удался никак, пришлось вносить счетчик в первый же цикл, даже симпатичнее получилось ))[/QUOTE]

а Вы в этой строчке
if ((a[i] % 2) == 1) m++;

поменяйте условие на такое:

if (!((a[i] % 2) == 0)) m++

и тогда ненужно никаких счетчиков...в этом случае будут и отрицательные отлавливаться


а для симпатичности (рас уж для Вас это важно), можно эти два оператора
i++;
if (a[i]==0) break;

заменить одним

if (a[i++]==0) break;

А зачем массив-то?
[CODE]#include <iostream>

int main(int argc, char *argv[])
{
int odds = 0;
int item;
std::cout << "Vvodite chisla. Esli hotite zakonchit', vvedite 0" << std::endl;
do
{
std::cin >> item;
if (item % 2 != 0)
{
++odds;
}
}
while (item != 0);
std::cin.get();

std::cout << "Kolichestvo nechetnyh chisel = " << odds;
std::cin.get();

return 0;
}[/CODE]

Люди помогите разобраться.

[HTML]cout<<"Введите буквы для американского формата (am, pm или no - для евро формата)\n";
do {
scanf ("%2s",f); //Ввод формата.
if (strcmp(f,"am")!=0) { //условие на правильность ввода формата.
cout<<"Вы ввели неправильный формат, введите заново\n";
}
}
while (strcmp(f,"am")!=0)[/HTML]

А если я хочу задать несколько условий, у меня ничего не выходит.

Например:

[HTML]cout<<"Введите буквы для американского формата (am, pm или no - для евро формата)\n";
do {
scanf ("%2s",f); //Ввод формата.
if ((strcmp(f,"am")!=0)||(strcmp(f,"am")!=0)||(strcmp(f,"am")!=0)) { //условие на правильность ввода формата.
cout<<"Вы ввели неправильный формат, введите заново\n";
}
}
while ((strcmp(f,"am")!=0)||(strcmp(f,"am")!=0)||(strcmp(f,"am")!=0));[/HTML]

Этот код не работает:(

а может здесь:
while ((strcmp(f,"am")!=0)||(strcmp(f,"am")!=0)||(strcmp(f,"am")!=0));
точка с запятой (в конце) лишняя?

[QUOTE=Aksined;20619539]Люди помогите разобраться.
......
Этот код не работает:([/QUOTE]
а в дебагер ты хоть смотрел? или это страшное ругательное слово...
этот код не работает правильно даже с одним условием...
введи больше двух символов... ;)
сравни со своим и найди 10 отличий...
[CODE]char f[10];
cout<<"Введите буквы для американского формата (am, pm или no - для евро формата)\n";
do {
scanf("%9s", f, 10); //Ввод формата.
if ((strcmp(f,"am")!=0)&&(strcmp(f,"pm")!=0)&&(strcmp(f,"no")!=0)) { //условие на правильность ввода формата.
cout<<"Вы ввели неправильный формат, введите заново\n";
}
}
while ((strcmp(f,"am")!=0)&&(strcmp(f,"pm")!=0)&&(strcmp(f,"no")!=0));[/CODE]

это судя по всему студентческие лабораторки?!

Всем спасибо, с предыдущей задачей я справился.

теперь просьба такая:

Ув. программисты у кого есть исходники на С++ простенького органайзера, могли бы выложить.
Можно в личку.
Буду также рад алгоритму написания проги.

Огромное спасибо.