프로그래밍

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

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

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

 

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

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

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

거기까지는 다 좋음

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

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

 

예를 들어

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

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

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

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

 

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
무분별한 사용은 차단될 수 있습니다.
번호 제목 글쓴이 추천 수 날짜 조회 수
5719 [프로그래밍] 하이브리드 웹뷰기반 앱은 rn이 정석이야? 3 잠적자 0 13 시간 전 149
5718 [프로그래밍] c# webview2 도움요청함.. 7 carpediem 0 19 시간 전 107
5717 [프로그래밍] 현업 개발자형들 맥씀? 8 이또히로부미 0 20 시간 전 194
5716 [프로그래밍] libtorch에서 cuda 안불러와지는거 도움! 2 Hakat 0 2 일 전 121
5715 [프로그래밍] 뭔가 게시판이 애매해서 그런데 gis 잘아는 사람? 1 하늘늑대 0 3 일 전 162
5714 [프로그래밍] 컴포즈가 프리뷰랑 폰에서 다르게 동작해요 1 집에가게해줘 0 3 일 전 116
5713 [프로그래밍] 난바보다) 크로미움 램사용량 문제 일단 해결 2 ye 0 4 일 전 282
5712 [프로그래밍] k8s DNS 이슈는 해결이 안되나보다. 잉텔 0 5 일 전 170
5711 [프로그래밍] 분노) 진짜 유튜브 구글 패악질 토나오네 씨발 17 ye 0 6 일 전 767
5710 [프로그래밍] 프론트엔드 공부하려는데 언어 추천좀 7 스트리플 0 7 일 전 348
5709 [프로그래밍] 객체지향 뽕에 취하지마라 8 69746974 2 8 일 전 407
5708 [프로그래밍] 요즘 앱개발 인력시장 어떰..3년차 2 센치해요 0 11 일 전 387
5707 [프로그래밍] 컴포즈 Box 컴포넌트가 안 나와... 1 집에가게해줘 0 12 일 전 148
5706 [프로그래밍] 아 ssl 적용햇는데 개정신없네 9 넌또화나있네 0 13 일 전 304
5705 [프로그래밍] 패스트 캠퍼스 <---- 얘내는 가격 인상 원툴임? 5 조강현 0 16 일 전 397
5704 [프로그래밍] 클라가 파이썬 셀레니움같은거 써서 클릭하고 그러는걸 감지 ... 5 리옴므 0 17 일 전 265
5703 [프로그래밍] leetcode 50일 달성 1 JimmyMcGill 1 17 일 전 237
5702 [프로그래밍] 그냥 개인공부용 git 만들건데 5 년째재수강 0 17 일 전 336
5701 [프로그래밍] html 자바스크립트 질문 19 책걸이 0 17 일 전 369
5700 [프로그래밍] 아니 시바 이게 무슨일이야 4 인간지표 0 18 일 전 380