gpt4 book ai didi

php - 这是太多行和太多嵌套 block 了吗?

转载 作者:行者123 更新时间:2023-12-01 22:29:35 24 4
gpt4 key购买 nike

我有一个函数,可以从数据库中加载事物列表并将它们放入选择列表中。函数如下:(伪代码)

 protected function Foo() 
{
try {

get pdo instance
prepare statement

if (pdo query executes)
{

while (row = fetched rows)
{
do stuff with row
}
}
}
catch (PDOException $ex)
{
do error stuff here
}
}

NetBeans 给出代码提示,提示行数过多,嵌套 block 过多。个人觉得功能应该可以接受。我还觉得将逻辑分解成更小的函数有点疯狂,但为什么 netbeans 会骗我:)?

所以我的问题如下:

这是错误的逻辑还是我可以继续?我欢迎任何关于如何重新设计函数以适应 NetBean 约束的建议。

编辑:

我不会回答我自己的问题,但在这种情况下,有一个不需要的嵌套 block 。 pdo 是从具有 try/catch block 的单例类中检索的。我不需要在此函数中再次重复它,因为异常已经被捕获。

编辑 2:

删除 try catch 代码块就像是以盗窃来支付 Paul。因此,如果在创建 pdo 实例时抛出异常,它不会停止执行。因此,我们尝试在未正确初始化的 PDO 对象上调用 prepare 语句。这迫使我们在 prepare 调用之前进行另一个测试,从而返回到原始函数的返工。

根据我的经验,这意味着我的逻辑在某处被打断了。如果我有什么值得说的话,我将回顾一下我的设计并打招呼。

再次感谢大家

最佳答案

你的代码很好。 NetBeans 建议的不一定是您应该遵循的规则,如果您使用 PHPStorm 等其他编辑器,您甚至不必担心(在 PHPStorm 中,您可以将编码风格设置为遵循 PSR 1/2)。

你至少可以通过使用一种叫做 guard clause 的东西来消除一个嵌套。 :

protected function Foo() 
{
try {
get pdo instance
prepare statement

if (! pdo query executes) return;

while (row = fetched rows)
{
do stuff with row
}

}
catch (PDOException $ex)
{
do error stuff here
}
}

我看到人们有不同的偏好,因为没有硬性规定。例如安东尼·费拉拉 personally thinks他不应该超过四个嵌套级别,我个人认为四个级别太多了。

重点是您应该尽可能减少嵌套和行数。当您的方法太大(有时称为上帝方法)时,这意味着您做错了。

您可能还想看看威廉·杜兰德 (William Durand) 撰写的这篇精彩文章,其中他讨论了杰夫·贝 (Jeff Bay) 在他的书中提出的几个(实际上是 9 个)建议 ThoughtWorks Anthology

Object Calisthenics

还有 PHP Coding Standards Fixer是你的 friend 。

所以:

  • 您当前的代码完全没问题
  • 创建您的个人偏好并尽量减少嵌套和行数,其中 possible .

关于php - 这是太多行和太多嵌套 block 了吗?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/30187135/

24 4 0
Copyright 2021 - 2024 cfsdn All Rights Reserved 蜀ICP备2022000587号
广告合作:1813099741@qq.com 6ren.com