Не получается наполнить массив внутри метода

@AMufu · Регистрация: 05.09.2012

Студворк — интернет-сервис помощи студентам

Уважаемые форумчане,

пробовал написать код игры Морской бой (пока упрощенная версия, где только 1 кораблик на 3 палубы, и 1 строчка, в которой он помещается. Часть брал из учебника часть пробовал придумывать сам. Возникла проблема с методом setLocationCells, который "устанавливает" кораблик (в виде массива). Метод я писал сам, и он не совсем работает. То есть отдельно, он как бы присваивает массиву нужные значения и даже может их распечатать, но остальные части программы данного массива int [] locationCells как бы не видят.
Подскажите пожалуйста, что сделал не так
Прилагаю код который написал я, и который рекомендуется учебником.
Мой код

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
package am;
import java.util.Random;
import java.io.*;
public class DotComGame {
    public static void main(String[] args) {
        int numOfGuesses = 0;
        SimpleDotCom theDotCom = new SimpleDotCom ();
        theDotCom.setLocationCells();
        GameHelper helper = new GameHelper ();
        boolean isAlive = true;
        while (isAlive == true) {
            String guess = helper.getUserInput("Введите число");
            System.out.println(guess);
            String result = theDotCom.checkYourSelf(guess);
            numOfGuesses++;
            if (result.equals("Потопил")) isAlive = false;
            System.out.println("Вам понадобилось " + numOfGuesses + "попытки(и)");
        }
    }
}
class GameHelper {
    public String getUserInput (String prompt) {
        String inputLine = null;
        System.out.println(prompt + "  ");
        try {
            BufferedReader is = new BufferedReader (new InputStreamReader (System.in));
            inputLine = is.readLine();
            if (inputLine.length() == 0) return null;
        } catch (IOException e) {System.out.println("IO Exception : " + e);}
        return inputLine;
    }   
} 
 
class SimpleDotCom  {
    int [] locationCells;
    int numOfHits = 0;
    public void setLocationCells () {
        int randomNum = (int) (Math.random ()* 5);
        int [] locationCells = {randomNum, randomNum+1, randomNum+2};
//        for(int i = 0; i <= locationCells.length  - 1; i++) {
//                System.out.print(locationCells[i] + "  ");
//        }    
    }
     public String checkYourSelf (String stringGuess) {
        int guess = Integer.parseInt(stringGuess);
        
        String result = "Промах";
        for (int cell : locationCells) {
            if (guess == cell) {
            result = "Попал";
            numOfHits++;
            break;
            }
        }
        if (numOfHits == locationCells.length) {
            result = "Потопил";
        }
        System.out.println(result);
        System.out.println(numOfHits);
        return result;
    }    
}

код из учебника

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
package simpledotcomtestdrive;
import java.util.Random;
import java.io.*;
public class SimpleDotComTestDrive {
    public static void main(String[] args) {
        int numOfGuesses = 0;
        SimpleDotCom theDotCom = new SimpleDotCom ();
        GameHelper helper = new GameHelper ();
        int randomNum = (int) (Math.random ()* 5);
        int [] locations = {randomNum, randomNum+1, randomNum+2};
        theDotCom.setLocationCells(locations);
        boolean isAlive = true;
        while (isAlive == true) {
            String guess = helper.getUserInput("Введите число");
            System.out.println(guess);
            String result = theDotCom.checkYourSelf(guess);
            numOfGuesses++;
            if (result.equals("Потопил")) isAlive = false;
            System.out.println("Вам понадобилось " + numOfGuesses + "попытки(и)");
        }
    }
}
class GameHelper {
    public String getUserInput (String prompt) {
        String inputLine = null;
        System.out.println(prompt + "  ");
        try {
            BufferedReader is = new BufferedReader (new InputStreamReader (System.in));
            inputLine = is.readLine();
            if (inputLine.length() == 0) return null;
        } catch (IOException e) {System.out.println("IO Exception : " + e);}
        return inputLine;
    }   
} 
 
class SimpleDotCom  {
    int [] locationCells;
    int numOfHits = 0;
    public void setLocationCells (int [] locs) {
        locationCells = locs;
    }    
    
     public String checkYourSelf (String stringGuess) {
        int guess = Integer.parseInt(stringGuess);
        
        String result = "Промах";
        for (int cell : locationCells) {
            if (guess == cell) {
            result = "Попал";
            numOfHits++;
            break;
            }
        }
        if (numOfHits == locationCells.length) {
            result = "Потопил";
        }
        System.out.println(result);
        System.out.println(numOfHits);
        return result;
    }    
}

Спасибо.

@Yoghurt_92 · 30.04.2014, 03:13

AMufu, а какие именно части не видят? подробнее опишите, что вы хотите сделать?

@AMufu · 30.04.2014, 03:24 **[ТС]**

Проблема с методом setLocationCells, который "устанавливает" кораблик (в виде массива). Метод я писал сам, и он не совсем работает. То есть отдельно, он как бы присваивает массиву нужные значения и даже может их распечатать, но остальные части программы данного массива int [] locationCells как бы не видят.
При попытке выполнить
Вот мой не работающий вариант метода

Java
1
2
3
4
5
6
public void setLocationCells () {
        int randomNum = (int) (Math.random ()* 5);
        int [] locationCells = {randomNum, randomNum+1, randomNum+2};
//        for(int i = 0; i <= locationCells.length  - 1; i++) {
//                System.out.print(locationCells[i] + "  ");
//        }

Программа с таким вариантом метода компилируется и даже начинает выполнятся, но выдает ошибку

Введите число
5
5
Exception in thread "main" java.lang.NullPointerException
at ua.am.SimpleDotCom.checkYourSelf(DotComG ame.java:48)
at ua.am.DotComGame.main(DotComGame.java:14 )
Java Result: 1

А метод описанный в учебнике такой проблемы не имеет.

Java
1
2
3
public void setLocationCells (int [] locs) {
        locationCells = locs;
    }

И не понимаю почему, и что я нарушил.

@Yoghurt_92 · 30.04.2014, 04:48

AMufu,

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
import java.util.Random;
import java.io.*;
public class DotComGame {
    public static void main(String[] args) {
        int numOfGuesses = 0;
        SimpleDotCom theDotCom = new SimpleDotCom ();
        theDotCom.setLocationCells();
        GameHelper helper = new GameHelper ();
        boolean isAlive = true;
        while (isAlive == true) {
            String guess = helper.getUserInput("Введите число");
            System.out.println(guess);
            String result = theDotCom.checkYourSelf(guess);
            numOfGuesses++;
            if (result.equals("Потопил")) isAlive = false;
            System.out.println("Вам понадобилось " + numOfGuesses + "попытки(и)");
        }
    }
}
class GameHelper {
    public String getUserInput (String prompt) {
        String inputLine = null;
        System.out.println(prompt + "  ");
        try {
            BufferedReader is = new BufferedReader (new InputStreamReader (System.in));
            inputLine = is.readLine();
            if (inputLine.length() == 0) return null;
        } catch (IOException e) {System.out.println("IO Exception : " + e);}
        return inputLine;
    }   
} 
 
class SimpleDotCom  {
    static int randomNum;
    
    static {
        randomNum = (int) (Math.random ()* 5);
    }
 
    int [] locationCells = {randomNum, randomNum+1, randomNum+2};
    int numOfHits = 0;
    public void setLocationCells () {
        for(int i = 0; i <= locationCells.length  - 1; i++) {
               System.out.print(locationCells[i] + "  ");
       }    
        System.out.println();
    }
     public String checkYourSelf (String stringGuess) {
        int guess = Integer.parseInt(stringGuess);
        
        String result = "Промах";
        for (int cell : locationCells) {
            if (guess == cell) {
            result = "Попал";
            numOfHits++;
            break;
            }
        }
        if (numOfHits == locationCells.length) {
            result = "Потопил";
        }
        System.out.println(result);
        System.out.println(numOfHits);
        return result;
     }
}

пояснить в чем была ошибка?

Добавлено через 1 минуту
AMufu, вы дважды объявляли массив с одним и тем же именем, один раз как член класса, а второй локально в методе, отсюда Вам и ошибка

Добавлено через 6 минут
AMufu, т.е. вы локальный массив (существующий на протяжении работы метода) вы заполняли необходимыми рандомными числами, а в методе checkYourSelf обращались к массиву принадлежащему объекту класса, но для него вы не определили никаких действий, он пустой, отсюда и исключение NullPointerException.

@AMufu · 30.04.2014, 05:03 **[ТС]**

Спасибо. Ето кое-что обяснило.
Но тогда еще вопрос возникает - почему если из моего варианта метода убрать int [] перед именем массива и сделать так, типа мы инициализируем этот массив (если я правильно использую термин)

Java
1
2
3
public void setLocationCells () {
        int randomNum = (int) (Math.random ()* 5);
        locationCells = {randomNum, randomNum+1, randomNum+2};

то все равно выдает ошибку

illegal start of expression

not a statement

';' expected

Добавлено через 5 минут
и если можно сдесь же задам вопрос по другой части того же кода.
В приведенном коде "ход" от игрока принимается с помощью класса GameHelper и использования входящего потока

Java
1
2
3
4
5
6
7
8
9
10
11
12
class GameHelper {
    public String getUserInput (String prompt) {
        String inputLine = null;
        System.out.println(prompt + "  ");
        try {
            BufferedReader is = new BufferedReader (new InputStreamReader (System.in));
            inputLine = is.readLine();
            if (inputLine.length() == 0) return null;
        } catch (IOException e) {System.out.println("IO Exception : " + e);}
        return inputLine;
    }   
}

можно ли сделать то же самое но с помощью метода Sytem.in.read (); ?

@Yoghurt_92 · 30.04.2014, 14:16

Сообщение от AMufu

locationCells = {randomNum, randomNum+1, randomNum+2};

почитайте литературу внимательнее, так можно инициализировать массивы только при объявлении

Java
1
unt[] locationCells = {randomNum, randomNum+1, randomNum+2};

так будет правильно

либо выделить память а потом в цикле заполнить.

Сообщение от AMufu

можно ли сделать то же самое но с помощью метода Sytem.in.read (); ?

Нет, так как Вам нужно считывать строку от игрока, а метода read принимает символ, поэтому Ваш вариант не подойдет

Вы читаете Кэти Сьерра, Берт Бейтс "Head First Java"?

@AMufu · 30.04.2014, 14:38 **[ТС]**

Вы читаете Кэти Сьерра, Берт Бейтс "Head First Java"

Да, именно её. Мучился несколько часов пробуя использовать system.in.read так и не смог (подсматривать не хотел, интерестно самому). А если изменить метод getUserInput на char, тогда должно получится с использованием system.in.read?

@Yoghurt_92 · 30.04.2014, 15:06

AMufu, зачем Вам этот System.in.read когда нужно считывать строку а не символ! я сам читаю эту книгу и тоже не стал пользоваться вариантом авторов, а написал свой, с консольной визуализацией, код конечно не очень, но для тренировки пойдет. Работайте с readLine и все будет хорошо, Вам чтение отдельных символов тут не нужно, так зачем заморачиваться?

@AMufu · 30.04.2014, 15:48 **[ТС]**

Вот попробовал обявить массив int [] locationCells и сразу его инициализировать. Но следующий метод checkYourSelf такого массива не видит.

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
class SimpleDotCom  {
    
    int numOfHits = 0;
    public void setLocationCells () {
        int randomNum = (int) (Math.random ()* 5);
        int [] locationCells = {randomNum, randomNum+1, randomNum+2};
        for(int i = 0; i <= locationCells.length  - 1; i++) {
                System.out.print(locationCells[i] + "  ");
        }    
    }
     public String checkYourSelf (String stringGuess) {
        int guess = Integer.parseInt(stringGuess);
        
        String result = "Промах";
        for (int cell : locationCells) {
            if (guess == cell) {
            result = "Попал";
            numOfHits++;
            break;
            }
        }
        if (numOfHits == locationCells.length) {
            result = "Потопил";
        }
        System.out.println(result);
        System.out.println(numOfHits);
        return result;
    }    
}

Как я понимаю, переменные объявленные в одном методе в других местах программы не доступны пока не вызвешь этот метод? Или как?

@Yoghurt_92 · 30.04.2014, 20:41

AMufu, конечно он не будет виден

я вам даже больше скажу, когда выполнение метода закончится, ваш массив будет уничтожен, если не сохранить больше никаких ссылок на него

Поэтому массив должен быть членом класса, а не объявлен локально в методе. Сделайте его приватным членом и вы сможете оперировать им в любом методе класса.

@AMufu · 30.04.2014, 23:16 **[ТС]**

AMufu, конечно он не будет виден я вам даже больше скажу, когда выполнение метода закончится, ваш массив будет уничтожен, если не сохранить больше никаких ссылок на него Поэтому массив должен быть членом класса, а не объявлен локально в методе. Сделайте его приватным членом и вы сможете оперировать им в любом методе класса.

Вот теперь мне понятно наконец. Спассибо )))

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .

