프로그래밍

변수명 이상하게 쓰는 동료 어떡하지?

나는 좀 변수명에 예민한 편인데

내가 감도를 좀 낮춰야할까?

 

오래된 서비스라 프로젝트 코드가 많음

웬만한건 쓰고 싶은 코드가 프로젝트 내에서 찾기 하면 나오는 수준임

그래서 작업의 대부분이 이전 코드 복사한 뒤에 작업하거나 기존 코드를 활용하는데

거기까지는 다 좋음

(다 이해하고 수정한다고 하니 더 좋음)

근데 볼때마다 거슬리는게 변수명이...

 

예를 들어

음료만 파는 자판기 로직이 있다고 쳐

지금까지는 음료를 꺼내서 마시기만 했다면

이제 자판기에서 도시락도 팔고 유저는 이걸 마시지만 않고 먹기도 하는거야. (그 외에는 거의 같다는 가정)

그렇게 됐을때 결과물이 이래

 

lunchBox = dispenser.pull(beverageType)

user.drink(lunchBox);

 

나는 여기서 beverageType 을 productType 등으로 확장한 변수명 (enum 이름도 바뀌어야겠지) 으로 썼으면 하는거고

drink 대신 consume 같은 다른 표현을 썼으면 함.

스펙이 확장되면 어색한 메서드명은 좀 바뀌었으면 함 (다른 타입이 들어왔을땐 똑같이 작동하더라도 내용이 추가되었으니까)

(하는일은 예전과 똑같고 drink 안에 lunchBox type 이면 어떻게 처리하는지 간단하게 추가되는 정도)

 

그렇게 큰 변경이 필요한것 같지도 않아

그냥 메서드명 변경인데.. (drink 를 한 20군데에서 쓴다고한들 그냥 변경하면 되는거잖아)

 

근데 매번 말하기도 그렇다

몇개는 거슬려도 넘어가자 싶어서 넘기는데

정 아닌거 같은거만 꼬집어서 말하는중임..

11개의 댓글

2024.01.05

아예 못알아보겠는 거 아니면 그냥 저냥 넘어가.... productType을 pt 로 안쓰는게 어디야

0
2024.01.05
@드론
0
2024.01.05

매번 코드리뷰 ㄱ?

0
2024.01.05
@야근좋아

저희는 항상 코드리뷰후 머지해야해요.. ㅠㅠ

0

남의 코드 수정하는거 꺼리면 그럴수있음

0
2024.01.05
@다르다르다르다

코드 자존감 같은건가

0
@Portrait

아 본인이 쓴건데 안바꾸는거면 왜 그랬는지 물어보고 너 의견도 말해주면 좋을듯 확실하게

0
2024.01.05

혹시 모를 사이드이펙트 땜시 나도 잘 손안대는 편이긴 함..

0
2024.01.05

여러가지 방안이 있지만 몇개 써봄

 

1. PR 리뷰를 더 적극적으로 한다

->"이러이러한 이유로 이런 네이밍이 더 적절하지 않을까요?" 하면 설득력 있으면 대부분 수용하고 바꿈

 

2. 회고나 미팅 시간에 네이밍 이슈 래이즈한다

-> 사람을 지적하지 말고 코드를 지적하고 설득력 있으면 대부분 바꾸자고 함

 

3. refactor/ 브랜치 새로 따서 직접 고치고 PR 올린다

-> 말마따나 별로 공수가 안드는 일이니까 꼴뵈기 싫으면 직접 하면 됨

-> PR 올리고 너 말이 맞는지 틀린지 리뷰 받아서 반영하면 됨

0
2024.01.05
@예스오얼노

내가 리뷰를 달면 팀전체한테 노티가 가는 시스템임

네이밍관련 코멘트는 할때마다 먼가 머쓱함

그냥 내 스스로 괜한걸로 예민하게 구는거같다는 느낌

한 2번정도는 적극적으로 리뷰해봤는데 습관이신건지 은연중에 자주 저런게 나오는 느낌

사실 이게 제일 큰거같음

내가 너무 예민한거 아닌가?

 

그냥 코멘트로만 말하지말고 1on1 신청해봐야겠다

0
2024.01.05
@Portrait

그니까 2, 3번을 달았자너 ㄱㄱ

0
무분별한 사용은 차단될 수 있습니다.
번호 제목 글쓴이 추천 수 날짜 조회 수
5711 [프로그래밍] 요즘 앱개발 인력시장 어떰..3년차 2 센치해요 0 23 시간 전 217
5710 [프로그래밍] 컴포즈 Box 컴포넌트가 안 나와... 1 집에가게해줘 0 2 일 전 120
5709 [프로그래밍] 아 ssl 적용햇는데 개정신없네 9 넌또화나있네 0 3 일 전 245
5708 [프로그래밍] 패스트 캠퍼스 <---- 얘내는 가격 인상 원툴임? 5 조강현 0 5 일 전 334
5707 [프로그래밍] 클라가 파이썬 셀레니움같은거 써서 클릭하고 그러는걸 감지 ... 5 리옴므 0 7 일 전 222
5706 [프로그래밍] leetcode 50일 달성 1 JimmyMcGill 1 7 일 전 194
5705 [프로그래밍] 그냥 개인공부용 git 만들건데 5 년째재수강 0 7 일 전 283
5704 [프로그래밍] html 자바스크립트 질문 19 책걸이 0 7 일 전 327
5703 [프로그래밍] 아니 시바 이게 무슨일이야 4 인간지표 0 8 일 전 344
5702 [프로그래밍] 아두이노 키트 아무것도 모르고 사도 될까? 6 그것 0 8 일 전 267
5701 [프로그래밍] 횽들 Vimeo에 올라가있는 동영상의 원본크기를 확인할 수 있... 13 카뜨만두 0 8 일 전 193
5700 [프로그래밍] c# 이벤트와 델리게이트 13 RX7900XTX 0 11 일 전 315
5699 [프로그래밍] Aws 람다에 파이썬 올려서 결과 받아오는데 11 아르피쥐 0 13 일 전 356
5698 [프로그래밍] 마리아DB mediumtext 그냥 쓰고 싶은데 21 잉텔 0 13 일 전 226
5697 [프로그래밍] 안드로이드 씹뉴비 질문이요 2 집에가게해줘 0 14 일 전 132
5696 [프로그래밍] c언어 7년했는데 이런게 되는거 처음알았음.. 4 케로로중사 0 15 일 전 985
5695 [프로그래밍] 파이썬 1도 모르는데 GPT로 프로그램 뚝딱 만듬 2 푸르딩딩 1 18 일 전 862
5694 [프로그래밍] 담주 면접잡혔는데 8 삐라루꾸 0 18 일 전 613
5693 [프로그래밍] 아두이노 부트로더를 구웠는데.. 4 렙이말한다ㅡ니가옳다 0 19 일 전 243
5692 [프로그래밍] IOS 개발자 있나여? 1 g4eng 0 21 일 전 265