КиноПоиск.ru (фильмы)

[rvc]

Очень давно прошу поправить добавление продолжительности фильма кинопоиска.
Сейчас не берется, а вчера отвалился с продолжительностью фильма и IMDB.

Исправим , только удивляет одно , я впервые вижу что вы написали об этой проблеме.

http://www.videodb.info/forum_ru/index.php/topic,116.msg12866.html#msg12866

самое раннее здесь

http://www.videodb.info/forum_ru/index.php/topic,116.msg10923.html#msg10923
 
У меня длительность фильма с кинопоиска не бралась с тех пор, спасал IMDB.

[djek-ural]

Я проверил у себя , у меня все забирается , не могли бы вы написать примеры фильмов на которых у вас не забирается время.

[parasite]

Фильмы Кинопоиска опять отвалились - при попытке импорта форвардят на 403.

[djek-ural]

Только что проверил на 2ух фильмах , все прошло удачно. Может у вас временные трудности с доступом к кинопоиску.

[parasite]

Да, действительно. Прямо сейчас всё работает, хотя полдня тупило. 

PS: а сейчас опять не работает. Опять 403...

[parasite]

Приветствую.

Попытка импорта следующих фильмов (как через поиск+клик на соотв.миниатюре, так и путем прямого указания УРЛа в карточке фильма) вызывает крах программы и выдачу окошка "Закрыть\Продолжить\Отправить отчет":

http://www.kinopoisk.ru/level/1/film/63991/
http://www.kinopoisk.ru/level/1/film/102124/
http://www.kinopoisk.ru/level/1/film/427076/
(это разные части одной квадрилогии)

При этом в статус-строке видно что всё прокачивается с сервера ОК, и ошибка возникает уже при парсинге результатов или уже при инсерте в БД...
Другие фильмы вроде качаются\добавляются ОК, проблема именно с тремя указанными. Почините, пожалуйста?

[djek-ural]

попробую отремонтировать

[djek-ural]

Проблема возникает при парсинге связей , но ошибку в коде найти пока не могу.

[unreal666]

djek-ural
я уже тоже определил, что ошибка при парсинге связей.
Возникает при отсутствии года у фильма. Также еще кажется косячок при парсинге фильма без оригинального названия, но не уверен, т.к. раньше кажется в таком случае было норм.
Место, вызывающее сбой, определил, но пока еще думаю, как обойти это.
Скорее всего фильмы без года вообще не добавлять в связи, т.к. в ф-и AddConnection он обязателен.

[djek-ural]

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

Код: [Выделить]

year := HTMLValues2(tmp, '<a', '</a>', 'class=orange', '</a>', ',', curPos);

[djek-ural]

Пример кода где он возможно не срабатывает.

Код: [Выделить]

                        <tr>
                                    <td width="5%" valign="top">2.</td>
                                    <td width="95%" align="left">
                                        <div style="position: relative; padding-right: 22px">
                                            <a href="/film/427076/" class="all">Пираты Карибского моря: На странных берегах</a>,&nbsp;<a href="/level/10/m_act%5Byear%5D/2011/" class=orange>2011</a></><br />
                                            <font color="#999999">Pirates of the Caribbean: On Stranger Tides&nbsp;</font>
                                           
                                        </div>
                                    </td>
                                </tr>
                           
                                <tr>
                                    <td width="5%" valign="top">3.</td>
                                    <td width="95%" align="left">
                                        <div style="position: relative; padding-right: 22px">
                                            <a href="/film/575180/" class="all">Пираты Карибского моря&nbsp;5</a></><br />
                                            <font color="#999999">Pirates of the Caribbean 5&nbsp;</font>
                                           
                                        </div>
                                    </td>
                                </tr>

[unreal666]

можно

Код: [Выделить]

           year := HTMLValues2(tmp, '<a', '</a>', 'class=orange', '</a>', ',', curPos);
           origtitle := ReplaceToSimple (HTMLValues2(tmp, '<font', '</font>', '#999999', '</font>', ',', curPos));
           if OrigTitle[length(OrigTitle)] = ' ' then
             OrigTitle := Copy (OrigTitle, 1, length (OrigTitle) - 1);
           if StringReplace (OrigTitle, ' ', '', true, false, false) = '' then
             OrigTitle := Title;
           if optConnection = Enabled then AddConnection(TVVideo(Title), TVVideo(OrigTitle), cat, url, year);
заменить на

Код: [Выделить]

           year := HTMLValues2(tmp, '<a', '</a>', 'class=orange', '</a>', ',', curPos);
           if StringReplace(year, ' ', '', true, false, false) <> '' then
             begin
              origtitle := ReplaceToSimple (HTMLValues2(tmp, '<font', '</font>', '#999999', '</font>', ',', curPos));
              if OrigTitle[length(OrigTitle)] = ' ' then
                OrigTitle := Copy (OrigTitle, 1, length (OrigTitle) - 1);
              if StringReplace (OrigTitle, ' ', '', true, false, false) = '' then
                OrigTitle := Title;
              if optConnection = Enabled then AddConnection(TVVideo(Title), TVVideo(OrigTitle), cat, url, year);
             end;
т.е. если год пустой или состоит из пробелов, то вообще игнорировать остальной код в if-е и, соответственно, данный фильм в связи не добавлять.

[unreal666]

хотя если можно вставлять с пустым годом, то и вправду лучше правильно обработать блок html-кода с отсутствующим годом.

[unreal666]

кажется нашел как сделать.
Заменить

Код: [Выделить]

           year := HTMLValues2(tmp, '<a', '</a>', 'class=orange', '</a>', ',', curPos);
на

Код: [Выделить]

           tmpCurPos := curPos;
           year := StringReplace(HTMLValues2(tmp, '<a', '</a>', 'class=orange', '</a>', ',', curPos), ' ', '', true, false, false);
           if year = '' then curPos := tmpCurPos;
ну и в блок переменных добавить

Код: [Выделить]

tmpCurPos            : Integer
ошибка не возникает и фильмы без года добавляются.

У себя его исправил. Заливать на сайт или сам другой вариант сделаешь?

[djek-ural]

Да у тебя нормальный код получился , заливай свой вариант.