코드 리뷰

Question

안녕하세요 강의 듣기 전에 혼자서 풀어보았는데 다음 코드도 가능할까요 ? function solution(s, arr) { let answer = Array.from({ length: s }, () => 0); for (let i = 0; i < arr.length; i++) { let p = answer.indexOf(arr); if (p > -1) { let tmp = arr[i]; for (let j = p; j > 0; j--) { answer[j] = answer[j - 1]; } answer[0] = tmp; } else { for (let j = answer.length - 1; j >= 0; j--) { if (answer[j] === 0 || j === answer.length - 1) continue; let tmp = answer[j]; answer[j + 1] = tmp; } answer[0] = arr[i]; } } return answer; }

인프런 AI 인턴 · Answer

안녕하세요, 인프런 AI 인턴입니다. 00님, 코드 리뷰 관련 질문 감사합니다. 주어진 코드를 살펴보았을 때, solution 함수는 어떤 특정 알고리즘을 구현하려고 하는 것 같습니다만, 코드에 오류가 있어보입니다. 지정된 arr 값을 indexOf 메서드로 검색하는 대신 arr[i] 를 사용해야 할 것 같습니다. 즉, let p = answer.indexOf(arr); 이 부분을 let p = answer.indexOf(arr[i]); 로 수정해야 할 것입니다. 또한, else 블록에서 continue 를 사용할 경우, let tmp = answer[j]; 에 이르기 전에 이미 반복문이 계속될 수 있습니다. 이는 배열의 재정렬로 이어지지 않고 예상치 못한 결과를 초래할 수 있습니다. 이 부분도 확인하실 필요가 있습니다. 수정된 코드는 다음과 같습니다. function solution(s, arr) { let answer = Array.from({ length: s }, () => 0); for (let i = 0; i < arr.length; i++) { let p = answer.indexOf(arr[i]); // 수정된 부분 if (p > -1) { let tmp = arr[i]; for (let j = p; j > 0; j--) { answer[j] = answer[j - 1]; } answer[0] = tmp; } else { for (let j = answer.length - 1; j > 0; j--) { // 수정된 부분 answer[j] = answer[j - 1]; // 수정된 부분 } answer[0] = arr[i]; } } return answer; } 해당 코드가 어떠한 문제를 해결하기 위한 것인지 정확한 상황을 알려주시면 좀 더 구체적인 조언을 드릴 수 있을 것 같습니다. 코드를 수정하여 원하는 결과를 얻을 수 있길 바랍니다. 혹시 추가적인 질문이 있으시면 언제든지 문의해주세요!