Weird NullPointerException

[Boreeas]

EDIT: It's not weird, I am weird, DarrylBurke says. I still need help though :P
EDIT EDIT: Solved - global variables get initialized before the class itself is initialized. Moving the initialization to a method solved the problem.

Hey, I am currently working on a chat bot with GUI. However, when I try to display the lines received in a pane, I get a NullPointerException

public void read(){

        try {
            
            reader = new BufferedReader(new InputStreamReader(socket.getInputStream()));
            writer = new BufferedWriter(new OutputStreamWriter(socket.getOutputStream()));

            while (true) {

                String line;

                if ((line = reader.readLine()) != null) {

                    System.out.println(line);
                    gui.appendRaw(line);
                }
            }
        } catch (IOException e) {

            gui.logError(e);
        } 
    }

System.out.println() works fine, but the line after it (gui.appendRaw()) causes the exception.
Whole class can be found here (Bot.java) and here (Line 325+) (GUI.java)

[Junky]

Then gui must be null. you will have to investigate why. We cannot tell from the small snippet you posted.

[sunde887]

Many people won't follow links, or wade through large amounts of code, you can post the full code if you'd like, but don't expect people to read it(some may). It would be helpful if you created a smaller version that illustrates your example.

Also, using print statements can be invaluable, what happens if you print the GUI object?

[DarrylBurke]

And there's nothing wierd about a NullPointerException, but I sometimes feel the people who regard exceptions as wierd are, well, you guessed it.

db

[Boreeas]

Many people won't follow links, or wade through large amounts of code, you can post the full code if you'd like, but don't expect people to read it(some may). It would be helpful if you created a smaller version that illustrates your example.

Also, using print statements can be invaluable, what happens if you print the GUI object?

Hmm, good point, thanks.
Printing the gui gives me - you guessed it - null.
I declare it using

private GUI gui = Instances.getGuiInstance();

and Instances.getGuiInstance() returns a static gui object (new GUI();) Is the "static" part the problem? I'm not good with static/nonstatic stuff :/

public static GUI gui = new GUI();

public static GUI getGuiInstance() {

    return gui;
}

[Norm]

Try adding a print to show if it is valid or changed

public static GUI getGuiInstance() {
    System.out.println("gui=" + gui); // show current contents of gui
    return gui;

[Boreeas]

Alright, I did that:

GUI = null
GUI = ircbotgui.GUI[frame0,0,0,1511x826,hidden,layout=java.awt.BorderLayout,title=,resizable,normal,defaultCloseOperation=EXIT_ON_CLOSE,rootPane=javax.swing.JRootPane[,8,30,1495x788,layout=javax.swing.JRootPane$RootLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=16777673,maximumSize=,minimumSize=,preferredSize=],rootPaneCheckingEnabled=true]
GUI = ircbotgui.GUI[frame0,0,0,1511x826,hidden,layout=java.awt.BorderLayout,title=,resizable,normal,defaultCloseOperation=EXIT_ON_CLOSE,rootPane=javax.swing.JRootPane[,8,30,1495x788,layout=javax.swing.JRootPane$RootLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=16777673,maximumSize=,minimumSize=,preferredSize=],rootPaneCheckingEnabled=true]

GUI = none when called from BaseBot.java (Note:As a global variable) and apparently correct the other two times when being called from GUI.java (From functions, that time)

[Boreeas]

Solved - global variables get initialized before the class itself is initialized. Moving the initialization to a method solved the problem.