가까운 1 찾기
문제 설명
정수 배열 arr가 주어집니다. 이때 arr의 원소는 1 또는 0입니다. 정수 idx가 주어졌을 때, idx보다 크면서 배열의 값이 1인 가장 작은 인덱스를 찾아서 반환하는 solution 함수를 완성해 주세요.
단, 만약 그러한 인덱스가 없다면 -1을 반환합니다.
제한사항
3 ≤ arr의 길이 ≤ 100'000
arr의 원소는 전부 1 또는 0입니다.
입출력 예
| arr | idx | result |
|---|---|---|
| [0, 0, 0, 1] | 1 | 3 |
| [1, 0, 0, 1, 0, 0] | 4 | -1 |
| [1, 1, 1, 1, 0] | 3 | 3 |
입출력 예 설명
입출력 예 #1
1보다 크면서 원소가 1인 가장 작은 인덱스는 3입니다. 따라서 3을 return 합니다.입출력 예 #2
4번 인덱스 이후에 1은 등장하지 않습니다. 따라서 -1을 return 합니다.입출력 예 #3
3번 인덱스의 값이 1입니다. 따라서 3을 return 합니다.
💻 내가 작성한 코드
function solution(arr, idx) {
let result = -1
for(let i = 0; i < arr.length; i++){
if(arr[i] === 1 && i >= idx){
return result = i
}
}
return result
}이번 문제는 문제 내용 부터 솔직히 이해가 가지 않아
다른 사람들의 질문 내용을 확인하면서 풀었다.
arr[i]가 1이고, i가 idx보다 크거가 같은 경우 result에 i를 할당해 반환하고 반복문을 종료한다.
만약 조건을 만족하는게 없을 경우 result를 반환한다.
🥹 실수 했던 점
function solution(arr, idx) {
let result = -1
for(let i = 0; i < arr.length; i++){
if(arr[i] === 1 && i >= idx){
result = i
}
}
return result
}for문 내부 조건문에 충족하면 result에 i를 할당하고 for문 밖에서 반환해 주었다.
처음에 왜 이게 안되지? 했다.
for문은 반복문으로 arr 길이 만큼 조건이 만족할 때 까지 반복해
원하는 답을 내놓지 않았던거였다.
💻 리팩토링 하기
function solution(arr, idx) {
for(let i = 0; i < arr.length; i++){
if(arr[i] === 1 && i >= idx){
return = i
}
}
return -1
}굳이 result라는 변수를 만들필요 없이 바로 값을 반환해 코드를 간결하게 작성할 수 있다.
💻 다른 사람이 작성하 코드
const solution=(a,i)=>a.indexOf(1,i);function solution(arr, idx) {
return arr.findIndex((v, i) => idx <= i && v === 1);
}indexOf, findIndex 메소드를 활용해 문제를 해결 할 수 있다는 점에서
다시 배워간다.
✏️ 마치며
이번 문제는 문제 내용 자체를 이해하는거 자체에서 부터 시간 소요가 되지 않았을까 싶다.
결국 다른 사람들의 질문을 확인하면서 이해하면서 풀었다...ㅠ
