【问题标题】:How can i improve these lines of code?我该如何改进这些代码行?
【发布时间】:2015-12-22 05:25:54
【问题描述】:

我有一个我认为可以改进的功能,代码非常幼稚。

public function isValid($data)
{
    $valid = true;

    $valid = parent::isValid($data) && $valid;
    $this->_errorsExist = !$valid;

    return $valid;
}

parent::isValid($data) 也会返回布尔值。

我通过删除 && 条件中的 $valid 来纠正它。

public function isValid($data)
{
    $valid = true;

    $valid = parent::isValid($data);
    $this->_errorsExist = !$valid;

    return $valid;
}

但我认为这仍然可以改进。任何建议将不胜感激。

【问题讨论】:

  • 可以去掉$valid = true;
  • 改进是什么意思? $valid 的初始分配在上述示例中没有任何作用。

标签: php if-statement boolean


【解决方案1】:

这段代码更中肯,但是否“改进”完全是主观的。

public function isValid($data) {
  $this->_errorsExist = !parent::isValid($data);
  return !$this->_errorsExist;
}

如果不查看所有代码,很难为您提供更多帮助。

【讨论】:

  • 我觉得这个主观上很好看。 +1
猜你喜欢
  • 1970-01-01
  • 2022-12-31
  • 1970-01-01
  • 2019-05-24
  • 2022-01-03
  • 2016-02-16
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多