2

我有一个从数据库加载事物列表并将它们放入选择列表的函数。函数如下：（伪代码）

 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 给出了一个代码提示，它有太多的行和太多的嵌套块。个人觉得功能应该可以接受。我也觉得将逻辑分解成更小的函数有点疯狂，但为什么 netbeans 会骗我:)？

所以我的问题如下：

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

编辑：

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

编辑2：

删除 try catch 块就像抢彼得付钱给保罗一样。因此，如果在创建 pdo 实例时抛出异常，它不会停止执行。因此，我们尝试在未正确初始化的 PDO 对象上调用 prepare 语句。这迫使我们在准备调用之前进行另一个测试，因此只是返回到原始函数的返工。

以我的经验，这意味着我的逻辑在某个地方被打破了。如果我有什么值得说的，我将回顾我的设计并回击。

再次感谢大家

4

2 回答 2

7

你的代码很好。NetBeans 建议的内容不一定是您应该遵循的规则，如果您使用 PHPStorm 等其他编辑器，您甚至不会担心（在 PHPStorm 中，您可以设置编码样式以遵循 PSR 1/2）。

您至少可以通过使用称为保护子句的东西来消除一个嵌套：

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
     }        
}

我见过人们有不同的偏好，因为没有硬性规定。例如 Anthony Ferrara个人认为他不应该超过四个嵌套级别，我个人认为四个级别太多了。

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

您可能还想看看 William Durand 的这篇不错的文章，关于这个主题，他讨论了 Jeff Bay 在他的《ThoughtWorks Anthology》一书中提出的几个（实际上是 9 个）建议

对象健美操

PHP Coding Standards Fixer也是你的朋友。

所以：

  • 您当前的代码非常好
  • 创建您的个人偏好并尽可能减少嵌套和行数。
于 2015-05-12T11:14:53.487 回答
0

如果条件不成立，您可以反转 if 语句并退出函数以消除一级嵌套。不过，逻辑看起来也不错。

于 2015-05-12T09:57:36.713 回答