我的第一个想法是方法中的代码行太多了。在大多数情况下，至少可以很容易地将某些东西拆分成多种方法。
我还有更多，但这似乎是最容易避免的事情之一。看到一个包含1000行代码的方法真的让我感到困惑，这似乎也是一个常见的问题。你需要向上/向下滚动的任何东西都是IMO应该乞求你把它分开。
但是列出一个列表问题可以帮助很多新开发人员学会识别问题并提供一些好的实践。
答案 0 :(得分：3)
我最喜欢的烦恼是有很多级别的缩进，即大量的嵌套ifs和循环。
比向上滚动更糟糕的是必须横向滚动！
我的经验法则通常是5级MAX。任何更多，他们应该被重构为新方法。
答案 1 :(得分：1)
代码中的注释：P
但严重的是，是的，过长的方法是一场噩梦。不正确的代码重用，或完全缺乏，可能是我最大的烦恼。
答案 2 :(得分：0)
不使用异常进行错误管理。使用异常是如此简单，如此难以检查每个函数返回代码（并忘记其中的一半）。
答案 3 :(得分：0)
我同意你的看法。长方法是最简单的方法，对于良好的代码至关重要。我的经验法则是，该方法的作用应该是一个显而易见的事情。如果另一个开发人员在阅读方法名称后必须询问方法的作用，根据经验，这样做太多了。当然它也可能只是一个坏名字：）
答案 4 :(得分：0)
SQL注入非常易于使用，我不明白为什么它会在屏幕上显示。
它也很难看。
我对它的非解决方案感到震惊，比如在语句中引用引号或替换INSERT / UPDATE / DELETE。该死的，只需使用占位符！
答案 5 :(得分：0)
单一责任原则，如此简单却经常不使用。