코드 리뷰 부탁드립니다 !!
336
작성한 질문수 10
아래 코드도 잘 짠게 맞을까요?
function solution(require, direct) {
let answer;
let queue = "";
for (let i = 0; i < direct.length; i++) {
if (require.includes(direct[i])) queue += direct[i];
}
if (queue === require) answer = "YES";
else answer = "NO";
return answer;
}답변 1
0
안녕하세요, 인프런 AI 인턴이에요.
코드를 보면, require라는 배열과 direct라는 배열이 있습니다. 이 두 배열을 비교하여 require에 모든 원소가 포함되어 있는지를 검사하는 코드입니다.
그렇다면 위 코드도 틀린 것은 아니지만 약간 개선할 여지가 있습니다. 예를 들어, 현재 코드는 queue 문자열을 만든 후 마지막에 문자열 비교를 통해 YES나 NO를 리턴하도록 되어 있습니다. 하지만 이는 require 배열에 있는 문자열 포함 여부를 검사하는 것이므로, 해당 문자열이 없을 때 바로 NO를 리턴하도록 코드를 수정하는 것이 좋을 것 같습니다. 또한 변수명도 직관적이지 않아서 어떤 용도로 쓰이는지 파악하기 어려울 수도 있습니다.
그래서 아래와 같이 코드를 개선해 보았습니다.
function checkRequiredModules(require, direct) {
for (let i = 0; i < require.length; i++) {
if (!direct.includes(require[i])) { // 하나라도 없을 때는 NO를 바로 리턴
return “NO”;
}
}
return “YES”; // 모두 있을 때는 YES를 리턴
}
위 코드에서는 바로 리턴하는 방법을 통해 코드를 간결하게 만들었고, 보다 직관적인 변수명과 함수명을 사용하여 가독성 또한 개선하였습니다.
해당 코드를 참고해 보시기 바랍니다.
continue를 사용하는 이유
0
82
2
정렬 가능 여부 판단하기
0
66
2
알고리즘 학습법 관련해서 질문드립니다.
0
86
1
코드 리뷰 부탁드립니다!
0
90
1
indexOf를 사용해서 풀어보았습니다 !!
0
69
1
저는 이런식으로 구현 해보았습니다 !!
0
65
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