PHP OOP 构造函数设计模式

Question

我有这个代码。它有效，但有可能做得更好吗？用更少的代码？

public function __construct($type ,$brand, $model, $year, $mileage, $height, $width, $depth, $ps, $color, $price, $value, $seats)
{
    if (is_int($type)) {
        $this->_type = $type;
    } else {
        throw new Exception('Wrong Value in Type. Expected integer');
    }
}

...

我创建一个这样的对象

try {
    $firstCar = new Car(1, 'Audi', 'A3', 2011, 94512, 2, 2, 2, 105, 'Black', 1000, 1920, 5);
} catch ( Exception $error) {
    die($error->getMessage());
}

我对 2 个类中的每个变量重复“IF-ELSE”语句。它的代码太多了。

这里有一些建议吗？

Answer 1

代码不多，就是乱七八糟。

可能问题是在一个地方有太多东西，将值对象中的属性“类型”分组，然后这些值对象中的每一个都会处理它们的“验证”

示例：

• 高度、宽度、深度应为尺寸类
• 类型、品牌、型号、年份应按CarModel
• 价格，价值应该去价格
• ...

或类似的东西，你应该知道它们之间的关系

会导致：

function __construct(CarType $model, Size $size, Price $price, $seats)
{
    // no need to validade CarType did that
    $this->model = $model;
    ....
}

为了帮助创建对象，您可以使用工厂或构建器

Answer 2

为了检查值是否为整数，您可以在回调方式中使用 filter_var 或者您可以将数组作为参数传递并按以下方式使用 filter_var_array 函数， 以下是我的示例代码：

$firstCar = new Car(1, 'Audi', 'A3', 2011, 94512, 2, 2, 2, 105, 'Black', 1000, 1920, 5);

public function __construct($type ,$brand, $model, $year, $mileage, $height, $width, $depth, $ps, $color, $price, $value, $seats) {
$arrInput = func_get_args();

// this is enough to check only all arguments are integer or not by comparing the counts by input and filter array
$arrFilter = array_filter(filter_var_array($arrInput,FILTER_VALIDATE_INT));

 if (count($arrFilter) < count($arrInput)) { // means there should be atleast one non-integer value
    // to get the exact values which are exaclty not interger
    $arrResult = array_diff($arrInput, $arrFilter);
    throw new('your error'); // or return $arrResult; // this return Audi, A3 and Black
 }

 }

您可以根据需要使用 try/catch 通过从构造函数中抛出错误来捕获错误，或者通过从构造函数返回 $arrResult 来获取非整数的结果值。

Answer 3

我以不同的方式解决了问题。

$this->setType($type);

protected function setType($worth)
{
    if (is_int($worth)) {
        $this->_type = $worth;
    } else {
        throw new Exception('Wrong Value in Type. Expected valid integer');
    }
}

这对于我传递给类的每个参数。

对于那些对此感兴趣的人。不是最好的解决方案，但它会缩短子类。

Answer 4

假设，每个字段都必须是int：

...

private function checkArgument($arg) {
    if (is_int($type)) {
        $this->_type = $type;
    } else {
        throw new Exception('Wrong Value in Type. Expected integer');
    }
}

...

public function __construct($type ,$brand, $model, $year, $mileage, ...) {
    checkArgument($type);
    checkArgument($brand);
    ...
}

您也可以使用func_get_args()，获取数组中的所有参数，然后循环它们。