Коллеги, приветствую.
У меня стоит задача в построении морского боя. Я начинающий и не понимаю ошибку и даже найти ее решения не могу.
Есть метод, в нем игрок указывает куда будет стрелять:
public void doShoot() {
turnScanner =new Scanner(System.in);
shoot=new Shoot();
System.out.println("Куда будем стрелять? Введите номер позиции и нажми Enter");
x= turnScanner.nextInt();
System.out.println("Вы стреляли в x="+ x);
shoot.doShoot();
}
Этот метод, преобразует ранее созданный двумерный массив в одномерный, чтобы игрок мог указать лишь 1 позицию. Например А1 будет 1, А2=2 и так далее.
public void rewriteArray(){
for (int k=0;k<counts; k++)
for (int i = 0; i < x; i++) {
for (int j = 0; j < fieldSize[i].length; j++) {
newFieldSize[k]=fieldSize[i][j];
System.out.print(newFieldSize[k]);
k++;
}
System.out.println();
}
}
Вот класс, в котором я создаю поля:
public class Field {
Scanner stepScanner;
Scanner turnScanner;
int x;
int y;
int counts;
int[][] fieldSize;
int[] newFieldSize;
public Field(){
counts=x*y;
newFieldSize=new int[counts];
stepScanner = new Scanner(System.in);
}
public int getNewFieldSize(int x) {
System.out.println(newFieldSize[x]);
return newFieldSize[x];
}
}
В этом классе, я делаю выстрел по кораблю:
public class Shoot {
Field fields;
public void doShoot() {
fields=new Field();
fields.getNewFieldSize(0);
}
}
Собственно при компиляции, у меня возникает ошибка:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 0
at ru.mega.Field.getNewFieldSize(Field.java:23)
at ru.mega.Shoot.doShoot(Shoot.java:14)
at ru.mega.Player.doShoot(Player.java:50)
at ru.mega.Main.main(Main.java:30)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:144)
Я понимаю, что можно сделать выстрел в классе Field и мороки не будет. А может даже и через ArrayList можно будет это сделать ( не пробовал и не хочу, так как хочу отработать все на примитивах сначала).
Помогите, пожалуйста, разобраться с этим.
Как сделать так, чтобы в классе Field мой массив newFieldSize получал значения из метода rewriteArray ( из этого же класса) и передавал их в класс Shoot?
Спасибо.
getNewFieldSize
newFieldSize[x]
, или даже в fields.getNewFieldSize(0); Логика не понятна, вы запрашиваете елемент с номером ноль, но массив newFieldSize у вас пустой. Поэтому возникает исключение. Вам нужно создать массив, до вызова ф-цииdoShoot
как минимум, либо пересмотреть логику алгоритма.System.out.println("Вы стреляли в x="+ x); shoot.doShoot();
вот тут лучше сделать проверку и выдать ошибку, мол вы указали некорректные координаты. повторите снова...