Victoree's Blog

함수 플래그 인수는 추하다 왠만하면 잘못되었을 확률이 큼 join_funding_channel() 한 함수에서 두 가지 작업을 하고 있음 (채널 추가 && 채널 추가 여부 확인) id 값을 kakao_account_id, talk_user_id 두 가지를 지원하고 있는데, 하나만 지원하도록 kakao_account_id 인 경우는 함수 밖에서 처리하고 인풋으로 넘기면 되지 않을까.. 인수가 3개 이상이 되면,, 캡슐화 시키는 게 나을까요? 보통 같이 응집되어 붙어다니는 이유가 있음 클래스나 하나의 오브젝트로 감싸는 방법도 좋음 (케바케) core나 앱의 utils에 type을 만들어서 관리하는 방법도 있음 물고있는 컨넥션이 있을 때는 클래스로 구성해서 분리해도 좋음 굳이 특정 값을 들고 있을 필요가 없을..

코드 리뷰 시간을 줄이기 위해 클린코드라는 책을 읽고, 매주 책 내용에 대한 토의 및 개발한 코드 중 개선할 부분을 찾아보기로 하였다. 1장. 깨끗한 코드 Software 유연해서 소프트웨어라지만 아니다! models 같이 low한 영역일수록 고치기가 더 어려워짐 코드는 항상 클린한 상태를 유지하도록! 더러운 코드에 기능만 붙이다가 어느 순간 기능을 추가할 수 없는 코드로 변질 -> 몰락 넷스케이프 흥행과 몰락 추상화는 어려워 Minter 앱 개발 당시, Queue -> Serial Pool로 변형 클래스의 상속이나 캡슐레이션 같은 추상화말고 기능적인 측면에서 어떤 부분을 추상화시킬지는 항상 고민하면서 구현해야 한다 2장. 의미있는 이름 data라는 변수를 툭 사용하지 말것 # data = { not_v..