[jdy8739] week 8 by jdy8739 · Pull Request #955 · DaleStudy/leetcode-study

jdy8739 · 2025-01-25T15:42:06Z

답안 제출 문제

232
244
259
274

체크 리스트

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

obzva

안녕하세요 이번 한 주도 고생 많으셨습니다.
설 연휴임에도 불구하고 4문제나 풀어주셨네요 :)
코멘트를 남겨 놓았으니 병합 전에 확인하고 병합 진행 부탁 드립니다
추가적인 코멘트 교환을 원하신다면 re-request review해주세요!

obzva · 2025-01-30T22:48:00Z

clone-graph/jdy8739.js

+
+// 재귀를 활용한 dfs를 사용한 풀이
+// 시간복잡도 O(v + e) -> v(노드의 수) + e(간선의 수) 만큼 탐색을 수행
+// 공간복잡도 O(n) -> map이 총 노드의 수 만큼 크기를 가짐


공간복잡도 분서 결과에는 영향을 미치지 않지만, 재귀호출 스택에 대한 언급도 해주면 좋을 것 같아요!

시간복잡도에서 노드의 수를 v라고 표현하셨으니까 공간복잡도도 O(v)라고 써주면 더 나을 것 같아요

obzva · 2025-01-30T22:53:19Z

clone-graph/jdy8739.js

+        nodeMap.set(clone.val, clone);
+
+        for (const nei of nodeParam.neighbors) {
+            clone.neighbors = [...clone.neighbors, dfs(nei)];


Array를 매번 새로 만들어주는 대신에 새 원소를 push해주는 방식은 어떤가요?

Suggested change

clone.neighbors = [...clone.neighbors, dfs(nei)];

clone.neighbors.push(dfs(nei));

넵 풀이에 그렇게 push로 원소를 넣어 주던데 그 방식이 좀 더 간결하고 성능도 나은 것 같습니다 :)

obzva · 2025-01-30T22:53:41Z

clone-graph/jdy8739.js

+
+// 큐를 활용한 bfs를 사용한 풀이
+// 시간복잡도 O(v + e) -> v(노드의 수) + e(간선의 수) 만큼 탐색을 수행
+// 공간복잡도 O(n) -> map이 총 노드의 수 만큼 크기를 가짐


시간복잡도에서 노드의 수를 v라고 표현하셨으니까 공간복잡도도 O(v)라고 써주면 더 나을 것 같아요

그러네요 수정해서 커밋하겠습니다. 감사합니다! :)

obzva · 2025-01-30T22:57:30Z

clone-graph/jdy8739.js

+    // 이 queue에는 클론이 아니라 원본 노드가 들어가야함(neighbors에 대한 참조 때문)
+    const queue = [node];


알고리즘은 정상적으로 동작하지만 해당 Array는 queue가 아니네요 (stack처럼 동작해요!)

obzva · 2025-01-30T23:13:43Z

longest-common-subsequence/jdy8739.js

+    return dfs(0, 0);
+};
+
+// 시간복잡도 O(m * n) -> 재귀를 통해 m과 n이 1씩 증가하면서 상대편의 m or n 번 째 인덱스 문자열을 비교하므로??? (잘 모르겠습니다. 리뷰어분이 여유되시면 설명부탁드립니다 ㅜㅜ)


어디까지 이해되셨고 어디부터 이해가 잘 안되시는지 Discussion에 올려주세요~!

obzva · 2025-01-30T23:19:14Z

longest-repeating-character-replacement/jdy8739.js

+        const countOfOthers = currLength - maxCountOfChar;
+
+        // 현재 문자열의 길이에서 가장 많이 나오는 문자열의 수를 뺸 값인 countOfOthers가
+        // k보다 작으면 현재 문자열에서 start번 째 문자의 수를 map에서 감소시키고 star의 값을 증가시킨다.


typo) k보다 크면

obzva · 2025-01-30T23:34:26Z

number-of-1-bits/jdy8739.js

+/**
+ * @param {number} n
+ * @return {number}
+ */
+var hammingWeight = function (n) {
+    const binary = [1];
+
+    while (binary[0] < n) {
+        const latest = binary[0] * 2;
+
+        binary.unshift(latest);
+    }
+
+    let count = 0;
+
+    for (let i = 0; i < binary.length; i++) {
+        if (binary[i] <= n) {
+            count++;
+            n = n - binary[i];
+        }
+
+        if (n === 0) {
+            break;
+        }
+    }
+
+    return count;
+};
+
+// 시간복잡도 O(logn) -> 이진탐색처럼 2씩 곱해가며 n을 넘어가는 가장 큰 수를 찾으므로


