システム管理者として、ときどき単純なアドホック プログラミングを行うことになります。私は進みながら学ぼうとしているので、一般的に、以下のコードの中に悪い習慣である、または不要であると思われるものはありますか?
特に、最後の 3 つの if ステートメントは、コードを不必要に複製しているように感じます。複雑に行き過ぎることなく、さらに短縮する方法はありますか?
<?php
define('TAKEN', 'Match: One');
define('AVAIL', 'Match: No Matches');
define('DATAMINE', 'Data mining count exceeded');
$ch = curl_init("http://co.za/cgi-bin/whois.sh?Domain=example");
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_HEADER, 0);
$output = curl_exec($ch);
function search_whois($findit) {
global $output;
if (strpos($output, $findit) === false)
return false;
if (is_int(strpos($output, $findit)))
return true;
}
if (search_whois(TAKEN))
echo "Domain is taken.\n";
if (search_whois(AVAIL))
echo "Domain is available.\n";
if (search_whois(DATAMINE))
echo "Blocked for datamining, try again later.\n";
// var_dump($output);
?>