황성민 과제제출합니다. #1
황성민 과제제출합니다. #1
Conversation
Main.java
Outdated
| for(int i=0; i<attackerlist.size(); i++) { | ||
| for(String shoot : attackerlist.get(i).ball) { | ||
| for(String goal : goalkeeperlist.get(0).keep) { | ||
| if(shoot.contains(goal)) miss[i] +=1; |
There was a problem hiding this comment.
삼중 for문을 사용해야할 이유가 있을까요? 차라리 goalkeeper의 keep을 for문으로 두고 attacker 두 명의 슛과 비교하는게 낫지 않을까요?
Main.java
Outdated
| int[] miss = new int[2]; | ||
| for(int i = 0; i < miss.length; i++) { | ||
| miss[i] = 0; | ||
| } |
There was a problem hiding this comment.
만약 배열을 0으로 초기화하고 싶으시다면 int[] miss = new int[]{0, 0};을 사용하여 선언하면서 초기화할 수 있습니다.
Main.java
Outdated
| balls[i] = sc.nextLine(); | ||
| attackerlist.get(i).ball = balls[i].split(","); | ||
| } |
There was a problem hiding this comment.
attackerlist.get(i).ball =sc.nextLine().split(",");
로 줄일 수 있습니다. 또한, 문제에서는 attackerlsit로 받지 않는다는 것을 명심해주세요.
There was a problem hiding this comment.
또한, attackerlist는 카멜케이스를 위배합니다.
Main.java
Outdated
| } | ||
| String[] balls = new String[2]; | ||
| String keepball; | ||
| Attacker[] attackers = new Attacker[2]; |
There was a problem hiding this comment.
attacker1과 attacker2를 각각 ArrayList로 선언하라고 되어있습니다.
Main.java
Outdated
| for(int i = 0; i < attackers.length; i++) { | ||
| attackers[i] = new Attacker(); | ||
| } | ||
| ArrayList<Goalkeeper> goalkeeperlist = new ArrayList<Goalkeeper>(); |
There was a problem hiding this comment.
goalkeeper 또한 goalkeeperlist가 아닌 ArrayList로 선언하라고 되어있습니다.
Main.java
Outdated
| class Attacker { | ||
| String name; | ||
| String[] ball = new String[5]; | ||
| } | ||
| class Goalkeeper { | ||
| String[] keep = new String[4]; | ||
| } |
There was a problem hiding this comment.
class가 아닌 attacker1과 attacker2, goalkeepr를 ArrayList로 선언하라고 되어있습니다.
Main.java
Outdated
| goalkeeperlist.add(new Goalkeeper()); | ||
| ArrayList<Attacker> attackerlist = Arrays.stream(attackers).collect(Collectors.toCollection(ArrayList::new)); |
Main.java
Outdated
| keepball = sc.nextLine(); | ||
|
|
||
| goalkeeperlist.get(0).keep = keepball.split(","); | ||
| for(int i=0; i<attackerlist.size(); i++) { |
There was a problem hiding this comment.
attackerList를 사용하지 않았더라면 위와 같은 쓸모 없는 for문이 감소했겠죠.
Main.java
Outdated
| if(miss[0]>miss[1]) System.out.println(attackerlist.get(1).name + "님이 승리하셨습니다."); | ||
| else if (miss[0]<miss[1]) System.out.println(attackerlist.get(0).name + "님이 승리하셨습니다."); | ||
| else System.out.println("무승부입니다."); | ||
| } | ||
| } |
There was a problem hiding this comment.
여기도 attackerlist가 아닌 attacker1과 attacker2라는 ArrayList를 생성한 뒤 각각처리해주세요.
No description provided.