NullPointerException

[MQue]

Hallo nochmal,

ich habe ein Problem mit meinem Observer, und zwar:
wie im HandBuch von Java (Guido Krüger) beschrieben, habe ich mir einen Observer programmiert.
In der 1. Klasse unten wird die Methode fireUpdate(..) von der Klasse 2 aufgerufen, in der Klasse 2 ruft die Methode fireUpdate - observers.update(this, index, Wert, iArr); auf, welche sich (die Methode update) in der 3. Klasse befindet.
Das ganze wird gestartet von der 3. Klasse ganz unten, durch den Aufruf z.B. von
if (getActionCommand().equals("Taster1"))
{
datenSpeicher.setArrayEintrag(1,4); /* Wert, Index */
}

Mein Problem ist jetzt, dass, wenn ich auf "Taster1" drücke und ich möchte die Farbe dieses Taste (Taster1) verändern (in Klasse 3) dann funktionierts, aber wenn ich auf "Schalter1" oder "Schalter2" drücke umd die hintergrundFarbe von Taster1 zu ändern, dann bekomme ich folgende NullpointerException: -> ich weiß aber nicht wieso, ich weiß nur, dass der Fehler in der dritten Klasse liegt.

der Sinn darin ist: ich möchte testen, wenn ich in der Klasse 1 die Methode setArrayEintrag(..) aufrufe, sollen sich die Hintergrundfarben der Buttons je nach Wert ändern.

Vielleicht könnte mir da jemand helfen -> ich wäre Euch sehr dankbar !!!
Danke
lg

PS: ich kann das ganze Projekt auch gerne schicken.

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at net.jank.ppos.component.JankButton.access$000(JankButton.java:14)
at net.jank.ppos.component.JankButton$1.update(JankButton.java:106)
at net.jank.ppos.datastore.Subject.fireUpdate(Subject.java:35)
at net.jank.ppos.datastore.DatenSpeicher.setArrayEintrag(DatenSpeicher.java:37)
at net.jank.ppos.component.JankButton.actionPerformed(JankButton.java:132)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6038)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3260)
at java.awt.Component.processEvent(Component.java:5803)
at java.awt.Container.processEvent(Container.java:2058)
at java.awt.Component.dispatchEventImpl(Component.java:4410)
at java.awt.Container.dispatchEventImpl(Container.java:2116)
at java.awt.Component.dispatchEvent(Component.java:4240)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4322)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3986)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3916)
at java.awt.Container.dispatchEventImpl(Container.java:2102)
at java.awt.Window.dispatchEventImpl(Window.java:2429)
at java.awt.Component.dispatchEvent(Component.java:4240)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:273)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:183)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:173)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:168)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:160)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:121)

in Klasse 1
public class DatenSpeicher  {
    public static int[] iArr = new int[100];
    Subject datenSpeicher = new Subject();
    int cnt = 0;
    
    public DatenSpeicher() {}
    
    public void attach(Observer observer)
	{
	datenSpeicher.attach(observer);
	}
    
    public void detach(Observer observer)
	{
	datenSpeicher.detach(observer);
	}
    /* wenn sich in dieser Klasse etwas tut dann datenSpeicher.fireUpdate() aufrufen */
    public void setArrayEintrag(int Wert, int index) 
	{
	System.out.println("DatenSpeicher: index: " + index + ", Wert: " + Wert);
        this.iArr[index] = Wert;
	datenSpeicher.fireUpdate(index, Wert, iArr);		
        } 
} 

Klasse 2

public interface Observer
    {
    public void update(Subject subject, int index, int Wert, int[] iArr);
    }

public class Subject
    {
    Observer[] observers = new Observer[5];
    int observerCnt = 0;
    public void attach(Observer observer)		// einen DatenSpeicherMethoden registrieren
	{
	observers[observerCnt++] = observer;
	}
    
    public void detach(Observer observer)		// einen DatenSpeicherMethoden deregistrieren
	{
	for (int i = 0; i < observerCnt; i++)
	    {
	    if (observers[i] == observer)
		{
		--observerCnt;
		for (;i < observerCnt; i++)
		    {
		    observers[i] = observers[i+1];
		    }
		break;
		}
	    }
	}
     public void fireUpdate(int index, int Wert, int[] iArr)
	{
	System.out.println("Ich bin in Subjekt");
	for (int i = 0; i < observerCnt; i++)
	    {
	    observers[i].update(this, index, Wert, iArr);
	   
	    } 
	}

    }

