Anfängerproblem Syntax if Schleife

[superping]

Nabend zusammen!

Hab was total peinliches am Start. Ich weiss nicht wie ich per if Abfrage rausbekommen kann, ob ein Panel connected ist. Ich bekomms einfach nicht hin.

Habe in einer Klasse:

public void setJPanelConnection(boolean x)
{
		tNPanel.setVisible(x);
}

...und greife aus einer anderen Klasse mit

tnDialog.add(tnPanel.getJPanelConnection());
tnPanel.setJPanelConnection(true);

drauf zu. Funktioniert auch.

Jetzt bräuchte ich eine if Abfrage die in etwa so aussieht:

if (tnPanel.setJPanelConnection() == true) {//mach irgendwas}

Leider bekomm ich´s einfach nicht hin, dass "richtig" zu coden. Compiler meckert bei o.g. Besipiel.

Bin für jede Hilfe dankbar!

 

[javimka]

Für eine Abfrage bräuchtest du ja eine get-Methode oder für ein boolean is-Methode. Wenn ich das richtig verstanden habe, müsste diese so aussehen:

public void isJPanelConnection()
{
        return tNPanel.isVisible();
}

und die if-Abfrage sieht dann so aus:

if (tnPanel.isJPanelConnection() == true) {
   ...
}

 

[superping]

*Verzweifel* Funktioniert leider auch nicht.

public void isJPanelConnection()
{
        return tNPanel.isVisible();
}

Compiler sagt: "cannot return a value from method whose result type is void"

 

[javimka]

ups, sorry, es muss natürlich boolean statt void heissen.

public boolean isJPanelConnection()
{
        return tNPanel.isVisible();
}

 

[superping]

Danke erstmal für Deine Hilfe!

Mit boolean klappts. Allerdings wirft mir der Compiler eine Null-Pointer-Exception wenn ich das in eine if Schleife packe.

if (tnPanel.isJPanelConnection() != true)
               {
               tnDialog.add(tnPanel.getJPanelConnection());
               tnPanel.setJPanelConnection(true);
               tnPanel.repaint();
               tnPanel.revalidate();
               tnPanel.setVisible(true);
               }

Noch irgendeine Idee?

 

[TobiTobsen]

Wo initialisierst du das Panel? Vor der schleife?

 

[Der Müde Joe]

>Noch irgendeine Idee?

Ja. tnPanel oder tnDialog ist nich initialisiert, dh: null. Wenn du nun auf null eine Methode ausführen willst --> Peng...NPE
(Und es motzt nicht der Compiler, sondern das geschieht zur Laufzeit ;-)

EDIT:
ach ja und:
if-schleife.de

 

[superping]

Also in "Klasse 1" erzeuge ich es.

"Klasse 1"

public void setJPanelConnection(boolean x)
{
	tNPanel.setVisible(x);
}

public boolean isJPanelConnection()
{
        return tNPanel.isVisible();
}

public static JPanel tNPanel = null;

public JPanel getJPanelConnection() {
        if(tNPanel == null) {...

und in "Klasse 2" initialisier ich (glaub ich) das.

"Klasse 2"

final Klasse1 tnPanel;

public Klasse2 {
...
tnPanel = new Klasse1();
...
if (tnPanel.isJPanelConnection() != true)
               {
               tnDialog.add(tnPanel.getJPanelConnection());
               tnPanel.setJPanelConnection(true);
               tnPanel.repaint();
               tnPanel.revalidate();
               tnPanel.setVisible(true);
               }
...
}

 

[superping]

@der müde joe

*lach kaputt*

if-schleife.de ist ein echt cooles Tutorial!!! Bringt die Sachen sofort auf den Punkt!

 

[superping]

...aber ist ist doch eigentlich beides initialisiert...

 

[Der Müde Joe]

Schwer zu beurteilen, woher die NPE kommt ohne Exception. Poste die mal. Dann weisst du ja woher sie kommt. Aus Klasse 1 oder 2.

 

[superping]

Na ja. So wie ich das sehe aus beiden:

TourNeu ist Klasse1
und
Main die Klasse2

Hier die Exception:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at connect.TourNeu.isJPanelConnection(TourNeu.java:160)
at connect.Main$15.actionPerformed(Main.java:2138)
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.AbstractButton.doClick(AbstractButton.java:357)
at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1225)
at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1266)
at java.awt.Component.processMouseEvent(Component.java:6134)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5899)
at java.awt.Container.processEvent(Container.java:2023)
at java.awt.Component.dispatchEventImpl(Component.java:4501)
at java.awt.Container.dispatchEventImpl(Container.java:2081)
at java.awt.Component.dispatchEvent(Component.java:4331)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4301)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3965)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3895)
at java.awt.Container.dispatchEventImpl(Container.java:2067)
at java.awt.Window.dispatchEventImpl(Window.java:2458)
at java.awt.Component.dispatchEvent(Component.java:4331)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

 

[Der Müde Joe]

hier fliegt sie:
connect.TourNeu.isJPanelConnection(TourNeu.java:160)

ausgelöst durch die ActionPerformed:
connect.Main$15.actionPerformed(Main.java:2138)

-->
tnPanel ist in isJPanelConnection() null.
Muss also noch initialisiert werden.
zb:

public static JPanel tNPanel = new JPanel();
anstatt:
public static JPanel tNPanel = null;

 

[superping]

Danke! Klappt jetzt schonmal... Jetzt hat sich da ein neues (wahrscheinlich) repaint Problemchen eingestellt.

Das aber in einem anderen Thread.

Danke euch!!!