Questa funzione fa una cosa sola?

Question

Questa funzione fa una cosa sola?

#1 da (3 voti)

0

Il seguente metodo è considerato come una cosa sola?

Mi sto chiedendo perché ci vuole un argomento facoltativo.

public function findErrors($name = null)
{
    if ($name) {
        return isset($this->errors[$name]) ? $this->errors[$name] : [];
    }

    return $this->errors ?: [];
}

In caso contrario, sarebbe meglio / importa averlo separato come segue:

public function findErrors()
{
    return $this->errors ?: [];
}

public function findErrorsOf($name)
{
    return isset($this->errors[$name]) ? $this->errors[$name] : [];
}

functions methods design-patterns php

posta Kid Diamond 28.08.2014 - 19:34

fonte

1 risposta

Leggi altre domande sui tag functions methods design-patterns php

Utilizzo di un binario da un progetto che viola GPL Come organizzare ed etichettare i metodi destinati al debug?

score 3 · Accepted Answer

No, non proprio. Ha chiaramente due percorsi indipendenti. Sarebbe meglio separarli in due funzioni (possibilmente sovraccariche) per disaccoppiarle meglio.

Ancora meglio, vorrei cercare di eliminare il comportamento specifico del nome a meno che non sia veramente quello comune. Cosa succede se vuoi trovare errori da giovedì? Che dire degli errori che contengono la parola "Banana"? Non dovresti tornare a modificare questo codice ogni volta che hai un nuovo criterio di ricerca e non dovresti avere un modo per trovare i nomi e un altro modo per trovare le banane.