Unikanie duplikatu kodu (PHP)

Question

Jako sysadmin, kończę wykonywanie prostych programów ad-hoc co jakiś czas. Próbuję się uczyć, kiedy idę dalej, więc ogólnie rzecz biorąc, czy jest coś w poniższym kodzie, które wyskakuje na ciebie jako zła praktyka lub w inny sposób niepotrzebny?

W szczególności 3 instrukcje if na końcu wydają się niepotrzebnie duplikować kod. Czy istnieje sposób, aby skrócić to jeszcze bardziej, nie przesadzając ze złożonością?

<?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); 

?> 

Answer 1

Nie powtarzasz się niepotrzebnie, ale byłem zdezorientowany, ponieważ search_whois nie ma domeny.

bym zreorganizować tak search_whois jest samowystarczalny

function search_whois($domain) { 
    $ch = curl_init("http://co.za/cgi-bin/whois.sh?Domain=$domain"); 

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

    $output = curl_exec($ch); 

    if (strpos($output, AVAIL) >= 0) { 
     echo "Domain is available.\n" 
     return true; 
    } 

    if (strpos($output, TAKEN) >= 0) 
     echo "Domain is taken.\n"; 
    else if (strpos($output, DATAMINE) >= 0) 
     echo "Blocked for datamining, try again later.\n" 

    return false; 
}