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

[sVasilich]

Ну дык покажи, как именно ты это написал и на что компилятор ругается.

[Igor95]

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

#include <iostream>//подключение заголовочного файла для операторов ввода/вывода
using namespace std;//использование стандартного пространства имен std для операторов ввода/вывода

FILE *file;//файловая переменная, описанная в структуре файл
char filename[100];//адрес файла
char digit[100];//входной поток, используемый для проверки и получения целого числа
int stream[100];//входной поток обработанных данных
int streamout[100];
int N;//целочисленная переменная для хранения количества вводимых элементов
int i;//итератор для прохода по массиву значений

//функция проверки на целое число
int IntegerCheck(char *digitstring) {
	int stepcurrent=0;//текущий шаг
	int state=1;//переменная флаг

	while ((stepcurrent<=strlen(digitstring)) && (!(state==0))) {
		if (digitstring[stepcurrent]<'0' || digitstring[stepcurrent]>'9') {
			state=0;
		}
		stepcurrent=stepcurrent+1;
	}
	if (state==1) {
		return state;
	}
	else return state;
	}

//функция вывода данных
void DataOutput(const int *matrix, const int amount) {
	int a;
	for (a=0; a<amount; a++)
		cout << "[" << a+1 << "]: " << matrix[a] << "\n";
}

//функция изменения данных
void DataChangeInFile(FILE *fileconst, int *buffer, const int amount) {
	int a;
	int tempdigit;
	int *pTempdigit=&tempdigit;
	for (a=0; a<amount; a++) {
		fseek(fileconst, sizeof(int)*a, 0);
		fread(pTempdigit, sizeof(int), 1, fileconst);
		if (*pTempdigit<0) 
			buffer[a]=0;
	}
	fseek(file, 0, 0);
	fwrite(buffer, sizeof(int), amount, fileconst);
}

void DataInput(char *charbuffer, int *buffer, const int amount) {
	int a;
	for (a=0; a<amount;) {
		gets(charbuffer);
		if (IntegerCheck(digit)==1) {
			buffer[a]=atoi(digit);
			a++;
			fflush(stdin);
		}
		else {
			cout << "Wrong. Re-enter: ";
			fflush(stdin);
		}
	}
}
void main() {
	cout << "Enter filename: ";
	gets(filename);
	file=fopen(filename, "w+b");
	if (file!=0) {
	cout << "Enter amount of digits: ";
	cin >> N;
	fflush(stdin);//очистка буфера
	DataInput(digit, stream, N);
	cout << "Data you have entered: " << "\n";
    DataOutput(stream, N);
	fwrite(stream, sizeof(int), N, file);
	DataChangeInFile(file, stream, N);
	fseek(file, 0, 0);
	fread(streamout, sizeof(int), N, file);
	cout << "Processed data written to the address: ";
	puts(filename);
	cout << "\n";
	for (i=0; i<N; i++) {
		cout << "element: " << streamout[i] << " ";
	}
	fclose(file);
	}
	else cout << "Error opening file..." << "\n";
	system("pause");
	}

[sVasilich]

Дело в том, что в конце каждой строки стоит символ '\0', который и даёт ложь при этой проверке

if (digitstring[stepcurrent]<'0' || digitstring[stepcurrent]>'9') {

Как вариант, можно не проверять последний символ. Либо игнорировать конец строки. Вот, подправил функцию (здесь не проверяю последний символ и убрал явно лишнюю state). В принципе, лучше возвращать значение типа bool, но я остаивл просто 0 и 1, как и было у тебя.

int IntegerCheck(char *digitstring) {
    size_t stepcurrent=0;//текущий шаг, тип size_t чтобы не было предупреждения в цикле

    while (stepcurrent<strlen(digitstring)) {
        if (digitstring[stepcurrent]<'0' || digitstring[stepcurrent]>'9') {
            return 0;
        }
        ++stepcurrent;
    }
    return 1;
}

[Igor95]

аааа))) я понял!))) спасибо)

[Igor95]

а что насчет такого варианта? тоже рабочий:

int IntegerCheck(char *digitstring) {
	int stepcurrent=0;//текущий шаг
	int state=1;//переменная флаг

	while ((stepcurrent<=strlen(digitstring)-2) && (!(state==0))) {
		if (digitstring[stepcurrent]<'0' || digitstring[stepcurrent]>'9') {
			state=0;
		}
		stepcurrent=stepcurrent+1;
	}
	if (state==1) {
		return state;
	}
	else return state;
	}

[Igor95]

int IntegerCheck(char *digitstring) {
	int stepcurrent=0;//текущий шаг
	int state=1;//переменная флаг

	while ((stepcurrent<=strlen(digitstring)-1) && (!(state==0))) {
		if (digitstring[stepcurrent]<'0' || digitstring[stepcurrent]>'9') {
			state=0;
		}
		stepcurrent=stepcurrent+1;
	}
	if (state==1) {
		return state;
	}
	else return state;
	}

//функция вывода данных
void DataOutput(const int *matrix, const int amount) {
	int a;
	for (a=0; a<amount; a++)
		cout << "[" << a+1 << "]: " << matrix[a] << "\n";
}

[Igor95]

второй вариант, с первым я ошибся (2 вместо 1 написал)

[sVasilich]

Да, тоже рабочий. Протестируй ещё на всякий случай

[Igor95]

момент пропустили один: ввода отрицательного числа не будет, т.к - находится вне промежутка от 0 до 9)) но эта проблема решаемая

[Igor95]

короче нужно еще думать