クラス内にたくさんのswitch文を使用する代わりに使用できますか？

Question

多くのswitch文を使用する代わりに使用できますか？

私は別のページで動作するクラスを持っていますが、同じことをします。 しかし、各ページには読み込むデータが異なります。 これは私がやっている方法です（色は単なる例に過ぎず、実際に私のアプリケーションにはありません）。 コンストラクタのページをチェックして、$ this-> page変数に入れます。それから私はこのような機能を持っています。

private function loadTable(){ 
    switch($this->page){ 
     case "green": 
      $check = $this->loadGreen(); 
      return $check; 
     break; 
     case "blue": 
      $check = $this->loadBlue(); 
      return $check; 
     break; 
     case "orange": 
      $check = $this->loadOrange(); 
      return $check; 
     break; 
    } 
} 

しかし、私はこのような何かをし、別の機能かもしれません：あなたは、私がswitch文の多くで終わるつもりです機能の多くと非常に素早く見ることができます

public function searchRecord(){ 
    $name = trim(strip_tags($_POST['term'])); 
    switch($this->page){ 
     case "green": 
      $query = "SELECT `green_ID`,Reason FROM `green`"; 
     break; 
     case "blue": 
      $query = "SELECT `blue_ID`,Reason FROM `blue`"; 
     break; 
     case "orange": 
      $query = "SELECT `orange_ID`,`colour`,`hat` FROM `orange`"; 
     break; 
     default: 
      return ""; 
    } 
    //rest of code removed for sanity 
} 

を、これは3ページ分です。何百枚もあれば

私はすべてのケースをすべてのページに持ち、変数を設定して私が現在いるページを言うようにすると考えましたが、その変数を確認するためにスイッチが必要なので機能しません。

私は関数を呼び出すことができ、その関数が私が現在いるページによって異なることをしたいと思っていますが、私の現在のシステムは、可能ならば減らしたいswitch文をたくさん用意しています。

これを行うには良い方法がありますか？ public function searchRecordGreen() をし、PHPで動作するはずです searchRecord$color() を経由して、それを呼び出す：

Answer 1

正しく実装OOPは、この問題の解決策です。それはsubtypingと呼ばれ、それは一種の多型です。

あなたはすべてのページタイプの機能を1つのクラスに詰め込んだので、たくさんのswitchステートメントで終わったのです。ページの種類の違いを特定し、機能ごとに1つの空の汎用関数を作成してください。次に、現在のページクラスを拡張し、各子クラスに1つのタイプのページに固有の機能を実装します。このような

何か：

abstract class Page 
{ 
    abstract public function loadTable(); 
    abstract protected function getSearchQuery($term); 

    public function searchRecord($term); 
    { 
     $query = $this->getSearchQuery($term); 
     // rest of search code 
    } 
} 

class GreenPage extends Page 
{ 
    public function loadTable() 
    { 
     // move the code of function Page::loadGreen() here 
    } 

    protected function getSearchQuery($term) 
    { 
     return "SELECT `green_ID`,Reason FROM `green`"; 
    } 
} 

class BluePage extends Page 
{ 
    public function loadTable() 
    { 
     // move the code of function Page::loadBlue() here 
    } 

    protected function getSearchQuery($term) 
    { 
     return "SELECT `blue_ID`,Reason FROM `blue`"; 
    } 
} 

次に、あなたはおそらくファクトリオブジェクトを作成する必要があります。

class PageFactory 
{ 
    public function createPage($name) 
    { 
     switch ($name) 
     { 
     case 'green': 
      return new GreenPage(); 
     case 'blue': 
      return new BluePage(); 
     case 'red': 
      return new RedPage(); 
     default: 
      // this either should never happen or you have a type of page as default 
     } 
    } 
} 


$factory = new PageFactory(); 
$page = $factory->createPage($name); 
$page->loadTable(); 
$page->searchRecord(trim($_POST['term'])); 

Answer 2

あなたはおそらく色に応じてクラス名書くことができます。

は、より具体的にするには：あなたがしたい名前を持つ関数を作成し、exessiveスイッチを保存するために、関数名として$this->pageを使用することができ

<?php 
class Foo 
{ 
    function Variable() 
    { 
     $name = 'Bar'; 
     $this->$name();  // This calls the Bar() method 
    } 

    function Bar() 
    { 
     echo "This is Bar"; 
    } 
} 

$foo = new Foo(); 
$funcname = "Variable"; 
$foo->$funcname(); // This calls $foo->Variable() 

?> 

：

Php.netは例があります文

あなたはちょうどこのようにそれを作成することにより、スイッチを使用せずにクエリを構築することができ

Answer 3

：

$query = "SELECT `".$this->page."_ID`,Reason FROM `".$this->page."`"; 

Answer 4

クエリを配列に格納して参照する方法はありますか？

public function searchRecord() { 
    $q = [ 
      'blue' => 'SELECT `blue_ID`, Reason FROM `blue`', 
      'orange' => 'SELECT `orange_ID`, `colour`, `hat` FROM `orange`' 
      //.... 
     ]; 
    if(array_key_exists($this->page, $q)) { 
     $query = $q[$this->page]; 

     //insert here the code you removed for sanity 
    } else { 
     //nothing found 
     return ""; 
    } 
} 

このようにして、各ページに1つのクエリをそれぞれ1行だけ定義することができます。
そのクエリ配列を別の場所に定義すると、コードを編集して1つのページを追加または編集する必要はありません。

編集：異なる機能のための別の方法

まず、あなたが奉仕したいページごとに一つの関数が含まれているプロバイダクラス定義：

class pageProvider { 
    public static function red() {} 
    public static function green() {} 
    //.... 
} 

をし、その後、あなたは明示的にあなたの呼び出しをマップしますそのクラスに：

直接入力コードを使用してcoul dはローミングを自由にするとセキュリティホールにつながりますが、定義されたクラスの公開されている静的メソッドへの呼び出しを明示的に制限しているため、攻撃者が突発して何か悪いことを行うことはできません。

も参照してください。

• is_callable
• call_user_func

Answer 5

は、あなたがこのような何かをしませんでした：

private function loadTable(){ 

    /* define what pages are acceptable*/ 
    $acceptablePages = array('green','red','blue'); 

    /* check if this->page is of an acceptable value*/ 
    if(in_array($this->page,$acceptablePages)) { 

     /* if it is acceptable then load a function with the page passed as a parameter*/ 
     return $this->loadColour($this->page); 
    } 

    return false; 

}