[ TIL ] 첫번째 과제에 대한 피드백 후 수용 / 추상화의 개념 / 24년 3월 8일

 

출처 : 핀터레스트

 

 

1주차 과제에 대한 피드백을 받았습니다.

생각보다 긴 글에 아주 긴장하며 천천히 읽어나갔어요!

 

헉..! 내가 이렇게 실수한 부분이 많았나..? 충격과 공포..

 

우선 세 가지의 피드백을 받았습니다.

첫 번째는 나눗셈 구현에 관한 피드백이었고

두 번째는 추상화에 관한 피드백, 세 번째는 Git에 관한 피드백이었어요.

그럼 이제 제가 해야할 일은 피드백 후 수용 입니다.

 

 

피드백 1)  나눗셈 구현 중 분모가 0일 때 고려

---

나눗셈 연산을 구현하는 중에 분모가 0일 때를 구현하지 않았어요.

아니.. 너 수학 강사라고 할 수 있는 거 맞아..? ㅋㅋㅋ ( 오늘부터 전 수학 강사 아닙니다. )

 

y를 0으로 해서 돌리니

Swift/arm64-apple-ios-simulator.swiftinterface:34781: Fatal error: Division by zero in remainder operation

이런 코드가 나오네요!

 

자 그럼 DivideOperation의 클래스 부터 고쳐봅시다.

우선 if문을 사용하여 고쳐보았습니다.

// 첫번째 값에서 두번째 값을 나눈 나눠주는 클래스
class DivideOperation: AbstractOperation {
    override func calculate(_ x: Int ,_ y: Int) -> Double {
        
        if y == 0 {
            print("분모가 0으로 나눗셈을 진행할 수 없습니다.")
            return 0.0
        }else{
            return Double(x) / Double(y)
        }
    
    }
}

 

나머지 연산도 엄밀히 말하면 나누기니깐 고쳐줘야겠죠?!

// 첫번째 값에서 두번째 값을 나눈 나머지를 구하는 클래스
class ModulusOperation: AbstractOperation {
    override func calculate(_ x: Int ,_ y: Int) -> Double {
        if y == 0 {
            print("분모가 0으로 나머지를 진행할 수 없습니다.")
            return 0.0
        }else {
            return Double(x % y)
        }
    }
}

 

guard 문을 사용해서도 시도해보았습니다.

class DivideOperation: AbstractOperation {
    override func calculate(_ x: Int ,_ y: Int) throws -> Double {

        guard y != 0 else { return nil }
        
        return Double(x) / Double(y)
        
    }
}

 

 

피드백 2) 추상화에 관한 피드백

---

저는 추상화에 대한 완벽한 이해를 하지 못하고 문제를 푼 것 같아요.

먼저 추상화에 대한 이해를 해봅시다.

 

추상화는 객체의 공통적인 속성과 기능을 추출하여 정의하는 것

 

 

많은 예시로 쓰이는 탈 것을 한 번 생각해봅시다.

여기 자동차, 자전거, 오토바이 클래스가 있어요.

 

Bicycle 클래스

class Bicycle {
    func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

 

MotorBike 클래스

class MotorBike {
    func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

 

Car 클래스

class Car {
    func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

 

 

근데 세 가지에는 공통점이 있죠.

바로 ' 앞으로 전진한다.' 입니다.

여기서 공통 부분인 ' 앞으로 전진한다. '를 추출하여 추상 클래스를 만드는 것입니다.

 

Vehicle 클래스

class Vehicle {
    func goForward() {}
}

 

탈 것들의 행위(역할)을 분리시켜 Vehicle 클래스를 만들었습니다.

 

Bicycle, MotoBike, Car 클래스

class Bicycle : Vehicle {
    override func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

class MotorBike : Vehicle {
    override func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

class Car : Vehicle {
    override func goForward() {
        print("\(Self.self)가 앞으로 전진한다.")
    }
}

 

이 행위를 직접 수행(구현)하는 bicycle, motobike, car 클래스는 추상클래스인 Vehicle클래스를 상속받아 위의 함수들을 재정의합니다. 이로써 객체(자동차, 자전거, 자동차)들의 공통적인 속성과 기능을 추출하여 관계를 맺도록 하는 것입니다.

 

어떤 객체가 수행해야하는 역할 부분과 그 역할을 구현하는 부분을 분리시켜 프로그램을 구현하는 것이죠.

 

위의 코드에서는 앞으로 전진한다는 공통적인 부분만 함수로 구현을 하였지만, 시동을 거는 행위에 대해서 생각해보면 탈 것들은 다 시동을 거는 행위를 하지만, 오토바이나 자동차나 자전거의 시동을 거는 방법은 다릅니다. 그 부분은 각자의 클래스의 내부에서 구현하는 것이죠. 

 

그래서 추상화에서 가장 중요한 건 뭐다?!

역할과 분리

이다.

 

 

 

제가 구현한 계산기 코드에서 저는 더하기, 빼기, 곱하기, 나누기, 나머지 클래스에 대한 구현과 그 클래스들의 추상메서드를 만들어 역할과 분리를 해주었습니다. 하지만, Calculator라는 클래스 내부에서의 클래스 들의 공통적인 부분인 calculate 함수를 사용할 때, 추상화와 어울리는 코드를 짜지 못했어요.

 

Calculator 의 calculate 메서드 구현부분인데요. 결론부터 적으면 작성하신 메서드는 추상화와 어울리지 않습니다. Calculator 는 연산이 어떻게 누가 수행하는지 관심이 없어야 합니다. Calculator 입장에서 '무엇', 여기서는 연산이 관심사 입니다. 연산이 필요할 때 협력자(collaborator)에게 메세지를 보내면 협력자는 연산을 수행합니다. Calculator 는 협력자가 '어떻게' 했는지는 관심이 없습니다. 즉, 추상화의 주요 목적 중 하나는 how 와 what 을 분리하는 것입니다. 하지만 calculate method 를 보면, 어떤 연산을 어떤 클래스가 수행하는지 Calculator 가 알게됩니다. 이런 걸 implementation detail(구현상세)가 클라이언트(서비스 또는 협력자를 사용하는 컴포넌트)에게 노출되었다고 말합니다.

 

튜텨님의 애정이 듬뿍담긴 피드백 중 일부입니다.

매정하시지만 그렇기에 더욱 감사한..

정신차리고 코드를 수정해봅시다.

 

AbstractOperation 클래스

class AbstractOperation {
    func calculate(_ x: Int ,_ y: Int) -> Double { 0 }
}

 

AddOperation 클래스

class AddOperation: AbstractOperation {
    override func calculate(_ x: Int ,_ y: Int) -> Double { Double(x + y) }
}

 

Calculator 클래스

// Calculator 클래스
class Calculator {
    
