inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

자바스크립트 알고리즘 문제풀이 입문(코딩테스트 대비)

8. 모든 아나그램 찾기(Hash & Sliding Window && Two Pointers Algorithm)

코드리뷰 부탁드립니다!

206

Chulwoong Choi

작성한 질문수 1

0

강사님, 안녕하세요.

문제를 미리보고 강의를 들었는데, 큰 개념은 비슷하지만

코드의 시간복잡도, 공간복잡도가 다를 수 있을 것 같아서 코드리뷰를 요청드립니다.

 

for문을 통해 슬라이딩 윈도우를 하면서, 문자열을 쪼개고 아나그램을 비교할 때도 매번 Map을 생성하는 부분에서 비효율적인 부분이 있을 것 같은데 한번 확인부탁드리겠습니다.

 

{
  function isAnagram(s: string, t: string) {
    const hash = new Map();

    for (let val of t) {
      if (!hash.has(val)) hash.set(val, 1);
      else hash.set(val, hash.get(val) + 1);
    }

    for (let val of s) {
      if (!hash.has(val) || hash.get(val) === 0) return false;
      else hash.set(val, hash.get(val) - 1);
    }

    return true;
  }

  function solution(s: string, t: string) {
    let answer = 0;
    let n = t.length;
    let substr = s.slice(0, n);

    if (isAnagram(substr, t)) answer++;

    for (let i = n; i < s.length; i++) {
      substr = substr.slice(1) + s[i];

      if (isAnagram(substr, t)) answer++;
    }

    return answer;
  }

  const s1 = "bacaAacba";
  const s2 = "abc";

  console.log(solution(s1, s2));
}

javascript 코딩-테스트

답변 1

0

인프런 AI 인턴

안녕하세요, 인프런 AI 인턴이에요.

코드리뷰에 대한 요청 감사합니다. 코드 전반적으로 깔끔하게 작성된 것 같아요!

하지만, 소중한 제 생각을 드리자면, 말씀하신 대로 시간복잡도, 공간복잡도에 대한 고민이 더 필요할 것 같아요. 코드를 보면, 슬라이딩 윈도우를 이용해서 문자열을 쪼개고, 매번 Map을 생성하는 부분에서 비효율적인 부분이 있어보입니다.

이 부분들을 개선하려면, 예를 들어 Map을 미리 생성하고, 값을 저장하여 재활용하는 것이 좋을 것 같습니다. 또한 슬라이딩 윈도우 값을 업데이트 하는데 substr 메소드를 사용하고 있지만, 이것은 시간 복잡도가 O(k)이므로 비효율적입니다. 따라서, 가독성 측면보다는 효율성 측면을 우선시하여 작성하는 것이 좋을 것 같습니다.

마지막으로, 변수명도 명확하게 작성하는 것이 좋습니다. 변수명은 코드의 가독성을 높여주는 중요한 역할을 하기 때문입니다.

제가 드릴 수 있는 조언은 여기까지이며, 좀 더 구체적인 조언이 필요하시면, 새로운 질문을 통해 다시 문의해주시면 성심성의껏 답변드리겠습니다. 감사합니다 :)

continue를 사용하는 이유

0

82

2

정렬 가능 여부 판단하기

0

66

2

알고리즘 학습법 관련해서 질문드립니다.

0

86

1

코드 리뷰 부탁드립니다!

0

90

1

indexOf를 사용해서 풀어보았습니다 !!

0

69

1

저는 이런식으로 구현 해보았습니다 !!

0

64

1

12,13,14 강의 소리만 나오고 검은 화면입니다

0

101

3

반복문 최소화하고 indexOf 사용해서 풀어봤습니다

0

63

1

영상 보기 전에 직접 풀어봤습니다.

0

75

1

섹션1의 17번문제 이 풀이로 풀어도 될까요?

0

136

2

정규표현식으로 처리해도 상관없나요 ?

0

120

2

3칸씩 건너뛸 수 있을 경우

0

126

2

강의에 대해 질문있습니다.

0

136

2

Object와 Set을 이용해 풀어봤습니다.

0

117

2

이렇게 해도 되나요?

0

102

2

선생님 중복 단어나 중복관련 문제들은 set을 이용하면 좋을것 같습니다.

0

145

2

이렇게 풀어도 괜찮을까요?

0

138

1

이렇게 풀어도 괜찮을까요?

0

113

1

모든 아나그램 찾기에서 시간복잡도

0

98

1

코드리뷰 부탁드립니다.

0

130

1

for loop 탈출은 return 문으로 해도 되지 않나요?

0

133

1

투포인트알고리즘으로 풀어봤습니다.

0

142

0

코드 리뷰 부탁드립니다.

0

120

1

코드 맞게 작성한 거 아닌가여??

0

146

1