Refactor: 객체 간의 협력 방식 개선

- TicketSeller가 Audience에게 티켓을 판매하도록 변경
- Audience가 Bag을 통해 티켓을 구매하도록 변경
- Bag이 초대장 소지 여부에 따라 티켓 구매 로직을 처리하도록 변경
- TicketOffice의 getTicket 메서드를 통해 티켓을 발급하도록 변경
- 관련 클래스들의 생성자를 간소화하고 접근 제어자 변경
- 이번 리팩토링을 통해 객체 간의 협력 방식을 개선하고, 불필요한 메서드 호출을 줄여 코드의 가독성과 유지보수성을 높였습니다.

Author: jihwooon

src/ticket/audience.ts

@@ -1,4 +1,5 @@
 import { Bag } from './bag';
+import { Ticket } from './ticket';
 
 export class Audience {
   private readonly bag: Bag;
@@ -7,7 +8,7 @@ export class Audience {
     this.bag = bag;
   }
 
-  get getBag(): Bag {
-    return this.bag;
+  buy(ticket: Ticket): number {
+    return this.bag.hold(ticket);
   }
 }

src/ticket/bag.ts

@@ -2,16 +2,9 @@ import { Invitation } from './invitation';
 import { Ticket } from './ticket';
 
 export class Bag {
-  private amount: number;
-
-  private invitation: Invitation;
-
   private ticket: Ticket;
 
-  constructor(invitation: Invitation, amount: number) {
-    this.invitation = invitation;
-    this.amount = amount;
-  }
+  constructor(private invitation: Invitation, private amount: number) {}
 
   public hasInvitation(): boolean {
     return this.ticket != null;
@@ -28,4 +21,17 @@ export class Bag {
   public plusAmount(amount: number) {
     this.amount += amount;
   }
+
+  public hold(ticket: Ticket) {
+    if (this.hasInvitation()) {
+      this.setTicket(ticket);
+      return 0;
+    }
+
+    this.setTicket(ticket);
+    this.minusAmount(ticket.getFee);
+    this.setTicket(ticket);
+
+    return ticket.getFee;
+  }
 }

src/ticket/theater.ts

@@ -2,21 +2,9 @@ import { Audience } from './audience';
 import { TicketSeller } from './ticketSeller';
 
 export class Theater {
-  private ticketSeller: TicketSeller;
-
-  constructor(ticketSeller: TicketSeller) {
-    this.ticketSeller = ticketSeller;
-  }
+  constructor(private ticketSeller: TicketSeller) {}
 
   public enter(audience: Audience): void {
-    if (audience.getBag.hasInvitation()) {
-      const ticket = this.ticketSeller.getTicketOffice.getTicket;
-      audience.getBag.setTicket(ticket);
-    } else {
-      const ticket = this.ticketSeller.getTicketOffice.getTicket;
-      audience.getBag.minusAmount(ticket.getFee);
-      this.ticketSeller.getTicketOffice.plusAmount(ticket.getFee);
-      audience.getBag.setTicket(ticket);
-    }
+    this.ticketSeller.sellTo(audience);
   }
 }

src/ticket/ticketOffice.ts

@@ -1,14 +1,7 @@
 import { Ticket } from './ticket';
 
 export class TicketOffice {
-  private amount: number;
-
-  private tickets: Array<Ticket>;
-
-  constructor(amount: number, tickets: Array<Ticket>) {
-    this.amount = amount;
-    this.tickets = tickets;
-  }
+  constructor(private amount: number, private tickets: Array<Ticket>) {}
 
   get getTicket(): Ticket {
     return this.tickets.pop();

src/ticket/ticketSeller.ts

@@ -1,13 +1,10 @@
 import { TicketOffice } from './ticketOffice';
+import { Audience } from './audience';
 
 export class TicketSeller {
-  private readonly ticketOffice: TicketOffice;
+  constructor(private ticketOffice: TicketOffice) {}
 
-  constructor(ticketOffice: TicketOffice) {
-    this.ticketOffice = ticketOffice;
-  }
-
-  get getTicketOffice(): TicketOffice {
-    return this.ticketOffice;
+  sellTo(audience: Audience) {
+    this.ticketOffice.plusAmount(audience.buy(this.ticketOffice.getTicket));
   }
 }