202202031 [20220203] 자바스크립트 스파게티 코드(?) 개선하기 개발자로 취직한 뒤 두 번째 유지보수 건이 들어왔다. 이 프로젝트는 스프링 부트로 구성되었지만, 생각보다 코드가 복잡하게 얽혀있었다. 아직 경험이 부족해 이것도 스파게티 코드라고 해야 할지는 모르겠는데, 왜 이렇게 짰는지 모를 부분이 꽤 있었다. 첫 번째로, 자바 코드의 Controller에서 DTO를 생성하고 DTO의 모든 변수들을 getter로 뽑아 변수를 만들어 놓았다. (왜죠?) DTO의 변수명이 마음에 안 들었던 걸까? 애초에 DTO의 목적 자체가 데이터베이스의 Row를 한 번의 쿼리로 가져와서 필요한 부분에 쓰기 위함이 아닌가? 변수로 선언하고 각각의 변수를 하나씩 다른 DTO에 setter로 넣어주고 그걸 ModelMap 객체로 내보낸다. 게다가 if-else if-else 문으로 경우에 따.. 2022. 2. 3. 이전 1 다음 728x90