Как записать символы в обратном порядке в каждой строке файла?

Question

[ZIK1337]

Нужно перевернуть символы (записать в обратном порядке) в каждой строке файла.
Есть такой набросок кода:

fseek(file_ptr, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr)) {
			_strrev(text);
			fseek(file_ptr, 0, SEEK_SET);
			fputs(text, file_ptr);
			fputs("\n", file_ptr);
			fseek(file_ptr, 0, SEEK_END);
		}

Но реверс происходит только первой строки, да еще и пустую строку оставляет (строку fputs("\n", file_ptr); в коде убирал, все равно остается пустая начальная строка):

spoiler

Подскажите, что я делаю не так?
Вот весь код:

spoiler

#include <stdio.h>
#include <string.h>
#include <locale.h>

int main(int argc, char **argv)
{
	setlocale(LC_ALL, "Ru");
	FILE *file_ptr; // указатель на файл (задание 1.1)
	int N; // количество строк и символов в строках (задание 1.1)
	int count=0; // для подсчета количества строк (задание 1.2)
	char ch; // для подсчета количества строк (задание 1.2)
	char text[50]="0"; // строка для чтения из файла

	file_ptr = fopen("D:\\f1.txt", "w+");

	if (file_ptr != NULL) {
		printf("Введите количество строк в файле и символов в них (не более 10): \n");
		scanf("%d", &N);
		
		if (N > 10) {
		do {
			printf("Нужно ввести число не более 10! Повторите ввод: \n");
			scanf("%d", &N);
		} while (N > 10);
		}

		printf("Введите строки не более %d символов.\n", N);

		for (int i = 0; i < N; i++)
		{
			printf("Введите %d-ю строку: \n", (i+1));
			scanf("%s", text);
			if (strlen(text) > N) {
				do {
				printf("Нельзя вводить больше %d символов! Введите строку еще раз: \n", N);
				scanf("%s", text);
				} while (strlen(text) > N);
				fputs(text, file_ptr);
				fputs("\n", file_ptr);
			}
			else
			{
				fputs(text, file_ptr);
				fputs("\n", file_ptr);
			}
		}

		// вывод числа строк в файле (задание 1.2)
		fseek(file_ptr, 0, SEEK_SET);
		while ((ch = fgetc(file_ptr)) != EOF) {
			if (ch == '\n')
				++count;
		}
		printf("Количество строк в файле: %d\n", count);

		// реверс строк в файле (задание 1.3)
		fseek(file_ptr, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr)) {
			_strrev(text);
			fseek(file_ptr, 0, SEEK_SET);
			fputs(text, file_ptr);
			fputs("\n", file_ptr);
			fseek(file_ptr, 0, SEEK_END);
		}
		
		// чтение файла
		//fseek(file_ptr, 0, SEEK_SET);
		//while (fgets(text, 50, file_ptr)) {
		//	printf("%s", text);
		//}

		fclose(file_ptr);
		_getch();
		return 0;

	}
	else {
		printf("Невозможно открыть файл\n");
		return 1;
	}

}</code lang="cpp"></spoiler>

Answer 1

[vanyamba-electronics]

char tmp;
...
tmp = text[0];
text[0] = text[2];
text[2] = tmp;
fputs(text, file_ptr);

Comments

[ZIK1337]

это сработает только для 3-х символьной строки, у меня же может быть до 10 рандомно

[vanyamba-electronics]

char tmp;
int n, len;
...
len = strlen(text);
for(n = 0; n < len / 2; ++n) {
    tmp = text[n];
    text[n] = text[len - 1 - n];
    text[len - 1 - n] = tmp;
}
fputs(text, file_ptr);

[ZIK1337]

vanyamba-electronics,

fseek(file_ptr, 0, SEEK_SET);
		fgets(text, 50, file_ptr);
		count = strlen(text);
		for (a = 0; a < count / 2; ++a) {
			tmp = text[a];
			text[a] = text[count - 1 - a];
			text[count - 1 - a] = tmp;
		}
		fputs(text, file_ptr);

Скрин ошибки:

spoiler

Я что-то делаю не так?

[vanyamba-electronics]

После записи в файл и перед чтением из него добавьте:
fflush(file_ptr);

[ZIK1337]

