[💬] 이런 코드가 정녕 맞는건지 확인좀..

[익명(1.225)]

팀 개발을 하는데, 개발자가 나 포함 2명이야

근데 내가 나중에 들어갔고, 기존 개발자가 코드를 다 이런식으로 짜놨거든

내 기준 위와 같은 코드는

변수, 함수 명명 규칙도 어긋나고

변수, 함수 뜻도 너무 모호하고

코드 가독성도 엄청 떨어지고

Start() 함수도 이상한데에 있고

뭐 기타등등 이런것들이 꼭 지키지 않다 하더라도 프로그램은 돌아가겠지만 이거는

협업은 물론이고 나중에 스스로 자기 코드 볼때도 햇갈릴거 같거든?

이게 몇십개의 스크립트에 몇천줄로 있는데 코드 볼때마다 머리 터질거같단말야

몇차례 말을 했는데도 이게 잘 고쳐지지가 않고 있거든

그래서 이쯤 내가 생각을 해봤는데

혹시 이건 사람마다 코드 스타일의 차이인걸까?

주관적인 판단으로는 이렇게 짜는건 진짜 아닌거 같은데, 나도 초보 개발자고 주변에 아는 사람이 없어서

혹시 내가 잘못생각하고 있는걸까 싶어 글 올려봄

[익명(58.124)]

대가리 뚝스딱스 깨버려도 됨. 협업할 때는 코딩 컨벤션 정해놓고 하는게 좋은데 유니티로 개발한다면 유니티에서 권장하는 c# 코딩 컨벤션대로 개발하는게 좋음

[익명(221.163)]

변수명을 OnDamge으로 한 것은 이상하네요. 보통 그런 이름은 이벤트 처리 함수에 붙입니다. bDamaged가 적절하겠네요. 너무 빡빡하게는 하지 말고요 몇 개 정도는 코드 규칙을 만드는 것이 좋습니다.

[흉황(smk4397)]

나같으면 진지하게 탈주 고민 해볼듯 - dc App

[익명(114.206)]

https://learn.microsoft.com/ko-kr/dotnet/csharp/fundamentals/coding-style/coding-conventions
이거
보여주고 이거 따르라고 하셈 혼자 코딩할거아닌데

[익명(222.117)]

컨벤션 짜서 맞추면 되는 일임. 라이더 저장시 포맷 기능도 활용하고

[익명(1.225)]

1년동안 컨벤션 언급했음

[익명(222.117)]

1년동안 이지랄 나도록 냅둔거임?

[익명(222.117)]

일단 줄임말이 너무 많고 변수명 오타도 심하고 (enegy가 아니라 energy) difens는 디펜스임? 그냥 소리나는대로 적었네 일단 PR 방식의 워크플로 도입하고 저지랄하면 코멘트 달고 빠꾸먹여서 다시 짜오라고 하셈 그래도 해결 안되면 퇴사가 답

[익명(1.225)]

나도 잘하는건 아니니까 견디면서 하긴 했는데 한계가 와서..

[익명(1.225)]

도저히 변수, 함수명이 뭔소린지 모르겠고 가독성 딸려서 주석 달아달라고 했는데 주석도 void enegy()//에너지

[익명(1.225)]

살짝 오바해서 이런식으로 써서 진짜 머리 터져버릴거같음

[익명(222.117)]

주석은 코드만으로 직관적이지 않은 복잡한 코드에나 적는 거임 보통은 코드 / 함수명 / 변수명이 직관적이라서 주석이 필요없어야 함 당장 해결해야할 문제임. 기존 코드도 다 리팩토링하고

[익명(222.117)]

솔직히 이 정도면 회사 대빵한테 찔러서 저 개발자 짤라도 합법이다

[익명(1.225)]

회사가 아니라 소규모 초보 팀이긴해

[익명(222.117)]

팀에서 나가지 않고 남아있고 싶으면 PR 작업 방식 꼭 도입해. 서로 코드 리뷰해주고 approve해주고.

[익명(1.235)]

심각한수준임

[익명(1.235)]

독학으로배워도 저거보단정리잘하겠다

[ㅇ(211.179)]

bool이면 보통 Is붙일텐데 On은 이벤트쪽에 붙이는데, 이정도개발자면 먼저 탈주해야됨 아니면 그개발자 나가면 똥처리다한다

[익명(211.196)]

거기서 뭐하세요?

[유타몰(ssamtoo)]

여백이 너무 화나요

[익명(183.100)]

나 현기증났어

[클라리네이(ggs7839)]

탭으로 들여쓰기 다르게 한거랑, if else 줄 바꿈 다 다른거보면 작정하고 일부러 저러는건데, 코딩 스타일이 아니지. 아예 고정적인 규칙 자체가 없는데 ㅋㅋ

[익명(1.225)]

일부러 그러는건 아닐걸 들여쓰기도 스페이스바 한번 쓸 때 있고 두번, 세번, 다섯번, 여섯번 등등 자유롭게 하심