Wasze rozwiązania - 6 grudnia (OrderController)

[lotyszm]

Hej ! Przesyłam moje rozwiązania - co myślicie?

enum InvalidState {
  unknown = "unknown",
}

export class OrderController {
  private machines = new Set<Machine>();

  registerMachine(machine: Machine) {
    this.machines.add(machine);
  }

  unregisterMachine(machine: Machine) {
    this.machines.delete(machine);
  }

  setState(state: string) {
    if (state === InvalidState.unknown) {
      throw new Error("Invalid state provided");
    }
    this.machines.forEach((machine) => {
      machine.state = state;
      machine.addToLog(state);
    });
  }
}

export class Machine {
  private statesLog: string[] = [];

  state: string | null = null;

  performAudit() {
    return this.statesLog.map((state, index) => {
      return `Order #${index + 1} - ${state}`;
    });
  }

  addToLog(state: string) {
    this.statesLog.push(state);
  }
}

Testy na zielono 😁

[przeprogramowani]

Wygląda dobrze, brawo :) Jako improvement można byłoby użyć funkcji setState w Machine żeby logika biznesowa audytu nie wyciekała, poza tym cool :)

audit: string[] = [];
state: string = '';

setState(newState: ...) {
 //set state, register entry in audit
}

[lotyszm]

Racja, dzięki :D

[KP1976]

export class OrderController {
  machines: Machine[] = [];

  registerMachine(machine: Machine) {
    this.machines.push(machine);
  }

  unregisterMachine(machine: Machine) {
    this.machines = this.machines.filter((m) => m !== machine);

    if (this.machines.length === 0) {
      throw new Error("No machines registered");
    }
  }

  setState(state: string) {
    if (state === "unknown") {
      throw new Error("Invalid state provided");
    }

    for (const machine of this.machines) {
      machine.setState(state);
    }
  }
}

export class Machine {
  state: string | null = null;
  states: string[] = [];

  setState(state: string) {
    this.state = state;

    if (!this.states.includes(state)) {
      this.states.push(`Order #${this.states.length + 1} - ${state}`);
    }
  }

  performAudit(): string[] {
    return this.states;
  }
}