Variablen Variable nimmt keinen Wert an (BlueJ)

[Graypaw]

Hallo Leute,
ich stehe vor einem Problem. Nach viel hin und her bin ich zu dem Schluss gekommen, dass ich die Variable "figur2" bzw. die Variable "figur" in eine andere Klasse übertragen muss. Jetzt kommt mir jedoch der Fehler unter, dass diese Variable keinen Wert sendet oder keinen Wert annimmt. Ich verstehe es nicht und bitte hier um Hilfe.
Dies ist die erste Klasse, bei der die Variable anscheinend keinen Wert hat:

import figuren.*;

public class Tisch extends Moebel
{
    private int xPosition;
    private int yPosition;
    private int orientierung;
    private String farbe;
    private boolean istSichtbar;
    private int breite;
    private int tiefe;
    private Object grafikObjekt;
    private Figur figur;
    private Figur TischFigur;
    private Figur figur2;
 
    public Tisch()  {
        xPosition = 100;
        yPosition = 100;
        breite = 120;
        tiefe  = 60;
        orientierung = 0;
        farbe = "rot";
        istSichtbar = false;
        setzeGrundform();
    }
 
    protected Figur setzeGrundform() {
        ZusammengesetzteFigur figur = new ZusammengesetzteFigur();
        figur.fuegeHinzu(xPosition, yPosition, new Rechteck(breite, tiefe), 0);
        this.figur = figur;
        return figur;
    }
 
    public void Figurendefinition(){
        figur2 = setzeGrundform();
    }

    public Figur getVariable() {
        return figur2;
    }
}

Und hier ist nun die Klasse, welche die Variable empfangen und weiter verarbeiten soll:

import figuren.*;

public class Tischgruppe extends Moebel
{
    private StuhlU2 stuhlu2;
    private StuhlO2 stuhlo2;
    private StuhlO stuhlo;
    private StuhlU stuhlu;
    private Tisch tisch;
    private Figur TischFigur2;
    private Figur figur;
 
    public Tischgruppe() {}
 
    public void Figuren() {
        TischFigur2 = tisch.getVariable();
    }
 
    protected Figur setzeGrundform() {
        ZusammengesetzteFigur figur = new ZusammengesetzteFigur();
        figur.fuegeHinzu(0, 0, TischFigur2, 0);
        return figur;
    }

    public void Tischgruppe() {
        Tisch tisch = new Tisch();
        tisch.zeige();
     
        StuhlO2 stuhlo2 = new StuhlO2();
        stuhlo2.zeige();
     
        StuhlO stuhlo = new StuhlO();
        stuhlo.zeige();
     
        StuhlU stuhlu = new StuhlU();
        stuhlu.dreheAuf(180);
        stuhlu.zeige();
     
        StuhlU2 stuhlu2 = new StuhlU2();
        stuhlu2.dreheAuf(180);
        stuhlu2.zeige();
    }
}

Der Fehler hierbei tritt auf, sobald ich versuche mithilfe einen in der dritten Klasse (Möbel) gespeicherten Befehl "zeige" auszuführen, welcher das Objekt in Tischgruppe (zweite Klasse) sichtbar machen soll. Der Fehler lautet wie folgt: "java.lang.NullPointerException: null (in figuren.ZusammengesetzteFigur)". Was der Fehler bedeutet ist mir klar, aber ich kann die Variable nicht mit dem füllen, womit sie gefüllt werden soll...
Hier nochmal der Quelltext zu Möbel:

import figuren.*;

/**
 * Abstract class Moebel - write a description of the class here
 *
 * @author (your name here)
 * @version (version number or date here)
 */
public abstract class Moebel
{
    public int xPosition;
    public int yPosition;
    public int orientierung;
    public String farbe;
    public boolean istSichtbar;
    public int breite;
    public int tiefe;
    public Object grafikObjekt;
    public Figur figur;
  
    protected abstract Figur setzeGrundform();
  
    /**
     * Mache dies Objekt sichtbar.
     * Wenn es bereits sichtbar ist, tue nichts.
     */
    public void zeige() {
        if (!istSichtbar) {
            istSichtbar = true;
            zeichne();
        }
    }
  
    /**
     * Mache dies Objekt unsichtbar.
     * Wenn es bereits unsichtbar ist, tue nichts.
     */
    public void verberge() {
        loesche(); // "tue nichts" wird in loesche() abgefangen.
        istSichtbar = false;
    }
  
    /**
     * Drehe dies Objekt auf den angegebenen Winkel
     */
    public void dreheAuf(int neuerWinkel) {
        loesche();
        orientierung = neuerWinkel;
        zeichne();
    }
  
    /**
     * Aendere die Breite dieses Objektes
     */
    public void neuBreite(int nBreite) {
        loesche();
        breite = nBreite;
        zeichne();
    }
  
    /**
     * Bewege dies Objekt horizontal um 'entfernung' Bildschirmpunkte.
     */
    public void bewegeHorizontal(int entfernung) {
        loesche();
        xPosition = xPosition + entfernung;
        zeichne();
    }
  
    /**
     * Bewege dies Objekt vertikal um 'entfernung' Bildschirmpunkte.
     */
    public void bewegeVertikal(int entfernung) {
        loesche();
        yPosition = yPosition + entfernung;
        zeichne();
    }  
  
    /**
     * Bewege dieses Objekt horizontal und vertikal um eine jeweils eigene "entfernung"
     */
    public void bewegeHorizontalundVertikal(int horizontal, int vertikal) {
        loesche();
        xPosition = xPosition + horizontal;
        yPosition = yPosition + vertikal;
        zeichne();
    }
  
