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

[moonhyeok] Week3 #801

Merged
merged 5 commits into from
Dec 28, 2024
Merged

[moonhyeok] Week3 #801

merged 5 commits into from
Dec 28, 2024

Conversation

mike2ox
Copy link
Contributor

@mike2ox mike2ox commented Dec 28, 2024

답안 제출 문제

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@mike2ox mike2ox self-assigned this Dec 28, 2024
@mike2ox mike2ox requested a review from a team as a code owner December 28, 2024 03:41
@github-actions github-actions bot added the ts label Dec 28, 2024
@mike2ox mike2ox requested a review from Yjason-K December 28, 2024 03:41
@mike2ox
Copy link
Contributor Author

mike2ox commented Dec 28, 2024

#254 같은 경우 의도적으로 재귀로 풀었습니다. (재귀를 써보고 싶었던?) 추후 제출형식은 아니더라도 최적화를 해보려고 합니다.

* source: https://leetcode.com/problems/product-of-array-except-self/
* 풀이방법: 왼쪽부터의 누적 곱과 오른쪽부터의 누적 곱을 이용하여 결과 계산
* 시간복잡도: O(n) (n: nums의 길이)
* 공간복잡도: O(1) (상수 공간만 사용)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

결과를 저장하는 곳이 n이라서 공간복잡도가 O(n)이어야 하네요;;

Copy link
Contributor

@Yjason-K Yjason-K left a comment

Choose a reason for hiding this comment

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

3주차도 수고 하셨습니다 :)


for (let i = 0; i < candidates.length; i++) {
const num = candidates[i];
const subCombos = combinationSum(candidates.slice(i), target - num);
Copy link
Contributor

Choose a reason for hiding this comment

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

코드가 가독성이 좋아서 읽기 편했어요! 감사합니다 :)

큰 차이는 없을 것 같지만 slice를 사용하면 배열 복사가 매번 발생하게 되어서 현재 idx를 파라미터로 넘기는 방법을 사용해도 괜찮을거 같아요!
로직 자체는 slice를 사용하는 것과 동일하게 현재 인덱스부터 끝까지 탐색한다는 면에서 같은 동작을 합니다.

function combinationSumHelper(candidates: number[], target: number, startIndex: number): number[][] {
  if (target === 0) return [[]];
  if (target < 0) return [];

  const result: number[][] = [];

  for (let i = startIndex; i < candidates.length; i++) {
    const num = candidates[i];
    const subCombos = combinationSumHelper(candidates, target - num, i); 
    // i를 넘기면, 중복 사용 가능하고 i 이전 것은 다시 보지 않음

    for (const combo of subCombos) {
      result.push([num, ...combo]);
    }
  }

  return result;
}

function combinationSum(candidates: number[], target: number): number[][] {
  return combinationSumHelper(candidates, target, 0);
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

문제에서 제공하는 입력 형식 그대로 하려다보니 저렇게 되었는데 올려주신 방법이 훨씬 효율적이네요!
감사합니다 :)

@mike2ox mike2ox merged commit 834ca1e into DaleStudy:main Dec 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

2 participants