작성
·
148
0
이렇게 풀었는데요 if부분이 저도 잘모르겠는데 작동은 하긴합니다..근데 일단 영한님처럼 두개로 나눠쓴다(꼭 나눌필요는없다하셨지만..)같은 차근차근 생각이 전혀 안납니다..ㅠㅠ 어떻게저떻게 머리 굴려서 해보긴했는데 뭔가 이상한 부분은 없는지 봐주셨으면 좋겠습니다.
그리고 이런..어떻게 코드를 작성해야할지 생각이 안나는 것은 어떻게 해결을 할 수 있을까요..?
package lang.Math.test;
import java.util.Arrays;
import java.util.Random;
public class Lotto {
public static void main(String[] args) {
Random random = new Random();
int[] arrays = new int[6];
for(int count = 0; count < 6; count++){
int num = random.nextInt(45) + 1;
if(num!=arrays[count]){
arrays[count] = num;
}
}
System.out.println("arrays = " + Arrays.toString(arrays));
}
답변 1
0
안녕하세요, 인프런 AI 인턴입니다. duswjdhh님.
작성하신 로또 번호 생성 코드에서 개선이 필요한 몇 가지 사항이 있습니다. 먼저, 현재 코드에서는 중복된 숫자가 배열에 포함될 수 있는 가능성이 있습니다. 이는 난수 발생 시 중복 검사를 제대로 수행하고 있지 않기 때문입니다. 아래에 개선 방안을 드리겠습니다.
중복 검사 로직의 부재: 현재의 if(num!=arrays[count])
조건은 배열 내의 값과 새로 생성된 num
값이 같지 않을 경우에만 값을 할당하고 있습니다. 이는 배열 내의 다른 값들과 중복을 허용할 수 있습니다. 따라서 배열에 이미 존재하는 값인지 확인하는 추가 로직이 필요합니다.
코드 개선 제안: 다음과 같은 방법으로 코드를 개선할 수 있습니다.
import java.util.Arrays;
import java.util.Random;
public class Lotto {
public static void main(String[] args) {
Random random = new Random();
int[] arrays = new int[6];
int count = 0;
while (count < 6) {
int num = random.nextInt(45) + 1;
boolean exists = false;
// 중복 체크
for (int i = 0; i < count; i++) {
if (arrays[i] == num) {
exists = true;
break;
}
}
// 중복되지 않는 경우만 배열에 할당
if (!exists) {
arrays[count] = num;
count++;
}
}
System.out.println("arrays = " + Arrays.toString(arrays));
}
}
유사한 질문들과 해결 방법을 공유한 게시물이 있었는데요:
- java중급 4강 레퍼클래스 로또 번호 생성기 문제
- 문제를 스스로 풀면서
위의 링크들이 추가적인 이해에 도움이 될 수 있습니다.
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.