선생님! 코드 한번만 봐주실 수 있을까요?
285
작성한 질문수 4
const solution = (nums) => {
let sumNum = 0
let minNum = Number.MAX_SAFE_INTEGER
let oddNums = nums.filter((num) => {
return num % 2 !== 0
})
oddNums.forEach((num) => {
sumNum += num
if (num < minNum) minNum = num
})
return [sumNum, minNum]
}위와 같이 짜봤는데 비효율적인 코드인가 싶어서 여쭤봅니다! (구지 filter와 forEach를 쓸 필요까지는 없었나 싶어서...)
효율성적인 측면에서 볼 때 어떤지 궁금합니다
답변 2
0
안녕하세요, 인프런 AI 인턴이에요.
주어진 문제를 추측해봤지만 원래 문제가 무엇인지 모르기 때문에 코드의 효율성을 정확하게 평가하기는 어렵습니다. 하지만 주어진 코드에서는 filter와 forEach를 사용하여 반복문을 두 번 사용합니다. 이렇게 할 경우 시간 복잡도가 높아져서 성능이 저하될 우려가 있습니다.
최적화를 위해 코드를 변경해보면 다음과 같습니다.
const solution = (nums) => {
let sumNum = 0
let minNum = Number.MAX_SAFE_INTEGER
for (let i = 0; i < nums.length; i++) {
if (nums[i] % 2 !== 0) {
let num = nums[i]
sumNum += num
if (num < minNum) minNum = num
}
}
return [sumNum, minNum]
}
이렇게 변경하면 filter와 forEach를 사용하지 않으므로 코드가 더 직관적이며, 실행 시간도 효율적으로 감소합니다.
이상입니다. 다른 질문이 있으면 언제든지 물어보세요. 감사합니다.
continue를 사용하는 이유
0
80
2
정렬 가능 여부 판단하기
0
64
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
74
1
섹션1의 17번문제 이 풀이로 풀어도 될까요?
0
136
2
정규표현식으로 처리해도 상관없나요 ?
0
120
2
3칸씩 건너뛸 수 있을 경우
0
126
2
강의에 대해 질문있습니다.
0
135
2
Object와 Set을 이용해 풀어봤습니다.
0
117
2
이렇게 해도 되나요?
0
102
2
선생님 중복 단어나 중복관련 문제들은 set을 이용하면 좋을것 같습니다.
0
145
2
이렇게 풀어도 괜찮을까요?
0
138
1
이렇게 풀어도 괜찮을까요?
0
112
1
모든 아나그램 찾기에서 시간복잡도
0
98
1
코드리뷰 부탁드립니다.
0
130
1
for loop 탈출은 return 문으로 해도 되지 않나요?
0
133
1
투포인트알고리즘으로 풀어봤습니다.
0
142
0
코드 리뷰 부탁드립니다.
0
120
1
코드 맞게 작성한 거 아닌가여??
0
146
1