프로그래밍

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

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

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

 

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

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

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

거기까지는 다 좋음

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

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

 

예를 들어

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

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

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

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

 

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
무분별한 사용은 차단될 수 있습니다.
번호 제목 글쓴이 추천 수 날짜 조회 수
5691 [프로그래밍] 공통코드테이블은 대체 왜 만드냐 9 잠적자 0 9 시간 전 247
5690 [프로그래밍] 토이프로젝트 주제 선정 2 개드립눈팅1세대 0 11 시간 전 136
5689 [프로그래밍] 엥 구글 플러터 유기각 재는거임?? 4 최수연 0 2 일 전 281
5688 [프로그래밍] 반도체 장비 업계인 있음? 9 캡틴띠모 0 3 일 전 249
5687 [프로그래밍] 안드로이드 책 추천좀 6 집에가게해줘 0 4 일 전 145
5686 [프로그래밍] 폰 스크리닝 해 본 사람 있어? 3 무지개빛푸딩 0 4 일 전 366
5685 [프로그래밍] jsp 트리메뉴 만들고있는데 구상한게가능한지 의견좀물어볼께 11 평택국 0 5 일 전 149
5684 [프로그래밍] JPA 도와줘어억 ㅠ 10 모그리또 0 5 일 전 238
5683 [프로그래밍] 의사는 뽑는 인원 제한하는데 부캠은 왜 제한 안 할까 5 조강현 0 7 일 전 360
5682 [프로그래밍] 그 혹시 게임쪽 종사자 있음? 17 god79ii 0 11 일 전 621
5681 [프로그래밍] 코린이 ㅅㅂ 뭐가 문젠지 모르겠어요 9 집에가게해줘 0 11 일 전 439
5680 [프로그래밍] Dear Imgui 라고 아시나요? 2 년째모쏠 0 11 일 전 243
5679 [프로그래밍] 현업개발자분들 주말엔 편하게 쉴수있나요? 10 키로 0 12 일 전 855
5678 [프로그래밍] 무엇이든 물어보세요. 28 변현제 0 14 일 전 410
5677 [프로그래밍] 개발자보단 엔지니어가 취업이 잘됨 5 iillillill 2 15 일 전 728
5676 [프로그래밍] 프론트엔드 개발자 연봉 1억 넘는 사람 있어? 13 잠적자 0 15 일 전 633
5675 [프로그래밍] Exiftool 이거 일본어 못 읽는데 13 부터시작하는이세... 0 17 일 전 234
5674 [프로그래밍] 반응형 웹페이지가 내가상상한거랑 좀 다르네 4 평택국 0 18 일 전 448
5673 [프로그래밍] 고졸 FE개발자 연봉, 상황에 조언좀.. 19 쾅꿍꿍 0 18 일 전 605
5672 [프로그래밍] 물경력들 보면 책임을 이해못하는것같음 5 mils 1 19 일 전 414