Код кладет сайт

Question

[HappyCougar]

Код парсера кладет сайт если сделать инклуд файла с ним:

<?php 
  $content = get_content(); 
  $pattern = "#<Valute ID=\"([^\"]+)[^>]+>[^>]+>([^<]+)[^>]+>[^>]+>[^>]+>[^>]+>[^>]+>[^>]+>([^<]+)[^>]+>[^>]+>([^<]+)#i"; 
  preg_match_all($pattern, $content, $out, PREG_SET_ORDER); 
  $dollar = ""; 
  $euro = ""; 
  foreach($out as $cur) 
  { 
    if($cur[2] == 840) $dollar = str_replace(",",".",$cur[4]); 
    if($cur[2] == 978) $euro   = str_replace(",",".",$cur[4]); 
  } 
  function get_content() 
  {  
    $date = date("d/m/Y"); 
    $link = "http://www.cbr.ru/scripts/XML_daily.asp?date_req=$date"; 
    $fd = fopen($link, "r"); 
    $text=""; 
    if (!$fd) {
		echo "Запрашиваемая страница не найдена"; 
	}
    else 
    { 
      while (!feof ($fd)) $text .= fgets($fd, 4096); 
    } 
    fclose ($fd); 
    return $text; 
  } 
?>

Ошибок в файле нету, более того он работает. Но если его поместить в целевую страницу он ломает её.

Comments

[Сергей Протько]

что значит ломает?

Answer 1

[darwin_usb]

$link = "http://www.cbr.ru/scripts/XML_daily.asp?date_req=".$date; 
$rss = implode("",file($link));

попробуйте так содержимое брать

Comments

[Mandor]

implode('', file()) - лишнее действие, file_get_contents() - лучше.

[darwin_usb]

А по мне implode лучше, он быстрей работает. Но это конечно для небольших файлов. Хотя он у меня в парсере на 2000 файлов отлично работает.

[Назар Мокринский]

@darwin_usb Зачем читать построчно и склеивать, если можно получить всё сразу???

Answer 2

[Mandor]

Кстати, cbr.ru умеет контент через SOAP отдавать.

Answer 3

[Назар Мокринский]

Можете точнее определить что значит "кладет сайт" и "ломает её"?

Answer 4

[Александр Кубинцев]

Use the cURL, Luke!

Answer 5

[Александр]

В логах что? Мы тут не ясновидящие)

Answer 6

[HappyCougar]

Код оказался целиком рабочим, оказалась трабла была в целевой странице. Там в фреймворке защита от использования функций запилена (непонятно зачем), достал код из функции и все заработало.