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; 
}