PHP最佳实践：给定参数是否应始终具有一致类型？

Question

我有一个函数接受checkGlossary bool param以及一个可选的glossary数组。
他们的州直接捆绑在一起。
如果bool为FALSE，则永远不需要词汇表，反之，如果bool为TRUE则始终需要词汇表。

对我来说，似乎可以很容易地简化：

// Current
function doSomething($param1, $param2, $checkGlossary=FALSE, $glossary=NULL){
    // blah blah blah
    if($checkGlossary)
        array_search($glossary[$param2]);
    // etc etc etc
}

... 至：

// Proposed
function doSomething($param1, $param2, $glossary=FALSE){
    // blah blah blah
    if($glossary)
        array_search($glossary[$param2]);
    // etc etc etc
}

我唯一的犹豫是因为$glossary （ bool或array ）的类型是不可预测的。
只要我不违反一些最佳实践指南，它就不会打扰我。

思考？

Answer 1

您可以使用is_array()和is_bool()来检查其类型！

Answer 2

使用PHP调用mixed数据类型的函数参数总是一个坏主意。 它需要函数中的附加代码来检查参数的类型，显然它会变得非常混乱。

在您的特殊情况下，最简单的解决方案可能是使用数组长度作为是否使用词汇表代码的指示器。 您需要一种方法来声明不应使用词汇表数组。 所以你应该问自己：什么时候使用术语表是没有意义的？ 当它是空的当然。 所以，我建议您摆脱该标志并将array()定义为其他参数的默认值：

function doSomething($param1, $param2, $glossary=array()) {
    if (count($array) > 0) {
        // do your glossary code here
    }
    // all the other stuff goes here
}

对我来说，这似乎在语义上是正确的，并且工作正常。

我不知道你在那里构建了什么，但另一种解决方案是将它全部放入一个类中并将词汇表作为实例变量。 如果您可以在多个函数调用中使用词汇表。 它大概如下：

 public class SomeAccurateClassName {
     private $glossary = array();

     function setGlossary(array $glossary) {
         $this->glossary = $glossary;
     }

     function doSomething($param1, $param2) {
         if (count($array) > 0) {
             // do your glossary code here
         }
         // all the other stuff goes here
     }
 }

考虑到你基本上有一个状态（使用词汇表或不使用词汇表），将一个类封装起来可能是一个好主意。