Skip to content

Conversation

@Taeyong98
Copy link
Collaborator

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 처음엔 어레이리스트로 고민했다가, 아이디어가 잘 떠오르지 않아
큐로 구현했는데, 이런 방법이 있었네요.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅋㅋㅋㅋ 저번주에 제가 toString변환 하신거에 대해 여쭤봤었는데
이번에는 안하셨네요

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네넵 할 필요가 아예 없더라구요.!!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Iterator라는걸 얼추 알고만 있었는데, listIterator()이라는 함수는 처음 보네요.
마찬가지로, hasPrevious(), Previous(), hasNext() 함수 모두
이번 PR을 통해 배웠습니다! 신기하네요

Copy link
Contributor

@growingJd growingJd May 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

키로거에서도 언급드렸다시피, hasPrevious와 previous 함수를 활용한게 인상깊네요.
그리고 저는 값을 입력 받을 때, br.readLine()으로 값을 받은 다음 문자열로 바로 변환하여 값을 비교했는데
생각해보니 태용님처럼 문자열 상태로 바로 비교하면 모든 문자열을 굳이 변환할 필요가 없으니
시간복잡도가 조금 더 줄겠네요.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체적으로 저랑 비슷하게 풀어서 딱히 말씀 드릴게 없네요!!
이 부분에서 바로 값을 비교해서 추가해주셨는데, 저도 바로 추가해서
시간 복잡도를 조금이나마 더 줄이는 방법들을 고민해봐야겠네요!!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전체적으로 저랑 비슷하게 해결하셔서 딱히 흠 잡을 점이 없네요...ㅋㅋㅋㅋㅋ
그나마..~ 아쉽?다기보단 궁금한 점이 있는데, 문제에서 값을 줄 때 '0 6' 과 같은 순서로 값을 주는데
순서와 반대로 배열을 선언하신 이유가 있는지 궁금합니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어.. 그냥 제 머릿속에는 "학년으로 나눈 후 성별을 구분 해야겠다"라는 그림이 그려져버려서 머릿속 그림대로 구현했습니다..

@lee-JunR lee-JunR merged commit 6cdba13 into lee-JunR:main May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants