[moin-review] 2021-10-07

김_리트리버·2021년 10월 8일
0

Facts

특정 송금 협력사의 경우 송금을 받는 사람 통장의 메모에 특수문자가 입력되면 안된다고 해서 수정작업을 진행함

기존 code 를 보니 메모내용에 사용자가 입력을 하지 않을 경우 송금을 보내는 사람의 전체 이름을 서버에 보내는 것으로 해놓았음

해당 송금협력사는 송금을 보내는 사람이름에는 특수문자가 가능하도록 해놓은 상태라 통장메모에 특수문자를 금지하는 정규식을 걸어놓아도 DB 에는 특수문자가 포함된 text 가 계속해서 넘어가고 있었음

때문에 현재 송금을 보내는 사람이름에 특수문자가 있을 경우에만 통장메모에 사람이름을 넣어서 정규식을 거치도록 하여 사용자가 직접 특수문자를 제거하도록 유도함

Findings

Feelings

legacy code 가 너무 중구난방이라 별거 아닌 거 수정하려고 해도 시간이 너무 걸린다.

관심사를 분리하지 않고 그냥 꼴리는 대로 여기저기 쑤셔박으면 이후 프로그램의 예측 가능성이 떨어져 유지보수하기 굉장히 힘들어 진다.

난 미래의 내 코드를 볼 사람들에게 욕먹지 않게 작성해야 겠다.

profile
web-developer

0개의 댓글