Hilfe! NullPointerException und Architektur Problem.

[Storyline1981]

Hallo,

derzeit schreibe ich an einem sehr kleinen Tool mit dem es möglich ist parameter in einem Bild zu verändern. Dabei tritt ein Kommunikationsproblem zwischen zwei Klassen auf. Das Programm ist noch nicht fertig und funktioniert nicht richtig. Dabei bitte Nachsicht walten lassen.

Hierbei brauch ich Hilfe von Euch da ich mit meinem "Latein" sprichwörtlich am Ende bin.

Der Chat hier geht leider nicht mehr. Ist es eine Einstellungssache oder kann mir eine ALternative empfohlen werden. Habe ein paar kleine Fragen noch zu einigen anderen Architekturproblemen.

Anbei lege ich einen Link mit dem vollständigen Programmcode und im folgenden kommt auszugsweise nur ein ganz kleiner auszug um die Problematik zu erklären.

Frage1: Architektur
Mein Programm besitzt einen Frame an dem beinahe alle Objekte angemeldet sind. Diese Objekte sind über getter erreichbar. Ist dies überhaupt sinnvoll? Der Frame wird instanziert über eine Klasse mit Main methode.
Jede Objektinstanz übergebe ich den Frame mit

Objekt name = new Objekt(this);

Ist dies überhaupt sinnvoll? Spricht etwas dagegen?



Frage2: NullPointer

public class MainFrame extends JFrame {
	private Slider slider;
	private PanelJTextField panelJTextField;
public MainFrame(){
this.slider = new Slider(this);                              
//liegt es daran das slider an diesem Punkt PanelJText noch nicht kennt? Wie umgehe ich das? Wo lege ich dann 
//klassen besser an wenn ich immer darauf achten muss wer wann wen kennt?
this.panelJTextField = new PanelJTextField(this);
}

	public Slider getSlider() {
		return slider;
	}
	public PanelJTextField getPanelJTextField() {
		return this.panelJTextField;
	}

                public void ForwardJSagA(){
		this.panelJTextField.sagA();
	}
}
---------------
public class PanelJTextField extends JPanel {
	private Slider slider;
private MainFrame frame;
public PanelJTextField(MainFrame frame) {
this.frame=frame;
this.slider=frame.getSlider();
	this.slider.sagA(); //funktioniert
		frame.getSlider().sagB();   //funktioniert
}
	public void sagA(){
		System.out.println("PanelA");
	}
	public void sagB() {
		System.out.println("PanelB");
		
	}
	public void sagC() {
		System.out.println("PanelC");
		
	}

--------
public class Slider {
	private Parameter parameter;

	private JSlider[] jSlider;

	private int sliderTicks;

	private PanelJTextField panelJTextField;
	private MainFrame frame;

	public Slider(MainFrame frame) {
		this.frame=frame;
		this.panelJTextField = frame.getPanelJTextField();
		this.parameter = frame.getParameter();
		this.jSlider = new JSlider[5];
		this.sliderTicks = 5;
	}
	public void sagA(){
		System.out.println("SliderA");
	}
	public void sagB() {
		System.out.println("SliderB");
		
	}
	public void sagC() {
		System.out.println("C");}

	public class ListenerSlide implements ChangeListener {  //innere Klasse

		public void stateChanged(ChangeEvent cE) {

			JSlider source = (JSlider) cE.getSource();

			if (!source.getValueIsAdjusting()) {
				int val = (int) source.getValue();
				String tool = (String) source.getToolTipText();
		
				if (tool.equals("SliderX")) { //funktioniert

					frame.ForwardJSagA(); //funktioniert ist doch genau das selbe wie darunter?
//IN FRAME: this.panelJTextField.sagA(); ist doch äquivalent zu IN SLIDER: frame.getPanelJTextField().sagA();
					panelJTextField.sagB();//NullPointer           <--------------------------- EXCEPTION
					frame.getPanelJTextField().sagC();//NullPointer       <------------------------- EXCEPTION
				}

			}
		}
	}

}
-----

An diesem Punkt komm ich mit meinem Verständnis nicht mehr weiter. Klasse PanelJTextField kann ohne weiteres auf slider zugreifen und dort A und B ausgeben lassen!

VOn Slider aus kann ich nur A von PanelJText field ausgeben lassen. B und C wirft eine Nullpointer Exception und ich weis nicht mehr weiter.

Im Anhang der restliche Code

http://rafb.net/p/MuDu3599.html


Ich hoffe auf Hilfe!

 

[SlaterB]

was ist denn immer so schwer daran, daraus ein fertiges Beispiel zu bauen:

public class MainFrame
    extends JFrame
{
    private Slider slider;
    private PanelJTextField panelJTextField;

    public MainFrame()
    {
        this.slider = new Slider(this);
        this.panelJTextField = new PanelJTextField(this);
    }

    public Slider getSlider()
    {
        return slider;
    }

    public PanelJTextField getPanelJTextField()
    {
        return this.panelJTextField;
    }

    public void ForwardJSagA()
    {
        this.panelJTextField.sagA();
    }

    public static void main(String[] args)
    {
        new MainFrame();
    }
}


class PanelJTextField
    extends JPanel
{
    private Slider slider;
    private MainFrame frame;

    public PanelJTextField(MainFrame frame)
    {
        this.frame = frame;
        this.slider = frame.getSlider();
        this.slider.sagA(); 
        frame.getSlider().sagB(); 
    }

    public void sagA()
    {
        System.out.println("PanelA");
    }

    public void sagB()
    {
        System.out.println("PanelB");

    }

    public void sagC()
    {
        System.out.println("PanelC");
    }
}

class Slider
{

