2012-04-12 78 views
-1

我知道我的代码很混乱,我对此非常陌生,并且随时随地学习。PHP XML失败

我的网站是这样的:​​(目前正在运行我的家用计算机,并使用tvdb api,所以它有点慢)。

大部分似乎工作,目前唯一没有的是NCIS如下所示:http://itsalllegit.com/season.php?sid=72108&season=1

我知道它的糟糕和长久,但这里是我的代码...我坚持这一个,并将不胜感激(如果有任何提示,使它在一般情况下更好)。

btw ...如果我echo $url3它提出了"(*#YR&(^#R(@$#%$&^@#%$*@#%$&^@$#&"等等。

<html> 
    <head> 
     <title>Saars - Homepage</title> 
      <link rel="stylesheet" type="text/css" href="default.css" /> 
    </head> 
     <body> 
      <?php 

       $series_id = $_GET["sid"]; 
       $season_number = $_GET["season"]; 
       $local_path = "images/Banner/".$series_id."-Banner.jpg"; 
       $url1 = file_get_contents("http://thetvdb.com/api/E676DF9578EF38D7/series/".urlencode($series_id)); 
       $xml = simplexml_load_string($url1); 
       $series_name = $xml->Series[0]->SeriesName; 
       $series_rating = $xml->Series[0]->Rating; 
       $series_network = $xml->Series[0]->Network; 
       $series_status = $xml->Series[0]->Status; 
        echo '<img src="'.$local_path.'"><br />'; 
        echo "Name: ".$series_name."<br />"; 
        echo "Rating: ".$series_rating."<br />"; 
        echo "Network: ".$series_network."<br />"; 
        echo "Status: ".$series_status."<br />"; 
        echo "<hr />"; 
         echo '<table bgcolor="#000000">'; 
          echo '<tr>'; 
           echo '<td width="30">'; 
            echo '<font color="white">#</font>'; 
           echo '</td>'; 
           echo '<td width="500">'; 
            echo '<font color="white">Episode Name</font>'; 
           echo "</td>\n"; 
           echo '<td width="100">'; 
            echo '<font color="white">First Aired</font>'; 
           echo "</td>\n"; 
          echo "</tr>\n"; 
         echo "</table>"; 
       for ($episode_number = 1; $episode_number <= 200; $episode_number++) { 
        $url2 = "http://thetvdb.com/api/E676DF9578EF38D7/series/".$series_id."/default/".urlencode($season_number)."/".urlencode($episode_number)."/en.xml"; 
        $handle = @fopen($url2, 'r'); 
        if ($handle === false) { 
         return; 
        } else { 
         $url3 = file_get_contents("http://thetvdb.com/api/E676DF9578EF38D7/series/".$series_id."/default/".urlencode($season_number)."/".urlencode($episode_number)."/en.xml"); 
         $xml = simplexml_load_string($url3); 
         $episode_name = $xml->Episode[0]->EpisodeName; 
         $episode_rating = $xml->Episode[0]->Rating; 
         $episode_firstaired = $xml->Episode[0]->FirstAired; 
         $episode_overview = $xml->Episode[0]->Overview; 
         $column1 = '<a href="episode.php?sid='.$series_id.'&season='.$season_number.'&episode='.$episode_number.'">'.$episode_number.'</a>'; 
         $column2 = '<a href="episode.php?sid='.$series_id.'&season='.$season_number.'&episode='.$episode_number.'">'.$episode_name.'</a>'; 
         $column3 = '<a href="episode.php?sid='.$series_id.'&season='.$season_number.'&episode='.$episode_number.'">'.$episode_firstaired.'</a>'; 
          echo '<table>'; 
           echo "<tr>\n"; 
            echo '<td width="30">'; 
             echo $column1; 
            echo "</td>\n"; 
            echo '<td width="500">'; 
             echo $column2; 
            echo "</td>\n"; 
            echo '<td width="100">'; 
             echo $column3; 
            echo "</td>\n"; 
           echo "</tr>\n"; 
          echo "</table>"; 
        fclose($handle); 
        } 
       } 
       ?> 
     </Body> 
    </html> 

回答

0

您正在打开每个URL两次,但只读取一次数据。服务器可能会拒绝这种行为,因为它会在一段时间后关闭并拒绝第二个连接。试着改变循环的内容,以这样的:

// Define the URL 
$url2 = "http://thetvdb.com/api/E676DF9578EF38D7/series/$series_id/default/".urlencode($season_number)."/".urlencode($episode_number)."/en.xml"; 

// Try and fetch the XML 
// You are returning a value - are you in a function? Should you maybe break; instead? 
if (!$xmlStr = file_get_contents($url2)) return FALSE; 
if (!$xml = simplexml_load_string($xmlStr)) return FALSE; 

// Get data 
$episode_name = $xml->Episode[0]->EpisodeName; 
$episode_rating = $xml->Episode[0]->Rating; 
$episode_firstaired = $xml->Episode[0]->FirstAired; 
$episode_overview = $xml->Episode[0]->Overview; 

// Output HTML 
// A single-row table per record? Maybe you should have one table with multiple rows? 
$epURL = htmlspecialchars("episode.php?sid=$series_id&season=$season_number&episode=$episode_number"); 
echo " 
<table> 
    <tr> 
    <td width='30'><a href='$epURL'>$episode_number</a></td> 
    <td width='500'><a href='$epURL'>$episode_name</a></td> 
    <td width='100'><a href='$epURL'>$episode_firstaired</a></td> 
    </tr> 
</table>"; 

如果还是不行,请尝试循环呼应了$url2看,当你错误它所包含的内容。

+0

谢谢,会做,但我不认为它会帮助很大,因为到目前为止它似乎只是NCIS多数民盟这是失败 – Saars 2012-04-12 12:00:40

+0

我知道这似乎很奇怪,但是当我加载的URL应该与您的代码生成第一个在我加载页面时不起作用的是ep 3:http://thetvdb.com/api/E676DF9578EF38D7/series/72108/default/1/3/en.xml)。 – DaveRandom 2012-04-12 12:10:43

+0

我知道......有时候是作品,有时候不是。我想我可能会把一个if语句放在下载它的地方,如果这个语句不是已经在本地的话 – Saars 2012-04-12 12:53:37