NullPointerException når jeg kalder min custom paintBG(Graphics g)

Du har kommenteret linje 40 i Window.java så g bliver aldrig initialiseret.

Det er jo fordi du først initialiserer panelet på linjen bagefter:

 g = panel.getGraphics();
 panel = new Panel(window);

Når du får en NullPointerException skyldes det næsten altid at du bruger en variabel du har glemt at initialisere. En strategi for at finde fejlen er at finde linjen du får problemet på. F.eks. fra før:

 g.drawImage(window.bg, 1, 1, null);

Det er ikke fordi du sender en null værdi med til drawImage du fik fejlen for så ville fejlen komme inde i drawImage. I stedet er det fordi du prøver at læse en null værdi. Så det kan kun være 'window' eller 'g'. Ved at "følge programmet tilbage" kan man så finde synderen.

Har lige læst lidt rundt og det anbefales ikke at man bruger getGraphics uden for tegnemetoderne. Siden du kalder getGraphics i constructoren på Window er det ikke sikkert at Window eller Panel er synlige endnu. Så kan getGraphics godt returnere null. Graphic's objektet den giver dig har også et kort liv, som du forlænger ved at have en reference til den 'g'. Men efter noget tid vil ting der bliver tegnet på den ikke længere have nogen virkning.

I stedet anbefales det at du overrider paintComponent og tegner derfra i stedet (den skal du ikke selv kalde, det gør Swing for dig):
http://stackoverflow.com/questions/5446396/javax-swing-jpanel-paintcomponent

Husk at kalde super.paintComponent(g);

Hvis du gør det på denne måde vil den automatisk gentegne sig selv når du flytter på vinduet eller resizer.

Når ja, da jeg prøvede at rette koden udkommenterede jeg og lavede paintBG(Graphics g) om til default paint(Graphics g)... - Når jeg fjerne kommenteringen, får jeg dette output:

Exception in thread "main" java.lang.NullPointerException
at Window.<init>(Window.java:40)
at Window.main(Window.java:102)


Får ikke engang skrevet Hello Woindow ud...

Aaaarh, ja selvfølgelig! :-) - Har rettet den fejl, men min kode et tydeligvis ikke særlig god, for får bare en ny fejl

Hello Woindow
Exception in thread "main" java.lang.NullPointerException
	at Panel.paintBG(Panel.java:25)
	at Window.<init>(Window.java:66)
	at Window.main(Window.java:103)

Nu kan den eksekvere koden ned til linje 66 i Window klassen, får fejl ved panel.paintBG(g);.. NullPointerException igen..

Overvejede om det måske kunne være fordi jeg gav den g.drawImage() en null værdi med som parameter, men det er ikke derfor, for jeg har brugt præcis samme kode før, altså g.drawImage(window.bg, 1, 1 null); i en tidligere version af min kode.. Har også initialiseret billederne før jeg kører paintBG(g) metoden, så det kan ikke være der jeg får NullPointerException.. Kan ikke finde ud af hvor jeg prøver at læse en null værdi.. Har tjekket om den fejler ved init af billederne, men det gør den ikke..

Hmm ved ikke om vi ser det samme kode eller linjenumre længere.

Okay opdaterer lige min kode

Window.java

 import java.awt.Color;
 import java.awt.Graphics;
 import java.awt.Image;
 import java.io.File;
 import java.io.IOException;
 
 import javax.imageio.ImageIO;
 import javax.swing.JFrame;
 import javax.swing.JLabel;
 import javax.swing.JTextArea;
 /**
  * Game DotMS
  * @author Jacob Andreas Sneding Rohde
  * @version 1.0
  */
 public class Window extends JFrame
 {    
     public static Window window;
     Panel panel;
     Keylistener keylistener; 
     Graphics g;
     
     int windowx, windowy; // Window size
     int charlesx, charlesy; // Where charles is positioned
     // For the prompt
     String command = "";
     String outputString = "";
     JTextArea output; // Output, where the response will get shown i:
     JLabel input; // For the commands
     
     
     public Image bg, charles, pants, shirt, socks, tanktop, underpants, wig;
     
     
     public Window() // ONLY MAKE ONE OBJECT OF EACH CLASS!!!!
     {
         
         keylistener = new Keylistener(window, panel);
         panel = new Panel(window);
         g = panel.getGraphics(); // Initiliazing the graphics object 
         
         // For the output stupid
         output = new JTextArea();
         output.setFocusable(false);
         output.setForeground(Color.WHITE);
         output.setBackground(Color.BLACK);
         output.setEditable(false);
         output.setText(outputString);
         output.setLineWrap(true);
         //panel.add(output, 0, 800);
         
         System.out.println("Hello Woindow");
         windowx = 1000;
         windowy = 700;
         
         charlesx = 50;
         charlesy = 60;
         
         init();
         
         this.setVisible(true);
         this.setDefaultCloseOperation(EXIT_ON_CLOSE);
         this.setSize(windowx, windowy);
         this.add(panel);
         
         panel.paintBG(g); // Putting the background to the screen
     }
     
     // Init pics
     public void init()
     {
                 try 
                 {
                     bg = ImageIO.read(new File("res/Background.png"));
                     charles = ImageIO.read(new File("res/Charles.png"));
                     pants = ImageIO.read(new File("res/Pants.png"));
                     shirt = ImageIO.read(new File("res/Shirt.png"));
                     socks = ImageIO.read(new File("res/Socks.png"));
                     tanktop = ImageIO.read(new File("res/Tanktop.png"));
                     underpants = ImageIO.read(new File("res/Underpants.png"));
                     wig = ImageIO.read(new File("res/Wig.png"));
                 } 
                 catch (IOException e) 
                 {
                     e.printStackTrace();
                     System.out.println("Failed when initializing pictures");
                 }
     }
     
     // Refreshing the output for the prompt
     public void refreshOutput()
     {
         output.setText(outputString.toUpperCase());
     }
     // Refreshing the input for the prompt
     public void refreshInput()
     {
         input.setText("\n> " + command);
     }
     
     public static void main(String[] args) 
     {
         window = new Window();
     }
 }

