が必要になることはありません、あなたは私が私に言ってPHPMDのメッセージを持って、他の

Question

せずに動作するようにコードを単純化することができます

他

が必要になることはありませんし、この部分に他の せずに動作するようにコードを簡素化することができますコードの：

if ($settings == null) { 
     $settings = new self($arrSettings); 
} else { 
     $settings->fill($arrSettings); 
} 
$settings->save(); 

return $settings; 

私の質問は：どのように私は）（それ以外は避けるべきです。私が見る唯一の方法は、$setting->save()を複製して戻すことです。

Answer 1

おそらくそれは

if ($settings === null) { 
    $settings = new self; // or new self([]); 
} 
$settings->fill($arrSettings); 
$settings->save(); 

return $settings; 

ように書き換えることができるので、しかし、クラスのインスタンスは、自身のクレート新しいインスタンスにできないようにする必要がありので、TBH、全体の事は、SRPの一つの大きな違反のように見えます。それだけで意味がありません..しかし、再び、私は "職人"ではありません。