search

f-lab-edu / used-market-server

[카카오, 넥슨 합격 / 이직 연봉 약 70% 상승] 번개장터 같은 중고거래 사이트
41 stars 3 forks source link

#30 웹소켓을 이용한 1:1 채팅 구현 #32

Open junshock5 opened 4 years ago

f-lab-dev commented 4 years ago

이 부분은 멘토링시에 말씀드린 내용대로 전체 재검토 부탁드려요~

f-lab-dev commented 4 years ago

리뷰를 하면서 느끼는게 리뷰에 나온 내용에 대해 알아보고 익히는게 아니라 그때그때 리뷰 내용을 반영하기만 하는 느낌입니다. 그렇게 하면 좋아보이는 코드는 만들 수 있겠지만 성장을 이뤄낼 순 없습니다. 특히 치명적인 코드가 몇개가 보이는데요. 꼼꼼함에 신경을 많이 쓰시는게 좋을 것 같습니다~

junshock5 commented 4 years ago

리뷰를 하면서 느끼는게 리뷰에 나온 내용에 대해 알아보고 익히는게 아니라 그때그때 리뷰 내용을 반영하기만 하는 느낌입니다. 그렇게 하면 좋아보이는 코드는 만들 수 있겠지만 성장을 이뤄낼 순 없습니다. 특히 치명적인 코드가 몇개가 보이는데요. 꼼꼼함에 신경을 많이 쓰시는게 좋을 것 같습니다~ -> 치명적인 코드가 어떤건지 명시해주시면 조금더 신경쓰겠습니다.

f-lab-dev commented 4 years ago

System.out.println가 치명적인 것 중에 하나입니다~ 쓰면 안돼는 메소드죠

junshock5 commented 4 years ago

우선 목요일 코딩테스트, 금요일 과제 이후에 기존에 리뷰주셨던 것들을 복습하면서 전체적으로 코드들을 다시 리뷰하겠습니다.