Week6 Solution

[YeonguChoe]

답안 제출 문제

• Valid Parentheses #222
• Container With Most Water #242
• Design Add And Search Words Data Structure #257
• Longest Increasing Subsequence #272
• Spiral Matrix #282

Design Add And Search Words Data Structure 에서 "."이 포함 되어있을때 구현하는 방법 빼고 어려운것 없었음.

체크 리스트

• PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요.
• 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
• 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
• 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

[bky373]

시공간 복잡도 계산이 빠져 있는 문제들이 있는데 추가해주시면 좋을 것 같습니다~

[bky373]

추가로, c# 은 자동 labeling 워크플로에서 감지하지 못하는데 주말 중 추가해두겠습니다~
그리고 java 와 cpp 파일이 있어서 관련 label 들은 남아있어도 될 것 같은데 지우신 이유가 있으실까요?

[haklee]

추가로, c# 은 자동 labeling 워크플로에서 감지하지 못하는데 주말 중 추가해두겠습니다~ 그리고 java 와 cpp 파일이 있어서 관련 label 들은 남아있어도 될 것 같은데 지우신 이유가 있으실까요?

이거 못 보고 java랑 c++ label 추가했다가 이거 보고 다시 지웠는데 다시 보니까 그냥 뒀어도 괜찮았을 뻔했네요...

[haklee]

코드 리뷰 이전에 이 PR을 먼저 살펴보았는데요,

• 이 PR 제목은 week 6라고 되어있습니다.
• PR 설명에도 week 6 문제만 적혀있습니다.
• 그런데 실제로 푸신 문제는 여러 주차에 걸쳐있는 것으로 보입니다.
• 그래서 프로젝트 iteration은 어떻게 설정하셨는지 궁금해서 가보았는데 설정이 되어있지 않았습니다.
• 그런데 PR 설명에 있는 PR을 프로젝트에 추가하고 Week를 현재 주차로 설정해주세요. 항목은 체크가 되어있습니다.

이렇게 되면 이 PR의 어디부터 어디까지 믿을 수 있는 것인지 리뷰어로 지정된 저는 헷갈립니다 @_@

이렇게 여러 주차에 걸친 문제를 올리는 상황이라면 현 시점의 iteration인 6주차로 설정하는 것이 맞는지, 아니면 각 주차에 따라 PR을 쪼개는 것이 맞는지 혹시 정해진 규칙 같은 것이 있을까요...? @DaleSeo