NullPointerException

[hasha]

Hallo!

Ich bin grade dabei Vier Gewinnt zu programmieren und habe ein Problem beim Programmieren einer Methode, die dafür sorgt, dass ein Stein in das VierGewinntFeld geworfen wird. Falls ich auf das Feld klicke, kriege ich eine NullPointerException.


Das ist die Klasse MouseListener, wo das Problem auftaucht:
(Die Variable access gibt an, ob der Spieler an der Reihe ist oder nicht. Die Variable red gibt an, ob der Spieler mit den roten Steinen oder den Gelben spielt.)

//MouseListener
  class CustomMouseListener implements MouseListener{
    public void mouseClicked(MouseEvent event) {
      taChat.append("Mouse Clicked: ("+event.getX()+", "+event.getY() +")" + "\n");
      
      if (access) {
        int x = event.getX();
        int y = event.getY();
        
        for (byte i = 0;i<7;i++) {
          if (x>i*89 && x<(i+1)*89) {
            byte j = this.searchJ(i);
            
            if (j == 6) {
              
            } // end of if
            else {
              byte key = 0;
              
              if (red == true) {
                key = 1;
                lbFeld[i][j].setIcon(new ImageIcon("red.png"));
                lbFeld[i][j].setKey(key);
              } // end of if
              else {
                key = 2;
                lbFeld[i][j].setIcon(new ImageIcon("yellow.png"));
                lbFeld[i][j].setKey(key);
              } // end of if-else
              
              hClient.sendTurn(i,j,key);
            } // end of if-else
            
            break;
          } // end of if
        } // end of for
        
        hClient.send(hClient.arrayToString(lbFeld));
      } // end of if
      
      else {
        taChat.append("Du bist nicht dran!" + "\n");
      } // end of if-else
      
      
    }
    
    public void mousePressed(MouseEvent e) {
    } 
    
    public void mouseReleased(MouseEvent e) {
    }
    
    public void mouseEntered(MouseEvent e) {
    }
    
    public void mouseExited(MouseEvent e) {
    }
    
    
    public byte searchJ (int pI) {
      byte j = 5;
      while (j>=0 && lbFeld[pI][j].getKey()==0) {
        j--;
      } // end of for
      if (j<0) {
        return 6;
      } // end of if
      else {
        return j;
      } // end of if-else
    }  
  }

Falls ich auf das Feld in der GUI klicke, wird im Editor folgendes angezeigt:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at FirstClientGUI$CustomMouseListener.mouseClicked(FirstClientGUI.java:310)
at java.awt.Component.processMouseEvent(Component.java:6508)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3320)
at java.awt.Component.processEvent(Component.java:6270)
at java.awt.Container.processEvent(Container.java:2229)
at java.awt.Component.dispatchEventImpl(Component.java:4861)
at java.awt.Container.dispatchEventImpl(Container.java:2287)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4832)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4501)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4422)
at java.awt.Container.dispatchEventImpl(Container.java:2273)
at java.awt.Window.dispatchEventImpl(Window.java:2719)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:735)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:694)
at java.awt.EventQueue$3.run(EventQueue.java:692)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
at java.awt.EventQueue$4.run(EventQueue.java:708)
at java.awt.EventQueue$4.run(EventQueue.java:706)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:705)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)







Vorhin sah die Methode mouseClicked noch so aus, da funktionierte es noch. Ich habe auch was an der Methode searchJ geändert, davon habe ich aber den ursprünglichen Code nicht mehr. Wenn ich mouseClicked wieder umändere, geht es immer noch nicht, also liegt es wohl an der Methode searchJ.

public void mouseClicked(MouseEvent event) {
taChat.setText("Mouse Clicked: ("+event.getX()+", "+event.getY() +")");
 
int x = event.getX();
int y = event.getY();
 
if (x>250 && x<330 && y>45 && y<480) {
int i = this.getI(x);
int j = this.searchJ(i);
lbFeld[i][j].setIcon(new ImageIcon("yellow.png"));
lbFeld[i][j].setKey(2);
} // end of if
}

Danke im Voraus!

EDIT:
Das VierGewinnt-Feld ist ein zweidimensionaler Array bestehend aus JLabeln mit den Längen 7 und 6. Jedes JLabel kriegt einen Key zugewiesen, falls kein Stein auf dem Array ist, ist der Key = 0, falls der Stein rot ist, ist der Key = 1 und falls er geld ist, ist der Key = 2.
Jedes JLabel hat eine Länge von 89 x 80.

 

[arilou]

Nun, das ist mit nur einem Codefragment nicht beantwortbar; die Fehlermeldung sagt

FirstClientGUI$CustomMouseListener.mouseClicked(FirstClientGUI.java:310)

der Fehler sei in Zeile :310, und auf jeden Fall in .mouseClicked (also nicht in .searchJ !), aber die Zeilennummern stimmen nunmal nicht mit dem hier von dir angegebenen Code überein...

 

