Evitare il codice duplicato (PHP)

Question

Come amministratore di sistema, che finisce per fare qualche semplice ad-hoc di programmazione ogni tanto un po '. Sto cercando di imparare come vado avanti, quindi, in generale, c'è qualcosa nel seguente codice che salta fuori a voi come essere cattiva pratica o in altro modo non necessario?

In particolare, il 3 se le dichiarazioni alla fine sente come sto duplicare il codice inutilmente. C'è un modo per accorciare ulteriormente senza andare in mare con la complessità?

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

Non stai ripetendo inutilmente, ma ero confuso perché search_whois non ci vuole un dominio.

mi piacerebbe riorganizzare in modo search_whois è a sé stante

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