Problem bei Übungsbeispiel

[homecomin]

Ich wollte ein kleines beispiel Programm meines Lehrers nach programmieren jedoch schaff ich es nicht. Also ganz kurz zu der Erklärung es soll auf einer Oberfläche eine zahl eingetragen werden die in der Arbeitsklasse quadriert wird und dann von 2 anderen klassen ausgegeben wird. Jetzt Sitz ich an der ersten Klasse und versuch dort den Wert aus meiner Arbeitsklasse in eine int variable zu packen damit ich den Wert ausgeben kann. Aber Eclipse möchte das ich die Methode in der Arbeitsklasse auf static setz. Gut möglich das es ein sehr einfacher Anfänger Fehler ist was ich dort versuche. Hier noch der Ausschnitt.

public class Arbeitsklasse {

    GUI_Controller GUI = new GUI_Controller();

    int dieZahl = GUI.lese();

    public int rechne() {
        int ergebnis = dieZahl * dieZahl;
        return ergebnis;

    }

public class GuterSchueler {

    int Antwort = Arbeitsklasse.rechne();
   
    public void Denke() {
        System.out.println("Ganz einfach");
    }

    public void GebeAntwort() {

        System.out.println( Antwort );
       
    }

}

Die Zeile int Antwort = Arbeitsklasse.rechne(); macht mir hier die Probleme.

Ich bedanke mich schon einmal für alle Antworten.

 

[Meeresgott]

Hallo homecomin,

es ist verwunderlich, dass Eclipse dir Arbeitsklasse.rechne() nicht als Fehler anzeigt.
Du musst (wenn du die Methoden nicht als static deklarierst) erst ein Objekt der Klasse erzeugen.
und auf der Referenz des Objektes dann die Methode aufrufen.

ArbeiterKlasse arbeit = new ArbeiterKlasse();

int antwort = arbeit.rechne();

Schau mal hier da geht es auch um ein ähnliches Problem

lg

 

[Kababär]

So wie du es jetzt stehen hast, sollte Arbeitsklasse static sein. Ist eine Klasse static, so greifst du über den Klassennamen (hier: Arbeitsklasse) auf die Methoden und Variablen zu. Allerdings müssen diese dann auch static sein, sonst stimmt der Kontext nicht mehr.
Im Gegensatz dazu kann man auf andere Klassen auch zugreifen, indem man ein Objekt dieser Klasse erstellt. Dies ist sinnvoll, wenn man verschiedene Objekte einer Klasse erstellen will, die alle ihre eigenen Attribute haben.

public class GuterSchueler{

int Antwort = new Arbeitsklasse().rechne();

public void Denke(){
System.out.println("Ganz einfach");
}
...

}

Damit dies auch weiterhin funktionert, sollte

GUI_Controller GUI = new GUI_Controller();

    int dieZahl = GUI.lese();

im Konstruktor der Arbeitsklasse stehen, denn sonst bekommst du eine Nullpointer-Exception.

@ Meeresgott: Eclipse und Netbeans tendieren dazu, dass in diesem Kontext der User Recht behält und in dem Fall wohl vom Nutzer das static vergessen wurde.


edit: Wo kommen denn die Variablen dieZahl und dieseZahl her? Aus dem GUI Objekt oder sind das Klassenattribute der Arbeitsklasse?

 

[Kababär]

Schon gut, habe jetzt erst gesehen, dass dieZahl über GUI.lese() zugewiesen wird.

 

[Meeresgott]

Ich kenne das nur von Netbeans und da wird dann der Fehler angezeigt wo man versucht eine nicht statische Methode aufzurufen.

Wäre es nicht sinnvoller die Variable "dieZahl" als Instanzvariable zu behalten damit die Methode rechne () drauf zugreifen kann?

Oder gibt es da einen besseren Weg?

 

[Kababär]

Ich denke nicht, dass TE beabsichtig, dass für jede Arbeitsklasse das selbe Ergebnis rauskommen soll.
Da er von der GUI einliest und der Benutzer wohl mehrere Eingaben machen kann, ist es schon sinnvoll, mehrere Objekte des Typs Arbeitsklasse zu erstellen, die dann den Input der GUI einlesen.
So kann mich sich beispielsweise auch merken, was der Benutzer so alles eingegeben hat.

@TE statt int Ergebnis = ...
Return Ergebnis
Würde ich lieber gleich
Return dieZahl*dieZahl; schreiben.

 

[Meeresgott]

Hmm ich habe eigentlich an sowie gedacht:

public class Arbeiterklasse
{
     Gui_Controller Gui = new Gui_Controller();

