Исправить/Указать ошибку

[scrinn]

С++ Работа с массивами
Задача:Даны 2 массива из n целых чисел (n<=10).Для каждого массива определить количество нечетных элементов.Описать в виде отдельной функции определение количества нечетных элементов данного массива.
Вот собственно код:

#include <iostream>
#include <stdio.h>
using namespace std;
int KolvoNe4Elem (char s[])
{
	int i,k=0;
	for (i=0;s[i]!='\0';i++)
		if (s[i]%2 !=0)k++;
	return k;
}
using namespace std;
void main()
{
	char s1[81],s2[81];
	cout<<"vvedite 2 stroki \n";
	gets(s1);
	gets(s2);
	printf (" v 1 stroke %d ne4 4isel\n",KolvoNe4Elem(s1));
    printf (" vo 2 stroke %d ne4 4isel\n",KolvoNe4Elem(s2));
	
}

Проблема:
Программа выполняет то что от нее требуется пока вводишь числа меньше 10, как только начинаешь вбивать числа которые больше она выдает абсолютно рандомный результат,нужно сделать чтобы количество членов массива было <=10
Спасибо

[s-andriano]

А это ничего, что вместо чисел ты работаешь с символами?
Пока числа однозначные, проблем нет, а как только появляются многозначные, программа начинает трактовать каждую цифру как отдельное число (благо, у пробела четный код, так что на результат это не сказывается).

[Stilet]

char s1[81],s2[81];

А зачем строкой? Почему не массивами типа int?
Вот же твоя проблема - символ строки не может содержать число.

[scrinn]

Сообщение от Stilet

А зачем строкой? Почему не массивами типа int?
Вот же твоя проблема - символ строки не может содержать число.

Сообщение от s-andriano

А это ничего, что вместо чисел ты работаешь с символами?
Пока числа однозначные, проблем нет, а как только появляются многозначные, программа начинает трактовать каждую цифру как отдельное число (благо, у пробела четный код, так что на результат это не сказывается).

Когда я заменил везде char на int

#include <iostream>
#include <stdio.h>
using namespace std;
int KolvoNe4Elem (int s[])
{
	int i,k=0;
	for (i=0;s[i]!='\0';i++)
		if (s[i]%2 !=0)k++;
	return k;
}
using namespace std;
int main()
{
	int s1[81],s2[81];
	cout<<"vvedite 2 stroki \n";
	gets(s1);
	gets(s2);
	printf (" v 1 stroke %d ne4 4isel\n",KolvoNe4Elem(s1));
    printf (" vo 2 stroke %d ne4 4isel\n",KolvoNe4Elem(s2));
	
}

то подчеркивается красным s1 и s2 в строках gets(s1); gets(s2);
и выдает следующее:
1>------ Построение начато: проект: lab_alg(4), Конфигурация: Debug Win32 ------
1> lab_alg(4).cpp
1>e:\my documents\visual studio 2010\projects\lab_alg(4)\lab_alg(4) \lab_alg(4).cpp(16): error C2664: gets: невозможно преобразовать параметр 1 из "int [81]" в "char *"
1> Типы, на которые указывают указатели, не связаны; для преобразования требуется reinterpret_cast, приведение в стиле С или приведение в стиле функции
1>e:\my documents\visual studio 2010\projects\lab_alg(4)\lab_alg(4) \lab_alg(4).cpp(17): error C2664: gets: невозможно преобразовать параметр 1 из "int [81]" в "char *"
1> Типы, на которые указывают указатели, не связаны; для преобразования требуется reinterpret_cast, приведение в стиле С или приведение в стиле функции
========== Построение: успешно: 0, с ошибками: 1, без изменений: 0, пропущено: 0 ==========

[Greblin]

Прочитайте те сообщения об ошибках, которые выдал Вам компилятор, и подумайте, какой логичный вывод можно из них сделать.
Правильно! gets считывает строки, и аргумент у неё тоже должен быть строкой, а вы подаёте int*. Следовательно, использовать gets нельзя. А как тогда? Правильно, запросить у пользователя, сколько он чисел введёт и считать их в цикле. Например так

int n1;
cout << "vvedite kolichestvo elementov v 1 massive: \n";
cin >> n1;
cout << "vvedite elementi massiva: \n";
for (int i = 0; i < n1; ++i) cin >> s1[i];

После этого со вторым массивом у Вас тоже всё получится, я в Вас верю!

[scrinn]

Сообщение от Greblin

Прочитайте те сообщения об ошибках, которые выдал Вам компилятор, и подумайте, какой логичный вывод можно из них сделать.
Правильно! gets считывает строки, и аргумент у неё тоже должен быть строкой, а вы подаёте int*. Следовательно, использовать gets нельзя. А как тогда? Правильно, запросить у пользователя, сколько он чисел введёт и считать их в цикле. Например так

int n1;
cout << "vvedite kolichestvo elementov v 1 massive: \n";
cin >> n1;
cout << "vvedite elementi massiva: \n";
for (int i = 0; i < n1; ++i) cin >> s1[i];

После этого со вторым массивом у Вас тоже всё получится, я в Вас верю!

Спасибо, но мне не хватило бы знаний чтобы сделать эти выводы.У вас отличительная особенность письма.