Conversation
hi-rachel
left a comment
hi-rachel
left a comment
There was a problem hiding this comment.
한 문제는 아직 안올리신 건가요?
코드가 깔끔해서 java인데 어느 정도 이해했습니다. 👍🏻
아뇨. 다른 문제에 비해서 좀더 빨리 푸시해놔서 그런건지 왜그런건지ㅠㅠ PR에 안올라가서 revert까지 했는데도, 파일로 반영이 안되네요ㅠ (solution: reverse linked list) 이 커밋의 파일인데.. 이유를 모르겠네요.. |
@njngwn revert를 해서 파일이 삭제된거 같은데요?? |
아 그 이후에 다시 커밋도 하셨었군요.. 뭔가 revert하면서 꼬인거 같은데.. 혹시 기록이 안남는다고 하시면 메인테이너분에게 문의하거나 다시 깔끔히 (로컬에서 revert 전으로 깔끔히 날려버리고 + 코드 백업은 하고) PR 올려서 저 지정해주시면 바로 승인해드릴게요! |
|
헤헤... 제가 한 문제 겨우 푼(?) 수준이라 코드 리뷰를 해도 되는지 잘 모르겠네요... 고생 많으셨습니다! |
| // if the letter is not in the hashset, then add it into hashset and move end pointer | ||
| if (!letterSet.contains(s.charAt(end))) { | ||
| letterSet.add(s.charAt(end)); | ||
| ++end; |
There was a problem hiding this comment.
제가 문제를 안 읽어서 잘은 모르겠는데 보통 후위 표기를 쓰는 걸 봤어서 전위 표기를 하신 이유가 궁금합니다~
There was a problem hiding this comment.
아, 그냥 제 코딩습관인데 ++i가 i++보다 조금 더 빠르다는 말을 들어서요..!! 근데 자바라서 상관없을 것 같기도 하고, 요즘 컴파일러는 상관없다구 하네요. https://ssocoit.tistory.com/51
There was a problem hiding this comment.
문제를 안 읽었지만 요구하는 게 뭔지 알 거 같네요! 주석의 설명이 짧지만 명확하고 코드 역시 깔끔해서 가독성이 좋다고 생각합니다.
There was a problem hiding this comment.
앗! DFS로 푸는 문제들이 나왔군요. 좀 더 시간을 들여서 저도 풀었다면 좋았을텐데... 쪼끔 아쉬운 마음이 남네요.
고생하셨습니다!
There was a problem hiding this comment.
꼼꼼하게 봐주셔서 감사해요!! 다음 기수에 한 번 더 도전하셔서 풀어보셔도 좋을 것 같아요!!
| ArrayList<Integer> rowList = new ArrayList<>(); | ||
| ArrayList<Integer> colList = new ArrayList<>(); | ||
|
|
||
| for (int m = 0; m < matrix.length; ++m) { |
There was a problem hiding this comment.
여기서도 전위 표기를 하셨네요! 선택한 이유가 있으신지 궁금합니다~
solution: reverse linked list 이 커밋으로 올라가 있어요..!! 감사합니다! |
3 participants
답안 제출 문제
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!