작성
·
10
0
Map과 for문으로 풀이를 했는데, 코드리뷰 부탁드립니다.
function solution(arr) {
let sumSet = new Map();
for (let i = 0; i < arr.length; i++) {
let sum = 0;
for (let j = 0; j < String(arr[i]).length; j++) {
sum += Number(String(arr[i])[j]);
}
sumSet.set(arr[i], sum);
}
let maxVal = 0;
let maxKey = 0;
for (const [key, value] of sumSet) {
if (value > maxVal) {
maxKey = key;
maxVal = value;
} else if (value === maxVal) {
if (key > maxKey) {
maxKey = key;
maxVal = value;
}
}
}
return maxVal;
}
console.log(solution([128, 460, 603, 40, 521, 137, 123]));
답변 1
0
안녕하세요, 인프런 AI 인턴입니다.
은우님, 작성해주신 코드 리뷰 요청에 대해 답변드리겠습니다. 전체적으로 작성하신 코드는 잘 구조화되어 있으며, Map을 사용하여 각 숫자와 그 자리수의 합을 저장한 후, 최댓값을 찾아내는 방식으로 구현되어 있습니다. 이는 매우 합리적인 접근 방식입니다. 다만, 마지막에 반환하시는 부분에서 착오가 있어 보입니다. 코드는 숫자 자체(maxKey
)를 반환해야 할 것 같은데 현재는 maxVal
을 반환하고 있습니다.
이를 해결하기 위해 다음 부분을 수정하시길 추천드립니다:
return maxKey;
관련해서 비슷한 문제 풀이 방식이나 코드 리뷰를 다룬 기존 게시물들을 확인하실 수 있습니다:
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.