    var operation: AbstractOperation?
    
    init(operation: AbstractOperation?) {
        self.operation = operation
    }
    
    func calculate( _ firstNumber: Int, _ secondNumber: Int) -> Double {
        return operation?.calculate(firstNumber, secondNumber) ?? 0
    }
}

 

추상화에 맞춰 Calculator라는 클래스는 비연산자만을 받아 함수를 넘겨주는 역할을 합니다.

 

이렇게 Calculator에서는 '무엇을' 에만 집중하는 클래스인거죠. '어떻게'는 오로지 AddOperation 클래스 만 관여를 합니다.

 

그 이외의 연산자를 지정해주는 방법을 이 Calculator의 클래스를 벗어난 곳에서 결정을 해야합니다.

이 부분에서 한가지 피드백을 추가로 받았어요.

 

연산을 "+","-","/" 의 Character 형태가 아닌 enum 타입으로 받아라

 

이유는 사용자로부터 Character로 입력을 받게되면 "^" 나 "_" 로 오타가 났을 때, 오류가 생길 수 있으니 코드가 안전하지 않기 때문입니다. 이 부분에 대한 제 생각을 아주 쉬운 예로 들자면 저에게 빨강, 파랑, 초록색의 풍선이 있는데, 상대방에게 '무슨 색의 풍선을 받을래?'라고 묻는 것보다 '빨강, 파랑, 초록의 풍선 중 무슨 색의 풍선을 받을래?' 라고 묻는게 코드 상으로 안전하다는 것이죠.

 

Operation 열거형

enum Operation {
    case add
    case subtract
    case divide
    case multiply
    case modulus
}

 

이렇게 구현을 하여 클래스의 인스턴스의 실행 부분에서 사용할 수 있었습니다.

 

 

피드백 3) Git에 관한 피드백

---

 

튜텨님께서 Git을 보고 뜨악! 하셨나봐요^^

왜냐면 제가 프로젝트 하나 다 만들고 커밋하고, 다 끝내고 커밋했거든요..

 

커밋도 습관이라 일찍부터 습관을 잘 들여놔야한다고 하셨어요.

내가 쪼개고 싶은 작은 단위( 논리적인 하나의 기능 )로 커밋을 올리라는 조언을 받았습니다!

아직 깃을 능숙하게 쓴지 얼마되지 않아 커밋에 습관이 안되어있었던 거 같아요.

 

또 제가 원래는 Github Desktop을 사용해서 커밋하였는데

알고보니 Xcode 안에서 opt + cmd + c 키만 눌러도 간단히 커밋을 할 수 있더라구요!

또 하나 배우고 갑니다~~!!!

 

그래서 코드 수정할 때는 커밋할거를 엄청나게 신경써서하고 최대한 자세하게 쓰려고 작성했던 거 같아요.

 

 

노력한 나의 증거..

내일..아니구 월요일날 또 피드백 받고 돌아오겠습니다!

 

---

 

피드백받고 공부하고 코드수정하고 다시 피드백받고 공부하고 코드수정하고 캠프동기들과 수다도 떨고 TIL 작성하니 벌써 1시를 달려가고 있네요.. 아침 9시 출근인 사람..? 나^^ㅎㅎㅎㅎㅎ 바쁘다 바빠 현대사회

블로그 글도 쓰다보니 가독성을 더 생각하게 되고 점점 글 실력도 느는 것 같아요!

오늘도 어제보다 더 성장했다^__^

내가 못한 것에 집중하기보다는 새로운 개념을 배웠다는 것에 집중할려구요!ㅎ