    /**
     * Drehe dies Objekt um den angegebenen Winkel
     */
    public void dreheUm(int addierterWinkel) {
        loesche();
        orientierung += addierterWinkel;
        zeichne();
    }
  
    /**
     * Aendere die Farbe dieses Objektes in 'neueFarbe'.
     * Muster "rot", "gelb" usw.
     */
    public void aendereFarbe(String neueFarbe) {
        loesche();
        farbe = neueFarbe;
        zeichne();
    }
  
    /**
     * Zeichne das Grafikobjekt mit seinen aktuellen Werten auf den Bildschirm.
     */
    protected void zeichne() {
        if (istSichtbar) {
            setzeGrundform();
            Zeichnung zeichnung = Zeichnung.gibZeichnung();
            figur = setzeGrundform();
            grafikObjekt =
            zeichnung.zeichne (// gibt das Objekt zurueck
              figur.gibForm(xPosition, yPosition, orientierung),    // definiert seinen grafischen Aspekt  
              farbe,              // definiert seine Zeichenfarbe
              false);          // keine Fuellung
            zeichnung.warte(10);
        }
    }
  
    /**
     * Loesche das Grafikobjekt vom Bildschirm.
     */
    public void loesche() {
        if (istSichtbar){
            Zeichnung zeichnung = Zeichnung.gibZeichnung();
            zeichnung.loesche(grafikObjekt);
        }
    }
  
}

Bei einer Klasse, die ebenfalls auf die Befehle in Möbel zugreift und eine ganz normale Figur hat tritt der Fehler nicht auf, weshalb ich dachte, dass er nicht in Möbel liegen kann.
Danke im Voraus!

 

[stg]

Poste bitte den gesamten StackTrace der Fehlermeldung und hebe die relevanten Zeilen deines Codes in geeigneter Weise hervor (den Wust überblickt ja kein Mensch...) und/oder bereite ein Minimalbeispiel auf.

 

[Graypaw]

protected Figur setzeGrundform() {
        ZusammengesetzteFigur figur = new ZusammengesetzteFigur();
        figur.fuegeHinzu(xPosition, yPosition, new Rechteck(breite, tiefe), 0);
        this.figur = figur;
        return figur;
    }
 
    public void Figurendefinition(){
        figur2 = setzeGrundform();
    }

     public Figur getVariable() {
         return figur2;
     }
}

public Tischgruppe() {}
 
    public void Figuren() {
        TischFigur2 = tisch.getVariable();
    }
 
    protected Figur setzeGrundform() {
        ZusammengesetzteFigur figur = new ZusammengesetzteFigur();
        figur.fuegeHinzu(0, 0, TischFigur2, 0); //<-- hier scheitert es, weil kein Wert vorhanden ist
        return figur;
    }

Hier der Versuch alles wichtige deutlich zu machen.
Was hierbei passieren soll ist das folgende:
Die Variable "figur2" soll an die zweite Klasse aus der ersten Klasse übergeben werden mithilfe des Konstruktors "getVariable()" (in der ersten Klasse) und der Methode "Figuren()" (in der zweiten Klasse).

//Sei dies nun der Konstruktor, welcher die Variable sendet in der ersten Klasse und sei die Variable figur1 bereits vordefiniert
public Figur figur() {
   return figur1;
}
//Dann wäre dies die Methode in der zweiten Klasse, welche die Variable empfangen soll
public void Methode() {
figur2 = klasse1.figur();
}
//Und genau so funktioniert es schonmal nicht ;)

Und direkt noch eine Frage hinterher: Was ist ein StackTrace einer Fehlermeldung?
Danke weiterhin!

 

[Joose]

Die Variable "figur2" soll an die zweite Klasse aus der ersten Klasse übergeben werden mithilfe des Konstruktors "getVariable()"

Du hast ein Verständnisproblem was ein Konstruktor ist und was eine Methode!
Der Konstruktor dient dazu das Objekt in einen gewissen "Startzustand" zu bringen. Der Konstruktor besitzt keine Rückgabetypen! Alles mit Rückgabetyp ist eine Methode.

getVariable kann daher kein Konstruktor sein, und mit Hilfe eine Konstruktors kann man auch nichts übergeben. Man kann nur an einen Konstruktor Parameter übergeben.

Und direkt noch eine Frage hinterher: Was ist ein StackTrace einer Fehlermeldung?

http://stackoverflow.com/questions/...w-can-i-use-it-to-debug-my-application-errors

Zu deinem Problem: Rufst du irgendwo die Methode Figuren() deiner Klasse Tischgruppe auf? Wenn nicht dann wird der Code auch nie ausgeführt und die Variable nicht initialisiert.

Anmerkungen zu deinem Code:
Nur Klassennamen werden in UpperCamelCase geschrieben, alles andere in lowerCamelCase
Methoden beschreiben eine Tätigkeit, daher ist Tischgruppe oder Figuren oder Figurendefinition eine schlechte Wahl für einen Methodennamen.
Nennen die Methoden daher um in zum Beispiel erstelleTischgruppe, initialisiereFiguren usw.

 

[Graypaw]

Ich danke an all die Antworten. Tatsächlich hat sich mein Problem gelöst, durch eine Überarbeitung des Codes. Ich habe nicht wirklich viel verändert und bin mir auch beim entscheidenden Faktor nicht sicher, aber jetzt geht es immerhin ... 🙂