Убивать…

[dil]

int size=0;
...
fread( &size, sizeof(int), 1, f);
while(size--)
{
...
        int size=0;
        ...
        fread( &size, sizeof(int), 1, f);
        while(size--)
        {

Оригинал этой записи. Комментировать можно тут или там.

Любые материалы из этого блога запрещается использовать на сайте livejournal.ru в любой форме и любом объёме

Comments

[motto.livejournal.com]

и работает? ,)

[blacklion.livejournal.com]

А что ему сделается?

[motto.livejournal.com]

Варианты-то можно придумать ,)

[dil.livejournal.com]

работает.. эти size больше никак не используются, чисто счётчик записей, которые надо прочитать из файла.

[blacklion.livejournal.com]

А в чём тогда проблема? структурный у нас язык или где?

[dil.livejournal.com]

Проблема в читабельности кода.

Там ещё красивое есть, я минуты две думал, чтО они имели в виду:

        FILE *f = fopen("/hdd/epg.dat", "r");
        if (f)
        {
                unlink("/hdd/epg.dat");

[blacklion.livejournal.com]

Хм. Это классика, нет? Открыть и удалить сразу.

[dil.livejournal.com]

Оно, конечно, от случая зависит. В данном случае я бы так делать не стал, потому что программа, которая это считывает, имеет обыкновение падать в корку, ничего не сохраняя. В том числе и этот файл. Я бы его стирал непосредственно перед записью нового экземпляра.

[blacklion.livejournal.com]

Тогда, если это не временные данные, я бы его вообще не стирал. А переименовывал в *.bak, а на его место переимновывал бы новый (который создавал бы как *.new)

[dil.livejournal.com]

это место для хранения кэша EPG на диске на время перезапуска програмы. Обычно он хранится в памяти. В какой момент он создаётся, я не знаю, но здесь он только считывается и сразу стирается с диска.

[blacklion.livejournal.com]

Проблема в читабельности кода.
Ну, и опять же, зачем вам/нам дали вложенные блоки? :)
Не, сам бы я так не написал, но совсем преступления (в отличие от memcmp()) я в этом не вижу.