Tránh mã trùng lặp (PHP)

Question

Như một sysadmin, tôi sẽ thực hiện một số chương trình ad-hoc đơn giản mỗi lần trong một thời gian. Tôi đang cố gắng tìm hiểu khi tôi đi cùng, vì vậy nói chung, có bất cứ điều gì trong đoạn code dưới đây nhảy ra ngoài bạn như là thực hành xấu hay không cần thiết?

Cụ thể, báo cáo 3 if ở cuối có vẻ như tôi đang sao chép mã không cần thiết. Có cách nào để rút ngắn hơn nữa mà không đi overboard với sự phức tạp?

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

Bạn không lặp lại không cần thiết, nhưng tôi đã nhầm lẫn vì search_whois không có miền.

tôi muốn tổ chức lại để search_whois là khép kín

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