vanyamba-electronics, к сожалению, не помогло(

spoiler

[vanyamba-electronics]

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

[ZIK1337]

vanyamba-electronics, все равно ошибка.
Если оставить так:

spoiler

то ошибка будет
Если же убрать fputs, то все нормально.
Режим доступа к файлу - w+ - верный?

[vanyamba-electronics]

Что там вообще делает fseek, я так и не понял. Вы же просто выводите в файл.

[ZIK1337]

vanyamba-electronics, но сначала мне же надо прочитать из файла, поэтому ставлю указатель в начало
если оставить так:

count = strlen(text);
		for (a = 0; a < count / 2; ++a) {
			tmp = text[a];
			text[a] = text[count - 1 - a];
			text[count - 1 - a] = tmp;
		}
		fseek(file_ptr, 0, SEEK_SET);
		fputs(text, file_ptr);

то результат таков:

spoiler

Если так:

count = strlen(text);
		for (a = 0; a < count / 2; ++a) {
			tmp = text[a];
			text[a] = text[count - 1 - a];
			text[count - 1 - a] = tmp;
		}
		fputs(text, file_ptr);

То такой:

spoiler

Мне нужно строки оставить в том же порядке, реверс сделать именно в самой строке (в каждой), а не в целом в файле. И чтобы кол-во строк осталось прежним. Без fseek тут не обойтись.

[vanyamba-electronics]

ZIK1337, проблема похоже в том, что вы просто ещё не усвоили принцип безопасного обращения с данными.
Программа не должна менять данные в исходном файле. Она должна создавать новый файл.
При необходимости старый можно потом переименовать, а новому присвоить старое имя.
В противном случае случайные потери данных без возможности восстановления неизбежны.

[ZIK1337]

vanyamba-electronics, то есть без второго файла здесь не обойтись?

[vanyamba-electronics]

ZIK1337, чтобы читать и писать в файл одновременно, его нужно открыть с ключом "r+", и использовать один указатель и для чтения, и для fseek, и для записи.
Если у вас длина строки будет изменяться, вам придётся каждый раз копировать данные в файле с места на место. Это очень медленно, если данных достаточно много.

[ZIK1337]

vanyamba-electronics, данные как раз ограничены, максимум N строк (N не больше 10) и максимум N символов в каждой строке (то есть тоже не больше 10)
понимаю, что очень медленно, но работа как раз направлена на то, чтобы изучить позиционирование..)

[vanyamba-electronics]

ZIK1337, зачем изучать позиционирование, если обработка файла в 10 Килобайт будет занимать 5-10 минут?
Осваивайте в таком случае memory mapping, он работает даже быстрее, чем fread/fwrite.
Отображение файла в память

[ZIK1337]

vanyamba-electronics, откуда я знаю, зачем?) таково задание, можете сами убедиться:

spoiler

[ZIK1337]

vanyamba-electronics,

fseek(file_ptr, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr)) {
			count = strlen(text);
			for (a = 0; a < count / 2; a++) {
				tmp = text[a];
				text[a] = text[count - 1 - a];
				text[count - 1 - a] = tmp;
			}
			fputs(text, file_ptr2);
		}

вот с помощью вспомогательного файла вышло, но только первая строка пустая становится, можете подсказать, как это исправить?

[vanyamba-electronics]

ZIK1337, пункт 4 что гласит? Из одного файла читать, в другой записать. Зачем Вам fseek понадобился?
Вот как это выглядит на С++.

[ZIK1337]

vanyamba-electronics, я же перед этим уже читал файл, значит указатель на конец файла встал, и если начать читать, то начнет читать с конца (то есть ничего не прочтет), разве не так?
вот, если убрать, в f3 ничего не заносится

spoiler

[vanyamba-electronics]

ZIK1337, просто закройте файл после записи и откройте для чтения.

[ZIK1337]

vanyamba-electronics, да все равно первая строка пустой остаётся

fclose(file_ptr);
		file_ptr = fopen("D:\\f1.txt", "r");
		// реверс строк в файле (задание 1.3)
		//fseek(file_ptr, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr)) {
			count = strlen(text);
			for (a = 0; a < count / 2; a++) {
				tmp = text[a];
				text[a] = text[count - 1 - a];
				text[count - 1 - a] = tmp;
			}
			fputs(text, file_ptr2);
		}

[vanyamba-electronics]

ZIK1337, скопируйте весь код в Joodle, и отправьте мне ссылку через Collaborate. Только страницу не закрывайте.

[ZIK1337]

vanyamba-electronics, https://www.jdoodle.com/c-online-compiler#&togethe...

[vanyamba-electronics]

ZIK1337, 51 и 52 строка

[ZIK1337]

