IMDB Grabber PHP - PullRequest
Новая
       17

IMDB Grabber PHP

1 голос
/ 23 марта 2010

Я получаю сообщение об ошибке:

Примечание: неопределенная переменная: содержимое в C: \ wamp \ www \ includes \ imdbgrabber.php в строке 17

При использовании этого кода: 

<?php
//url
$url = 'http://www.imdb.com/title/tt0367882/';

//get the page content
$imdb_content = get_data($url);

//parse for product name
$name = get_match('/<title>(.*)<\/title>/isU',$imdb_content);
$director = strip_tags(get_match('/<h5[^>]*>Director:<\/h5>(.*)<\/div>/isU',$imdb_content));
$plot = get_match('/<h5[^>]*>Plot:<\/h5>(.*)<\/div>/isU',$imdb_content);
$release_date = get_match('/<h5[^>]*>Release Date:<\/h5>(.*)<\/div>/isU',$imdb_content);
$mpaa = get_match('/<a href="\/mpaa">MPAA<\/a>:<\/h5>(.*)<\/div>/isU',$imdb_content);
$run_time = get_match('/Runtime:<\/h5>(.*)<\/div>/isU',$imdb_content);

//build content


line 17 -->  $content.= '<h2>Film</h2><p>'.$name.'</p>';
    $content.= '<h2>Director</h2><p>'.$director.'</p>';
    $content.= '<h2>Plot</h2><p>'.substr($plot,0,strpos($plot,'<a')).'</p>';
    $content.= '<h2>Release Date</h2><p>'.substr($release_date,0,strpos($release_date,'<a')).'</p>';
    $content.= '<h2>MPAA</h2><p>'.$mpaa.'</p>';
    $content.= '<h2>Run Time</h2><p>'.$run_time.'</p>';
    $content.= '<h2>Full Details</h2><p><a href="'.$url.'" rel="nofollow">'.$url.'</a></p>';

    echo $content;

//gets the match content
function get_match($regex,$content)
{
    preg_match($regex,$content,$matches);
    return $matches[1];
}

//gets the data from a URL
function get_data($url)
{
    $ch = curl_init();
    $timeout = 5;
    curl_setopt($ch,CURLOPT_URL,$url);
    curl_setopt($ch,CURLOPT_RETURNTRANSFER,1);
    curl_setopt($ch,CURLOPT_CONNECTTIMEOUT,$timeout);
    $data = curl_exec($ch);
    curl_close($ch);
    return $data;
}
?>

Ответы [ 4 ]

6 голосов
/ 23 марта 2010

Вы добавляете контент в переменную, которая не существует. Измените строку 17 на присвоение: 

$content = '<h2>Film</h2><p>'.$name.'</p>';

Вы также можете изменить этот раздел кода на следующий, что немного лучше: 

$content = '<h2>Film</h2><p>'.$name.'</p>'
         . '<h2>Director</h2><p>'.$director.'</p>'
         . '<h2>Plot</h2><p>'.substr($plot,0,strpos($plot,'<a')).'</p>'
      // etc
3 голосов
/ 23 марта 2010

Вы не получаете сообщение об ошибке, вы получаете уведомление, потому что вы пытаетесь объединить что-то с несуществующей переменной. Удалите точку из .= в строке 17 или поставьте $content = '' перед строкой 17.

3 голосов
/ 23 марта 2010

Вы пытаетесь добавить что-то в переменную $content, когда она еще не существует, это естественно вызывает ошибку.

Попробуйте заменить $content.= на $content= в строке 17.

1 голос
/ 23 марта 2010

Помимо того, что говорили другие, есть еще одна проблема с вашим кодом, которая требует внимания. Вы не проверяете возвращаемое значение preg_match, прежде чем вернуть значение из функции get_match. Вы должны сделать что-то вроде: 

if(preg_match($regex,$content,$matches))
  return $matches[1];
else
  // return some default
...