섣부른 디자인패턴의 표본을 보고 있습니다.

프로젝트 같은분 개발자 소스 보는데 패키지명이 디자인패턴명이더라구요. 당당히 적어놨길래 소스 까봤더니 if와 else로만 되어있는 분기문을 전략패턴이라고 클래스 2개 만들어서 써놓고, 추상 팩토리 패턴이라고 해놓고 클래스 2개만 만들어서, 클래스 한개에다가

분기문으로 메서드 하드 코딩해놓았더라구여. 람다식으로 쓰면 되는 한줄짜리를 굳이 메서드로 따로 빼서 foo1, foo2, foo3, foo4, foo5, foo10 으로 하드코딩해서 한번만 쓰고, 버립니다.

뭐 잘돌아가면 그럴수 있는데 개발 초반이라 developer한테 직접푸시권한을 줬더니, 개발 브랜치에 올릴때 서버 구동안되게 푸시를 해가지고 소스 까다보니 나온겁니다.

돌아가면 당장 문제가 안되는데 돌아가지도 않고, 제대로된 디자인패턴 적용도 못해놓고 대놓고 쓰니 어이가 없더라구요.

오늘도 개발서버 젠킨스 빌드 오류 내놓고 안고치고 칼퇴했습니다. 테스트는 해보고 올리라고 주의는 몇번 줬는데 안고치네요.