forked from qiwsir/ITArticles
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
32 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
#腐败代码的警告 | ||
|
||
如果在代码中,存在下面的情况之一者,则说明代码需要清除腐败。 | ||
|
||
1、代码中存在许多大型的类和复杂的函数 | ||
|
||
2、函数的名称很晦涩或者会误导人。如果函数名称没有良好地反应函数的用途,那么会造成惊人的副作用。 | ||
|
||
3、没有任何结构:应该在哪里找某个功能非常不清晰。 | ||
|
||
4、内容重复:有许多相互独立的代码在做着同一件事情。 | ||
|
||
5、高耦合性:复杂的模块相互连接和依赖关系,将意味着某个地方的小小改动就会影响整个代码,甚至会影响看起来不相关的模块。 | ||
|
||
6、当数据流过系统时,它将在各个表示方法之间转化。 | ||
|
||
7、API变得模糊不清。由于考虑不周,增加了新功能,曾经整洁的接口现在在范围上变得宽泛。 | ||
|
||
8、API在不同代码版本中快速地变换。 | ||
|
||
9、公共API中泄露了部分私有实现,以便可以进行别的快速的修改。 | ||
|
||
10、代码中到处都是权宜之计:治标不治本的修改。它们隐藏了真正的问题。系统的外围尽是这种修改,结果导致缺陷隐藏在系统的核心。 | ||
|
||
11、有些函数的参数太多。这些函数有许多并不使用的参数,而只是将他们传递给下一级函数调用。 | ||
|
||
12、你发现代码太可怕了,以至于你都不想去修改它。你不知道它是会改善的,还是会轻微地破坏它,或是它更糟糕。 | ||
|
||
13、添加新功能时没有提供任何支持文档:现有的文档过时了。 | ||
|
||
14、你发现有的注释说:“不要动这段代码......” | ||
|