リファクタリング - 属性をセットに変換する-jsチュートリアル-php.cn

リファクタリング - 属性をセットに変換する

Barbara Streisand

リリース： 2024-10-20 11:33:02

オリジナル

608 人が閲覧しました

属性をセットに変換して不変性を優先する

TL;DR: 属性にセットを使用すると、コードが簡素化され、状態管理が容易になります

対処された問題

可変性
複雑さ
属性が汚染される
セッター

コードスメル 35 - プロパティとしての状態

マキシ・コンティエリ・ 2020年11月27日

#おっと＃チュートリアル #webdev ＃プログラミング

Refactoring - Convert Attributes to Sets

コードの匂い 28 - セッター

マキシ・コンティエリ・ 2020年11月19日

#おっと #コード初心者＃プログラミング #webdev

コードの匂い 34 - 属性が多すぎます

マキシ・コンティエリ・ 2020年11月25日

#おっと #コード初心者＃プログラミング

ステップ

状態を表す属性を特定する
属性をセットに置き換えます: 各状態に 1 つ
属性を変更するのではなく、セット間で項目を移動するようにメソッドを調整します

サンプルコード

前に

class Bill {
  amount: number;
  paid: boolean;

  constructor(amount: number) {
    this.amount = amount;
    this.paid = false;
  }

  pay() {
    if (!this.paid) {
      this.paid = true;
    }
  }
}

const bill = new Bill(100);
console.log(bill.paid); // false
bill.pay();
console.log(bill.paid); // true

ログイン後にコピー

後

// 1. Identify attributes representing states

class Accountant {  
   // 2. Replace the attributes with sets: one for each state
  unpaidBills: Set<Bill>;
  paidBills: Set<Bill>;

  constructor() {
    this.unpaidBills = new Set();
    this.paidBills = new Set();
  }

  addBill(bill: Bill) {
    this.unpaidBills.add(bill);
  }

  payBill(bill: Bill) {    
    // 3. Adjust methods to move items
    // between sets instead of mutating attributes
    if (this.unpaidBills.has(bill)) {
      this.unpaidBills.delete(bill);
      this.paidBills.add(bill);
    }
  }
}

class Bill {
  amount: number;

  constructor(amount: number) {
    this.amount = amount;
  }
}

const bill = new Bill(100);
const accountant = new Accountant();
accountant.addBill(bill);
console.log(accountant.unpaidBills.has(bill)); // true
accountant.payBill(bill);
console.log(accountant.paidBills.has(bill)); // true

ログイン後にコピー