2012-12-22 96 views
0

我剛開始使用json進行實際工作,並試圖儘可能地學習!我想分享我所做的這項工作,我覺得這可能需要一些改進,如果不是很多。json_decode響應是否爲空

那麼好吧,我用twitch.tv REST_API。這是我的代碼。基本上我想通過我的網絡託管公司運行這個每分鐘作爲一個crontab。我知道你可以通過這種方式獲得(編碼)JSON數據:「http://api.justin.tv/api/stream/list.json?channel=example,example2,example3」;以及。它可能更快?但後來我不知道如何在數據庫中設置我的流離線。

所以我想我是問我如何能改善這一點。

$result = mysql_query("SELECT streamname FROM streams") or die(mysql_error()); 

$ids=array(); 
while($row = mysql_fetch_assoc($result)) 
{ 
    $ids[]=$row["streamname"]; 
} 

$mycurl = curl_init(); 
for($i=0;$i<count($ids);$i++) 
{ 

    curl_setopt ($mycurl, CURLOPT_HEADER, 0); 
    curl_setopt ($mycurl, CURLOPT_RETURNTRANSFER, 1); 

    $url = "http://api.justin.tv/api/stream/list.json?channel=$ids[$i]"; 
    curl_setopt ($mycurl, CURLOPT_URL, $url); 

    $web_response = curl_exec($mycurl); 
    $result = json_decode($web_response); 

    if(empty($result)) 
    { 
     $sql = "UPDATE streams SET online = '0' WHERE streamname = '" . $ids[$i] . "'"; 
    } 
    else 
    { 
     $sql = "UPDATE streams SET online = '1' WHERE streamname = '" . $ids[$i] . "'"; 
    } 
    mysql_query($sql) or die(mysql_error()); 
} 
+5

您正在使用[an **過時的**數據庫API](http://stackoverflow.com/q/12859942/19068)並應使用[現代替換](http://php.net/manual/) EN/mysqlinfo.api.choosing.php)。你也**易受[SQL注入攻擊](http://bobby-tables.com/)**,現代的API會使[防禦]更容易(http://stackoverflow.com/questions/60174/best-way-to-prevent-sql-injection-in-php)自己從。 – Quentin

回答

1

顯然要離線渠道不出現在API結果,以及其他方式,標記通道在線大關,這仍然會出現。

首先這是已經在評論中說,一個音符。 請不要再使用PHP的mysql擴展。這被棄用,並將在PHP的未來版本中刪除我推薦的MySQLi:http://php.net/manual/en/book.mysqli.php

每個通道,這當然減慢的過程和不必要的放justin.tv服務器的負載當前正在獲取數據。

查詢狀態時,限制是GET請求的最大大小,在大多數服務器上爲8192字節。

現在,取而代之的是針對empty檢查結果,您可以將所有通道都視爲離線,然後循環顯示結果並再次將結果中的通道標記爲在線。在數組或對象中執行此操作(可以是您爲頻道列表獲取的數據庫結果),並在一個查詢中更新所有頻道。

+0

謝謝你的幫助,現在看來很明顯,我應該這樣做!但MySQL LI我不知道,謝謝。我會看看。 :) – mpj