Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

# 20231219 doozuu (이주희) 문제 풀이 업로드 #138

Merged
merged 1 commit into from
Dec 21, 2023
Merged

Conversation

Doozuu
Copy link
Contributor

@Doozuu Doozuu commented Dec 19, 2023

📚 풀이 문제

  1. 기둥과 보 설치 - 오답
  2. 광물 캐기
  3. 풍선 터뜨리기 - 메모리 초과
  4. DFS와 BFS
  5. 단지 번호 붙이기

🔏 궁금한 점

기둥과 보 설치 문제에서 요소를 삭제하는 경우를 제대로 고려하지 못한 것 같습니다.
저도 풍선 터뜨리기 문제는 메모리 초과가 떴습니다.😂

🔍 체크 리스트

  • PR 제목을 규칙에 맞게 작성
  • label 설정
  • 작업한 사람 모두를 Assign
  • Code Review 요청

@Doozuu Doozuu added the 📕 Unsolved 금주차 문제를 완전히 해결하지 못했습니다. 풀지 못한 문제에 대한 질문을 원할때도 쓰입니다. label Dec 19, 2023
@Doozuu Doozuu requested a review from stakbucks December 19, 2023 07:33
@Doozuu Doozuu self-assigned this Dec 19, 2023
@github-actions github-actions bot requested a review from Hyeple December 19, 2023 07:33
Copy link
Contributor

@stakbucks stakbucks left a comment

Choose a reason for hiding this comment

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

예전 부터 느꼈지만, 배열/객체 구조분해할당 및 상수를 자주 사용하시면 훨씬 좋을 것 같습니다! 저도 코드 읽는 데 시간이 덜 걸리고 아마 주희님도 디버깅 하기도 편하시고 실수도 덜 하시게 되지 않을까 싶네요.
예를 들어, 길이가 정해진 배열은 인덱스 대신 구조분해할당 해서 각각 변수명 할당해주기만 해도 더 보기 좋을 것 같아요 😃
고생하셨습니다!

Doozuu/231219/boj_DFS와BFS.js Show resolved Hide resolved
Doozuu/231219/boj_DFS와BFS.js Show resolved Hide resolved
Doozuu/231219/pro_광물캐기.js Show resolved Hide resolved
Doozuu/231219/pro_광물캐기.js Show resolved Hide resolved
Doozuu/231219/pro_기둥과보설치.js Show resolved Hide resolved
@Doozuu Doozuu merged commit ad65c79 into main Dec 21, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📕 Unsolved 금주차 문제를 완전히 해결하지 못했습니다. 풀지 못한 문제에 대한 질문을 원할때도 쓰입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants