foreachでクリーンコードでメソッドを呼び出す

Question

私はDomDocumentを使ってHTMLページにアクセスし、オブジェクトに変換しますが、別のタグでノードを検索する必要があるたびに呼び出されます。私は、ロジック名をパラメータとして持つメソッドを必要としますが、各ロジックには異なる数のパラメータがあります。

public function foo(){ 
    $this->findTag('bar', $parameters); 
} 
public function findTag($logic, $parameters){ 
    foreach ($this->dom->getElementsByTagName($this->tag) as $node) { 
     $this->$logic($node, $parameters); 
    } 
} 
public function logic1($node, $foo, $bar){ 
    //something with $foo and $bar 
} 
public function logic2($node, $fooBar){ 
    //something with $fooBar 
} 

これは悪いアプローチですか？このforeachを、別のタグを使用して、DomObjectからの情報が必要なときに常に呼び出される一意の関数にするにはどうすればよいですか。 @JustOnUnderMillionsコメントの後

作品

class pei 
{ 
    public function classe($nome) 
    { 
     $this->findTag('logic1', $nome); 
    } 

    public function classe2($porta, $janela) 
    { 
     $this->findTag('logic2', $porta, $janela); 
    } 

    public function findTag($logic, ...$parameters) 
    { 
     $array = [1, 2, 3, 4]; 
     print_r($parameters); 
     foreach ($array as $node) { 
      $this->$logic($node, $parameters); 
     } 
    } 

    public function logic1($node, $nome) 
    { 
     print_r($node); 
     print_r($nome); 
    } 

    public function logic2($node, $porta) 
    { 
     print_r($node); 
     print_r($porta[0]); 
     print_r($porta[1]); 
    } 
} 
(new pei)->classe('pablito'); 
(new pei)->classe2('janela1', 'janela2'); 

これは、これをコーディングするクリーン/ドライストラテジーのですか？私は重複していくつかの問題の後で学びたい。

ここ

Answer 1

は、私はあなたの質問から取る ルールは、私からの例です：

class pei 
{ 
    /** 
    * @param $logic array|string [logicname,tagname] or "logicname" 
    * @param $arguments the argument list for the logic 
    **/ 
    public function findTag($logic,...$arguments) 
    { 
     if(is_array($logic)){ 
      list($logic,$tag)=$logic; 
     } else { 
      $tag=$this->tag;//Note: your example dont show where this is coming from!? 
     } 

     if(!method_exists($this,$logic)){ 
      throw new Exception("Logic '$logic' not found!"); 
     } 

     $result=array(); 
     //Note: Is also unclear where and how $this->dom is set!? 
     foreach ($this->dom->getElementsByTagName($tag) as $node) { 
      //here we expand the args to $arg0,$arg1,... 
      $result[]= $this->{$logic}($node,...$arguments); 
     } 

     return $result; 
    } 

    //logic definitions 
    public function logic1($node, $arg0){} 
    public function logic2($node, $arg0, $arg1){} 
    public function logic3($node, $arg0, $arg1, $arg2){} 
} 

異なるタグを使用してパラメータととして

の論理名今、我々はできる一つの方法を持っていますパラメータの設定に応じて、他のメソッドに委譲します。

$result = (new pei($Dom))->findTag('logic1','pablito'); 
$result = (new pei($Dom))->findTag('logic2','janela1', 'janela2'); 
$result = (new pei($Dom))->findTag(['logic2','tagname'],'janela1', 'janela2'); 

• だから今、あなたは直接ロジックメソッドを呼び出すかfindTag()メソッドを使用することができます。
• あなたはあなたがlogicname &タグ名のみlogicname

とメソッドを使用することができます。しかし、これはすべて行うことができるもののためだけの例である

を必要とするなど、多くのロジック・メソッドを定義することができます。私はこれを別のやり方で行い、現時点では、ここでどんな種類の論理が必要かを完全には知りません。私の例では、メソッドの呼び出しを異なるパラメータのサブメソッドに処理してデリゲートする方法のみを示しています。

例には、エラー処理のようなサーバーの機能がないことに注意してください。

これは方向を示します。