인프런 커뮤니티 질문&답변

정현님의 프로필 이미지
정현

작성한 질문수

김영한의 실전 자바 - 중급 1편

java중급 4강 레퍼클래스 로또 번호 생성기 문제

작성

·

38

0

4강 레퍼클래스 로또 번호 생성기 질문 있습니다.

 

package class04wrapperClass;
import java.util.Random;

public class Lottery {
    public static void main(String args[]) {
        //랜덤 생성자 선언 ==> 로또 값이 그때 그때 바뀜
        Random random = new Random();

        //로또 범위 지정 배열
        int[] randomArray = new int[6];
        

        //로또 배열에 로또 값 넣기
        for (int i = 0; i < randomArray.length; i++) {
            int randomValue=random.nextInt(45)+1;
            randomArray[i]=randomValue;
            if (randomArray[i]==randomArray[i++]||randomArray[i]==0)
                randomArray[i]=random.nextInt(45)+1;
        }

        for(int i =0; i<randomArray.length; i++){
            System.out.print(randomArray[i]+" , ");
        }

            }
}

이렇게 풀었는데 이것도 올바른 풀이 일까요???

답변 1

0

안녕하세요. 정현님, 공식 서포터즈 OMG입니다.

올리신 코드의 경우, 중복 값을 제대로 걸러내지 못합니다.

사례)

image.png

 

설명드리기 위해 코드 정리를 하면

int randomValue=random.nextInt(45)+1;

randomArray[i]=randomValue;

randomValue는 배열에 값을 할당하기 위해 사용하고 그 외에는 사용하지 않으므로 불필요(=코드를 이해하기 위해 도움이 되지 않음) 하여 동일한 동작을 하지만, 아래처럼 별도 변수를 사용하지 않겠습니다.

for (int i = 0; i < randomArray.length; i++) {
    randomArray[i]= random.nextInt(45)+1;
    if (randomArray[i]==randomArray[i++] || randomArray[i]==0)
        randomArray[i]=random.nextInt(45)+1;
}

그리고 난수 생성을 random.nextInt(45)+1;

이렇게 1~45가 나오도록 했기 때문에 다음 if 조건문은 불필요(=항상 거짓)하여 제거하겠습니다. randomArray[i]==0

for (int i = 0; i < randomArray.length; i++) {
    randomArray[i]= random.nextInt(45)+1;
    if (randomArray[i]==randomArray[i++])
        randomArray[i]=random.nextInt(45)+1;
}

 

그럼 위 코드가 어떤 동작을 하는지 설명드리면,

if 비교문에서 i++문은 후위 증가이므로 == 비교를 할 시점에는

if (randomArray[i]==randomArray[i]) // 항상 참. i가 0일 때(처음 반복문 시작이라 가정)

이렇게 항상 참인 비교 결과를 갖고 비교 연산을 합니다. 그리고 후위증가로 인해 i값은 1이 증가하게 됩니다.

그 상태에서 다음 연산이 실행됩니다.

randomArray[i]=random.nextInt(45)+1; // i는 1이 되고 i가 1인 배열의 값에 난수 생성

무조건 참인 조건문으로 인해 첫번째 반복문이 실행될 시점에 i가 0, 1일 때의 난수가 생성됩니다.

for (int i = 0; i < randomArray.length; i++) {
    randomArray[i]= random.nextInt(45)+1; // i가 0일 때 난수가 한번 생성되고
    if (randomArray[i]==randomArray[i++])
        randomArray[i]=random.nextInt(45)+1; // 비교 연산으로 i가 1이 반드시 되고 난수가 생성됨
}

올리신 코드를 정리하면 반복문은 배열의 길이의 절반만 실행되는 코드와 같고, 중복 체크는 없는 코드라고 볼 수 있습니다. 아래와 같은 동작을 한다고 봐도 무방하다고 볼 수 있습니다.

for (int i = 0; i < randomArray.length; i++) {
    randomArray[i]= random.nextInt(45)+1;
    i++;
    randomArray[i]=random.nextInt(45)+1;
}

코드가 어떤 동작을 하는지 정리하여 설명드렸으니 보완해보셨으면 좋겠습니다 😀

감사합니다.

정현님의 프로필 이미지
정현

작성한 질문수

질문하기