# 은행창구 매니저 [STEP 3] Comment</br>

---

## 보완되면 좋을 부분</br>

- 우선 현재 실제 구동시켜보면 딜레이가 되지 않고 1초만에 바로 끝나게 됩니다ㅠ 다만 업무 처리 총 시간과 고객 수 등 결과 프린트는 정상적으로 찍히는데 실제로 업무 처리되는 부분이 뭔가 이상한듯 해요🥲 한번 런 시켜서 확인해보시면 좋을듯 합니다~!

---

### 1


✏️ 답변</br>

- `return`으로 변경 진행했습니다. 🙌 값이 존재하지 않는데 `continue`를 타면 문제가 있습니다. 그 이유는 어떤 업무를 처리해야 할지 모르기 때문입니다. 즉, "예금" 업무인지, "대출" 업무인지를 판별할 수 없습니다.🥲

---

### 2


✏️ 답변</br>

<img src = "https://github.com/devKobe24/images/blob/main/CustomerGithub.png?raw=true">

- 프로젝트 파일에서 코드를 봤을 때는 두 프로퍼티의 띄어쓰기 컨벤션이 정상적입니다 어떤 문제일까요 ?? 🥲

---

### 3


✏️ 답변</br>

- `TimeCheck`메서드에서 `guard let` 시에 `else`를 개행했었는데요 한줄에 100자가 넘을꺼같아 개행을 진행했었습니다. 이로 인해 `bankingServiceTimeConverter`에서도 개행을 진행했었습니다. 지금 확인해보니 `TimeCheck`에서 100자가 안되고 97자가 되기에 `guard let` 사용 부분에 있어 컨벤션을 맞추기위해 else을 개행 하지 않고 진행하고자 변경 했습니다!

---

### 4


✏️ 답변</br>

 for i in 1...Int.random(in: 10...numberOfCustomers){}
private var bank = BankService(numberOfCustomers: 30)

- 이렇게 고객수를 랜덤하게 정하여 업무를 진행 할수있게 변경하였습니다.

---

### 5


✏️ 답변</br>

<img src = "https://github.com/devKobe24/images/blob/main/EnumErrorCase.png?raw=true">

- Apple의 Docs를 참고하여 에러 케이스 명을 `convertedString`으로 수정해봤습니다. 그 이유는 `NumberFormatError`이 `Error`임을 이미 명시해주고 있으므로 케이스 명까지 `Error`이 들어가면 중복적인 단어가 쓰이는 것 같아 위와 같이 수정했습니다. 따라서 에러를 던져줄 때 `throw NumberFormaterError.convertedString` 과 같은 형태가 됩니다.🙌

---