如何將布爾參數傳遞給函數以充當開關？

Question

如果標題不清楚，我很抱歉，但我覺得這很難描述。基本上，我有一個函數可以查找類（學校類）的實例，並給出班級ID號和日期。如果需要，該函數還可以創建一個新的類實例。

function get_class_instance($class_id, $date, $create) 

該函數內部使用類標識碼和日期作爲參數的class_instances數據庫中選擇表。如果找到匹配的class_instance，則返回其ID。如果沒有找到，則有一個條件可以使創建參數。如果是，則使用數據庫插入創建新的class_instance，並返回其ID。如果爲false，則數據庫中沒有任何更改，並返回false。

我還是一個初學PHP和一般編碼，所以我想這可能是一個更好的方法。問題是，在調用函數時，可能不清楚爲什麼有一個布爾值被傳遞。

$original_cinstance_id = get_class_instance($original_class_id, $original_date, 1); 

Answer 1

將函數布爾標誌傳遞給兩個不同的事物而不是一個，被認爲是的的Clean Code book。建議更好的選擇將是有兩個功能get_whatever和create_whatever。

雖然Code Smell取決於上下文，但我認爲布爾標誌的氣味適用於這種情況，因爲創建一些東西與僅僅讀取它不同。前者是命令，後者是查詢。 So they should be separated。一個改變狀態，另一個不改變狀態。它使更好的語義和separation of concerns拆分它們。此外，它將減少函數的Cyclomatic Complexity一個分支，所以您將需要一個單元測試來減少它。

報價https://martinfowler.com/bliki/FlagArgument.html

布爾參數大聲宣佈這個功能確實不止一兩件事。他們混亂，應該被淘汰。

這裏引用http://www.informit.com/articles/article.aspx?p=1392524

我的理由是，不同的方法傳達更清楚我的意圖是什麼，當我撥打電話。當我看到book(martin, false)時，我不需要記住標誌變量的含義，我可以輕鬆閱讀regularBook(martin)。

附加討論和閱讀材料：

• https://softwareengineering.stackexchange.com/questions/147977/is-it-wrong-to-use-a-boolean-parameter-to-determine-behavior
• https://medium.com/@amlcurran/clean-code-the-curse-of-a-boolean-parameter-c237a830b7a3
• https://8thlight.com/blog/dariusz-pasciak/2015/05/28/alternatives-to-boolean-parameters.html

Answer 2

而是傳遞一個數值，你可以通過這樣一個真正的布爾值：

$original_cinstance_id = get_class_instance($original_class_id, $original_date, true); 

另外在你的函數的聲明，你可以指定一個默認值，以避免傳遞布爾每次：

function get_class_instance($class_id, $date, $create = false) 

Answer 3

您可以使用創建一個默認值，所以如果你通過什麼，它充當數據庫的一個正常的「獲取」操作。就像這樣：

function get_class_instance($class_id, $date, $create = false); 

您可以查詢您的ID是這樣的：

$class_id = get_class_instance(1, "18-10-2017"); 

然後，您可以通過「真」到它時，你需要在數據庫中創建它：

$class_id = get_class_instance(1, "18-10-2017", true); 

Answer 4

一種選擇是創建一個枚舉器，如下所示：

abstract class create_options { 
    const no_action = 0; 
    const create = 1; 
} 

所以現在你的函數調用是這樣的：

$original_cinstance_id = get_class_instance($original_class_id, $original_date, create_options::no_action); 

在實踐

只要你的代碼有很好的註釋，那麼這是不是布爾標誌一個主要問題，但如果你有一打有可能的選擇不同的結果，那麼這可能是有用的。

正如其他人所提到的，在很多語言中，您還可以使參數成爲可選參數，並具有默認行爲，除非調用方明確定義了該參數。