in Klasse 3 ist im Konstruktor folgendes Implementiert:
datenSpeicher.attach(new Observer(){  
	   public void update(Subject subject, int index, int Wert,  int[] iArr)
		{
	       System.out.println("Index: " + index + ", Wert: " + Wert);
		switch (index) {		
		    case 3: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Taster1").iArrMoeglWerteInt[i])   
				    buttons.get("Taster1").setBackground(colorMap.get(buttons.get("Taster1").iArrFarbeZuWerte[i]));
				} break;
		    case 4: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Schalter1").iArrMoeglWerteInt[i])   
				  buttons.get("Schalter1").setBackground(colorMap.get(buttons.get("Taster1").iArrFarbeZuWerte[i]));
				 break;
				}
		    case 5: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Schalter2").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter2").setBackground(colorMap.get(buttons.get("Schalter2").iArrFarbeZuWerte[i]));
				} break;	
		    default:  				
		    }
		}
	    });
und auch in dieser Klasse:
public void actionPerformed(ActionEvent event) 
	{
	if (getActionCommand().equals("Taster1")) {
	    datenSpeicher.setArrayEintrag(1,4);    /* Wert, Index */
	    }
	if (getActionCommand().equals("Schalter1")) {
	    System.out.println(buttons.get("Schalter1").getBackground());
	     datenSpeicher.setArrayEintrag(1,4);
	    }
	if (getActionCommand().equals("Schalter2")) {
	    System.out.println(buttons.get("Schalter2").getBackground());
	     datenSpeicher.setArrayEintrag(1,4);
	    }
	}

 

[Wildcard]

Bei NPEs ist Selbsthilfe angesagt.
Die Exception sagt dir das in JankButton.java in Zeile 14 versucht wird auf ein null-'Objekt' zuzugreifen.
-> Zeile 14 anschauen und herausfinden warum da eine Exception fliegt.
Tipp:
Debugger und println's können sehr hilfreich sein.

 

[MQue]

in Zeile 14 kanns nicht sein aber in Zeile 106 (also 2 FehlermeldungsAusgabe)
ich hab schon alles probiert, ich hab keine Ahnung, warum ich eine NullPointerException bekomme,
Es sind alle 3 Buttons in der Klasse Subjekt registriert worden.


Also: Zeile 106 ist diese Zeile: if (Wert == buttons.get("Schalter1").iArrMoeglWerteInt)
ich habe also mit if (getActionCommand().equals("Taster1")) {
datenSpeicher.setArrayEintrag(2,4);
versucht, die hintergrundfarbe von Schalter1 zu setzen, aber da bekomme ich eben die NPE, ich habe aer echt keine Ahnung und schon alles, was mit bekannt ist, versucht.

Vielleicht könnte ja irgendwer mein Projekt mal ausprobieren? (ich trau mich eh nur leise zu fragen)

Vielen dank auf jeden Fall!!


lg
der verzweifelte Michl

 

[Wildcard]

in Zeile 14 kanns nicht sein

Erzähl mir doch nichts :wink:

at net.jank.ppos.component.JankButton.access$000(JankButton.java:14)

 

[Roar]

"JankButton.access$000" die klasse wird vom compiler erzeugt

 

[WieselAc]

Gelöscht: wegen Langsamkeit!

 

[Wildcard]

"JankButton.access$000" die klasse wird vom compiler erzeugt

Was ändert das an der angegebenen Zeilennummer?

 

[MQue]

ja das war dann eine Ausnahme, Meistens bekomme ich die NPE:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at net.jank.ppos.component.JankButton$1.update(JankButton.java:111)
at net.jank.ppos.datastore.Subject.fireUpdate(Subject.java:35)
at net.jank.ppos.datastore.DatenSpeicher.setArrayEintrag(DatenSpeicher.java:37)
at net.jank.ppos.component.JankButton.actionPerformed(JankButton.java:132)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6038) usw.


