0

Здравствуйте, Я только начал изучать Java (примерно как месяц) и сейчас застрял на коллекциях. Причём застрял так основательно - с пробуксировкой. Решил написать игрушку консольную "Поле Чудес", но есть запара. Подскажите, подалуйста, форумы, где мне могут помочь. Не думаю, что кому то из местных захочется разбираться в моём "творчистве", поэтому прошу лишь указать где искать помощи. В гугле посмотрел, но там то просто статьи с примерами задач, но то какие это уж слишком крутый форумы. Спасибо. Cyrus.

UPD Вот у меня в main.java есть кусок:

Player player = new Player();
        player.setName("Player1");
        game.AddPlayer();
        player.setLetter('a');
        game.Check();

Как из него может быть некоторым понятно происходит слкдующее: 1. создаём объект класса Плеир 2. через сеттер заносим в него имя игрока 3. добавляем игрока в коллекцию (код ниже) 4. игрок вводит букву 5. происходит некая операция с буквой

Теперь "код ниже":

void AddPlayer() {
        System.out.println("Регистрирую игрока с id " + globalId);
        playersCollection.put(globalId, player.getName());
        globalId++;
    }

Ошибка происходит как раз в третий строчке при попытке внести игрока в коллекцию - NullPointerException, т.е., как я понимаю, из player.getName() нифигашеньки не возвращается, верно?

UPD2 Вот инициализация, @JEcho

HashMap<Integer, String> playersCollection = new HashMap<>();
Player player;
int globalId = 0;

UPD3 Вот класс Player, который по нашим ожиданиям "сыпится":

package main;

/
@date Sep 24, 2013
@author Cyrus S
*/

class Player { private String name; private char letter;

void setName(String name) {
    this.name = name; 
}

String getName() {
    return name;
}

void setLetter(char letter) {
    this.letter = letter; 
}

char getLetter() {
    return letter;
}

}

Хотя там и сыпаться-то нечему.

8
  • Напишите конкретно какая вам коллекция непонятна - попробуем помочь. Только не надо все валить в кучу - одна коллекция. И не надо лепить много исходников. Попробуйте вкратце и внятно объяснить что непонятно. Если вы внятно объясните что вам непонятно, то вам помогут, а если начнете писать: млин завтра здавать курсач, препод казёл - поможите!!!! - вас пошлют и причем далеко
    – Barmaley
    26 сен 2013 в 16:50
  • а можно поподробнее про "рассыпание" player? Что мне стоит проверить, на что заострить внимание?
    – P. Savrov
    26 сен 2013 в 17:49
  • player не инициализирован. int по умолчанию "0" , HashMap, не ругается, если в нее null пихать, т.е. такая конструкция к ошибке не приведет: map.put(null, null); остается player) вообще не показательный код;-) пишите более подробно (но без фанатизма)
    – JEcho
    26 сен 2013 в 17:52
  • @JEcho, инициальзировал player'a, с инта снял ноль - поезд двинулся, спасибо! А почему с инта нужно было убрать ноль? Потому что при каждом вызове объекта класса Game он обнуляется, верно?
    – P. Savrov
    27 сен 2013 в 3:28
  • 1
    Сыпаться тут и правда нечему. Давайте-ка вы возьмёте отладчик, и скажете, какая именно строка вылетает. Ставлю на то, что ваша переменная, в которой должен быть Player, равна null.
    – VladD
    27 сен 2013 в 9:49

2 ответа 2

3

Советую переписать функцию добавления игрока, слово 'add' в названии функции предполагает передачу параметров. Вместо: game.AddPlayer(); Лучше: game.AddPlayer(player);

3
  • 2
    И заодно перечитать Code Conventions for the Java Programming Language
    – wwvv
    27 сен 2013 в 9:55
  • 1
    это сионисты)..в смысле сишники...у них обычно методы с заглавной))..тоже глаз режет))
    – JEcho
    27 сен 2013 в 11:31
  • понял. уже читаю.
    – P. Savrov
    27 сен 2013 в 11:33
1

стэктрейс бы...NullPointerException значит что-то инициализированно.. playersCollection,player,globalId - где и как инициализируются? Npe возникает когда Вы пытаетесь обратиться, к методу не инициализированного объекта, т.е. я почти уверен, что трабла ни в том, что getName возвращает нал, скорее всего сыпится player, на попытке вызвать метод getName, либо playersCollection на попытке дернуть put

2
  • знать бы что такое этот "стэкрейс". в гугле на первых трёх страницах одно и тоже: "вот мой стекрейс", "дай стекрейс", а что это из себя представляет и как его вызывать - не понятно upd. нашёл. не нужно в русском сегменте искать было.
    – P. Savrov
    27 сен 2013 в 10:25
  • стэк вызова....эм...как-бы попроще...а придумал! Та хрень которая пишется когда вываливается ошибка, просто описание ошибки обычно многое может рассказть, кто куда и что впихнул невпихуемое)
    – JEcho
    27 сен 2013 в 11:29

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.