Panel.java

 import java.awt.Graphics;
 import java.awt.Image;
 import javax.swing.*;
 /**
  * Game DotMS
  * @author Jacob Andreas Sneding Rohde
  * @version 1.0
  */
 public class Panel extends JPanel
 {    
     Window window;
     
     public Panel(Window w)
     {
         window = w;
     }
     
     public void addClothes(Graphics g, Image img)
     {
         g.drawImage(img, window.charlesx, window.charlesy, null);
     }
     
     public void paintBG(Graphics g)
     {
         g.drawImage(window.bg, 1, 1, null);    // BACKGROUND
         g.drawImage(window.charles, window.charlesx, window.charlesy, null); // CHARACTER CHARLES!!
     }
 }

Keylistener.java

 import java.awt.event.KeyListener;
 import java.awt.event.KeyEvent;
 /**
  * Game DotMS
  * @author Jacob Andreas Sneding Rohde
  * @version 1.0
  */
 public class Keylistener implements KeyListener
 {
     boolean bpants, bshirt, bsocks, btanktop, bunderpants, bwig;
     
      Window window; 
      Panel panel;
      
      public Keylistener(Window w, Panel p)
      {
          window = w;
          panel = p;
      }
      
      public void keyPressed(KeyEvent e) 
      {
          int keyTyped = e.getKeyCode();
  
          if(keyTyped == KeyEvent.VK_1)
          {
             window.command += "1";
             window.refreshInput();
          }
          
          else if(keyTyped == KeyEvent.VK_2)
          {
               window.command += "2";
             window.refreshInput();
          }
          
          else if(keyTyped == KeyEvent.VK_3)
          {
                window.command += "3";
             window.refreshInput();
          }
          
          else if(keyTyped == KeyEvent.VK_4)
          {
                window.command += "4";
             window.refreshInput();
          }    
          
          else if(keyTyped == KeyEvent.VK_5)
          {
                window.command += "5";
             window.refreshInput();
          }
          
          else if(keyTyped == KeyEvent.VK_6)
          {
                window.command += "6";
             window.refreshInput();
          }
          else if(keyTyped == KeyEvent.VK_7)
          {
             window.command += "7";
             window.refreshInput();
          }
           
         else if(keyTyped == KeyEvent.VK_8)
         {
             window.command += "8";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_9)
         {
             window.command += "9";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_0)
         {
             window.command += "0";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_COMMA)
            {
             window.command += ",";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_PERIOD)
            {
             window.command += ".";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_A)
            {
             window.command += "A";
             window.refreshInput();
         } 
          
         else if(keyTyped == KeyEvent.VK_B)
            {
             window.command += "B";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_C)
            {
             window.command += "C";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_D)
            {
             window.command += "D";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_E)
            {
             window.command += "E";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_F)
            {
             window.command += "F";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_G)
            {
             window.command += "G";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_H)
            {
             window.command += "H";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_I)
            {
             window.command += "I";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_J)
            {
             window.command += "J";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_K)
            {
             window.command += "K";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_L)
            {
             window.command += "L";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_M)
            {
             window.command += "M";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_N)
            {
             window.command += "N";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_O)
            {
             window.command += "O";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_P)
            {
             window.command += "P";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_Q)
            {
             window.command += "Q";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_R)
            {
             window.command += "R";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_S)
            {
             window.command += "S";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_T)
            {
             window.command += "T";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_U)
            {
             window.command += "U";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_V)
            {
             window.command += "V";
             window.refreshInput();
         }  
          
         else if(keyTyped == KeyEvent.VK_W)
            {
             window.command += "W";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_X)
            {
             window.command += "X";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_Y)
            {
             window.command += "Y";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_Z)
            {
             window.command += "Z";
             window.refreshInput();
         }
          
         else if(keyTyped == KeyEvent.VK_SPACE)
          {
             window.command += " ";
             window.refreshInput();
          }
          
         else if(keyTyped == KeyEvent.VK_BACK_SPACE)
         {
             int delete = window.command.length() - 1;
             window.command = window.command.substring(0, delete);
             
                window.refreshInput();
         }
          
         /* if(keyTyped == KeyEvent.VK_ENTER)
         {
                          
            String addOutput
             
             window.outputString += "\n>>> ";
             window.outputString += addOutput;
             window.refreshOutput();
             window.command = "";
             window.refreshInput();
         } */
                  
      }
      
      public void keyReleased(KeyEvent e) 
      {        
          
      }
      
      public void keyTyped(KeyEvent e) 
      {        
          
      }
      
 }

Her er mit output

Hello Woindow
Exception in thread "main" java.lang.NullPointerException
	at Panel.paintBG(Panel.java:25)
	at Window.<init>(Window.java:66)
	at Window.main(Window.java:103)

Okay fik det til at virke, men var desværre lidt doven.. Det blev for besværligt med et JPanel, så har bare valgt at ligge billederne direkte på JFramet...

- Ellers mange tak! Det var nogle gode svar