und 111 ist genau hier: case 5: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
if (Wert == buttons.get("Schalter2").iArrMoeglWerteInt) //hier
buttons.get("Schalter2").setBackground(Color.MAGENTA);
} break;

Es muss irgend etwas mit buttons.get("Schalter") zu tun haben,
ich hab mir in der Klasse 3 eine HashMap (wie vorgeschlagen) gemacht

public HashMap<String, JankButton> buttons = new HashMap<String, JankButton>();

und im Konstruktor beim Erzeugen des Buttons dann

buttons.put(beschriftung, this);

in die Map hinzugefügt



Das wäre die ganze Klasse:

Vielen Dank für eure Hilfe!!!!

lg

public class JankButton extends JButton implements ActionListener 
    {  
    private ImageIcon buttonIcon;
    private Color bgCol, fgCol;
    private int iArrIndex;
    private String[] iArrMoeglWerte;
    private String[] iArrFarbeZuWerte;
    public int[] iArrMoeglWerteInt = new int[20];
    public HashMap<String, Color> colorMap;
    public HashMap<String, JankButton> buttons = new HashMap<String, JankButton>();
    private DatenSpeicher datenSpeicher = new DatenSpeicher();
    private int cnt = 0;
    private Subject sub;
   
    
    public JankButton(final String beschriftung, String icon, String font, String style, String umrandung, String hintergrundFarbe, String textFarbe,
	              String schriftGroesse, String buttonBreite, String buttonHoehe, String enable, String xPos, String yPos, String textPos, 
		      String textBorder, String durchsichtig)
	{
	buttons.put(beschriftung, this);
	setContentAreaFilled(true);
	setFocusPainted(false);
	setLocation(new Integer(xPos).intValue(), new Integer(yPos).intValue());			
	setFont(new java.awt.Font(font, new Integer(style).intValue(), new Integer(schriftGroesse).intValue()));
	
	if (durchsichtig.equals("durchsichtig"))
	    setOpaque(false);
	else if (durchsichtig.equals("null"))
	    setOpaque(true);
	
	setText(beschriftung);
	
	if (!icon.equals("null"))
	    {
	    buttonIcon = new ImageIcon(System.getProperty("user.dir") + "\\pics\\" + icon);
	    setIcon(buttonIcon);
	    setVerticalTextPosition(AbstractButton.BOTTOM);
	    setHorizontalTextPosition(AbstractButton.CENTER);
	    setSize(buttonIcon.getIconWidth(), buttonIcon.getIconHeight());
	    setIconTextGap(new Integer(textPos).intValue());
	    }
	else 
	    setSize(new Integer(buttonBreite).intValue(), new Integer(buttonHoehe).intValue());	
	
	if (enable.equals("enabled"))
	    setEnabled(true);
	else if (enable.equals("disabled"))	    
	    setEnabled(false);	    
	else 
	    setEnabled(true);
	
	if (umrandung.equals("BeveledBorder"))
	    setBorder(javax.swing.BorderFactory.createBevelBorder(javax.swing.border.BevelBorder.RAISED));
	else if (umrandung.equals("SoftBeveledBorder"))
	    setBorder(new javax.swing.border.SoftBevelBorder(javax.swing.border.BevelBorder.RAISED));
	else if (umrandung.equals("EtchedBorder"))
	    setBorder(javax.swing.BorderFactory.createEtchedBorder());
	else if (umrandung.equals("TitledBorder"))
	    setBorder(javax.swing.BorderFactory.createTitledBorder(textBorder));
	else if (umrandung.equals("LineBorder"))
	    setBorder(javax.swing.BorderFactory.createLineBorder(new java.awt.Color(0, 0, 0)));
	
	colorMap = new HashMap<String,Color>();
	colorMap.put("hellgrau", Color.LIGHT_GRAY);
	colorMap.put("grau", Color.GRAY);
	colorMap.put("gruen", Color.GREEN);
	colorMap.put("rot", Color.RED);
	colorMap.put("blau", Color.BLUE);
	colorMap.put("weiss", Color.WHITE);
	colorMap.put("gelb", Color.YELLOW);
	bgCol = Color.WHITE; 
	fgCol = Color.BLACK;
	if(colorMap.containsKey(hintergrundFarbe))		    	    
	     bgCol = colorMap.get(hintergrundFarbe);	    	    
	setBackground(bgCol);
	
	if(colorMap.containsKey(textFarbe))		    	    
	     fgCol = colorMap.get(textFarbe);	    	    
	setForeground(fgCol); 
	
	addActionListener(this); 
	
	datenSpeicher.attach(new Observer(){  
	   public void update(Subject subject, int index, int Wert,  int[] iArr)
		{
	       System.out.println("Index: " + index + ", Wert: " + Wert);
		switch (index) {		
		    case 3: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Taster1").iArrMoeglWerteInt[i])   
				    buttons.get("Taster1").setBackground(Color.WHITE);			    //colorMap.get(buttons.get("Taster1").iArrFarbeZuWerte[i]));
				} break;
		    case 4: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Schalter1").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter1").setBackground(Color.YELLOW);
				 break;
				}
		    case 5: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Schalter2").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter2").setBackground(Color.MAGENTA);			    //colorMap.get(buttons.get("Schalter2").iArrFarbeZuWerte[i]));
				} break;	
		    default:  				
		    }
		}
	    });
	}
    
    public void setLogic(String iArrIndex, String iArrMoeglWerte1, String iArrFarbeZuWerte1)
	{
	this.iArrIndex = new Integer(iArrIndex).intValue();
	this.iArrMoeglWerte   = iArrMoeglWerte1.split(",");
	this.iArrFarbeZuWerte = iArrFarbeZuWerte1.split(",");
	for (int k = 0; k < iArrMoeglWerte.length-1; k++)
	    this.iArrMoeglWerteInt[k] = new Integer(iArrMoeglWerte[k]).intValue();
	}
    
    public void actionPerformed(ActionEvent event) 
	{
	if (getActionCommand().equals("Taster1")) {
	    datenSpeicher.setArrayEintrag(2,5);    /* Wert, Index */
	    }
	if (getActionCommand().equals("Schalter1")) {
	    System.out.println(buttons.get("Schalter1").getBackground());
	     datenSpeicher.setArrayEintrag(2,5);
	    }
	if (getActionCommand().equals("Schalter2")) {
	    System.out.println(buttons.get("Schalter2").getBackground());
	     datenSpeicher.setArrayEintrag(2,5);
	    }
	}       
    }

 

