Задать вопрос
Sergey_Uspenskiy
@Sergey_Uspenskiy
Веб-дизайнер, front-end разработчик
php

Сохраняет битый jpg,скрипт парсинга новостей, в чем ошибка?

Скрипт парсит новость, скачивает картинку по link, сохраняет на сервер битый файл, в чем ошибка? :

if($f_Link){
$c=file_get_contents($f_Link);

$fh=fopen($_SERVER['DOCUMENT_ROOT']."/scripts/news.txt","w");
fwrite($fh,$c);
fclose($fh);
preg_match("/b-material-head__title\">(.{1,})<\/h1>/", $c, $h1);
preg_match("/class=\"b-material-head__date-day\">(.{10})<\/span>/", $c, $date);
preg_match("/link rel=\"image_src\" href=\"(.*)\">/U", $c, $img); -сохранение картинки
preg_match("/(\C*<\/article>)/U", $c, $text);

$id=mysql_insert_id($LinkID);
if($img[1]){
$ext= strrchr($img[1],".");
mysql_unbuffered_query("INSERT INTO Message5 (User_ID,Subdivision_ID,Sub_Class_ID,Priority,Checked,Created,parent_class_id,parent_mess_id,foto) VALUES(1,'19','19',20,1,NOW(),'2','".$message."','str".$ext.":image/jpeg:1');", $LinkID);
$id=mysql_insert_id($LinkID);
copy($img[1],$_SERVER['DOCUMENT_ROOT']."/sysfiles/45_".$id.$ext);
}
}
--
d22fdb0c5a274ab3b2cb1a19298e9e1e.jpga0b2cf7262e44243a1085b5f6636c87a.jpg

--
мини-картинка есть, только она в самом теле новости, а должна быть наверху в заголовке.
Скрипт раьнше работал без проблем, обновилась верстка на сайте Русской газеты, правлю регулярные выражения и не могу никак решить задачу с изображением. Может, кто видит ошибку?
  • Вопрос задан
  • 296 просмотров
8 комментариев
Подписаться 1 Оценить 8 комментариев
Пригласить эксперта
Ответы на вопрос 1
slo_nik
@slo_nik Куратор тега PHP
Доброй ночи.
Проблема у Вас в строке:
copy($img[1],$_SERVER['DOCUMENT_ROOT']."/sysfiles/45_".$id.$ext);

В $img['1'] попадает '//cdnimg.rg.ru/img/content/138/31/30/pr_t_650x433.jpg'.
Попробуйте сделать так:
copy('https:' . $img[1],$_SERVER['DOCUMENT_ROOT']."/sysfiles/45_".$id.$ext);

Файл будет копироваться без проблем.
И делайте проверки, чтобы можно было самостоятельно определить ошибку.
Сделайте сначала как у Вас сейчас:
if(!copy($img[1],$_SERVER['DOCUMENT_ROOT']."/sysfiles/45_".$id.$ext)){
   echo 'error';
}
else{
 echo 'success';
}

Без указания протокола передачи данных должно быть предупреждение, добавите https и всё должно заработать.
И посмотрите логи сервера, там должно быть это предупреждение.
И сам pattern регулярного выражения можно подправить.

p.s.
И в строке
preg_match("/link rel=\"image_src\" href=\"(.*)\">/U", $c, $img); -сохранение картинки

нет никакого сохранения/копирования изображения)

p.s.
Этот код
$fh=fopen($_SERVER['DOCUMENT_ROOT']."/scripts/news.txt","w");
fwrite($fh,$c);
fclose($fh);

можно заменить на file_put_contents()
Ответ написан
Комментировать
Комментировать
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы
Вакансии с Хабр Карьеры
PHP Developer
YCLIENTS • Москва
от 200 000 до 350 000 ₽
PHP разработчик
Ведисофт • Екатеринбург
от 25 000 ₽
Midlle PHP developer (backend)
ИТЦ Аусферр • Магнитогорск
от 100 000 до 160 000 ₽
Ещё вакансии
Заказы с Хабр Фриланса
Дизайн ios приложения утилиты
25 апр. 2024, в 12:20
15000 руб./за проект
Куплю аккаунт Linkedin
25 апр. 2024, в 12:08
300 руб./за проект
Помощь СММ-специалиста
25 апр. 2024, в 11:49
25000 руб./за проект
Ещё заказы