什麼最簡單的可避免的醜陋的代碼問題？

Question

我的第一個想法是在一個方法中有太多的代碼行。在大多數情況下，至少可以很容易地將某些東西分解成多個方法。

我還有更多，但似乎是最簡單的避免之一。它真的讓我看到有1000行代碼的方法，這似乎也是一個常見問題。任何你需要向上/向下滾動的地方是IMO應該要求你將它分開。

但是，列出問題可能會幫助很多新開發人員學習識別問題並提供一些良好實踐。

Answer 1

評論代碼：P

但嚴重的是，是的，過長的方法是一個噩夢。不正確的代碼重用，或完全缺乏，可能是我最大的煩惱。

Answer 2

我最喜歡的寵物尿道有許多級別的縮進，即大量的嵌套ifs和循環。

比滾動向上和向下更糟的是必須側身滾動！

我的經驗法則通常是5級MAX。還有，他們應該被重構成新的方法。

Answer 3

不使用異常進行錯誤管理。使用異常非常簡單，而且很難檢查每個函數的返回碼（並忘記了其中的一半）。

Answer 4

我同意你的意見。對於優秀的代碼來說，長方法是最簡單的避免和必不可少的。我的經驗法則是，這種方法應該是一件顯而易見的事情。作爲一個經驗法則，如果另一個開發人員在閱讀方法名稱後不得不問一下某個方法的作用，那麼這個方法做得太多了。當然，它也可能只是一個壞名字:)

Answer 5

SQL注入是如此容易aviodable，我不明白爲什麼它曾經在屏幕上顯示。

而且它也很醜。

而我非常喜歡用引號引用或替換語句中的INSERT/UPDATE/DELETE。該死的，只是使用佔位符！

Answer 6

單一責任原則，這麼容易，但經常不使用。