Сейчас обсуждают

[RoMeN]

всем привет, очень прошу помощи, что в инициализации я сделал не так? (забыл приписать что Java)

Код

public class Queue
{
    public int GROUP_ID;
    public int ITEM_ID;

    Queue()
    {
        GROUP_ID = 0;
        ITEM_ID = 0;
    }
}



public class DataBaseControl
{
    final private int queueSize = 10; //максимальный размер очереди

    private Queue[] theQueue;   //массив, представляющий псевдобазу для хранения очереди
    private int addPointer;     //указатель для добавления новых элементов
    private int queuePointer;   //указатель для извлечения элементов


    DataBaseControl()
    {
        theQueue = new Queue[queueSize];
        for(int i = 0; i < queueSize; i++)
        {
            theQueue[i].GROUP_ID = 0;    //почему здесь выходит nullpointerexception?? я же инициализировал двумя строками выше?
            theQueue[i].ITEM_ID = 0;
        }

        addPointer = 0;
        queuePointer = 0;
    }

    //.....

}

Сообщение отредактировал RoMeN - 29.1.2014, 17:49

[HiSER]

Не знаю про синтаксис Java, не пользовался.
Но на Си было бы помойму так:

Код

theQueue[i]->GROUP_ID = 0;
theQueue[i]->ITEM_ID = 0;

Сообщение отредактировал HiSER - 29.1.2014, 21:19

[shouhei]

С точки зрения синтаксиса — всё правильно. В яве все объекты — указатели, потому конкретно указателей то там и нет. Как нет и такой математики вида pointer++ как в Цэ. Точка там корректна. Только такое впечатление, что массив нормально проинициализироваться не может. Может быть из-за queueSize, может быть ещё по какой причине.

ТС, вот тупо, ради отладки, сделай вывод queueSize первой строкой цикла.

И ещё одно, я не вижу никакого смысла в этом куске кода, ибо каждый объект из массива итак инициализируется (0,0)

[RoMeN]

ТС, вот тупо, ради отладки, сделай вывод queueSize первой строкой цикла.

с queueSize всё норм, проверял

И ещё одно, я не вижу никакого смысла в этом куске кода, ибо каждый объект из массива итак инициализируется (0,0)

так или иначе ошибки сыпятся и дальше, и бо выскакивают уже на [0] итерации

==============================================================
PS: добрый человек подсказал решение:

Нужно заполнить массив элементами сперва.
theQueue = new Queue[10]; создает массив для 10 обьектов Queue, но не создает сами обьекты.
вот так должно быть:

Код

public class DataBaseControl
{
    final private int queueSize = 10; //максимальный размер очереди

    private Queue[] theQueue;   //массив, представляющий псевдобазу для хранения очереди
    private int addPointer;     //указатель для добавления новых элементов
    private int queuePointer;   //указатель для извлечения элементов


    DataBaseControl()
    {
        theQueue = new Queue[queueSize];
        for(int i = 0; i < queueSize; i++)
        {
            theQueue[i] = new Queue();
            theQueue[i].GROUP_ID = 0;   //почему здесь выходит nullpointerexception?? я же инициализировал двумя строками выше?
            theQueue[i].ITEM_ID = 0;
        }

        addPointer = 0;
        queuePointer = 0;
    }

    //.....

}

Сообщение отредактировал RoMeN - 29.1.2014, 23:38

[shouhei]

PS: добрый человек подсказал решение:

Кстати да, про такое я и забыл. Всё, закрываемся?

Но и опять таки, не нужны тебе присваивания 0. У тебя он при создании нулями инициализируется.