     private int dieZahl;

    public Arbeiterklasse()
    {
        Gui.lese ()
    }

    //weitere Code
}

Oder meinst du dasselbe ?

 

[homecomin]

Zuerst danke für die Antworten aber das Problem besteht weiterhin. Ich kann in der Klasse GuterSchueler kein Objekt der Arbeitsklasse erzeugen da es sonst beim Ausführen zu einem StackOverFlow kommt, weil ich hab bereits ein Objekt von GuterSchueler im GUI_Controller hab. Aber die Methode rechen statisch zu machen kann ich auch nicht weil Eclipse sonst will das alles statisch ist und dann kommt es zu einem NullPointerException beim auslesen der Zahl aus dem Textfeld.

Danke für eure Geduld mit so einem Anfänger.
Gruß, homecomin.

public class GUI_Controller {

    GuterSchueler GS = new GuterSchueler();

    @FXML
    private TextField fxTFzahl;

    @FXML
    private Button fxBschlechterschueler;

    @FXML
    private Button fxBguterschueler;

    @FXML
    void fxBguterschueler(ActionEvent event) {
        GS.Denke();
        GS.GebeAntwort();
    }

    @FXML
    void fxBschlechterschueler(ActionEvent event) {

    }

    public int lese() {
        int Zahl = Integer.parseInt(fxTFzahl.getText());
        return Zahl;
    }

}

 

[Meeresgott]

Ich habe dein Problem gefunden:

Du erzeugst in GUI_Controller ein GuterSchueler-Objekt
Im GuterSchueler-Objekt erzeugst du ein Arbeitsklassen-Objekt
und zu guter letzt erzeugst du im Arbeitsklassen-Objekt ein GUI_Controller.

Da es sich um einen Kreis handelt hört das nie auf bis die Exception geschmissen wird.

Um deinem Problem zu entgehen habe ich deine Klassen und Methoden leicht verändert. Ich hoffe du verstehst alles

public class Arbeitsklasse
{
    public int rechne(int dieZahl) //Liefert die Zahl dierekt zurück
    {
        int ergebnis = dieZahl * dieZahl;
        return ergebnis;
    }
}

public class GuterSchueler
{
    private int dieZahl; //Namen von Variabeln sollten immer mit kleinem Buchstaben anfangen.
    Arbeitsklasse arbeit;
    public GuterSchueler()
    {
        arbeit = new Arbeitsklasse();
    }
 
    public void denke(int dieZahl) //denke() nehme ich als input, später wirst du die Methode setDieZahl() nennen.
    {
        System.out.println("Ganz Einfach");
     
        this.dieZahl = dieZahl;
    }

    public void gebeAntwort() //Methoden Namen sollten auch immer mit einem kleinen Buchstaben anfangen.
    {
        System.out.println(arbeit.rechne(dieZahl));
    }
}

public class GUI_Controller
{
    private GuterSchueler gs;

    public GUI_Controller()
    {
        gs = new GuterSchueler();
    }
 
    public void fxBguterschueler() //Simuliert den Button
    {
        int zahl = lese(); //Die Zahl einlesen
     
        gs.denke(zahl);    //Die Zahl dem Schueler übergeben (Ihn nach denken lassen)
        gs.gebeAntwort();  //Das Ergebnis preisgeben
    }
 
    public int lese()
    {
        return 20; //Einfach um keinen Gui Code schreiben zu müssen.
    }

}

Und zum testen der Klassen habe ich folgende Klasse geschrieben:

public class Main
{
    public static void main(String[] args)
    {
        GUI_Controller gui = new GUI_Controller(); //Die "GUI" erstellen
     
        gui.fxBguterschueler(); //Den Button-Press simulieren
    }
}

Ich denke jetzt dürfte es für dich kein Problem mehr sein dein Programm richtig zu schreiben.
Hoffe mal ich habe dir nicht zu viel vorweg genommen.

lg,
Meeresgott

 

[homecomin]

Vielen Dank an alle und mit der Vorlage wird es auf jeden fall funktionieren.