[bygones]

Es muss irgend etwas mit buttons.get("Schalter") zu tun haben,

noe.. dieses array dingsens da ist auch moeglich

 

[Wildcard]

Schonmal dran gedacht das kein Button 'Schalter2' in der Map liegt?
Wie gesagt, benutz einen Debugger, dann siehst du sehr schnell was falsch läuft.

 

[MQue]

naja das Array habe ich in der 3. Klasse definiert mit:

public int[] iArrMoeglWerteInt = new int[3];

ich habs jetzt ausprobier: wenn ich if(Wert == 2) schreibe dann bekomme ich in der nächsten Zeile eine NPE
ich hab keine Ahnung mehr wieso.

for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert ==  2) //buttons.get("Schalter2").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter2").setBackground(Color.MAGENTA);	 // hier eine NPE		    //colorMap.get(buttons.get("Schalter2").iArrFarbeZuWerte[i]));
				} break;

 

[Lim_Dul]

Wo wird den ein Button mit dem Key Schalter2 eingefügt? Kannst du die Codezeile zeigen?

 

[MQue]

Also Wildcard hat wieder mal recht, wenn ich auf Schalter 2 drücke, dann gibt es Schalter 2 aber Schalter1 und Taster1 nicht, wenn ich Schalter1 drücke, dann gibt es Schalter1 aber die anderen 2 nicht in der HashMap, bzw. ich kann sie in dieser Methode (inneren Klasse) nicht sehen.

Kann mir da noch jemand einen Tipp geben, werden die beiden anderen Buttons nicht in die HashMap hinzugefügt (keine Ahnung wieso) oder sehe ich nur die beiden Buttons nicht in datenSpeicher.attach(...);

