TL;DR：用对象交互代替静态函数。

解决的问题

相关代码异味

步骤

示例代码

前

class CharacterUtils {
    static createOrpheus() {
        return { name: "Orpheus", role: "Musician" };
    }

    static createEurydice() {
        return { name: "Eurydice", role: "Wanderer" };
    }

    static lookBack(character) {
      if (character.name === "Orpheus") {
        return "Orpheus looks back and loses Eurydice.";
    } else if (character.name === "Eurydice") {
        return "Eurydice follows Orpheus in silence.";
    }
       return "Unknown character.";
  }
}

const orpheus = CharacterUtils.createOrpheus();
const eurydice = CharacterUtils.createEurydice();

后

// 1. Identify static methods used in your code.
// 2. Replace static methods with instance methods.
// 3. Pass dependencies explicitly through
// constructors or method parameters.

class Character {
    constructor(name, role, lookBackBehavior) {
        this.name = name;
        this.role = role;
        this.lookBackBehavior = lookBackBehavior;
    }

    lookBack() {
        return this.lookBackBehavior(this);
    }
}

// 4. Refactor clients to interact with objects 
// instead of static functions.
const orpheusLookBack = (character) =>
    "Orpheus looks back and loses Eurydice.";
const eurydiceLookBack = (character) =>
    "Eurydice follows Orpheus in silence.";

const orpheus = new Character("Orpheus", "Musician", orpheusLookBack);
const eurydice = new Character("Eurydice", "Wanderer", eurydiceLookBack);

类型

[X] 半自动

您可以逐步进行替换。

安全

这种重构通常是安全的，但您应该彻底测试您的更改。

确保代码的其他部分不依赖于您替换的静态方法。

为什么该代码更好？

您的代码更易于测试，因为您可以在测试期间替换依赖项。

对象封装行为，提高内聚力并减少协议过载。

您可以删除隐藏的全局依赖关系，使代码更清晰，更易于理解。

使用 AI 进行重构

标签

相关重构

参见

致谢

该图片由Menno van der Krift在Pixabay上发布

本文是重构系列的一部分。