    private JSlider[] jSlider;

    private int sliderTicks;

    private PanelJTextField panelJTextField;
    private MainFrame frame;

    public Slider(MainFrame f)
    {
        this.frame = f;
        this.panelJTextField = f.getPanelJTextField();
        this.jSlider = new JSlider[5];
        this.sliderTicks = 5;

        Runnable r = new Runnable()
            {
                public void run()
                {
                    try
                    {
                        Thread.sleep(1000);
                    }
                    catch (InterruptedException e)
                    {
                        e.printStackTrace();
                    }
                    frame.ForwardJSagA();
                    panelJTextField.sagB();
                    frame.getPanelJTextField().sagC();
                }

            };
        new Thread(r).start();
    }

    public void sagA()
    {
        System.out.println("SliderA");
    }

    public void sagB()
    {
        System.out.println("SliderB");

    }

    public void sagC()
    {
        System.out.println("C");
    }
}

kopieren, starten, Exception,
so leicht kann ein Fehler in Java sein,

aber ne, alles selber mühsam zusammenflicken..

also die Exception bei
panelJTextField.sagB();
ist ja klar, panelJTextField ist noch null zum Zeitpunkt der Speicherung im Slider,
der Befehl danach ist der interessantere,
kann eigentlich keine Exception sein, frame wird korrekt gesetzt und getPanelJTextField() liefert auch des korrekte JPanel,
folgerichtig konnte ich bei mir auch keine Exception feststellen

---------

die Initialisierungsprobleme könntest du teilweise umgehen, indem du leere Konstruktoren verwendest:

this.slider = new Slider();
this.panelJTextField = new PanelJTextField();
this.slider.init(this); // hier panelJTextField bereits gesetzt
this.panelJTextField.init(this);

---------

dass das MainFrame die Objekte sammelt ist gar nicht so schlecht,
man muss mit so einer Architektur natürlich Erfahrung haben oder bei Fehlern wie obigen sammeln,
wenn es dir nicht möglich ist, durch logisches Denken oder zumindest Debuggen herausfinden, welches Objekt wann wo gesetzt ist, dann hast du es natürlich schwer

--------

noch eine kleine Korrektur:
dass die Unterobjekte wie Slider selbstständig das MainFrame nach anderen Objekten abgrasen, ist teilweise elegant, aber auch etwas unübersichtlich,

eine Alternative wäre, dass das MainFrame alle benötigten Objekte übergibt:

this.slider = new Slider();
this.panelJTextField = new PanelJTextField();
this.slider.init(this, this.panelJTextField);
this.panelJTextField.init(this, this.slider);

 

[Guest]

Ich danke Dir vielmals! ... Aye Chef Beim nächsten mal ist das Beispiel vollständig.

Diese Architektur hatte ich vorher verwendet, das ich den ganzen "Frame" + das Objekt übergab. Damals funktionierte es auch. Nur dachte ich das es eher von der Architektur ein Nachteil ist und absolut keinen Unterschied macht wenn nur der Frame übergeben wird und dann per "get" auf das Objekt verweist.

 

[SlaterB]

falls du keinen get-Zugriff erlaubst, bevor das Objekt nicht initialsiert ist, was einigermaßen Sinn macht,
dann kannst du in der get-Operation eine Sicherung einbauen:

public PanelJTextField getPanelJTextField()
{
if ( this.panelJTextField == null) {
throw new RuntimeException("halt, hier stimmt was nicht");
}
return this.panelJTextField;
}

das empfände ich dann auch als relativ sicher für dieses aktuelle Problem,
aber was man nun generell als Architektur toll ist will ich mal lieber doch nicht mit so einfachen Argmumenten entscheiden