一种方法有多个参数还是多种方法有一个参数？

Question

想象一下用于跟踪对手之间游戏的“游戏”类。使用 1 种基于用户输入参数检索游戏的方法是 OOP 更好，还是使用特定于检索目标的多种方法更好？

class Games {
  function get_games($game_id = NULL, $stadium_id = NULL, $start_date = NULL,   
      $end_date = NULL, $count = 999); {}
}

VS

class Games {
  function get_all_games($count = 999); {}
  function get_game_by_id($game_id = 1); {}
  function get_games_by_stadium($stadium_id = 1); {}
  function get_games_by_dates($start_date = NULL; $end_date = NULL) {}
}

好处的解释和任何编码/snytax 技巧将不胜感激。谢谢。

Answer 1

我练习 OOP 的次数越多，我就越发现自己遵循将参数传递给方法的规则。有点像有很多级别的嵌套 if 语句，我发现如果我有两个以上，我可能会做错什么。

保持代码简单。您正在编写一个执行某些操作的方法，而不是执行所有操作的过程代码块。如果你想玩游戏，那就玩游戏吧。如果您想获取某个日期范围的列表，那么就这样做。

不过我想指出的是，您实际上并不需要 get_all_games() - 您可以只允许 get_games_by_dates() 不带参数地传递。如果它没有得到任何，那么它会得到从永远开始的每个日期的游戏（所有游戏）

Answer 2

我总是倾向于 OOP 代码。原因是它使您的代码更易于维护和阅读。拥有的功能越多，以后执行代码就越容易

Answer 3

“特定于检索目标的多种方法”的好处是您可以添加/删除目标。使用带有一堆参数的单一函数的问题在于，如果您决定添加/删除一种获取游戏的方式，则必须更改界面。这会破坏任何使用它的代码。

每种方法应尽可能简洁，仅执行一项功能。

Answer 4

我会选择单独的方法，因为您使用了大量具有默认值的参数。

如果你想获得所有游戏，你必须这样做：

$games->get_games(NULL, NULL, NULL, NULL, 999);

Answer 5

假设您的 get_....() 函数返回所有游戏数据，我会根据传入的 id 编写一个函数来返回此数据，并编写一系列 find_...() 函数返回找到的 id 数组。这将带来额外的好处，即可以更轻松地重写后代类中的数据检索代码。

class Games {
    public function get_game($game_id) {
        // Return game details (array/object) for $game_id, or FALSE if not found.
    }

    public function find_all_games() {
        // Return array of ids for all games.
    }

    public function find_games_by_dates($start_date = NULL, $end_date = NULL) {
        // Return array of ids between $start_date and $end_date unless NULL.
    }
}

然后您可以致电：

$oGames = new Games() ;
$aGames = $oGames->find_all_games() ;
foreach($aGames as $id) {
    $aGame = $oGames->get_game($id) ;
    if($aGame !== FALSE) {  // This check might be skipped if you trust the array of ids from find_all_games().
        // Assuming an array is returned.
        echo "Game Found: ".$aGame['name']."\n" ;
    }
}