[211219] 나누어 떨어지는 숫자 배열

문제 설명

array의 각 element 중 divisor로 나누어 떨어지는 값을 오름차순으로 정렬한 배열을 반환하는 함수, solution을 작성해주세요.
divisor로 나누어 떨어지는 element가 하나도 없다면 배열에 -1을 담아 반환하세요.

제한사항

• arr은 자연수를 담은 배열입니다.
• 정수 i, j에 대해 i ≠ j 이면 arr[i] ≠ arr[j] 입니다.
• divisor는 자연수입니다.
• array는 길이 1 이상인 배열입니다.

입출력 예

[5, 9, 7, 10]	5	[5, 10]
[2, 36, 1, 3]	1	[1, 2, 3, 36]
[3,2,6]	10	[-1]

입출력 예 설명

입출력 예#1
arr의 원소 중 5로 나누어 떨어지는 원소는 5와 10입니다. 따라서 [5, 10]을 리턴합니다.

입출력 예#2
arr의 모든 원소는 1으로 나누어 떨어집니다. 원소를 오름차순으로 정렬해 [1, 2, 3, 36]을 리턴합니다.

입출력 예#3
3, 2, 6은 10으로 나누어 떨어지지 않습니다. 나누어 떨어지는 원소가 없으므로 [-1]을 리턴합니다.

 

채점시 오류 (틀린 코드)

import java.util.*;
class Solution {
    public int[] solution(int[] arr, int divisor) {   
        List<Integer> list = new ArrayList<Integer>();
        int cnt = 0;
        for(int i = 0; i < arr.length; i++){
            if(arr[i]%divisor == 0){
                cnt++;
                list.add(arr[i]);
            }
            if(cnt == 0){
                list = new ArrayList<Integer>();
                list.add(-1);
            }
        }
        Integer[] iArr = new Integer[list.size()];
        list.toArray(iArr);      
        Arrays.sort(iArr);
        int[] answer = new int[list.size()];
        int idx = 0;
        for(Integer i : iArr){
            answer[idx] = i.intValue();
            idx++;
        }
        
        return answer;
    }
}

 

수정한 코드

cnt로 판별할 것 없이 list 비었다면 (나누어 떨어지는 수가 없는 경우) list.isEmpty() 함수 쓰면 된다. 

리뷰 : list가 integer일 경우 list.toArray(new int[list.size()]); 를 사용하여 list -> array 로 변환할 수 없다.

상기 코드에서처럼 integer 배열을 만들어서 넣어 주어야 하고, integer를 나중에 .intValue() 를 이용해 int로 형변환해야한다. 

그냥 아래 코드 처럼 integer list 를 for 문 바로 돌려서 list.get(index) 메소드로 추출하면 알아서 int로 반환된다.

import java.util.*;
class Solution {
    public int[] solution(int[] arr, int divisor) {   
        List<Integer> list = new ArrayList<Integer>();
        for(int i = 0; i < arr.length; i++){
            if(arr[i]%divisor == 0){
                list.add(arr[i]);
            }  
        }
        if(list.isEmpty()){
            list.add(-1);
        }    
        int[] answer = new int[list.size()];
        for(int i = 0; i < list.size(); i++){
            answer[i] = list.get(i);
        }
        Arrays.sort(answer);
        return answer;
    }
}