Выдается исключение NullPointerException, и программа не завершается должным образом

Question

Я создаю программу типа фондовой биржи, и пока у меня работает ввод, так что он правильно воспринимает команду от пользователя. Однако то, что он делает с входом, не работает должным образом. Первое, что меня смущает, это то, почему при запуске кода возникает исключение NullPointerException.

В основном программа будет принимать 3 вещи, а затем пробел между ними. Так, например, я хочу купить 30 акций по 20 долларов каждая, я бы набрал следующую информацию:

б 30 20

Затем он разделится на 3 части и сохранится в массиве. После этого он будет сравнивать первый индекс массива, чтобы увидеть, что должна делать программа, в этом примере он будет покупать, так что он вызовет метод buy и сохранит количество акций и их стоимость в моем CircularArrayQueue.

Он получает значение общего ресурса и количество общего ресурса, сохраненные в Node, но когда я пытаюсь вызвать метод enqueue с моим CirularArrayQueue для сохранения Node в очереди, он дает мне исключение NullPointerException.

Другой проблемой, с которой я столкнулся, было прекращение работы программы. Программа должна завершиться, когда увидит, что первое значение индекса ввода - "q". Я сделал цикл while, заявив, что он будет повторяться, когда логическое завершение будет ложным. Затем в цикле while я сделал оператор if, проверяющий, является ли значение stockParts[0] значением "q". Если это так, он изменит значение quit на true, чтобы завершить цикл, но по какой-то причине он не завершается и продолжает цикл.

Я несколько часов ломал голову над этими вопросами, но, похоже, не могу найти корень проблемы. Может ли кто-нибудь помочь мне в этом? Ниже приведен код из моего основного класса и класса CircularArrayQueue:

import java.util.Scanner;
import java.lang.Integer;

public class StockTran {
String command = "";
String[] stockParts = null;
CircleArrayQueue Q;
boolean quit = false;

public StockTran(String inputCommand) {
    try {
        Scanner conReader = new Scanner(System.in);
        this.command = inputCommand.toLowerCase();
        this.stockParts = command.split("\\s"); // splits the input into three parts

        buyShares(Integer.parseInt(stockParts[1]), Integer.parseInt(stockParts[2]));        //testing purpose only

        while (quit == false) {
            if (this.stockParts[0] == "q") {        // ends transaction and terminates program
                System.out.println("Share trading successfully cancelled.");
                quit = true;    
            }

            if (this.stockParts == null || this.stockParts.length > 3) {
                throw new Exception("Bad input.");
            }

            if (stockParts[0] == "b") {     // checks to see if it is a buying of shares
                int shares = Integer.parseInt(stockParts[1]);       // stores share amount
                int value = Integer.parseInt(stockParts[2]);        // stores selling value
                buyShares(shares, value);       // calls buyShares method and adds share to queue
            }
            else if (stockParts[0] == "s") {        // checks to see if it is a selling of shares
                int shares = Integer.parseInt(stockParts[1]);       // stores share amount
                int value = Integer.parseInt(stockParts[2]);        // stores selling value
                sellShares(shares, value);      // calls sellShares method
            }
            else if (stockParts[0] == "c") {        // checks to see if it is capital gain
                capitalGain();      // calls capitalGain and calculates net gain
            }
            System.out.println("Enter your next command or press 'q' to quit: ");
            command = conReader.nextLine().toLowerCase();
            stockParts = command.split("\\s");
        }

    } catch (Exception e) {
        e.printStackTrace();
    }
}


public void buyShares(int shareAmout, int shareValue) {     // takes in share total and values for each share
    Node temp = new Node(shareAmout, shareValue);       // stores values into node
    try {
        Q.enqueue(temp);        // enqueues the node into the CircularArrayQueue
        //System.out.println(Q.toString());

    } catch (FullQueueException e) {
        e.printStackTrace();
    }

}

public void sellShares(int shareAmount, int sharePrice) {   // ToDo

}

public int capitalGain() {  // ToDo
    return 0;
}

public static void main(String[] args) {
    String inputCommand = "";
    Scanner mainReader = new Scanner(System.in);

    System.out.println("Enter 'b' to purchase share, 's' to sell share, 'c' for capital gain, or 'Q' to quit: ");
    inputCommand = mainReader.nextLine();

    StockTran tran = new StockTran(inputCommand);
}
}

public class CircleArrayQueue implements Queue {
protected Node Q[];     // initializes an empty array for any element type
private int MAX_CAP = 0;        // initializes the value for the maximum array capacity
private int f, r;

public CircleArrayQueue(int maxCap) {
    MAX_CAP = maxCap;
    Q = new Node[MAX_CAP];  // sets Q to be a specific maximum size specified
    f = 0;      // sets front value to be 0
    r = 0;      // sets rear value to be 0;
}

public int size() {
    return (MAX_CAP - f + r) % MAX_CAP;     // returns the size of the CircularArrayQueue
}

public boolean isEmpty() {      // if front and rear are of equal value, Queue is empty
    return f == r;
}

public Node front() throws EmptyQueueException {        // method to get the front value of the CircularArrayQueue
    if (isEmpty()) throw new EmptyQueueException("Queue is empty.");
        return Q[f];        // returns object at front of CircularArrayQueue
}

public Node dequeue() throws EmptyQueueException {  // method to remove from the front of the CircularArrayQueue
    if (isEmpty()) throw new EmptyQueueException("Queue is empty.");
        Node temp = Q[f];       // stores front object in local variable
        Q[f] = null;        // sets the value to be null in the array
        f = (f + 1) % MAX_CAP;      // sets the new front value to be this
        return temp;        // returns the object that was originally in the front
}

public void enqueue(Node element) throws FullQueueException {       // method to add to the end of the CircualarArrayQueue
    if (size() == MAX_CAP - 1) throw new FullQueueException("Queue has reached maximum capacity.");
        Q[r] = element;     // stores the new element at the rear of array
        r = (r + 1) % MAX_CAP;      // sets the new rear value to be the location after element insertion
}
}

Answer 1

Вы не инициализировали ссылку Q. Поскольку это переменная поля, по умолчанию она инициализируется значением null.

    CircleArrayQueue Q;

Когда вы сталкиваетесь с такой проблемой, вы должны отладить ее. Одним из источников информации является трассировка стека от исключения, которая сообщит вам, где было сгенерировано исключение. Вы также можете попросить отладчик в вашей среде разработки автоматически остановиться в тот момент, когда выдается исключение.

Во-вторых, когда вы сравниваете строки в Java, используйте метод equals() вместо оператора ==. Метод equals() сравнивает значения объекта. Оператор == сравнивает значения ссылок, которые указывают на объекты. Вы можете иметь два одинаковых объекта с разными ссылочными значениями.

Answer 2

Инициализируйте ваш CircleArrayQueue Q. если вы не инициализируете его. в качестве значения по умолчанию принимается ноль.

CircleArrayQueue q= new CircleArrayQueue(size);