qq_42889517的博客

一个方法频繁访问其他对象的数据，比访问自己对象的数据还多。

提取变量提取方法重命名方法引入断言内联类合并继承结构封装字段封装集合提取方法移动方法移除设置方法隐藏方法删除参数内联类折叠继承结构。

当你修改一个类的时候，发现这个类里很多不相关的方法都要改，就说明这个类的职责太杂了！🧠 举个例子：你加一个产品类型，却要修改同一个类中用于查找、显示、下单的多个方法。你要加个新功能或改个逻辑，结果需要同时改好几个类里的一小部分，像打霰弹枪一样“到处改”。🧠 举个例子：想改个价格逻辑，结果 Product 类、Cart 类、Order 类都要动。当你给某个类添加一个子类时，发现另一个相关的类也必须添加对应的子类，两个继承结构就像“并排走路”。Animal->CatDog同时->只要新增Fish。

如果你写了很长的或if-else结构，很容易出问题。每加一种判断条件，就要改很多地方，难维护。有些类里的变量，只在特定情况下有值，大多数时候是空的。这些就是“临时字段”。你写了一个子类，它继承了父类，但只用了父类的一部分功能，很多继承的方法根本没用上，甚至要手动重写。你发现两个类做的事一模一样，但方法名不一样。坏味道表现怎么办Switch 语句写很多 case 分支用“子类”或“策略模式”代替临时字段变量只偶尔才用拆到新类里，或用空对象拒绝继承子类没用多少父类的东西。

一个函数（方法）写得太长了，比如超过了10 行代码，就要开始警惕了。一个类（类似于一块代码模块）里面包含太多变量、方法、代码行。就是你过度使用 int、string 这些基本类型，而不愿意封装成对象。电话号码用 string，金额用 float，而不是专门建个或Money类。用常量1, 2, 3来表示权限等级，而不是定义角色类。如果一个方法需要传超过 3~4 个参数，那就太多了。如果你在多个地方看到总是成组出现的变量。