《代码整洁之道》中示例为何属于Feature Envy代码异味？

特性羡慕的核心问题是：一个类过度依赖另一个类的内部细节（字段、方法），本该属于被依赖类的核心逻辑，却被放到了外部类中，既违背了封装原则，也不符合单一职责。

我们来拆解示例代码的问题：

• HourlyPayCalculator的calculateWeeklyPay方法，所有计算逻辑完全依赖HourlyEmployee的tenthRate和tenthsWorked数据，它需要反复调用HourlyEmployee的getter方法来获取内部信息才能完成计算。
• 计算周薪的逻辑，本质上是HourlyEmployee自身应该具备的能力——员工的薪资计算规则和自身的工作时长、时薪息息相关，理应由HourlyEmployee自己负责，而不是交给一个外部的“计算器”类来处理。
• 这个外部计算器类就像“羡慕”HourlyEmployee拥有这些核心数据，硬生生把本该属于员工类的逻辑抢了过来，导致HourlyEmployee只负责存储数据，丢失了自己的行为能力，同时HourlyPayCalculator也因为过度依赖其他类的内部细节，变得耦合度极高。

如果重构消除这个异味，应该把计算逻辑移到HourlyEmployee内部：

public class HourlyEmployee {
    private Money tenthRate;
    private int tenthsWorked;

    public Money calculateWeeklyPay() {
        int tenthRatePennies = tenthRate.getPennies();
        int straightTime = Math.min(400, tenthsWorked);
        int overTime = Math.max(0, tenthsWorked - straightTime);
        int straightPay = straightTime * tenthRatePennies;
        int overtimePay = (int)Math.round(overTime * tenthRatePennies * 1.5); 
        return new Money(straightPay + overtimePay);
    }

    // 按需保留getter方法
}

重构后，HourlyEmployee对自己的薪资计算逻辑负责，符合封装原则，HourlyPayCalculator也可以被移除（或承担更通用的薪资计算协调职责），代码的内聚性和可维护性都会提升。

内容的提问来源于stack exchange，提问作者shirin monzavi