코드리뷰 부탁드립니다!
206
작성한 질문수 1
강사님, 안녕하세요.
문제를 미리보고 강의를 들었는데, 큰 개념은 비슷하지만
코드의 시간복잡도, 공간복잡도가 다를 수 있을 것 같아서 코드리뷰를 요청드립니다.
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)이므로 비효율적입니다. 따라서, 가독성 측면보다는 효율성 측면을 우선시하여 작성하는 것이 좋을 것 같습니다.
마지막으로, 변수명도 명확하게 작성하는 것이 좋습니다. 변수명은 코드의 가독성을 높여주는 중요한 역할을 하기 때문입니다.
제가 드릴 수 있는 조언은 여기까지이며, 좀 더 구체적인 조언이 필요하시면, 새로운 질문을 통해 다시 문의해주시면 성심성의껏 답변드리겠습니다. 감사합니다 :)
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