Vielen dank für die Hilfe.

lg

datenSpeicher.attach(new Observer(){  
	   public void update(Subject subject, int index, int Wert,  int[] iArr)
		{
	       System.out.println("Index: " + index + ", Wert: " + Wert);
		switch (index) {		
		    case 3: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Taster1").iArrMoeglWerteInt[i])   
				    buttons.get("Taster1").setBackground(Color.WHITE);			    //colorMap.get(buttons.get("Taster1").iArrFarbeZuWerte[i]));
				} break;
		    case 4: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert == buttons.get("Schalter1").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter1").setBackground(Color.YELLOW);
				 break;
				}
		    case 5: for (int i = 0; i < iArrMoeglWerteInt.length-1; i++) {
				if (Wert ==  2) //buttons.get("Schalter2").iArrMoeglWerteInt[i])   
				    buttons.get("Schalter2").setBackground(Color.MAGENTA);//colorMap.get(buttons.get("Schalter2").iArrFarbeZuWerte[i]));
				System.out.println("Taster1: " + buttons.get("Taster1") + ", Schalter1: " + buttons.get("Schalter1") + ", Scahlter2: " + buttons.get("Schalter2") );
				} break;	
		    default:  				
		    }
		}
	    });

public class JankButton extends JButton implements ActionListener 
    {  
    
    public HashMap<String, JankButton> buttons = new HashMap<String, JankButton>();
     
    
    public JankButton(String beschriftung, String icon, String font, String style, String umrandung, String    hintergrundFarbe, String textFarbe,
	              String schriftGroesse, String buttonBreite, String buttonHoehe, String enable, String xPos, String yPos, String textPos, 
		      String textBorder, String durchsichtig)
	{
	buttons.put(beschriftung, this);  //hier wird bei jedem Aufruf der Button in die HashMap hinzugefügt (dachte ich mir)

 

[Wildcard]

Wenn ich eine 3-Zeilige Parameter Liste sehe vergeht mir schon alles.
Du hast doch in den letzten Tagen schon eingies gelernt, was hälst du davon das gelernte umzusetzen, dich in Ruhe hinzusetzen, ein UML-Diagramm zu zeichnen und dir dabei eine sinnvolle Aufgabenverteilung für deine Objekte zu überlegen.
Wenn du mit dem Ergebnis zufrieden bist (nachfragen ist auch erlaubt), dann schreib das Ding von vorne.
Nicht persönlich nehmen, für mich sieht das einfach nach ziemlichem Foobar aus und es ist nur eine Frage der Zeit bis dir deine Architektur unter den Füßen wegbricht.

 

[MQue]

das ist eigentlich das letzte Problem, ich hab sonst alles unter Kontrolle, aufteilung passt, Konzept find ich auch ok, nur eben dieses Problem ist ein bisschen hartnäckig.
Mir ist schon klar, dass ich viel schreib hier, das wird sich aber mit sicherheit sehr bald aufhören, ich bin stark am üben.

Vielen Dank auf jeden Fall, das Forum (bzw die Schreiber) hier sind echt super (vor allem Wildcard!!)

lg
Michl

 

[Roar]

"JankButton.access$000" die klasse wird vom compiler erzeugt

Was ändert das an der angegebenen Zeilennummer?

der code in der von dem compiler automatisch generierten klasse liegt nich als sourcecode in der datei vor, die zeilennummer ist also vermutlich die zeile der class definition.

 

[Wildcard]

der code in der von dem compiler automatisch generierten klasse liegt nich als sourcecode in der datei vor, die zeilennummer ist also vermutlich die zeile der class definition.

Aber die Fehlermeldung verweist auf die .java da vermutlich mit Debug Informationen kompiliert wurde.

 

[MQue]

und habt ihr noch einen Tipp für mich, was kann ich da machen, damit ich die HahMap in der inneren Klasse sehen kann,
wie kann das eigentlich sein, dass ich nur den Button in der inneren Klasse sehe, welcher gedrückt wurde?

Bitte noch einen Hinweis

Vielen Dank!!

lg