idinaloq와 Hemg 팀이 만든 쥬스 메이커 프로젝트 입니다.
-
쥬스 메이커는 원하는 맛의 쥬스를 만들고, 원하는 과일 재고 수량을 변경할 수 있습니다.
-
주요 개념:
Initialization,Access Control,Nested Types,Type Casting,Error Handling
| idinaloq | Hemg |
|---|---|
 |
 |
| Github Profile | Github Profile |
| 쥬스 메이커 |
|---|
 |
| 쥬스 주문 |
|---|
 |
| 쥬스 주문시 재고 부족 |
|---|
 |
| 재고 수정 |
|---|
 |
| 날짜 | 내용 |
|---|---|
| 2023.05.08. | 팀 룰, 쥬스 메이커 구현 고민 |
| 2023.05.09. | FruitStore 객체, JuiceMaker 객체 생성 |
| 2023.05.10. | FruitStore 개방폐쇄원칙 피드백 진행 |
| 2023.05.11. | 전반적인 코드 리팩토링 |
| 2023.05.12. | PR, 피드백 코멘트 작성 및 README 작성 |
| 2023.05.15. | 두번째 화면 생성기능, 화면 전환기능, 알럿컨트롤 추가 |
| 2023.05.16. | 쥬스 주문 시 재고에 수량 반영, 주문완료 또는 재고 부족 시 알럿추가 |
| 2023.05.16. | FruitStore의 stock은닉화 후 재고수량 받아오는 메서드 추가 |
| 2023.05.18. | 피드백 반영하여 코드 리팩토링 진행 |
| 2023.05.19. | 코드 리팩토링 완료, README |
| 2023.05.22. | 두번째 화면 과일수량 반영, 과일별 StepperAction 함수 추가 |
| 2023.05.24. | 함수 네이밍 변경, [fruit: UIlabel] 중복사용으로 인해 전역 변수 변경(lazy활용) |
| 2023.05.25. | orderJuice() 쥬스주문 방식switch문에서 -> guard 문으로 변경 showOrderedAlert() 알림표시창 if -> guard로 변경 |
| 2023.05.26. | 코드 리팩토링 완료, README |
- 기존 과일 재고에 대해 저장 하기 위해 저장 프로퍼티를 사용하여 하나의 저장 프로퍼티에 하나의 값을 저장 하여 진행 했었는데 이부분에 있어 조금더 효율적으로 진행하고자 하여
Int->Dictionary로 변경하여 재고를 저장 하게 되었습니다.
class FruitStore {
var strawBerry = 10
var banana = 10
var pineApple = 10
var kiwi = 10
var mango = 10
} var stock: [Fruit: Int] = [
.strawBerry: 10,
.banana: 10,
.pineApple: 10,
.kiwi: 10,
.mango: 10
]- 기존 Juice 타입에서 Juice 주문시에 주문에 맞게 재고를 깍는 개념으로
decreaseStock()를 실행 시키는 함수였습니다. 하지만 이렇게하게 되면FruitStore에서Juice까지 알아야 하기에 문제점이 발생 할 수있기에 타입을 변경 했어야 했습니다.
- 결합도를 낮추기 위해 하나의 객체가 여러개의 일을 하면 문제가 될 수있다고 생각을 했으며 1,2,3,4,5 에서 1=2, 2=3, 3=4, 4=5 이런식으로 단일 책임을 갖게 되면 좋을꺼 같아 타입을 변경하여 진행 하게 되었습니다.
func decreaseStock(with juice: Juice) {
let recipe: [Fruit: Int] = juice.recipe
for (fruit, quantity) in recipe {
if let stock = self.stock[fruit] {
self.stock[fruit] = stock - quantity
}
}
}- 기존
Juice타입에서 ->Fruit타입으로 변경 하여decreaseStock()함수를 재구현 했습니다.
func decreaseStock(with fruits: [Fruit: Int]) {
for (fruit, quantity) in fruits {
if let stock = self.stock[fruit] {
self.stock[fruit] = stock - quantity
}
}
}- 기존에 각각의 쥬스제조 버튼에 모든 액션함수들을 따로 선언했었습니다.
- 이렇게 할 경우, 가독성의 문제와 같은 코드들이 반복적으로 사용되는 문제가 있었고 결국 코드의 유지보수성도 떨어뜨리는 문제가 있었습니다.
@IBAction func strawBerryBananaJuiceButton(_ sender: UIButton) {
if juiceMaker.order(.strawBerryBananaJuice) == true {
showFruitStockOnLabel()
enoughFruitStock(sender)
} else {
notEnoughFruitStock()
}
}
@IBAction func bananaJuiceButton(_ sender: UIButton) { ... }
@IBAction func pineappleJuiceButton(_ sender: UIButton) { ... }
@IBAction func kiwiJuiceButton(_ sender: UIButton) { ... }
@IBAction func mangoKiwiJuiceButton(_ sender: UIButton) { ... }
@IBAction func mangoJuiceButton(_ sender: UIButton) { ... }- 기존의 액션함수들을 하나로 합치고
sender로 들어오는UIButton타입의Label을 읽어서 구분하도록 했습니다. 따라서 중복되는 기능을 제거하는 한편 코드의 유지보수성도 높아질 수 있도록 수정할 수 있었습니다.
@IBAction func orderJuice(_ sender: UIButton) {
guard let senderTitle = sender.titleLabel?.text else { return }
let juiceNameFromSendTitle = senderTitle.components(separatedBy: "쥬스").first
switch juiceNameFromSendTitle {
case "딸기":
showOrderedAlert(sender, by:.strawBerryJuice)
case "딸바":
showOrderedAlert(sender, by:.strawBerryBananaJuice)
case "바나나":
showOrderedAlert(sender, by:.bananaJuice)
case "파인애플":
showOrderedAlert(sender, by:.pineAppleJuice)
case "키위":
showOrderedAlert(sender, by:.kiwiJuice)
case "망키":
showOrderedAlert(sender, by:.mangoKiwiJuice)
case "망고":
showOrderedAlert(sender, by:.mangoJuice)
default:
print("없는 메뉴입니다.")
}
modifyFruitStockOnLabel()
}- 코드 가독성, 중복되는 코드(1)에서
switch-case구문으로 변경을 진행했는데, 이 것 역시 각각의 쥬스가 추가될 때 마다 case를 하나씩 추가해줘야되는 문제가 있었습니다. - case의 조건 특성상 어쩔 수 없이 하드코딩이 된다는 느낌을 받아서 수정을 진행하였습니다.
Juice타입에rawValue를 다음과 같이 할당하였습니다.
enum Juice: String {
case strawberryJuice = "딸기"
case bananaJuice = "바나나"
case pineappleJuice = "파인애플"
case kiwiJuice = "키위"
case mangoJuice = "망고"
case strawberryBananaJuice = "딸바"
case mangoKiwiJuice = "망키"
}- 그 다음
Juice의rawValue값을 활용하여 기존 switch처럼 case를 따로 추가하지 않아도 되도록 수정을 하였습니다.
@IBAction func orderJuice(_ sender: UIButton) {
guard let senderTitle = sender.titleLabel?.text,
let juiceNameFromSendTitle = senderTitle.components(separatedBy: "쥬스").first,
let juice = Juice(rawValue: juiceNameFromSendTitle) else { return }
showOrderedAlert(by: juice)
modifyFruitStockOnLabel()
}viewController특성상 규모가 커지고 그와 동시에if-else의 빈번한 사용, 중복된 코드로 인해 가독성이 떨어지고 속도에도 영향을 미칠 수 있는 문제가 있었습니다.changeStockViewController에서UILabel,UIStepper를 배열로 사용할 때, 각각의 함수에서 다음과 같이 선언하여 사용하였습니다.
final class ChangeStockViewController: UIViewController {
lazy var fruitAndLabel: [Fruit: UILabel] = [
.strawberry: strawberryStockLabel,
.banana: bananaStockLabel,
.pineapple: pineappleStockLabel,
.kiwi: kiwiStockLabel,
.mango: mangoStockLabel
]
private func updateFruitStockOnStepper() {
let fruitAndStepper: [Fruit: UIStepper] = [
.strawberry: strawberryStepper,
.banana: bananaStepper,
.pineapple: pineappleStepper,
.kiwi: kiwiStepper,
.mango: mangoStepper
]
}
@IBAction func changeStockOnLabel(by sender: UIStepper) {
let stepperAndLabel: [UIStepper: UILabel] = [
strawberryStepper: strawberryStockLabel,
bananaStepper: bananaStockLabel,
pineappleStepper: pineappleStockLabel,
kiwiStepper: kiwiStockLabel,
mangoStepper: mangoStockLabel
]
}
}typealias와tuple을 사용하여FruitComponent라는 타입을 만들고, 이를 활용해서 기존의 선언된UIStepper,UILabel,Fruit을 사용하는 배열을 합칠 수 있었고,if-else구문들을 전부guard로 변경을 진행하였습니다.
final class ChangeStockViewController: UIViewController {
typealias FruitComponent = (fruit: Fruit, label: UILabel, stepper: UIStepper)
lazy var fruitComponents: [FruitComponent] = [(.strawberry, strawberryStockLabel, strawberryStepper),
(.banana, bananaStockLabel, bananaStepper),
(.pineapple, pineappleStockLabel, pineappleStepper),
(.kiwi, kiwiStockLabel, kiwiStepper),
(.mango, mangoStockLabel, mangoStepper)]
}- 기존
delegate패턴에서singleton으로 변경을 진행하였는데,ChangeStockViewController에서MakeJuiceViewController로 화면이 넘어가는 동시에 데이터는 전달되나, 화면이 업데이트가 되지 않는 문제가 있었습니다.
- 첫 번째 문제로는
present로 화면을 보여줄 때, 두 번째 화면인ChangeStockViewController가 전체화면이 아닌, 위에 팝업창 같이 나타나서 첫 번째인MakeJuiceViewController가 종료되지 않는 문제를 해결하기위해
private func presentChangeStockViewController() {
changeStockNavigationController.modalPresentationStyle = .fullScreen
}presentChangeStockViewController()메서드에fullScreen기능을 사용하여 두 번째화면이 팝업이 아닌, 전체화면으로 보이고 그와 동시에 첫 번째 화면이 종료될 수 있도록 하였습니다.- 두 번째 문제로는
ViewController의 생명주기와 관련이 있었습니다. - 기존에
viewDidLoad()에 수량 레이블을 최초에 업데이트만 했기 때문에, 화면이 다시 전환된 후 재고수량이 변경될 수 있도록viewWillApear()에서 업데이트를 진행하도록 다음과 같이 추가하였습니다.
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)
modifyFruitStockOnLabel()
}- 🍎Apple Docs: Initialization
- 🍎Apple Docs: Access Control
- 🍎Apple Docs: Nested Types
- 🍎Apple Docs: Type Casting
- 🍎Apple Docs: Error Handling
- 🍎Apple Docs: API Design Guidelines
ifswitch조건문을 최대한 사용하지 않았다.- 최소한의 코드 구현으로 프로젝트 구현을 진행했다.
- 코드의 가독성을 생각한 부분이 부족했다.
- 네이밍이 어색한 부분 개선이 필요하다.
- 함수의 기능을 분리하는 점에서 개선이 필요하다.
- 서로 다른 생각을 가지고 있었기에 문제 해결 진행에 있어 여러가지 방안이 제시 되었다.
- 프로젝트 끝까지 포기하지 않고 늦은 시간까지 텐션을 잃지 않아서 열심히 할 수 있었다.
- 리팩토링 부분을 찾는 시간을 갖지 못했다
- STEP2를 진행할 때 커밋 관리가 잘 안되었던 점 (STEP1, STEP3에서는 이상 없었다.)