vanyamba-electronics,
первая строка все равно пустая и последняя теперь склеивается с предпоследней, а также кол-во строк на 1 меньше считает

[vanyamba-electronics]

ZIK1337, я переставил fopen fclose, чтобы там можно было разобраться, где что, и поубирал плюсы в параметрах fopen.

[ZIK1337]

vanyamba-electronics, все равно первая строка пустая..)
ваш код полностью скопировал и заменил свой

[vanyamba-electronics]

ZIK1337, в строке 66 добавил -1, чтобы символ переноса строки не переставлялся в нулевую позицию.

[ZIK1337]

vanyamba-electronics, вот теперь все хорошо, ох уж эти коварные единицы..)
спасибо большое

[ZIK1337]

ZIK1337, кому интересно, через fseek тоже все работает, если "-1" добавить в том же месте

spoiler

#include <stdio.h>
#include <string.h>
#include <locale.h>

int main(int argc, char **argv)
{
	setlocale(LC_ALL, "Ru");
	FILE *file_ptr; // указатель на файл (задание 1.1)
	int N; // количество строк и символов в строках (задание 1.1)
	int count = 0; // для подсчета количества строк (задание 1.2)
	char ch; // для подсчета количества строк (задание 1.2)
	char text[50] = "0"; // строка для чтения из файла
	FILE *file_ptr1; // указатель на f2 (задание 1.4)
	int a; // для задания 1.3
	char tmp; // для задания 1.3
	FILE *file_ptr2; // вспомогательный файл для задания 1.3

	file_ptr = fopen("D:\\f1.txt", "w+");
	file_ptr1 = fopen("D:\\f2.txt", "w+");
	file_ptr2 = fopen("D:\\f3.txt", "w+");

	if (file_ptr != NULL) {
		printf("Введите количество строк в файле и символов в них (не более 10): \n");
		scanf("%d", &N);

		if (N > 10) {
			do {
				printf("Нужно ввести число не более 10! Повторите ввод: \n");
				scanf("%d", &N);
			} while (N > 10);
		}

		printf("Введите строки не более %d символов.\n", N);

		for (int i = 0; i < N; i++)
		{
			printf("Введите %d-ю строку: \n", (i + 1));
			scanf("%s", text);
			if (strlen(text) > N) {
				do {
					printf("Нельзя вводить больше %d символов! Введите строку еще раз: \n", N);
					scanf("%s", text);
				} while (strlen(text) > N);
				fputs(text, file_ptr);
				fputs("\n", file_ptr);
			}
			else
			{
				fputs(text, file_ptr);
				fputs("\n", file_ptr);
			}
		}


		// вывод числа строк в файле (задание 1.2)
		fseek(file_ptr, 0, SEEK_SET);
		while ((ch = fgetc(file_ptr)) != EOF) {
			if (ch == '\n')
				++count;
		}
		printf("Количество строк в файле: %d\n", count);


		// реверс строк в файле (задание 1.3)
		fseek(file_ptr, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr)) {
			count = strlen(text) - 1;
			for (a = 0; a < count / 2; a++) {
				tmp = text[a];
				text[a] = text[count - 1 - a];
				text[count - 1 - a] = tmp;
			}
			fputs(text, file_ptr2);
		}
		// копирование из вспомогательного файла в исходный
		fseek(file_ptr, 0, SEEK_SET);
		fseek(file_ptr2, 0, SEEK_SET);
		if (file_ptr2 != NULL) {
			while (fgets(text, 50, file_ptr2)) {
				fputs(text, file_ptr);
			}
		}
		else {
			printf("Невозможно открыть файл f3\n");
			return 1;
		}


		// вывод в f2 (задание 1.4)
		/*fseek(file_ptr, 0, SEEK_SET);
		if (file_ptr1 != NULL) {
			while (fgets(text, 50, file_ptr)) {
				fseek(file_ptr1, -(count1*N), SEEK_SET);
				count1++;
				fputs(text, file_ptr1);
			}
		}
		else {
			printf("Невозможно открыть файл f2\n");
			return 1;
		}*/


		// вывод содержимого f2 на экран (задание 1.5)
		/*fseek(file_ptr1, 0, SEEK_SET);
		while (fgets(text, 50, file_ptr1)) {
			printf("%s", text);
		}*/

		fclose(file_ptr);
		fclose(file_ptr1);
		fclose(file_ptr2);
		_getch();
		return 0;

	}
	else {
		printf("Невозможно открыть файл f1\n");
		return 1;
	}

}