bitwise연산자에 대해서 한 번 찾아보시는 걸 추천드립니다 e.g. Right shift. bitwise 연산자를 이용한 풀이가 떠오르지 않는다면 다른 분들 풀이를 참고해주세요~!

제가 알기론 unshift의 시간 복잡도는 O(1)이 아닙니다. (이 부분도 찾아보시는게 좋을 것 같아요) 따라서 해당 풀이의 시간 복잡도 분석도 다시 이뤄져야할 것 같다는 생각이 듭니다.

js의 배열이 원소들이 연달아 주소값을 가지는 형태가 아니다보니 O(1)라고 생각했는데요, 한 번 알아보고 주석을 다시 달아 커밋하겠습니다. 코멘트 감사합니다!

js의 unshift는 배열의 요소가 맨 앞에 추가될 때마다 뒤 모든 요소들의 index가 1 씩 추가되므로 O(n)의 복잡도를 갖는다는 것을 알게되었습니다!!

JavaScript 배열의 내부 구조
JavaScript 엔진(V8 기준)에서 배열은 두 가지 형태로 저장될 수 있습니다.

✅ (1) Packed Elements (연속된 메모리, 빠름)
배열이 숫자로만 이루어져 있고, 인덱스가 연속적이면
내부적으로 일반적인 동적 배열처럼 연속된 메모리에 저장됨
O(1)로 빠른 조회가 가능

let arr = [1, 2, 3, 4]; // Packed Elements 사용
✅ (2) Sparse Elements (비연속적, 느림)
배열이 중간에 빈 요소가 있거나, 숫자가 아닌 객체가 포함된 경우
내부적으로 해시 테이블(Hash Map)처럼 동작
요소를 조회할 때 해시 테이블에서 검색해야 하므로 느려짐 (O(1) → O(n))

let arr = [];
arr[0] = 1;
arr[100] = 2; // 희소 배열 (Sparse Array)
✅ 위처럼 인덱스가 연속되지 않으면 메모리가 연속적이지 않음!

number-of-1-bits solution

a46a3b1

jdy8739 requested a review from a team as a code owner January 25, 2025 15:42

github-actions bot assigned jdy8739 Jan 25, 2025

github-actions bot added the js label Jan 25, 2025

jdy8739 requested a review from imsosleepy January 25, 2025 15:43

jdy8739 added 6 commits January 27, 2025 00:25

longest-repeating-character-replacement solution

7f18137

fix: 244번 풀이 개행추가

031f8ca

clone-graph solution

d2f5691

fix: clone-graph 개행추가

8043d8e

fix: clone-graph 파일명 수정

4177b2f

longest-common-subsequence solution

3eadf05

obzva approved these changes Jan 30, 2025

View reviewed changes

jdy8739 added 2 commits January 31, 2025 22:02

clone-graph 주석 수정

6e3faf4

number-of-1-bits 시간복잡도 주석 수정

550375d

jdy8739 merged commit a747d1f into DaleStudy:main Jan 31, 2025
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[jdy8739] week 8#955

[jdy8739] week 8#955
jdy8739 merged 9 commits intoDaleStudy:mainfrom
jdy8739:main

jdy8739 commented Jan 25, 2025 •

edited

Loading

Uh oh!

obzva left a comment •

edited

Loading

Uh oh!

obzva Jan 30, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

jdy8739 Jan 31, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

jdy8739 Jan 31, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

obzva Jan 30, 2025

Uh oh!

jdy8739 Jan 31, 2025

Uh oh!

jdy8739 Jan 31, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Milestone

	clone.neighbors = [...clone.neighbors, dfs(nei)];
	clone.neighbors.push(dfs(nei));

		// 이 queue에는 클론이 아니라 원본 노드가 들어가야함(neighbors에 대한 참조 때문)
		const queue = [node];

Conversation

jdy8739 commented Jan 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

답안 제출 문제

체크 리스트

Uh oh!

obzva left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Milestone

jdy8739 commented Jan 25, 2025 •

edited

Loading

obzva left a comment •

edited

Loading