使用面向对象设计 PHP 避免 if 语句

Question

我基本上是为我创建的广告系统创建一个显示模块。

我试图避免使用重复的 if 语句进行以下构造。

我的直觉告诉我有一种更聪明的方法可以做到这一点，也许是通过多态性？

<?php

class Ad { 
    public $adState = 'active'; 
} 

class AdWriter { 
    public function displayAd(Ad $ad, $viewmode = 'visitor') { 
        if ($viewmode =='visitor') { 
            if ($adState == 'active') {} 

            else if ($adState == 'paused') {} 

            else if ($adState == 'inactive') {} 

        } 

        else if ($viewmode = 'owner') { 
            if ($adState == 'active') {} 

            else if ($adState == 'paused') {} 

            else if ($adState == 'inactive') {} 
        } 

        else if ($viewmode == 'administrator') { 
            if ($adState == 'active') {} 

            else if ($adState == 'paused') {} 

            else if ($adState == 'inactive') {} 
        } 
    } 
}  

?>

Answer 1

应用重构 将条件替换为多态性并查看状态模式。

Answer 2

您可以创建一个 工厂（模式），使用 viewmode 开关并创建实现 界面例如具有简单的功能“显示”。

伪示例：

class AdFactory { 

    public static function getAd($sType) {
        switch($sType) {
            case "AdOne":
                return new AdOne();
            case "AdTwo":
                return new AdTwo();
        }
    }
    throw new Exception("Unknown ad!");
}

class AdOne implement AdInterface {
    public function display() {
        // All that AdOne does when displaying.
    }
}

interface AdInterface {
    public function display() { }
}

$oAd1 = AdFactory::getAd('typeOne');
$oAd1->display();

$oAd2 = AdFactory::getAd('typeTwo');
$oAd2->display();

Answer 3

不要传递 $viewmode，而是传递一个封装此 viewmore 逻辑的对象，并调用其执行该工作的方法。这样您就可以避免使用 if 语句。

Answer 4

我在工作中偷偷访问 StackOverflow，所以没有时间写下所有可能性的详细回复。

但要“整理”这些 if，您可以这样做：

switch $viewmode {
  case 'visitor':
    your_code_here;
    break;
  case 'owner':
    your_code_here;
    break;
  default:
    will_run_if_nothing_above_matches;
    break;
}

Answer 5

  switch($viewmode){
    case "visitor":
        switch($adstate){
            case "active": 
                //statement
                break;
            case "paused": 
            break;
            case "inactive": 
            break;
        }
        break;
    case "owner":
        break;
    case "administrator":
        break;
}

Answer 6

在本书的第8章中，您可以找到问题的非常详细的答案。

简而言之：使用 组合 或工厂。 （参见韦斯利·范·奥普多普的回答）。

另外，避免使用字符串参数作为可枚举的：
$viewmode = '访客'
对于这个参数，您必须记住该参数的所有可能值。或者查看函数代码来记住它们。这些值是字符串 - 很容易出现拼写错误。此外，更改功能中的值将非常困难，因为此方法的所有调用都将包含硬编码字符串。
使用类常量：

class AdWriter { 
const view_mode_visitor = 1;
...

另外，$adState - 错误的代码，应该是 $ad->state。但使用公共字段也是不好的做法:)