작성
·
175
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 인턴이에요.
코드리뷰에 대한 요청 감사합니다. 코드 전반적으로 깔끔하게 작성된 것 같아요!
하지만, 소중한 제 생각을 드리자면, 말씀하신 대로 시간복잡도, 공간복잡도에 대한 고민이 더 필요할 것 같아요. 코드를 보면, 슬라이딩 윈도우를 이용해서 문자열을 쪼개고, 매번 Map을 생성하는 부분에서 비효율적인 부분이 있어보입니다.
이 부분들을 개선하려면, 예를 들어 Map을 미리 생성하고, 값을 저장하여 재활용하는 것이 좋을 것 같습니다. 또한 슬라이딩 윈도우 값을 업데이트 하는데 substr 메소드를 사용하고 있지만, 이것은 시간 복잡도가 O(k)이므로 비효율적입니다. 따라서, 가독성 측면보다는 효율성 측면을 우선시하여 작성하는 것이 좋을 것 같습니다.
마지막으로, 변수명도 명확하게 작성하는 것이 좋습니다. 변수명은 코드의 가독성을 높여주는 중요한 역할을 하기 때문입니다.
제가 드릴 수 있는 조언은 여기까지이며, 좀 더 구체적인 조언이 필요하시면, 새로운 질문을 통해 다시 문의해주시면 성심성의껏 답변드리겠습니다. 감사합니다 :)