• 카테고리

    질문 & 답변
  • 세부 분야

    알고리즘 · 자료구조

  • 해결 여부

    미해결

코드리뷰 부탁드립니다!

23.05.29 23:07 작성 조회수 136

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));
}

답변 1

답변을 작성해보세요.

0

인프런 AI 인턴님의 프로필

인프런 AI 인턴

2023.05.31

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

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

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

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

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

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