Ray`s Tech Lab

스프링에서 사용할 객체를 위해 클래스를 만들어줬다고 해도 그 녀석을 사용하려면 스프링 빈으로 등록을 해야 한다. 만약 빈으로 등록하지 않으면, 스프링에서 사용할 수 없게 된다. 근데 빈이란 것이 도대체 무엇일까?

스프링 빈(Bean)

스프링 컨테이너가 관리하는 Java객체. 즉 스프링에 의해 생성되고 관리되는 Java 객체라고 보면 된다. 스프링 컨테이너가 일종의 바구니 같은 느낌이라면, 빈은 그 바구니에 담기는 콩들이라고 생각할 수 있다. 한마디로 이 콩으로 등록을 해야 스프링에서 이용가능하게 된다는 것..! 원래는 개발자가 직접 코드를 통해 new연산자나 팩토리 패턴 등으로 객체를 생성하고 소멸시키는데, 스프링은 컨테이너라는 바구니가 이 역할들을 대신해준다고 생각할 수 있다.

예를 들어 내가 MemberController를 만들고, 이 컨트롤러가 MemberService를 통해 회원가입이나 회원조회를 할 수 있게끔 만들려고 한다면 내가 만든 MemberController와 MemberService를 빈으로 등록을 해야 스프링에서 이용 가능하다. 이 때 다음과 같은 2가지 방법으로 스프링 빈 등록이 가능하다.

1. 어노테이션을 통한 등록

빈으로 등록할 클래스에 @Component라는 어노테이션을 멕이면 된다. 이렇게 해두면 스프링으 컴포넌트 스캔을 통해 지가 알아서 Bean으로 등록해준다. @Controller, @Service, @Repository등과 같이 @Component를 포함하는 어노테이션들도 컴포넌트 스캔을 통해 Bean으로 등록된다. 근데 그렇다고 아무데서나 저런 어노테이션 멕인다고 죄다 Bean으로 등록되는건 아니고, 컴포넌트 스캔을 하는 범위가 따로 있다. 기본적으론 @ComponentScan 어노테이션이 있는 파일의 패키지 아래를 모두 스캔하며, @SpringBootApplication(scanBasePackages ={{경로설정}}) 옵션으로 컴포넌트 스캔 시작 범위를 직접 설정할 수 있다. (이 경우 설정한 경로 기준으로 그 하위 애들만 스캔)  

암튼 어노테이션을 통한 등록법의 예시는 다음과 같다.

@Controller
public class MemberController {
    private final MemberService memberService;

    @Autowired
    public MemberController(MemberService memberService) {
        this.memberService = memberService;
    }
}

@Autowired : 이 어노테이션이 보시다시피 생성자에 있으면 스프링에 컨테이너에 있는 memberService 빈을 찾아서 자동으로 연결해주게 한다(즉 이를 위해서 MemberService가 당연히 빈으로 등록돼있어야 한다). 이렇게 의존관계를 갖는 객체를 외부에서 넣어주는 이런 모습을 DI(Dependency Injection)이라고 한다. 여기서는 개발자가 직접 코드를 통해 주입하는 게 아니라 @Autowired 어노테이션에 의해 스프링에 알아서 주입해준다.

2. 자바 코드로 직접 등록

하나하나 직접 Bean으로 등록하는 방식이다. ~~Config라는 자바 파일을 만들고 거기에 다음과 같이 빈을 등록한다.

@Configuration
public class SpringConfig {
    @Bean
    public MemberService memberService() {
        return new MemberService(memberRepository());
    }

    @Bean
    public MemberRepository memberRepository() {
        return new MemoryMemberRepository();
    }
}

참고로 MemberService가 MemberRepository에 의존하기 때문에 MemberService의 경우 생성자에서 memberRepository를 넣어주는 모습.

@Bean이란 어노테이션을 보고 스프링이 이 객체들을 Bean으로 등록해준다. 

두 방법 중 그런 어떤 방법이 더 좋은 것일까?

상황에 따라 다르다.

주로 정형화된 컨트롤러나 서비스, 리포지토리 같은 경우는 어노테이션을 통한 등록이 편하다.

그러나 정형화되지 않았거나 상황에 따라 구현 클래스를 변경해야 하는 경우 등엔 코드로 직접 등록하는 게 더 편할 것이다. 왜냐하면 어노테이션을 통한 등록의 경우 변경할 때마다 새로 어노테이션을 작성해줘야 하는 번거로움이 있지만 직접 등록하는 경우는 1번만 바꾸면 되기 때문.

스프링 웹 개발은 다음과 같은 3가지 방법이 있다.

1. 정적 컨텐츠 : 서버에서 특별히 뭔가 하는 거 없이 단순히 html파일을 클라이언트 쪽으로 넘기는 방식이다. 
2. MVC & 템플릿 엔진 : 서버에서 특정한 작업을 통해 html파일을 동적으로 만들어 넘기는 방식이다. JSP, PHP 등의 템플릿 엔진이 사용되며 이걸 하기 위해 MVC패턴이 도입된다.
3. API : JSON 포맷 등으로 '데이터'를 클라이언트에 전달하는 방식이다. 서버 간의 통신에도 활용되며, 요즘은 React같은 애들한테 API로 데이터를 넘기고 그걸 통해 화면을 클라이언트가 그리는 식으로도 활용된다.

정적 컨텐츠

resources폴더의 static폴더에 있는 파일을 찾아 넘긴다. 다음 순서로 동작한다고 이해할 수 있다.

1. url를 입력
2. 내장된 톰캣이 요청을 받고 스프링 컨테이너에게 요청을 넘김
3. 컨트롤러가 url에 매핑되는 메서드가 있는지 먼저 찾음
4. 없으면 내부의 resources/static에서 찾아서 넘김

MVC & 템플릿 엔진

MVC는 Model, View, Controller로 각자의 역할을 구분해 개발하는 방식을 일컫는다. Model과 Controller는 비즈니스 로직과 내부 로직을 처리하는 역할을 맡고, View는 화면을 그리는 역할을 맡는다. 옛날엔 JSP를 통해서 View에서 Controller의 역할도 수행했는데 이를 model 1방식이라 부르기도 함.

@Controller
public class HelloController {
    @GetMapping("hello")
    public String hello(Model model) {
        model.addAttribute("data", "hello!!");
        return "hello";
    }
}

• GetMapping : GET POST 할 때의 get이며, /hello로 get요청이 들어왔을 때 hello라는 메서드를 실행한다는 거다. 
• Model : View를 그리기 위해 사용되는 데이터들의 박스(?)로, key-value형태로 데이터들을 저장한다. 내부적으론 Map이 사용된다고 함.
• return "hello" : 데이터들을 보낼 View이름을 지정하는 거라고 보면 된다.

이 방식의 동작순서는 다음과 같다.

1. url를 입력
2. 내장된 톰캣이 요청을 받고 스프링 컨테이너에게 요청을 넘김
3. 컨트롤러가 url에 매핑되는 메서드가 있는지 먼저 찾음
4. 있으면 해당 메서드를 호출
5. viewResolver가 지정된 이름의 View를 찾아주고 템플릿 엔진과 연결시켜줌
6. 템플릿엔진이 렌더링한 html파일을 반환

API

이 방식은 html파일이 아니라 데이터를 넘기는 방식이라고 볼 수 있다. 결국 정적 컨텐츠를 넘기는게 아니라면

• html로 내려주든가
• api로 (즉 데이터로) 주든가

둘 중에 하나인 거다.

@Controller
public class HelloController {
    @GetMapping("hello-string")
    @ResponseBody
    public String helloString(@RequestParam("name") String name) {
        return "hello" + name;
    }
}

• GetMapping : GET POST 할 때의 get이며, /hello-string로 get요청이 들어왔을 때 helloString라는 메서드를 실행한다는 거다. 
• @ResponseBody : 이 어노테이션을 쓰면 viewResolver를 쓰지 않고, 대신에 Http Response의 BODY에 데이터를 담게 된다.
• @RequestParam("name") String name : 쿼리에서 "name"에 해당하는 값을 name이란 파라미터로 받아온다는 거다.
• return "hello"  + name : 이 문자열 자체를 데이터로 준다는거다.

참고로 객체를 반환하면 객체가 JSON으로 변환되서 보내진다!

@Controller
public class HelloController {
    @GetMapping("hello-api")
    @ResponseBody
    public Hello helloApi(@RequestParam("name") String name) {
        Hello hello = new Hello();
        hello.setName(name);
        return hello;
    }

    static class Hello {
        private String name;

        public String getName() {
            return name;
        }

        public void setName(String name) {
            this.name = name;
        }
    }
}

이 방식의 동작순서는 다음과 같다.

1. url를 입력
2. 내장된 톰캣이 요청을 받고 스프링 컨테이너에게 요청을 넘김
3. 컨트롤러가 url에 매핑되는 메서드가 있는지 먼저 찾음
4. 있으면 해당 메서드를 호출
5. 근데 @ResponseBody가 있으면 viewResolver에게 전달하지 않고 httpMessageConverter에게 전달함
6. 문자는 그대로 보내고, 객체는 JSON으로 바꿔서 전달한다.

용어 정리

Gradle : 라이브러리들을 버전 설정하고 가져와주는 빌드 도구. 은 의존관계가 있는 라이브러리들도 함께 다운해준다(A라이브러리가 B에 의존하면 자동으로 B도 땡겨줌).

resources폴더 : 자바 파일(*.java)를 제외한 나머지 파일들을 위한 폴더

템플릿 엔진 : 지정된 템플릿 양식과 데이터를 합쳐 HTML 문서를 출력하는 SW. 예를 들어  그냥 a.html을 작성하기만 하면 서버에선 그냥 걔를 클라이언트쪽으로 넘기기만 하는데, 템플릿 엔진을 쓰면 html에서 루프를 쓴다던가 어떤 데이터값을 넣는다든가 할 수 있는 것이다. 

<html xmlns:th="http://wwww.thymeleaf.org">

html태그에 이렇게 작성해두면 해당 html파일에서 타임리프(템플릿 엔진) 문법을 사용 가능하다. 이를 활용해 서버에서 받아온 특정 데이터 값을 넣고 싶으면

<p th:text="'안녕하세요' + ${data}">안녕하세요.손님</p>

이런 식으로 하면 된다. 참고로 태그 사이엔 딱히 뭔가를 쓸 필욘 없음. 근데 템플릿 엔진을 거치지 않고 html파일을 열거나 할 때 태그 사이의 값이 나온다. 이미지 태그의 alt값과 비슷한 역할이라고 생각할 수 있겠다.

스프링 강의를 수강하기 시작하며 Optional이란 걸 마주쳤다! null값으로 올 수 있는 애들을 위해서 실무에서 많이 쓴다던데?

이제 이런 것들 안 지나치고 내 것으로 만들어야겠다는 생각이 든다. 바로 알아봤음.

일단 이 녀석은 Integer클래스처럼 일종의 Wrapper클래스다. 포장해주는 느낌. 그리고 그 알멩이로는 모든 타입의 변수가 저장될 수 있다.

내부 알멩이로 null 역시 저장될 수 있기 때문에, null값과 관련된 예상치 못한 NPE(NullPointerException)들을 피할 수 있다는게 가장 큰 장점이다. 예를 들면 어떤 저장소에서 find를 했는데 내가 찾는 게 없어서 null값이 반환된 경우 등..이런 상황을 방지하려면 find한 결과가 null인지 아닌지를 if문으로 파악해야 했지만 Optional을 사용하면 그런 걸 간단히 처리할 수 있다는 게 장점쓰. 애당초 null일 수도 있는 값을 다루기 위한 클래스이기 때문에 그와 관련된 다양한 메서드들을 제공한다.

그러면 일단, 이 Optional 객체를 어떻게 만드는가? Optional클래스를 뜯어보면 다음과 같이 of와 ofNullable이라는 스태틱 메서드가 정의돼있는 것을 볼 수 있다.

두 메서드 모두 Optional을 리턴하는 걸 볼 수 있다. 즉 이 두 메서드를 통해서 Optional객체를 생성할 수 있다! of메서드는 인자로 'null이 아닌 값'을 받고, ofNullable메서드는 인자로 'null값일 수도 있는 값'을 받는다.

of메서드의 경우 인자로 준 값이 null이면 NPE예외가 발생한다. 또한, ofNullable메서드의 경우 인자로 준 값이 null이면 EMPTY, 즉 빈 Optional객체가 리턴된다. 아니 그냥 내부에 null값 넣어서 리턴해주지 왜 약속된 빈 Optional객체(= EMPTY)를 리턴하도록 했을까? 답은 메모리 절약을 위해서. 어차피 알멩이가 null인 Optional 객체는 솔직히 의미없는 객체인데, 여러 곳에서 이 깡깡이 Optional객체를 만들었다면 메모리 낭비가 있을 것이다. 이를 위해 Optional클래스가 싱글턴 인스턴스로 EMPTY 객체를 가지는거고, 얘 하나만을 빈 객체로 리턴해주는 것이다. 개발자 분들이 메모리 절약을 위한 센스있는 선택을 하신 것 같다. 캬..

그럼 이 Optional객체가 감싸고 있는 알멩이는 어떻게 꺼낼 수 있을까? 간단하다. get메서드로 꺼낼 수 있다. 근데 음..이렇게 꺼내쓰는 건 권장되지 않는다고 한다. 만약 Optional객체가 감싸고 있는 값이 null이면 NoSuchElementException예외가 발생하기 때문. 즉 다짜고짜 꺼내는 게 아니라 안에 있는 알멩이가 null인지를 확인하고 꺼내는 게 좋다. isPresent메서드가 이 기능을 해준다.

그럼 만약 알멩이가 null인지를 확인하고, null이 아니면 바로 뭔가 해주고 싶으면? ifPresent메서드를 사용할 수도 있다. 이 메서드는 알멩이가 null이 아니면 파라미터로 받은 action을 실행해준다.

만약, 알멩이를 꺼내고 싶은데 알멩이가 null이면 내가 지정한 값을 쓰고 싶다면? orElse메서드를 쓰면 된다.

이외에도 여러 메서드들이 있다. 실무에서는 null값이 될 수도 있는 애들을 Optional로 래핑해서 처리하는 경우가 많다고 한다. 나도 이 방법을 숙지하도록 하자. 아자아자..!

※ 참고 : orElse vs orElseGet에 관련된 좋은 글

https://ysjune.github.io/posts/java/orelsenorelseget/

 어느덧 마지막 주차다. 3주 전 긴장 반 설렘 반으로 프리코스를 시작하던 때가 엊그제 같은데 벌써 마지막 주차 미션이라니..

 3주차 미션을 다시금 돌아보면 부셔지고 깨지는 순간들이었다. 고통스러웠던 시간이었다고 해도 될 것 같다. 그러나 포비 코치님이 코수타에서 고통 속에서 학습이 이뤄진다고 말씀해주셨다. 나중에 이런 고통들을 이겨내고 뒤를 돌아보면 이 산을 많이 올라왔음을 내려다볼 수 있을 때가 있을 거다. 목표로 가는 길이 쉽고 평탄하지 많은 않고 당연히 가시밭길도 있을 거다. 예방주사 한 번 맞았다고 생각하고 다시금 재정비할 때다.

 3주차 미션을 진행하며 고민했던 요소들이 공통 피드백에 녹아있었다. 대표적으론 private메서드를 어떻게 테스트할지?와 같은 내용이 있었고 테스트하기 쉬운 구조로 코드를 짜는 팁도 있었다. 저번 미션에 어설프게 TDD쓰려 하느니 미션의 요구사항과 목표에만 집중하자!라는 생각으로 기능 구현이 다 끝날 때까지 테스트 코드를 1개도 짜지 않았었다..덕분에 프로덕션 코드를 테스트하려 할 때 이걸 어떻게 테스트해야하지..하면서 고민하다가 접근지정자를 전부 다 public으로 바꿔야 했던 뼈아픈 실책을 내기도 했던 만큼, 이번 피드백 역시 피가 되고 살이 되고 뼈가 되는 내용들이었다. 

 어느덧 4주차, 프리코스의 마지막 미션.  마무리 매듭을 짓는 단계를, 즉 끝을 어설프게 끝내고 싶진 않다. 3주차에 대한 조금의 아쉬움이 있는 만큼, 마지막 4주차만큼은 스스로가 만족할 수 있는 완벽한 미션진행을 하고 싶다. 3주차를 통해 예방주사를 맞았겠다, 후회없이 모든 걸 쏟아내고 싶다. 모든 걸 갈아넣고 최선을 다 해야 후회가 안 남고, 최종선발여부를 떠나 프리코스가 끝나고 뒤를 돌아봤을 때 내가 산을 올라왔음을, 성장했음을 느낄 수 있을 것이다.  

저번 주에 인사를 나눴던 MVC와 친해지기로 하다

완벽하게 하고 싶단 욕심이 들었던 만큼, 우선 전달받은 공통 피드백을 보며 내가 신경쓰지 못한 부분들과 좀 더 신경써야 할 부분들을 보기로 했다. 그리고 요구사항에 명시된 제한들을 파악하고 먼저 정리해보기로 했다.

• "변경을 못 하게 막으려는 경우 final키워드 붙이기"
• "객체는 객체스럽게 쓰기"
• "인스턴스 변수 개수 줄이기"

• "메서드 라인길이 10라인 안 넘도록 구현"
• "파라미터 3개로 제한"

등등.. 이걸 어떻게 해? 라는 수준의 경악스러운 조건은 없었다고 생각했다. 오히려 좀 더 빡세진 만큼 내가 더 잘하면 된다! 보여주면 된다! show and prove하면 된다! 라는 생각을 했고, 더 어려워진 만큼 완벽하게 해냈을 때 스스로 느낄 수 있는 만족감과 가치가 더욱 더 커져서 좋다는 생각이 들었다.

 암튼 그러면서 어떻게 만들지 계속 생각해봤다. 이번 미션에서의 유별난 특징 중 하나는 InputView, OutputView등의 클래스를 제시해주며 특정 기능들은 어느 클래스에서만 구현하라는 조건들이 많이 있었다는 점이었다.

"음..? 클래스 이름이 View.. 입력은 무조건 InputView에서만 한다..?"

"결과 출력하는 역할을 OutputView가 하게 한다..?"

"BridgeGame클래스는 InputView와 OutputView를 사용할 수 없다..?

..이거 어디서 많이 보던 문장인데..?

설마..

이거, MVC를 써보란거 아니야 ..? 

 물론 MVC패턴을 쓰란 말이 직접적으로 있던 것도 아니고, 특히나 저번 주에는 "내가 잘 모르는 것들(TDD, MVC같은..)에 집중하기보다는 요구사항을 완벽히 지키는 것에 초점을 맞추자!"라는 생각을 하긴 했다. 근데 제시된 내용에 따르면 단순 클래스 분리가 아닌 역할에 따른 분리를 요구하고 있고, 이런 분리에 알맞는 놈은 내가 잘은 몰라도 저번 주에 나랑 가볍게 인사한 사이인 MVC가 있다는 생각이 들었다.

 "그래, MVC 이 참에 배워서 적용해보자!"

물론 제일 첫번째가 되야 하는 것은 요구사항들과 기능들을 명확히 지키는 것. 그러나 이 정도의 제한이 없던 3주차와는 달리 MVC를 배워서 도입하는게 오히려 그런 요구사항과 기능을 지키는데 도움이 되리라 판단했다. 구현을 시작해야 한다는 마음을 내려놓고, 달리기를 시작하기 전 몸풀기가 먼저다라는 생각으로 MVC를 구글링하며 공부했다. 그리고 유튜브 테코톡으로 올라온 MVC영상들을 보며, MVC가 뭔지 그리고 어떻게 적용할지 구상을 하고 구현단계로 들어갔다.

어떻게 만들어볼까?

 뭐 잘 모르긴 하지만, 일단 view의 역할은 InputView와 OutputView가 한다. 그리고 컨트롤러를 둬서 모델과 뷰 사이에서 이런저런 고생을 시켜야 한다! 모델에는 뷰나 컨트롤러의 코드가 들어있으면 안 된다!! 등등을 숙지한 상태. 작성한 기능목록을 토대로 어떤 클래스들이 이 기능들을 수행하게 할까를 고민한 다음 이들을 적절히 model, controller, view로 나누기로 했다.

 이 단계에서 가장 고민했던 건 "검증기능을 어디로 부여할 것인가"였다. 예외처리 기능이 있어야 하기 때문에 Validate만 따로 하는 클래스도 필요하다고 생각했지만, 저번 주 로또게임을 하면서 우테코가 제시한 Lotto클래스에 검증기능이 있던 걸 봤었기 때문에 검증기능을 따로 클래스를 만들어빼낼지, 아니면 클래스들 내부로 넣을지를 많이 고민했다. 이번 미션이 가지는 기존 미션과의 또 다른 차이점 중 하나가 입력으로부터 예외가 발생하면 메시지를 출력하고 종료하는게 아니라 그 페이즈부터 다시 입력을 받게 하는 것이었기 때문에, 입력한 값을 검증하고 처리하는 것도 도메인 로직 중 하나로 생각했었다. 이를 위해서 클래스 내부로 검증기능을 넣어야 할 것 같다곤 생각했는데, 문제는 다리 길이를 입력받는 부분이었다. 위 아래 칸 선택 입력과 게임커맨드 입력(재시작/종료)는 문자열 그대로 받으니까 상관없지만 다리 길이는 정수로 바꿔서 받아야했다. 즉 정수가 입력된 게 아니라면 바꿔서 받는 과정에서 오류가 날 것이고, 이를 막으려면 Bridge클래스 외부에서 문자열 여부를 검증해야 한다..! 

 즉 검증기능을 도메인 로직으로 생각해서 이를 클래스 내부로  넣어야 할 것 같지만, 다리 길이의 경우 외부에서 1차 검증(문자열이냐 아니냐)를 마쳐야 하기 때문에 깔끔하지 않다고 생각했다(검증을 두 곳에서 나눠서 하는 꼴이 되므로..). 차라리 아예 클래스 외부에서 검증기능을 하는게 더 깔끔하지 않나라는 생각이 들었다. 그러나 숫자로 들어온 다리 길이를 검증하는 것은 도메인 로직으로 보는게 맞지만 문자로 들어온 다리 길이를 검증하는 것은 기대하는 상황이 아니므로 도메인 로직이 아닌 것으로 봐야 한다는 생각이 들었기 때문에, 일단 좀 찝찝하지만 다리 길이만 그렇게 검증을 분리해서 진행하는 쪽으로 방향을 잡았다.

만들고, 점검하고, 고치고!

 저번 주 내가 했던 뼈아픈 실수를 복기해보자면..잘 모르는 TDD같은 거 어설프게 하려 하지 말고, 요구사항 지키는 데에 집중하자! 라는 생각을 가진 건 좋았다. 요구사항 지키기도 빡센데 제시되지 않은 영역까지 다 가져가려 하는건 되려 독이 될 수 있다고 생각했기 때문이다. 그러나 문제는, 그렇다곤 하지만 제시된 기능 조건들이 모두 동작하는 코드들이 완성될 때까지 테스트 코드를 하나도 만들지 않았다는 것이었다. 

그래서 그런지, 테스트 코드를 작성하는 과정에서 어려움을 겪을 수밖에 없었다. 왜냐하면 테스트를 고려하면서 만들어나가지 않았기 때문! 중간중간 테스트 코드를 작성했다면 프로덕션 코드의 리팩터링을 거치면서 진행했었겠지만, 이미 덩치가 커질대로 커진 상태에서 할려고 했던 게 문제였다. 결국에 private를 전부다 public으로 풀어서 진행했다는..

 암튼 이번엔 그런 뼈아픈 역사를 다시 한 번 저지르지 않고자 각 기능묶음별로 구현이 끝나면 테스트 코드를 작성해 잘 동작하는지 테스트한 다음, 구현한 프로덕션 코드들을 리팩터링하는 식으로 진행했다. 이 과정에서 프로덕션 코드가 좀 더 테스트하기 쉬운 구조로 만들어져 가는 것도 느꼈고, 리팩터링이 끝난 후에도 만들어둔 테스트 코드들을 딸깍 클릭하는 것으로 구현한 기능들이 그대로 동작하는지 확인할 수 있어 정말 편했다. 

 이런 과정에서 가장 좋았던 점은 테스트의 강력함을 계속해서 느낄 수 있었다는 거다. 일단 첫 번째로, 테스트 코드를 작성하며 내가 미처 신경쓰지 못한 예외상황을 마주할 수 있었다.

이건 내가 작성한 Bridge클래스에 대한 테스트 코드로, 플레이어가 다리를 다 건넜는지 확인하는 도메인 로직에 대한 테스트를 진행한다. U, D, D형태 다리를 건너다가 마지막에 U로 건너가서 떨어지는 상황을 연출했으니, 플레이어의 도착여부는 False가 될 것이다.

그러나 위 테스트는 통과하지 않았었다. 즉, 플레이어의 도착여부가 true로 돼있었다. 즉 내가 짠 프로덕션 코드에 문제가 생겼던 것! 당시에 내가 짰던 isArrived메서드는 다음과 같았다.

바로 플레이어의 현재 위치와 다리의 길이와 같다면, 즉 플레이어가 다리 끝에 있다면 거기서 살아있던 죽어있던 true를 리턴하도록 코드가 짜여있던 거다. 그러나 플레이어가 다리 끝까지 갔어도 그 마지막 칸에서 죽었다면, 도착한 게 아니여야 한다. 나는 이 부분을 놓쳤던 거다. 내가 짠 프로덕션 코드에서 내가 신경 쓰지 못한 부분들을 알게 해줬다. 만약 테스트를 안 했다면..나는 이런 오류가 있는지도 모르고 지나쳤을 수도 있었다.

 두 번째로 느낀 강력함은 테스트 코드도 리팩토링이 가능하다는 것. 피어리뷰를 하다가 ParameterizedTest를 통해 여러 값들에 대한 테스트를 편하게 할 수 있음을 알게 됐고(참고로 이 내용은 3주차가 끝나고 제공된 공통 피드백에도 있었다), 이를 통해 비슷한 케이스에 대한 여러 개의 테스트 코드를 짜둔 것을 보기 좋게 리팩터링해갈 수 있었다. 그리고 오류가 나는 상황은 isInstanceOf로 확인할 수 있던 것과는 달리 오류가 나지 않는 상황은 "오류가 안 난다"라는 걸 테스트하는 방법을 몰라서 메서드의 리턴값이 확인되는지 정도로만 테스트했었는데, doesNotThrowAnyException을 통해 오류가 안 나는 상황을 테스트할 수도 있음을 알게 되기도 했다. 정말이지 테스트 도구가 제공해주는 강력한 편의성을 느낄 수 있었다..저번 미션들에선 사용자가 입력하는 과정 자체를 하나의 테스트로 만들 수 있다는 점에서도 그 편의성에 놀랐지만, 테스트 작성 자체에서도 상당한 편의성을 제공해준다니..이렇게 테스트 코드들도 리팩터링을 하고 보니, 다른 사람이 볼 때 내가 어떤 것들을 테스트하는건지 쉽게 이해할 수 있게 만들 수 있겠다는 생각이 들었다. 덕분에 이전까지는 느끼지 못했던 문장인 "테스트 코드가 일종의 문서로 쓰일 수 있다"라는 것이 어떤 걸 의미하는지 조금은 이해할 수 있었다. 요구하는 기능이 어떻게 동작해야 하는지, 어떤 상황에서 어떤 값들이 쓰이느냐에 따라 오류가 날 수 있는지를 명시해줄 수 있기 때문인 거다. 단순히 테스트 코드는 내가 만든 기능이 잘 동작하는지 쉽게 파악할 수 있게 해줄 뿐만 아니라, 하나의 문서가 될 수 있다는 것. 사실 다른 것도 많겠지만 이번 프리코스에서의 최대 수확인 뭐니뭐니해도 테스트 코드가 아닐까. 라는 생각이 들 정도로 이 강력함에 매료되고 있는 것 같다. 얘는 정말 내가 어떤 걸 공부하던 계속 들고 가야할 동반자가 되야 할 것 같다.

딱 보니, 이 놈만 고치면 되겠구먼

 이번 주 미션은 MVC에 최대한 입각한 방법으로 (물론 고수들이 보면 어설픈 구조일 수 있지만..) 클래스 분리를 하고자 했다. 물론 처음에도 어느 정도 클래스들을 어떻게 분리할지 생각해둔 상태로 구현을 시작했지만, 구현 중간마다 필요성을 느끼면 좀 더 세분화해서 클래스들을 쪼개나갔다. 또한 model, view, controller 외에도 service라는 역할을 하는 애들을 둘 수도 있음을 알게 되고, 전역적으로 쓰이는 애들을 util 안에 둬서 관리할 있다는 것 등을 알게 된 후엔 최대한 계층을 생각해서 수정을 해나갔다.

 이 방식의 장점은, 역시나 리팩터링할 때 피부로 느낄 수 있었다. 바로 "고치기 쉽다"는 것! 특정 기능을 고치거나 리팩터링을 할 땐 딱 그 놈이 있는 클래스만 고쳐주면 됐다. 예를 들어, BridgeGame에서 Player의 생사여부(떨어지면 죽는다는 식으로 구현함)를 확인하여 전달하는 코드가 있고, 만약 죽는 기능을 수정해야 하는 상황이라면 BridgeGame은 건드리지 않고 Player의 코드만 건드리면 됐던 거다. 사용하는 쪽(BridgeGame)은 건드리지 않고, 그 로직을 제공하는 놈(Player)만 건드리면 되도록 구조가 잡혔기 때문일 것이다. 즉 각각이 하는 역할에 따라 클래스들을 분리한 덕에 "난 네가 내부적으로 어찌 동작하는지는 모르지만, 그냥 너한테서 ~라는 정보만 얻으면 땡이야!" 라고 말하는 듯한 구조로 짜여진 거다.

4주차 종료 및 회고

 결국 그렇게, 4주차 미션까지 종료됐다. 뭔가 시원섭섭한 느낌. 6년전 이맘때쯤이었을까? 수능이 끝나고나서 느꼈던 감정이 조금은 다시 느껴졌던 것 같다. 뭔가 몰두해서 하던 것이 끝난 느낌. 

  떠올려봤다. 프리코스를 시작할 때, 그리고 우테코를 지원할 때의 내 마음가짐. 성장을 위해 후회없이 이 과정에 임하자고 다짐했던 각오. 난 정말 최선을 다 했을까? "최선을 다 해 임할 것이다"라고 말할 수는 있어도, "최선을 다 했다"라고 말을 꺼내는 것은 그 무게가 사뭇 다를 것이다.  학교 과제? 다 포기했다. 학교에서 받는 학점보다 우테코로 얻는 가치가 더 크다고 생각했으니까. 사실상 알바와 팀플을 제외하고 잠도 줄여가며 남는 시간을 대부분 프리코스 미션에 투자했다. 바쁜 날도 하루 최소 2시간씩은 투자했고, 알바를 갈 때도 노트북을 챙겨가서 주문이 뜸한 동안 틈틈이 미션을 하곤 했다.

그러나 정말 최선을 다 했다고 말할 수 있을까?

 부끄럽게도...물론 이렇게 한 것도 힘들었지만..고통스러웠지만, 더욱 더 쥐어짤 수 있지 않았을까라는 생각이 든다. 좀 더 짜낼걸..이란 아쉬움이 조금은 남는다. 최선을 다 했다는 말을 스스로에게 해주며 토닥거리고 싶지만, 함부로 그럴 수 없을 것 같다. 더 할 수 있는 여지가 분명 있었다.

 뒤를 돌아봤다. 내 프리코스 목표였던 성장을 나는 이뤄냈을까? 내가 얻은 건 뭐가 있을까?

 우선 첫 번째. 몰입하는 경험을 쌓을 수 있었다. 사실 나는 프로그래밍 쪽으로는 깊은 몰입 경험이 있긴 있어도 많지 않다. 그러나 이번 우테코기간은 정말 그 미션들에 내가 몰입하고 있다는 걸 느끼는 순간들이 많았다. 버스를 타는 길에 창밖을 보며 "음 집가서 이런 식으로 만들어볼까.."하던 순간들도 있었고, 알바 출근길에는 클래스를 이렇게 바꾸는 게 더 나을 것 같다고 생각하던 순간들도 있었다. 잠자리에 누웠다가 번뜩하고 생각난 것들을 바로 적어두는 순간들 역시 있었다. 

 내가 프로그래밍을 해오면서 이런 몰입을 스스로 느껴본 적이 얼마나 있었을까? 많지 않다. 솔직히 드물다. 그러나 프리코스 과정을 통해 내가 몰입하고 있음을 느낄 수 있었다. 사실 내 단점 중 하나가 주의력이 좀 낮다는 것이라고 생각했는데, 이번 우테코를 통해 내가 뭔가에 집중하고 있다, 몰입하고 있다는 순간들을 느낄 수 있어 스스로에게 너무 놀라웠다. 역시 나도 하면 하는 놈이구나 라는 것도 느껴진 것 같다. 이런 귀중한 경험은 분명 쉽게 쌓을 수 있는 것이 아니다. 

 그리고 두 번째. 테스트 코드를 공부하게 됐다는 것. 뭐 길게 말할 필요 있을까? 사실상 이번 프리코스 최대 수확이라 해도 과언이 아니다. 이 맛있는 놈을 이제서야 알게 되다니..지금껏 이런 명품을 모르고 있던 게 부끄러울 지경. 

 마지막 세 번째. 내 목표에 한 발짝은 다가가지 않았을까.  내가 우테코를 지원한 궁극적인 이유는 "읽기 좋고 재사용성이 좋은 실전적 코드를 작성하는 개발자가 되고 싶어서"이다. 지금의 내겐 원하는 기능을 어떻게든 꾸역꾸역 구현할 수 있는 능력이 있다고 생각하지만, 코드를 고칠 때나 며칠이 지나고 내가 짠 코드를 볼 때 "이게 뭐하는 거지..어딜 고쳐야 하지.." 라는 생각에 현타가 많이 오곤 했었다.

 그러나 이번 프리코스 미션들을 통해, 자바 컨벤션도 찾아보고, 메서드 네이밍을 고민하고 메서드 분리를 연습하고, 10줄 제한을 통과하려고 낑낑대보기도 하고, 클래스 분리도 연습하고, MVC패턴에 입각한 분리도 연습해봤다. 물론 아직도 많이 부족하지만 각각의 연습들을 통해 내 코드가 읽기 좋은 코드가 되고 있음을 스스로가 느낄 수  있었다.

 뭐 별 거 아니라고 할 수 있다. 그게 뭐 어쨌다고? 그게 그렇게 좋았냐(?)고 할 수도 있고..ㅋㅋㅋㅋ. 그러나 지금까지 하나의 파일에서 대부분 코드를 작성하던 내게, 구조를 생각하지 않고 구현만 하던 내가 처음으로 나름대로 읽기 쉽고 재사용하기 쉬운 구조로 만들겠다고 낑낑대며 만든 프로그램들이다. 네이밍? 개인적으론 꽤나 맘에 든다. 팔은 안으로 굽는다고 내 코드들이 이렇게 예뻐보일수가 없다. 메서드 분리를 연습하고, 클래스 분리를 연습하고, MVC패턴 도입을 연습해보며 고치기 쉬운 구조가 만들어졌다고 스스로 느끼기도 했다. 학교 수업시간엔 교수님이 학생들에게 자신이 코드를 짜는 걸 보여줄 땐 "저렇게 짜면 다른 사람이 보기 불편할텐데..분리를 좀 해야할텐데.."등등의 생각이 들기도 했다. 읽기 좋고 재사용성이 좋은 코드를 짜야된다는 일종의 마인드셋을 조금은 장착한게 아닐까? 일단 원하는 기능이 동작하면 된다는 마인드로 주먹구구식 구현을 하던 과거의 나에 비하면 정말 유의미한 성장을 거뒀다고 생각한다.

 그럼 프리코스 진행기간 동안 내가 부족한 점은 뭐였을까?

 일단 뭐, 자바 문법을 잘 몰랐다는 것에 한 표 던진다. 이번 프리코스를 통해 자바를 처음 공부하게 됐으니 당연하다고도 생각하지만..자바의 stream이라든지, stringBuilder라든지 이런 놈들을 통해 내가 원하는 기능들을 좀 더 편하게 만들 수 있었다. 일단 모르는 게 생기면 그 부분만 공부한다는 마인드로 프리코스를 임하긴 했고, 나도 프론트를 공부하던 시절의 경험들을 통해 이런 학습방법이 좀 더 효율적이란 건 알고 있지만 역시 기본서 하나를 읽으며 어떤 문법들이 있는지 알아두면 좋을 것 같다. 프리코스를 시작하며 구매한 자바의 정석을 지금까진 필요한 부분들만 봤지만, 한 번 처음부터 끝까지 정독하며 문법을 좀 더 공부하고 제공하는 기능들이 뭐가 있는지도 살펴볼 필요가 있을 것이다.

 두 번째로, 도메인 등에 대한 확실한 개념을 잡아야 할 것이다. 대략적으로 어떠한 것이라는 것만 알지, 도메인을 아직도 정확히 뭐다라고 설명할 수 없고, 이에 대한 나만의 기준도 확실히 잡히지 않은 상태다. 업계에서 자주 쓰이는 용어인 만큼 이 개념을 좀 더 공부해야 할 필요가 있다.

  세 번째, 결국 읽기 좋고 결국은 재사용성이 좋은 코드를 짜는데 미숙하다. 물론 이 부분에 대한 성장을 목표로 우테코에 지원하긴 했고, 이 부분에 대한 성장을 느끼긴 했지만 아직 많이 부족하다. 솔직히 피어리뷰를 하며 다른 사람의 코드를 보면 정말 감탄할 때가 많았다. 이 방법을 어떻게 생각했을까 싶은 부분이 있기도 하고, 디자인 패턴 시간에 배우는 패턴 등을 접목해 만든 사람들도 보였다.. 그 사람들은 그만큼 뼈를 깎는 노력을 많이 해온 사람들이 아닐까. 나 역시 계속해서 이런 부분들을 의식적으로 연습해야 한다.

 길다면 길고 짧다면 짧은 4주였다. 솔직히 휙 지나간 듯 하고, 내가 최종선발테스트에 뽑힐지는 모르겠다. 그러나 이번 우테코를 통해 읽기 좋고 재사용성이 좋은 코드를 짜는 법에 대한 힌트를 얻을 수 있었고, 몰입하는 경험도 쌓을 수 있었다. 물론 아쉬운 점들이 있긴 했지만 그래도 내 목표에는 한 발짝은 다가선 것 같다. 그러나 아직 부족한 점이 많기에, 이 한 발짝에 만족하지 않고 더 큰 보폭으로 걸어가고 뛰어가고 싶다.  그게 우테코를 통했으면 좋겠지만.. 설사 최종테스트선발이 안 되더라도, 테스트를 보고 떨어지라도, 내 목표를 향해 달려나가자. 

3주차를 시작하며 - 앞으로는 <할 수 있는 부분>에 집중하기로

 어느덧 프리코스의 절반이 지났다. 2주차 미션이 끝나고 다른 사람들이 제출한 코드들과 회고들을 살펴봤다. 우테코가 선발 과정이 아니라 양성 과정임을 잘 알곤 있지만, MVC패턴이라든지 stream이라던지 내가 잘 모르는 것들을 도입해서 미션을 해결하는 사람들을 보며 조금은 주눅이 들었다. 열정있고 실력있는 사람들과 함께하는 환경에서 공부하고 싶은 맘에 우테코를 지원하기도 했었고, 저런 사람들을 보면서 자극도 받으면서 "자바를 이번에 시작해서 잘 모를 뿐 내가 공부하면 더 예쁘게 코드 짤 수 있어!"라는 나름의 의욕도 생겼지만, 한 편으로는 어찌됐든 나보다 더 실력있어보이는 이 사람들 사이에서 내가 선발될 수 있을까라는 불안감이 들었다.

 그러다가 2시에 코수타를 통해 포비 님의 조언을 듣게 됐다. 나처럼 주눅이 든(?) 사람들에게 해주는 말이었는데, 바로 "할 수 있는 것에 집중해서 해봐라!"라는 말이었다. (feat. TDD는 쓰레기다) 괜히 뭐 클린코드를 지향한다고 처음부터 이것저것 좋은 것들을 싹 다 할라고 하지 말고, 우테코 미션에서 요구하는 내용들에만 집중해서 구현하려고 해봐라! 라는 의미였다.

 돌이켜보면, 나 역시 1 ~ 2주차 미션들을 진행할 때 각 미션의 요구사항들 뿐만 아니라 그 외에 추가적으로 이것저것 시도할려고 했었던 것 같다. "있어 보이기 위해" 커밋메시지를 죄다 영어로 작성했고, 잘 할 줄도 모르는 TDD 한 번 해본다고 테스트 먼저 만들고 프로덕션 만들기를 어설프게 시도했었다. 그런데 이것들이 해당 주차에서 우테코가 요구하던 것들이었는가? 커밋메시지 컨벤션을 지키라는 말이 있었을 뿐 한글/영어에 대한 제한은 없었고, TDD를 사용해라 라는 말 역시 없었다. 우테코가 원하던 것은 코치님들이 메일로 명시했던 "함수 분리 연습 및 테스팅 도구에 익숙해지기"이지 영어로 뽀대나게 커밋메시지를 작성하는 것이나 TDD를 도입한 개발에 익숙해지기가 아닐 것이다. 물론 내 딴에는 "이번 기회를 통해 성장하기 위해서, 습관을 들이기 위해서"라는 명목으로 저런 것들을 시도할려고 했던 것이고, 그것들이 결코 안 좋은 일들은 아닐 것이다. 그러나 되려 왠지 모를 조바심에 내가 너무 많은 걸 할려고 했던 게 아닐까.

 생각해봤다. 내가 지금까지 프로그래밍을 공부하면서 함수 분리에 신경을 쓰려 했던 적이 있었나? 테스트란 걸 해볼려고 했던 적이 한 번이라도 있나? 이 질문들은 우테코의 2주차 미션의 목표였다. 함수 분리? 이렇게까지 한 함수가 하나의 기능만 하게 하려고 기를 쓰고 해본건 이번이 처음이다. 테스트? 테스트 코드 자체를 이번에 처음 해봤다. 그리고 이걸 통해서 "한 함수가 하나의 기능만 하게 만드는 걸 통해 좀 더 읽기 좋고 고치기 쉬운 코드가 만들어지는군! " 이란 걸 느낄 수 있었고, "내가 직접 프로그램 수행해서 입력해볼 필요 없이 클릭 한 번으로 테스트가 가능하다고? 너무 편하잖아! "도 느낄 수 있었다. 굳이 뭐 커밋메시지 영어로 쓰고 TDD할려고 하려고 하고 이런 거 필요없이, 우테코에서 제시한 목표를 이루려고 하는 과정에서 조금씩 성장한 내 모습을 볼 수 있었다.

 솔직히 말하면, 나는 우테코에서 제시한 목표들을 파고드는 것도 빡세다. 저번 2주차만 해도 내가 함수 분리에 얼마나 많은 시간과 고민을 투자했었는가. 그런데 내가 커밋메시지를 영어로 쓴다고 계속 papago번역기 돌리면서 얼마나 많은 시간을 "날렸는가". (뭐 내가 영어를 잘 못하기 때문인 것 같기도..)  사실 한글로 써도 되는 부분이다. 배보다 배꼽이 더 커지면 안된다. 미션의 목표를 달성하려는 과정에서 성장할 것이고, 거기에 집중해야 한다는 생각이 들었다. 그 외에 집중하는 게 더 많아지면 안 된다.

 MVC패턴, stream..다른 사람들의 코드엔 뭐 SOLID의 ocp원칙에 위배된다는 류의 피드백이 달리는 것도 봤다. 지금의 나에겐 다른 차원의 피드백으로 느껴진다. 솔직히 대단하다. 그 정도로 코드를 보는 안목이 있다는 건 그 사람의 실력이 그만큼 엄청나다는 반증이다. 그렇다고 해서, 내가 MVC같은 걸 꼭 써야 한다는 강박은 갖지 말자. 당장 이번 3주차 미션의 목표는 클래스 분리 연습과 도메인 로직에 대한 단위 테스트 작성 연습이다. 이 두 목표를 달성하려는 과정에서도 나는 성장할 거다. 왜? 저런 걸 안 해봤으니까. 그 과정에서 MVC를 도입하는게 더 좋을 것 같다 이런 생각이 들면 써도 되겠지만, 처음부터 "다들 MVC쓰는 거 같으니까 나도 해야 돼!! "라는 생각으로 잘 모르는 MVC를 어설프게 하려고 하지 말자.  그거야말로 배보다 배꼽이 커지는 상황일 것이다.

 지금의 내가 할 수 있는 것에 최대한 집중하자. 미션의 목표로 주어진 내용을 달성하려고 하는 것에 집중하자. 저번 2주차에서 느꼈듯이, 그 과정에서 분명히 성장할 것이다. 최대한 우테코가 제시하는 목표를 달성하려고 하다보면, 선발 여부에 관련없이 스스로 돌이켜봤을 때 전에 비해 많은 성장을 했을 것이고, 내가 나아간 방향도 올바른 방향일 것이다.

클래스 분리 및 도메인 로직 분리 - 오랜만에 마주한 벽

이번 주 학습 목표는 

1. 클래스 분리 연습
2. 도메인 로직에 대한 단위 테스트 연습

이다. 이 둘을 중심으로 파고들기로 맘먹고 클래스 분리부터 해보자고 생각했다.

 2주차 미션을 진행하며 야구게임기 / 검증기 / 출력기 라는 3개의 객체로 분리해 코드를 짜긴 했지만, 이것보다 좀 더 심화된 객체분리를 연습해보는 것이 미션 취지에 맞다고 생각했다. 역할에 따라 분리하면 되겠거니 했지만 감이 잘 잡히지 않았다. 해본 적이 없었기 때문이다..그러다가 2주차 피드백에 실려있던 강의영상을 떠올렸다. 제이슨 코치님이 기능 목록을 정의한 후 각 기능들을 어떤 클래스가 수행하게 할지 정하고 클래스를 만드셨는데, 이 방법을 내 기능 목록에도 적용시켜 보기로 한 것이다.

 이 방법을 통해 기능 목록의 각 기능들을 어떤 클래스가 수행하게 할지, 메서드 이름은 뭘로 할지 가닥을 잡을 수 있었나. 크게 입력기 클래스 / 출력기 클래스 / 검증기 클래스/ 생성기 클래스 등으로 역할에 따라 분류했는데, 머지 않아 안개에 가려져 잘 안 보였던 큰 산 하나에 도착하게 됐다. 바로

도메인 로직과 UI로직을 분리해 구현하기

 정말 그게 뭔데 그 잡채였다. 내가 아는 도메인은 url창에 관련된 그 놈밖에 없는데..분위기상 그걸 말하는 건 아닌 것 같았다. 저번 주에 테스트 코드를 접하면서 테스트 코드에 대해 공부했듯이, 이번에도 도메인 로직이나 UI로직이 뭘 말하는 건지부터 공부해야겠다는 생각이 들었고 구글링부터 시작했다.

근데 시간이 그리 많이 걸릴 줄 몰랐지..

 도메인 로직, 즉 비즈니스 로직이 뭔지 설명하는 글들은 제법 있었지만, "결국 추상적인 말이다"라는 느낌을 세게 받았다. UI로직이 사용자와 상호작용하는 로직, 즉 입출력에 관한 것을 말한다는 건 금방 이해됐지만 도미엔 로직 이 놈이 문제였다. 일단 도메인이란 "해결하고자 하는 문제 영역"이란 의미를 가진다고 한다. 그러면 도메인 로직은 "해결하고자 하는 문제 영역에 대한 로직 주어진 문제에 대한 해결을 이끌어내는 로직"이 된다. 

 처음에는 음~ 이었다. 5초 뒤 응? 이었다. 해결해고자 하는 문제 영역에 대한 로직이란 건 알겠는데, 그 범위를 어디까지 봐야 하는지를 모르겠다. 한 마디로 추상적인 개념이라고 느껴졌다. 유효성 검사 로직을 예로 들면, 유효성 검사 자체가 문제에 대한 해결은 아니니 도메인 로직으로 보면 안 된다고 생각할 수도 있지만 문제에 대한 해결을 이끌어내는데 필요하니 도메인 로직이라고 볼 수도 있다고 느껴졌다. 코에 붙이면 코걸이, 귀에 붙이면 귀걸이처럼 생각하는 관점에 따라 도메인 로직이라고도 아니라고도 부를 수 있는 것 같아 엄청난 혼란이 느껴졌다. 

 개인적으론..이런 추상적인 개념들은 어떻게든 이해를 하고 넘어가야 하는 성격이다. 애시당초에 이 개념에 대한 이해가 없으면 이번 미션 진행이 힘들기도 할 것이다. 이대로는 안된다는 생각이 들어 구글링을 계속 했다..

 그러나 답을 찾지 못 했다. 심지어는 도메인을 명사로 부르는 경우도 있는 것 같아 더욱 혼란스러웠다. 해결하고자 하는 문제 영역을 도메인이라 하는 것 같은데, ~~하는 것의 도메인은 OO, OO, OO입니다 이런 식으로 명사로 부르기도 하는 걸 봤다. 혼란은 더욱 가중됐다. 도대체 도메인이라는게 구체적으로 뭘 말하는건지? 도메인 로직은 그럼 구체적으로 뭘 말하는건지?

 여기서 끝이 아니었다. MVC패턴 같은 걸 남들 다 쓰니까 나도? 라는 생각으로 쓰지 말자고 다짐했건만, 도메인 로직을 알아보던 중 MVC패턴이 도메인 로직과 UI로직을 분리하기에 좋다는 말을 듣게 됐다. 나도 모르게 무의식적으로 MVC패턴을 써야한다는 생각이 스며들기 시작했다. 그러면 따로 도메인 패키지를 만들어서 클래스들을 넣어야 할 것 같은데, 어떤 파일들을 넣어야 하지? 라는 생각이 들었다. 아직 도메인이 뭔지도 정확히 잘 모르겠는데? 라는 상황이었고, 잘 모르고 써본 적도 없는 MVC패턴을 써야 한다고 생각하니 너무 답답했다.. 

 프로그래밍을 공부하고 얼마 지나지 않아 마주친 포인터의 벽. 알고리즘 공부를 시작하며 느꼈던 재귀의 벽. 그리고 이번에 또 하나의 벽을 마주하게 되는 느낌이었다. 내가 잘 모르는 개념들, 별로 해본 적 없는 구조를 주어진 시간 내에 이해하고 구현할 수 있을까? 라는 생각이 들기 시작했다.

 그러다 포비 코치님이 해주신 말씀이 떠올랐다. 1주차가 끝나고 코수타를 진행할 때 해주신 "기능 목록이나 메서드 분리의 답은 없다! 스스로가 자신의 기준을 만들어 나가야 한다!" 라는 말씀이었다. 결국에 추상적인 개념이라면, 스스로의 기준을 만들어 가야 하는 것이고 도메인 로직 역시 내가 나만의 기준을 만들어야 한다는 생각이 들었다. 이게 맞는지 모르겠지만, 물론 맞다 틀리다가 있는지도 모르겠지만, 나름대로 사용자와 상호작용하는 로직을 UI로직으로 분류하고 원하는 것을 도출하는 로직을 도메인 로직으로 분류하기로 일단 기준을 잡아보기로 했다. MVC패턴? 처음에 3주차 미션을 시작하며 했던 다짐인 "할 수 있는 것에 집중하기"를 다시 한 번 상기시켰다. 이번 주 목표인 클래스 분리 연습에 초점을 최대한 두고, 메서드 분리나 함수 네이밍같은 부분(2주차가 끝나고 피어리뷰를 할 때 네이밍을 잘 한 것 같다고 칭찬들을 해주셔서 메서드 네이밍만큼은 남들보다 잘 하고 싶다는 생각이 있기도 했다)을 좀 더 신경쓰자!!라고 생각하며 다시금 멘탈을 잡을 수 있었다.

 확실히 느낀 건, 메서드 분리보다 클래스 분리가 훨씬 더 어렵다. 한 메서드가 하나의 기능을 하도록 분리하는 것도 물론 쉽지 않지만, 클래스 분리는 메서드 분리보다 개인의 주관이 좀 더 많이 들어가는 문제라고 느껴졌기 때문이다. 대표적으로 이걸 느꼈던 건 "검증(validation)"이었다. 금액 입력에 대한 유효성 검사, 당첨 번호 및 보너스 번호 입력 시 유효성 검사가 필요해서 나는 이걸 검증기 클래스들을 따로 만들어 구현했었다. 왜냐면 내 클래스 분리의 기준은 "역할"이었으니까. 그러나 이번 미션에서는 Lotto클래스에서 validate메서드를 호출해 검증하는 작업을 해야 했었다.! validate메서드의 내부에서는 실제로 검증을 수행하는 코드가 들어있었다. 내 생각은 검증은 따로 클래스를 만들고 그 클래스를 통해 수행하는 게 맞는 것 같은데, 왜 로또 객체 내부적으로 검증을 수행하게끔 하는거지? 라는 생각이 들었다. "로또 번호 검증"에서 "검증"에 포커스를 두면 검증기 클래스에서 담당하는게 맞고, "로또"에 포커스를 두면 로또 클래스에서 담당하게 할 수도 있다는 생각이 들었다. ..이런 부분에서 개인의 주관이 좀 더 들어가는 문제라고 느껴진 것이다. 이것 역시 나만의 기준을 만들어야 한다는 생각이 들었고, 지금 당장은 역할에 따라 구분하는 게 좋다는 생각이 들었던만큼 검증기 클래스를 따로 만드는 식으로 진행했다. Lotto의 validate는 검증기 객체를 이용한 검증을 하는 식으로 바꾸고.

클래스 분리를 이번에 연습하며 느낀 장점? 일단 확실히 한 파일의 길이가 짧아진다. 그리고 나중에 기능을 수정할 땐 그 역할에 맞는 클래스만 수정하면 됐다. 근데 이런 장점보다도 솔직히 고통스러웠던 시간의 임팩트가 훨씬 길었다..고통이 장점으로 얻는 달콤함을 압도하는 느낌..

도메인 로직에 대한 단위 테스트 작성 - private vs public..이거 뭔가 잘못됐다..?

 저번 주 미션에서는 TDD를 한 번 해보겠다고 설치면서 테스트 코드를 먼저 만들고 프로덕션 코드를 나중에 만드는 식으로 진행했었다. 그러나 이번 주부터는 "할 수 있는 것에 집중하기"를 다짐한 만큼 어설픈 TDD를 잘 해보겠다고 깝치지 말고 우테코에서 제시하는 요구사항과 목표들에 포커스를 맞추기로 했다. 즉 테스트 코드 먼저 만들고 프로덕션 코드 만들기라는 방식을 쓰지 않았다.

근데 문제는, 그렇다고 내가 아예 테스트 코드를 모든 기능의 구현이 다 끝날 때까지 한 개도 만들지 않았다는 것..

 모든 기능의 구현이 끝나고 나서야 테스트 코드의 작성을 시작했다. 1주차 미션을 통해서 기능 목록을 정의하는 나만의 기준을 세웠었는데, 그건 바로 목록의 각 기능들이 하나의 메서드 또는 코드에 대응하도록 만드는 것이었다. 2주차 때 이런 식으로 기능 목록을 세우는 걸 연습했던 만큼, 이번 3주차 역시 기능목록의 각 항목들이 하나의 메서드에 대응하는 구조였다. 즉 이 메서드들을 테스트 코드로 테스트하는 식으로만 만들면 그것이 단위 테스트가 될 것이다!

 라고 생각하고, 내가 생각한 도메인에 해당하는 각 클래스별로 테스트클래스를 만들면서 테스트코드들을 만들었지만..문제에 봉착했다. 테스트를 진행할 수 없었다. 왜냐하면..2주차 미션의 피어리뷰 당시 죄다 public으로 작성한 점을 지적받았어서, 이번엔 외부에서 호출할 수 있는 메서드말고 내부적으로 쓰는 메서드들은 죄다 private로 만들었기 때문이다.. 물론 다시 public으로 돌리면 될 일이다. 다 public으로 돌린다고 본 미션에서의 지장은 없다. 그러나 죄다 public으로 할 때 드러나는 단점이 너무나도 명확하지 않은가. 외부에서 함부로 다 쓸 수 있게 하는 건..좋지 않다. 내가 채택한 클래스 분리가 이런 결과를 도출해낸 건가? 다른 식으로 클래스 분리를 해야했던 걸까? 라는 생각들이 들었다. 그러나 시간이 충분하지 않았다. 클래스 분리 과정에서 그 죽일 놈의 도메인이 뭔지 따지던 과정에서 시간을 너무 많이 한 탓일까..팀플이 너무 많았던 탓일까....암튼, 다시 다 뒤엎고 처음부터 새 판을 짤 수 있을 시간은..솔직히 없었다..  할 수 없이 울며 겨자먹기로 public으로 돌리기를 선택했다. 단 멤버변수들은 private로 그대로 둬서 나름대로의 양심(?)은 챙겼다. 그럼에도 불구하고 메서드 단위로 테스트할 수 없는 부분이 존재했다. 예를 들어 LottoGameResult의 멤버변수로 earningRate(수익률) 변수를 뒀는데, calcaulateEarningRate를 수행한 다음 수익률을 확인할 수 있는 방법이 없었다. 왜냐면 내가 멤버변수들의 접근지정자마저 public으로 돌릴 순 없었서 private로 남겨뒀으니까!! 내 마지막 자존심이니까!! ㅠㅜ.. 이 부분은 할 수 없이 calculateEarningRate의 수행을 포함하는 더 큰 메서드를 수행하고 내가 원하는 로직이 이뤄지는지 확인하는 식으로 코드를 짰다.

 도메인 로직에 대한 단위 테스트 작성에 대한 연습이라..클래스 별로 단위 테스트를 진행하도록 만들면서 테스트 코드가 좀 더 깔끔해지는 느낌을 받았고, 도메인 로직들에 대해서만 테스트하며 중요한 로직에 대한 테스트 위주로 진행하는 게 중요하겠다라는 느낌도 받았지만..결국 아쉬운 감정이 크게 남는다. 좀 더 클래스 분리를 잘했더라면 어땠을까. 기능 구현 중간중간에 지금까지 만든 기능에 대한 테스트 코드를 작성하는 식으로 했으면 어땠을까. 등등과 같은..

그래도 성장했고, 배운 건 많다. 

 정말 힘든 고통의 1주일을 보낸 것 같다. 4개의 팀플, 가족모임, 알바를 제외한 모든 시간을 쏟아부었고 학교 개인과제는 전부 버리면서까지 시간을 투자했지만, 벽을 느끼는 순간들이 있었다고 할 수 있을 정도로 고통스러웠고 마무리한 후 뒤돌아보는 지금 이 시점에서도 개운함보다는 아쉬움과 찝찝함이 더 남는다. 지금 이 순간조차도 도메인에 대한 스스로의 기준이 구체적이지 않다. 이 죽일 놈의 도메인..

 그러나 뒤를 천천히 돌아보면, 물론 고통스러웠지만 그럼에도 배운 게 많다. 새로 배운 개념부터만 해도 enum이란 것이 있다. 열거형 이런 거 내가 3주차 미션을 시작하기 전까지는 잘 모르던 놈이었다. 그래도 이번 미션을 통해 enum이 뭔지를 알게 되고, 능숙하진 않아도 다루는 법을 알게 됐다. 클래스 분리를 통해 좀 더 읽기 좋고 재사용성이 좋은 코드를 짤 수 있다는 것도 알게 됐고, 구현이 다 끝나고 테스트 코드를 만드는 게 아니라 중간중간 만드는게 좀 더 좋다는 것도 느낄 수 있었다.

 고통스러운 시간이었고, 깨지고 부셔지는 시간들이었다. 그러나 전에 비해 성장한 게 느껴지고, 이 깨지고 부셔지는 것도 성장하는 과정에 포함되는 요소일거다. 성장하는 시간들이 달콤함만 있지는 않고 고통을 많이 수반할 거다. 그럼에도 포기하지 말고 나아가면, 더 큰 성장 더 맛있는 달콤함을 느낄 수 있을 것이다.

  이번 년도 롤드컵에서 DRX라는 팀이 우승했고, 데프트라는 선수가 데뷔 10년만에 우승하면서 "중요한 건 꺾이지 않는 마음"이라는 말을 했었다. 나도 꺾이지 않는 맘을 가지고 계속해서 나아가야겠다.

음..뭔가 오글거리는 것 같기도..