PHP變量太多寫

Question

我有以下代碼：

require("class.XMLHttpRequest.php"); 
function hot($news){ 
    $url="https://localhost/search.aspx?search=".$news.""; 
$ajax=new XMLHttpRequest(); 
$ajax->setRequestHeader("Cookie","Cookie: host"); 
$ajax->open("GET",$url,true); 
$ajax->send(null); 
if($ajax->status==200){ 
    $rHeader=$ajax->getResponseHeader("Set-Cookie"); 
    if(substr_count($rHeader, "Present!")>0) { return true; } 
}else{ return false; } 
} 

$content1= hot("britney") ? "britney found" : ""; 
$content2= hot("gaga") ? "gaga found" : ""; 
$content3= hot("carol") ? "carol found" : ""; 

$filename = 'result.txt'; 
$handle = fopen($filename, 'a'); 
fwrite($handle, "$Content1\r\n"); 
fwrite($handle, "$Content2\r\n"); 
fwrite($handle, "$Content3\r\n"); 
fwrite($handle, "$Content4\r\n"); 
fclose($handle); 

我想縮短腳本cuz我有很多$ ContentN變量 也許類似的foreach？

Answer 1

我會做這樣的：

$celebrities = array('britney','gaga','carol'); 
$filename = 'result.txt'; 
$handle = fopen($filename, 'a'); 

foreach($celebrities as $celebrity) 
{ 
    if(hot($celebrity)) { fwrite($handle, "{$celebrity}\r\n"); } 
} 

fclose($handle); 

如果你需要更多的名人，只是將它們添加到陣列。

Answer 2

這樣的事情將是非常接近你的實際代碼，但

for($i = 1 ; $i <= 4 ; $i++) 
fwrite($handle, "${Content$i}\r\n"); 

它使用可變的變量可能不被recomanded：http://php.net/manual/en/language.variables.variable.php

這裏不是最好的解決辦法：你爲什麼不只需使用一個數組？

$content[1]= hot("britney") ? "britney found" : ""; 
$content[2]= hot("gaga") ? "gaga found" : ""; 
$content[3]= hot("carol") ? "carol found" : ""; 
for($i = 1 ; $i <= 4 ; $i++) 
fwrite($handle, $Content[$i]."\r\n"); 

或者以前更好，使用captaintokyo的解決方案，因爲你可能不希望在文本文件中的空行。

Answer 3

下面是一個簡短的代碼重構你。更新的變量名稱，數組中保存的趨勢主題以及您詢問的foreach。代碼未經測試。

require("class.XMLHttpRequest.php"); 

$result_filename = 'result.txt'; 

$hot_topics = array(
    'britney', 
    'gaga', 
    'carol' 
); 

$handle = @fopen($result_filename, 'a+'); 
if (!$handle) { 
    exit("Unable to open $result_filename"); 
} 

foreach($hot_topics as $topic) { 
    if (is_hot($topic)) { 
    fwrite($handle, "$topic found\r\n"); 
    } 
} 

fclose($handle); 

exit("\ncomplete"); 

function is_hot($news) { 
    $url = "https://localhost/search.aspx?search=".$news; 

    $ajax = new XMLHttpRequest(); 
    $ajax->setRequestHeader("Cookie", "Cookie: host"); 
    $ajax->open("GET", $url, true); 
    $ajax->send(null); 

    if ($ajax->status == 200) { 
    $rHeader = $ajax->getResponseHeader("Set-Cookie"); 
    if (substr_count($rHeader, "Present!") > 0) { 
     return true; 
    } 
    } 
    return false; 
}