Модераторы: Daevaorn
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Последний символ - два раза, fgets 
V
    Опции темы
Валерия
Дата 7.4.2007, 14:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 598
Регистрация: 10.3.2007

Репутация: нет
Всего: 8
Код

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

int main(void)
{
    FILE *stream;
    char string[]=" \n";
    char *msg;

    stream = fopen("D:\\1.txt", "r");

    while (!feof(stream)){
        fgets(msg, strlen(string), stream);
        printf("%s", msg);
    }

    fclose(stream);
   return 0;
}


Файл 1.txt:
Код
First
Second
Third


Программа выдает
Код
First
Second
Thirdd


Почему последний символ выводиться на экран дважды?
PM MAIL   Вверх
apook
Дата 7.4.2007, 14:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 9
Всего: 23
Код

    while (!feof(stream))
    {
        fgets(msg, strlen(string), stream);
        if(!feof(stream))
           printf("%s", msg);
         }

попробуй так т.е проверку на конец файла сделать дважды

Это сообщение отредактировал(а) apook - 7.4.2007, 14:32


--------------------
Мои руки из дуба, голова из свинца ну и пусть ...
PM MAIL   Вверх
Валерия
Дата 7.4.2007, 14:25 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 598
Регистрация: 10.3.2007

Репутация: нет
Всего: 8
Спасибо, помогло.
PM MAIL   Вверх
codelord
Дата 7.4.2007, 14:32 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 777
Регистрация: 7.5.2005
Где: ты моя темноглаза я где?!

Репутация: 1
Всего: 39
что то не понятно как у тебя вообще заработала эта программа, у меня ошибки, собственно и понятно, зачем тебе
string если ты берешь его длину, при этом его длина как раз заведомо меньше даже небольшой строки, делать проверки два раза
тоже не красиво.
Код

#include <string.h>
#include <stdio.h>
#define strl 256
int main(void)
{
    FILE *stream;
    char msg[strl];
    stream = fopen("D:\\1.txt", "r");
    if(!stream)return 0;
    while (!feof(stream)){
        fgets(msg, strl-1, stream);
        printf("%s", msg);
    }
    fclose(stream);
   return 0;
}


Добавлено через 3 минуты и 53 секунды
char *msg;
где собственно память под это?
возьми файл где строки скажем хотя бы длины так по 100 символов, и это должно будет вылезти.

Это сообщение отредактировал(а) codelord - 7.4.2007, 14:33


--------------------
Доступен поиск по исходным кодам в GOOGLE.
http://www.google.com/codesearch
PM MAIL   Вверх
Валерия
Дата 9.4.2007, 00:00 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 598
Регистрация: 10.3.2007

Репутация: нет
Всего: 8
Тоесть если я просто объявляю
char *var1;

то не резервирую память, и оно может полезть на память скажем, компилятора и зависнуть все нафик?

Добавлено через 6 минут и 36 секунд
...
char *buff;
memset(buff, 0, 256):
...


такое резервирование катит?
PM MAIL   Вверх
nickless
Дата 9.4.2007, 00:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Гентозавр
****


Профиль
Группа: Участник Клуба
Сообщений: 2976
Регистрация: 29.8.2005
Где: Germany

Репутация: 19
Всего: 181
Валерия, да, память нужно выделять отдельно. 
Не инициализированный указатель указывает неизвестно куда, если попытаться записать туда что-нибудь на память других программ не залезешь (у каждой программы своё адресное пространство), но можно затереть себе что-нибудь (хуже) или схлопотать access violation/segfault (немного лучше)

Цитата(Валерия @  8.4.2007,  23:00 Найти цитируемый пост)
такое резервирование катит? 

Нет, это как раз наглое затирание нулями чего-то.
Память нужно или выделить динамически (malloc в C/new в C++) и не забыть освободить потом (free/delete) или указать адрес существующей переменной, например.
Код

char bla[100];
char * ptr = bla;


Это сообщение отредактировал(а) nickless - 9.4.2007, 17:12


--------------------
user posted image

Real men don't use backups, they post their stuff on a public ftp server and let the rest of the world make copies
- Linus Torvalds
PM MAIL   Вверх
Валерия
Дата 9.4.2007, 00:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 598
Регистрация: 10.3.2007

Репутация: нет
Всего: 8
Поняла когда побольше заперла туды, выкинуло.
А вот FILE *fp;
такое резервирование тоже получаться не правильное...
Ведь те несколько байт, в которых содержится дескриптор, тоже могут полезть не на то что надо?

Добавлено через 4 минуты и 32 секунды
А вот это написано по всем правилам? (учусь правильно писать).
И будет ли оно везде одинаково работать?
Скажем там под линуксом...
Код

#include <stdio.h>
#include <string.h>
#define SIZE 256

void main()
{
    FILE *fpin, *fpout;
    char *fnamein="D:\\1.txt";
    char *fnameout="D:\\2.txt";
    char buff[SIZE];
    int sz;
    fpin = fopen(fnamein, "r");
    fpout = fopen(fnameout, "w");

    memset(buff, 0, SIZE);

    while (!feof(fpin)){
        fgets(buff, SIZE, fpin);
        sz = strlen(buff)-1;
        if (buff[sz] == '\n'){
            buff[sz] = '\0';
        }

        fputs(buff, fpout);
        printf("%s", buff);
    }

    fclose(fpin);
    fclose(fout);
}
PM MAIL   Вверх
nickless
Дата 9.4.2007, 00:19 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Гентозавр
****


