pile·
백엔드·LINE EngineeringLINE Engineering·

코드 품질 개선 기법 29편: 고르디우스 변수

문제로컬·원격 데이터 동기화 클래스에서 ID 집합 비교 결과를 `localEntryIds`, `remoteEntryIds` 변수에 담아 추가·업데이트·삭제 분기를 만들면, 변수 하나로 여러 의미가 묶여 "고르디우스 변수" 가 된다.
접근의도별로 변수를 잘라낸다. `idsToAdd`, `idsToUpdate`, `idsToDelete` 처럼 결과 집합을 미리 계산한 뒤 각 처리에 그대로 전달한다. 비교식 `id !in localEntryIds && id in remoteEntryIds` 같은 표현을 변수명으로 흡수.
결과분기 조건이 변수명으로 자기 설명적이 되고, 분기마다 의미가 명확해진다. 코드 리뷰 시 변수 한 개에 여러 책임이 묶인 패턴을 식별하는 안티패턴으로 정착시킨다.
LINE Engineering
LINE Engineering 블로그
원문은 여기서 이어서 읽을 수 있어요
원문 읽기
읽음 (0)
·takedown 요청

이 글과 비슷한

  1. 백엔드·cloudflare-blogCloudflare Blog·

    hyper HTTP 라이브러리의 버그를 발견한 방법

    Cloudflare의 Images 서비스를 Unix 소켓 기반 아키텍처로 재구성한 후, 대용량 이미지 응답이 중간에 잘리는 버그가 발생했다. 14.8MB 응답에서 219KB만 전달되고 HTTP 200 OK는 정상 반환되어 애플리케이션 레벨에서 탐지가 불가능했다. 원인은 hyper 라이브러리의 dispatch 루프에서 flush 완료 여부를 확인하지 않고 연결을 종료하는 경쟁 조건이었으며, strace로 커널 호출 순서를 추적해 root cause를 특정했다. 최종 수정은 upstream PR #4018로 hyper 레포에 병합됐다.

    #rust#debugging#race-condition+2
  2. 백엔드·stackoverflow-blogStack Overflow Blog·

    CherryScript — 데이터 파이프라인을 위한 커스텀 Python 인터프리터 설계

    CherryScript는 데이터 기반 워크플로우 최적화를 위한 커스텀 DSL로, Python 기반 인터프리터로 구현됐다. 일반 Python 인터프리터의 메모리 병목과 AST 트리워킹 성능 문제를 극복하기 위해 스트리밍 렉서, 바이트코드 컴파일, 불변 상태 관리의 세 가지 최적화 전략을 채택했다.

    #dsl#python#interpreter+2