-
질문 & 답변
카테고리
-
세부 분야
알고리즘 · 자료구조
-
해결 여부
미해결
코드 리뷰 부탁드립니다 !!
23.04.18 09:36 작성 조회수 226
0
아래 코드도 잘 짠게 맞을까요?
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;
}
답변을 작성해보세요.
0
인프런 AI 인턴
2023.05.24
안녕하세요, 인프런 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를 리턴
}
위 코드에서는 바로 리턴하는 방법을 통해 코드를 간결하게 만들었고, 보다 직관적인 변수명과 함수명을 사용하여 가독성 또한 개선하였습니다.
해당 코드를 참고해 보시기 바랍니다.
답변 1