[Joose]

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at FirstClientGUI$CustomMouseListener.mouseClicked(FirstClientGUI.java:310)

Hier steht genau in welcher Datei und welcher Zeile die Exception auftritt.
Leider können wir nicht nachvollziehen um welche Zeile es sich bei deinem Code hier handelt.

Aber suche einfach die Zeile und überlege warum dort ein Objekt "null" ist.

 

[hasha]

Die Zeile 310 entspricht der Zeile 38. (hClient.send(hClient.arrayToString(lbFeld));) Die Methode arrayToString ist dafür da, das Feld in einen String umzuwandeln, um ihn dann an den Server schicken zu können. In der Anwendung kann man nähmlich nur Strings an den Server schicken.

Ich habe die Methode arrayToString nun folgendermaßen geändert und frage einfach ab, ob das Feld gleich null ist (pBaustein ist ein JLabel):

public String arrayToString(Baustein[][] pBaustein){
    if (pBaustein != null) {
      String erg = "Feld:";
      for (int i = 0;i<7;i++) {
        for (int j=0;j<6;j++) {
          int key = pBaustein[i][j].getKey();
          erg += i + j + key; 
        } // end of for
      } // end of for
      return erg;
    } // end of if
    else {
      return "";
    } // end of if-else
  }

Aber auch das hilft nicht..

Der Array sollte aber eigentlich sowieso nie null sein. In der GUI wird er folgendermaßen erstellt:

public Baustein[][] lbFeld = new Baustein[7][6];
public FirstClientGUI(){

//Spielfeld
    for (int i=0;i<7;i++) {
      for (int j=0;j<6;j++) {
        lbFeld[i][j] = new Baustein("empty.png");
        lbFeld[i][j].setKey(0);
        //panelFeld.add(lbFeld[i][j]);
        lbFeld[i][j].setBounds(290 + i*89, 75+j*80, 89, 80);
        contentPane.add(lbFeld[i][j]);
      } // end of for
    } // end of for
    panelFeld.setBounds(290, 75, 623, 480);
    panelFeld.addMouseListener(new CustomMouseListener());
    contentPane.add(panelFeld);
}

Die Klase pBaustein sieht so aus:

 import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.io.File;
import javax.imageio.ImageIO;
import java.awt.image.BufferedImage;


class Baustein extends JLabel
{
  private int key;                         
  //private JLabel label;
  
  public Baustein(String pImage){
    super.setIcon(new ImageIcon (pImage));
  }
  
  public void setIcon (int pKey) {
    if (pKey == 0) {
      super.setIcon(new ImageIcon ("empty.png"));
    } // end of if
    else if (pKey == 1) {
      super.setIcon(new ImageIcon ("red.png"));
    } // end of if
    else if (pKey == 2) {
      super.setIcon(new ImageIcon ("yellow.png"));
    } // end of if
  }
  
  public void setKey(int pKey){
    key = pKey;
  }
  
  public int getKey(){
    return key;
  }
  
}

Ganz abgesehen davon, habe ich die Zeile 310 mal auskommentiert, die NullPointerException kriege ich nicht mehr, aber in der GUI tut sich sowieso nichts wenn ich auf das Feld klicke. Es werden lediglich die Koordinaten des Klickpunktes angezeigt, aber keines der JLabel wird geändert.
Wenn ich aber beispielsweise lbFeld[2][3].setIcon(2); in den ActionListener eines Knopfes reinpacke, dann funktioniert das Ganze per Knopfdruck.


Ich habe die Zeilen 22 und 27 des MouseListeners (ganz oben im ersten Beitrag) folgendermaßen geändert, aber auch das bringt nichts..:

 lbFeld[i][j].setIcon(key);
lbFeld[i][j].setIcon(key);

(Die Variable access habe ich übrigens auf true gesetzt.)
Sonst habe ich keine Idee warum nicht einmal das funktioniert. :S


So das ist jetzt einiges an Quelltext, ich hoffe das schreckt euch nicht ab. ^^

 

[Joose]

Die Zeile 310 entspricht der Zeile 38. (hClient.send(hClient.arrayToString(lbFeld));)

Passt, es ist nicht die Methode "arrayToString" das Problem sondern das Element "hClient" ist ganz einfach "null"!
Erklärung:
Der Fehler tritt in Zeile X auf, daher muss du in Zeile X schauen welche Objekte verwendet werden. In diesem Fall ist es nur "hClient".
Daher muss dieses Objekt null sein.

 

[hasha]

Das macht Sinn, danke! Ich habe einfach einen weiteren boolean ergänzt, der erst dann auf true gesetzt wird, wenn hClient intialisiert wurde, jetzt taucht keine NullPointerException auf. Die Zeile 6 (erster Beitrag) sieht jetzt so aus:

 if (access==true && verbindung == true) {

Allerdings ändert sich an der GUI sowieso nichts, es wird "kein Stein in das Feld geworfen". Hast du/Habt ihr eine Idee warum?

EDIT:
So, habe den Fehler jetzt gefunden! Danke nochmal.