@Yoghurt_92 51 / 51 / 42 Регистрация: 06.09.2013 Сообщений: 188
	30.04.2014, 03:13
	AMufu, а какие именно части не видят? подробнее опишите, что вы хотите сделать? 0

@Yoghurt_92 51 / 51 / 42 Регистрация: 06.09.2013 Сообщений: 188
	30.04.2014, 15:06
	AMufu, зачем Вам этот System.in.read когда нужно считывать строку а не символ! я сам читаю эту книгу и тоже не стал пользоваться вариантом авторов, а написал свой, с консольной визуализацией, код конечно не очень, но для тренировки пойдет. Работайте с readLine и все будет хорошо, Вам чтение отдельных символов тут не нужно, так зачем заморачиваться? 0

@Yoghurt_92 51 / 51 / 42 Регистрация: 06.09.2013 Сообщений: 188
	30.04.2014, 20:41
	Сообщение было отмечено AMufu как решение Решение AMufu, конечно он не будет виден я вам даже больше скажу, когда выполнение метода закончится, ваш массив будет уничтожен, если не сохранить больше никаких ссылок на него Поэтому массив должен быть членом класса, а не объявлен локально в методе. Сделайте его приватным членом и вы сможете оперировать им в любом методе класса. 1

Не получается наполнить массив внутри метода

Решение