Профиль
Группа: Участник Клуба
Сообщений: 2976
Регистрация: 29.8.2005
Где: Germany

Репутация: 19
Всего: 181
С FILE *fp всё нормально, fopen аллокирует память сам, тут нужно по идее только проверять открылся ли файл 
Код

fp = fopen(...);
if (!fp)
{
    // делаем что-нибудь, ошибка при открытии файла
}



--------------------
user posted image

Real men don't use backups, they post their stuff on a public ftp server and let the rest of the world make copies
- Linus Torvalds
PM MAIL   Вверх
zkv
Дата 9.4.2007, 00:23 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата



****


Профиль
Группа: Участник Клуба
Сообщений: 2133
Регистрация: 23.7.2006
Где: Санкт-Петербург

Репутация: 26
Всего: 92
Цитата(Валерия @  9.4.2007,  00:00 Найти цитируемый пост)
char *buff;
memset(buff, 0, 256):
...
такое резервирование катит? 

нет, не катит,  memset() не выделяет память, а устанавливает в определенное значение. Юзай malloc() или new (в С++).

Цитата(Валерия @  9.4.2007,  00:12 Найти цитируемый пост)
А вот FILE *fp;
такое резервирование тоже получаться не правильное...
Ведь те несколько байт, в которых содержится дескриптор, тоже могут полезть не на то что надо? 

тут другой случай, система открывает заданный файл, а тебе дает его handle, а ты дальше за этот handle им (файлом) управляешь.
PM MAIL   Вверх
nickless
Дата 9.4.2007, 00:35 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Гентозавр
****


Профиль
Группа: Участник Клуба
Сообщений: 2976
Регистрация: 29.8.2005
Где: Germany

Репутация: 19
Всего: 181
Цитата(Валерия @  8.4.2007,  23:12 Найти цитируемый пост)
будет ли оно везде одинаково работать?
Скажем там под линуксом

Не будет, в линуксе нет диска D:  smile  Если убрать его, будет
А вообще main должна по стандарту возвращать int, и добавь в конце цикла while 
Код
buff[0] = 0;
, чтоб последняя строка второй раз не записывалась в файл.

Это сообщение отредактировал(а) nickless - 9.4.2007, 00:36


--------------------
user posted image

Real men don't use backups, they post their stuff on a public ftp server and let the rest of the world make copies
- Linus Torvalds
PM MAIL   Вверх
apook
Дата 9.4.2007, 04:53 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 9
Всего: 23
Код

char *msg="V etom sluchae ni hrena ne nado pamjft videljat";

nickless
Код

char bla[]="ABCDEFG";
char *ptr = &bla;

и какой компилятор на это не ругнулся?
тагда уж
Код

char *ptr = &bla[0];

или
Код

char *ptr = bla;


Это сообщение отредактировал(а) apook - 9.4.2007, 05:17


--------------------
Мои руки из дуба, голова из свинца ну и пусть ...
PM MAIL   Вверх
Валерия
Дата 9.4.2007, 13:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 598
Регистрация: 10.3.2007

Репутация: нет
Всего: 8
Как ни странно но последняя строка не записалась в файл два раза smile

Добавлено через 2 минуты и 7 секунд
А вот еще такой вопрос.
Если я не делаю free для возврата памяти, и fclose для закрытия файла.
Что в этом будет плохого?
По идее после завершения работы программы она сама всё закроет и высвободит.
Или времени больше на это нужно будет?
PM MAIL   Вверх
nickless
Дата 9.4.2007, 17:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Гентозавр
****


Профиль
Группа: Участник Клуба
Сообщений: 2976
Регистрация: 29.8.2005
Где: Germany

Репутация: 19
Всего: 181
Цитата(apook @  9.4.2007,  03:53 Найти цитируемый пост)
и какой компилятор на это не ругнулся?

Сорри, & лишнюю написал  smile 

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

Цитата(Валерия @  9.4.2007,  12:10 Найти цитируемый пост)
Как ни странно но последняя строка не записалась в файл два раза
А у меня записалась, странно... 

Это сообщение отредактировал(а) nickless - 9.4.2007, 17:22


--------------------
user posted image

Real men don't use backups, they post their stuff on a public ftp server and let the rest of the world make copies
- Linus Torvalds
PM MAIL   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "С++:Общие вопросы"
Earnest Daevaorn

Добро пожаловать!

  • Черновик стандарта C++ (за октябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика(4.4мб).
  • Черновик стандарта C (за сентябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика (3.4мб).
  • Прежде чем задать вопрос, прочтите это и/или это!
  • Здесь хранится весь мировой запас ссылок на документы, связанные с C++ :)
  • Не брезгуйте пользоваться тегами [code=cpp][/code].
  • Пожалуйста, не просите написать за вас программы в этом разделе - для этого существует "Центр Помощи".
  • C++ FAQ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Earnest Daevaorn
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | C/C++: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0540 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.