뭐 구현할때마다 유지보수 이런건 생각안하고 하나에 다때려박네 시발진짜
예를 들면 데이터 조회해서 가공하는 메소드가 그냥 최신 30개 조회해서 처리하도록 되있길래 싸해서
~씨 이거 처리개수나 인덱스도 매개변수로 받도록 해야 확장할때 편하지 않을까요?? 수정좀 부탁드릴게요 하고 작업사항 대충 정리해서 지라에 남겨줬는데
난당연히 오버로딩으로 짤줄알고 그정도는 지라에 안남겨놨는데
나중에 코드 보니까 그냥 개수 인덱스 받을 매개변수 2개 추가해놓고 메소드에서 변수 입력여부 일일히 검증하고 있음...
기본적인 개발론 자체를 모르는거같음....
이거 그냥 내가 지라에 업무지시 존나게 상세하게 안써준게 문제냐 이새끼가 문제냐
나도 좃고수는 아니지만 좃소신입에 이런것도 바라면 안되는거엿냐고 ㅋㅋ
허접한 우리가 미안해....너는 얼른 대기업가라....
해당 댓글은 삭제되었습니다.
이새기 코딩컨벤션도 맨날 안지키고 컨트롤시프트알트L도 모르는지 맨날 탭 띄어쓰기 줄바꿈 씹창내는데 AS로 1인분개발자 만들기... 가능할까...?
신입이니까 그래도 갱생 여지가 있지 니보다 나이 많은 경력도 똥 푸짐하게 쌀 수 있다
슬픕니다...
컨쉬알L이 포매팅 단축키인가? 그 새기 ide 들어가서 format on save 같은거 키게 해라 ㅋㅋㅋ
맞음 ㅋㅋㅋㅋ 나중에 시간날때 내가 설정해줘야겠네..
해당 댓글은 삭제되었습니다.
SI는 아니라서 그정도는 아닌데 회사 윗놈들이 맨날 재촉 ㅈㄴ해대서 느긋하게 이것저것 가르쳐줄 시간 없는게 아쉽긴 한듯 내업무 끝내기도 바쁘니까... ㅅㅂ 맨날 똥싼거 나중에 찾아내고 알려주고 대공사하고 끔찍하다 이거에요
신입이니까 지적하고 가르치면 되잖아 나는 경력많은 시니어가 저러고있어서 그래도 수평적인 조직이라 지적을 하긴 했지만 매번 지적하는것도 모양새 이상해서 힘들었다 다행히 그사람 퇴사함
근데 또 신입인데 나보다 나이많아서 계속 뭐라고 하면 좀 미안해지긴함... 진짜 소통이 제일어렵네..
나이많은건 무시해 감정없이 지적할건 지적해야지 사실 코드리뷰 절차가 있어야 이런 피드백이 자연스러운데 그런 절차를 만드는건 안됨?
좃소라 인력이 부족해서 주기적으로 코드봐줄 시간이 따로 없는게 큰듯 내 작업 좀 더 빨리 처리하고 나라도 코드리뷰 피드백좀 자주해야겠다...
근데 그냥 궁금한건데 매개변수로 인덱스랑 개수 넘기는 방법은 유지보수에서 어떤점이 단점인거임?
나중에 쓸때 사용하지 않을 변수까지 다 지정해줘야 하고 만약에 새로운 매개변수를 사용할 필요가 있게 된다면 결국 또 오버로딩으로 구현해야 하니까 처음부터 나누는게 낫지 않나? 라는 생각이였는데 그럴 필요 없다는 의견도 있네요
좋은 유지보수: 나중을 대응하지 않는다.
나중에 갯수를 변경할 필요가 있어서... 나중에 인덱스를 변경할 필요가 있어서 ... 나중에 소팅할 수 있어서 ... 나중에 소팅에 필요한 인자를 바꿀 필요가 있어서 ... 나 중에 페이지네이션도 해야해서... 나중은 끝이없다
그리고 웬 오버로딩이여. 이정도는 걍 디폴트 매개변수 쓰면 되겠구만. 오버로딩으로 처리하려면 벌써 함수가 2개가 늘어나네. 쓸데없이 함수 늘어나는거 좀 ㅋ
그..런가? 저는 함수 늘어나도 나중에 추가로 뭔가 만들때 편해지는게 좋아서 오버로딩을 하는 쪽으로 개발했는데 아닐수도 있겠네요 역시 더 공부해야겠음... 감사합니다
회사 레거시코드가 (이거랑은 다른환경임) 유지보수때문에 개쌉스파게티 상태라서 좀 더 신경쓰면서 만들고 있긴해요 예전부터 자꾸 운영에서 이상한 요청 들어와서 최대한 대응 가능하도록 만들고 있었는데 역시 컴팩트하게 필요한 부분만 개발하고 나중에 추가하는게 맞는건가 싶기도 하고 크아악
유지보수에 대한 관점은 서로가 다를 수 있고 꼭 정답이 있는건 아니라고 생각. (유지보수에 대한 본인만의 생각이 있을수도있고, 아니면 생각이 없을수도있고..) 이게 옳은 유지보수 방법이니 내 방식을 따라라 라고 말하기보단 내가 생각하는 상황과 도구에 다른 운영 비용등을 설명하면서 이야기해주면 더 좋지않음/
좋은말씀 감사합니다 제 의견낼때도 주장을 좀 더 뒷받침해주는 이야기를 추가해야겠네요
나라면 어떻게 했을까 고민하면서 클린코드 책을 편다.. 큼큼..
오버로딩으로 하는이유가 기존 호출방식도 호환시키려고 하는건가요
인덱스값으로 처리하는 컨트롤러가 있고 개수로 처리하는 컨트롤러가 있어서 (스프링임) 그렇게 만들려고 했었네요 이게 최선일지는 저도 잘 몰름
그런 방법론 같은거 볼만한 책 추천좀 해주실수있나요 - dc App
글삭 ㄴㄴ 내용하고 댓글하고 생각하면서 한번 나중에 읽어보게 - dc App
예에전에 전공서에서 봤던거같은데 기억이 잘 안나네요 좋은책 발견하면 달아드리겠습니다
코딩에 정답은 없지만... 상수 박아넣는 건 피하는 게 좋긴 하지
상수는 ㄹㅇ 인정!!
여기에 한풀이 글쓰고 댓글놀이 할 시간에 10분 들여서 피드백 했겠다 쩝ㅠ 같이 배우고 어깨동무하며 도와가요 그렇게 개발자 인맥쌓는거지 별거 있남
에이 제 생각 얘기하고 이런점때문에 나누는게 좋다고 생각한다 이렇게 수정하면 어떻겠냐 얘기해주긴 했죠 다른분들 댓글보니까 저도 아직 멀었는듯 갤에서도 배워갑니다
관점을 바꿔. 신입은 갱생이 불가능할 수도 가능할 수도 있음. 그 신입에게 니 생각을 불어넣는 과정 또한 너한테 단련이 됨. 신입을 납득시킬 수 있게 설득한다고 생각해봐. 당연하게 생각했던 부분도 이게 맞나? 하고 고민하게 되고 얘가 못하는 이유에 대해서 고민하다 보면 내가 못하는 이유도 알게 됨.
수정 부탁하는거 좀 귀찮다고 생각하는 부분도 있었는데 생각해보니 제 공부도 되는 것이였네요...
댓글 단거 딴거 쭉 읽어 보니까 형이 말한 구조가 더 좋아보이네. 근데 관점을 바꾸는것만으로도 형이 대단한거임. 신입은 아무 생각 없이 그렇게 만들었고 형은 최소 세단계를 더 고려하고 있으니까
난 신입인데 1년차 선임이 그래서 넘 슬프다.. Vo가 100줄이 넘어가는데 정상인가 싶다..