프로그래머스 제일 작은 수 제거하기(javascript)

이진우·2022년 7월 2일
1

문제
정수를 저장한 배열, arr 에서 가장 작은 수를 제거한 배열을 리턴하는 함수, solution을 완성해주세요. 단, 리턴하려는 배열이 빈 배열인 경우엔 배열에 -1을 채워 리턴하세요. 예를들어 arr이 [4,3,2,1]인 경우는 [4,3,2]를 리턴 하고, [10]면 [-1]을 리턴 합니다.

제한 조건
arr은 길이 1 이상인 배열입니다.
인덱스 i, j에 대해 i ≠ j이면 arr[i] ≠ arr[j] 입니다.

1. 배열이 비어있을때와 그렇지 않을때 조건문을 만든다.

2. 비어있을때는 [-1]을 return하고 그렇지 않을때는 제일 작은 수를 뺀 나머지 배열을 return해준다.

3. 제일 작은수를 Math.min으로 구해주고 그숫자의 index값을 indexOf()로 구해서 배열에 splice()메소드로 뺀후 반환한다.

function solution(arr) {

	let answer = [];
	if (arr.length <= 1){
		return [-1]
	} else {
		let minNum = Math.min(arr)
		let idx = arr.indexOf(minNum)
		arr.splice(idx,1)
        return answer = arr
	}
    
    return answer
}

그래서 처음 작성한 코드인데 코드실행은 통과인데 유효성검사에서 막혔다...
처음 겪어보는 빨간글씨의 체점판😱😱😱

그래서 검색해보니 코드가 작동해도 효율성이 떨어지면 탈락되는것 같아서 좀더 빨리 작동할 수 있게 코드를 수정하려하다보니 삼항조건문이 생각 나서 삼항조건문으로 하니 통과하였다...휴

function solution(arr) {
    
    arr.splice(arr.indexOf(Math.min(...arr)),1);
    arr.length <= 1 ? answer = [-1] : answer = arr
    
    return answer
}

오늘의 교훈 : 코드가 작동한다고 장땡이 아니다 실용적인지 다시한번 보자...입니다😂

profile
초보개발자의 개발일기

1개의 댓글

comment-user-thumbnail
2023년 6월 30일

function solution(arr) {
let answer = [];
if(arr[0] === 10) answer.push(-1);
else{
let minNum = Math.min(...arr);
let idx = arr.indexOf(minNum);
arr.splice(idx,1);
for(let k of arr){
answer.push(k);
}
}
return answer;
}
이렇게 하면 처음 설계가 잘 동작하긴 하네요 ㅎㅎ

답글 달기