Оцените работу функции

[Igor95]

Привет!
написал функцию для проверки на то, что пользователь ввел целое число...при этом, если число - не целое, то заставляем пользователя по новой его вводить
Вызов функции Integer Check(строка, количество цифр для ввода);

void IntegerCheck(char *digitstring, int digitsamnt) {
	int iterator=1;
	int step=0;
	int stepcurrent;
	int state=1;
	int retval;
	while (step<digitsamnt) {
	cout << "Enter digit string: ";
	cout << "[" << iterator << "]: ";
	gets(digitstring);
	stepcurrent=0;
	while ((stepcurrent<=strlen(digitstring)) && (!(state==0))) {
		if (digitstring[stepcurrent]=='.') {
			state=0;
		}
		stepcurrent=stepcurrent+1;
	}
	if (state==1) {
		retval=atoi(digitstring);
		step=step+1;
		iterator=iterator+1;
		cout << "digit: " << retval << "\n";
	}
	else {
		cout << "Wrong format! Re-enter: ";
		state=1;
	}
	}
	}

Пример вызова функции:
IntegerCheck(digit, 3);

Использование: вставьте функцию в ваш код (не в main!)
вызовите в main'е)
Все)
Откомментите работу, т.к функции пишк только второй день как) укажите на ошибки тоже))

[Igor95]

int IntegerCheck(char *digitstring) {
	int stepcurrent;
	int state=1;
	cout << "Enter digit string: ";
	gets(digitstring);
	stepcurrent=0;
	while ((stepcurrent<=strlen(digitstring)) && (!(state==0))) {
		if (digitstring[stepcurrent]=='.') {
			state=0;
		}
		stepcurrent=stepcurrent+1;
	}
	if (state==1) {
		return state;
	}
	else
		return 0;
	}

теперь ф-ция фозвращает значения) так что ее вполне можно использовать))

[Blind Guard]

Что делает эта функция я так и не понял, точнее ваше объяснение не совпадает с кодом

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

теперь ф-ция фозвращает значения) так что ее вполне можно использовать))

Её можно было и при void использовать

[sVasilich]

Эм. Долго думал что оно делает. Вставил функцию из второго поста в main таким образом:

int main()
{
    char* dstr;
    int d = IntegerCheck(dstr);

    std::cout<<d;

    std::cin.get();

    return 0;
}

вот чего получилось на выходе:

[svasilich@localhost debug]$ ./test_pr 
Enter digit string: 123
1
[svasilich@localhost debug]$ ./test_pr 
Enter digit string: 123.123
0
[svasilich@localhost debug]$ ./test_pr 
Enter digit string: qwerty
1
[svasilich@localhost debug]$ ./test_pr 
Enter digit string: qwert.
0
[svasilich@localhost debug]$ ./test_pr 
Enter digit string: .
0

Оно что, просто проверяет есть ли точка в предложении? Или я чего не понял?

[Perchik71]

Поскольку я автора не понял, я прошёл мимо.
но ребят да точка ищиться

if (digitstring[stepcurrent]=='.')

[Blind Guard]

Использование: вставьте функцию в ваш код (не в main!)
вызовите в main'е)

Видно, что вы недопоняли принцип и сущность понятия "функция".
Вы запоминаете частные моменты, вместо того, чтобы выучить общие правила, которые объяснят любой частный момент.

Другими словами main это тоже функция; все функции вызываются так как вы описали, а не только ваша и т.д.
У меня такие же проблемы были — не понят материал (точнее даже не пробовал понять), а тупо заучен алгоритм

[Igor95]

Да, фактически, просто точку в предложении ищет...
Чтобы, затем использовать это число нужно применить функцию atoi
Буду учить, спасибо))
Все, исправил описание работы функции)

[Perchik71]

а если там будет текст? или 0px или 0em.
самы что не наесть. способ только один.. "мягкое конвертирование"
берём массив digits[] = "0123456789"; <-- все эти символы мы ищем.
способ простой идём до тех пор пока не увидим, то что нет в digits.
после чего.. строку преобразуем... от 0 до позиции.
если строка получилась пустая.. возвращать 0. или MAX_INT

[sVasilich]

В принципе, символы от нуля до девяти в таблице ASCII идут подряд, поэтому проверку можно упростить. Вместо поиска по массиву символов, проверять лежит ли текущий символ в интервале от 48 (0), до 57 (9)

const int minDidgit = 48; //0
const int maxDidgit = 57; //9

....
int currentCode = static_cast<int>(digitstring[stepcurrent]);
if(currentCode<minDidgit || currentCode>maxDidgit){
    //это не число и можно завершить работу, вернув нулевое состояние
    return 0; //state
}

[Igor95]

не работает такое((