[2023-09-15] dohyun #208#234
Conversation
Woo-Yeol
left a comment
There was a problem hiding this comment.
적절한 자료구조를 선택하고 List 와 Set를 사용해서 풀이를 진행하신 부분이 해당 자료구조를 잘 이해하고 있다고 와닿아서 좋았습니다.
There was a problem hiding this comment.
해당 식을 is_prime_number에 녹여낼 수 있으면 반복문 횟수를 줄여내기 유리할 것 같습니다!
There was a problem hiding this comment.
검사하는 부분이 저와 거의 유사한 것 같습니다! 고생하셨어요 도현님! 적절한 자료구조를 만들어주셔서 더 쉽게 이해했습니다!
There was a problem hiding this comment.
각 변수에 대한 주석이 있으면 좋을 것 같아요!
set 자료형으로 또 소수를 추가해줬는데 밑에 코드에서 O(1)로 탐색하기 위해 사용하신게 맞으실까요?!
그럼 우열님 코드처럼 dictionary 자료형을 써도 좋을 것 같아요!
There was a problem hiding this comment.
아 지금보니까 위에 적혀있네용! 🤣 고생하셨습니다 도현님~~!
There was a problem hiding this comment.
이 부분은 무한으로 초기화하지않고, num정도로 초기화해도 상관없을것 같아요!
There was a problem hiding this comment.
아 지금보니까 위에 적혀있네용! 🤣 고생하셨습니다 도현님~~!
zsmalla
left a comment
There was a problem hiding this comment.
우열님 말씀과 비슷하게, 차라리 get_prime_numbers라는 메서드를 정의해서 안에 is_prime_number 로직을 녹여내는 편이 조금 더 깔끔해 보일 것 같기는 합니다. 그리고 도현님 풀이처럼, 제한이 조금 널널한 문제 조건에서는 일반 소수 판별을 사용해도 좋을 것 같습니다. 고생하셨습니다!
There was a problem hiding this comment.
리스트를 set으로 변환하는 오버헤드가 그렇기 크지 않은 걸로 알고있어서, 그냥 List로 다 구하셨으면 set으로 변환하셔서 사용하셔도 좋을 것 같습니다!
PR Summary
풀이시간
접근법
회고
ISSUE NUMBER