Hilfe bei Instanzmethoden

[platonderweise]

Ich habe ein kleines Problem mit folgender aufgabenstellung

Folgende Prozesse beim Start und bei der Landung eines Flugzeugs sollen in der Klasse Flugzeug
mittels Instanzmethoden abgebildet werden:
• Vergabe einer Landebahn
Eine Methode vergebeLandebahn() soll die zu belegende Landebahn übergeben werden. Dieser
Wert soll die Instanzvariable landebahn überschreiben. Außerdem soll der Status des Flugzeugs
auf “Landeanflug” gesetzt werden.
• Vergabe einer Parkstelle
Der Methode vergebeParkstelle() soll die zu belegende Parkstelle übergeben werden. Zusätzlich
soll der Status auf “Gelandet” gesetzt werden.
• Meldung einer erfolgreichen Landung
Einer weiteren Methode meldeGelandet() wird die Istzeit der Landung übergeben, die in istzeitLandung
gespeichert werden soll. Der Status des Flugzeugs wird auf “Geparkt” gesetzt.
• Vergabe einer Startbahn
Die Methode vergebeStartbahn() setzt die zu verwendende Startbahn, die erwartete Sollzeit
für den Start sowie den Status auf “Startvorbereitung”.
• Meldung eines erfolgreichen Starts
Der Methode meldeGestartet() wird die Istzeit des Starts übergeben. Der Status wird auf “Gestartet”
gesetzt.
Als letztes soll der Klasse Flugzeug eine Methode ausgabe() hinzugefügt werden. Diese Methode dient
dazu, den aktuellen Zustand eines Flugzeuges auf dem Bildschirm auszugeben.
Alle Methoden sollen durch den Modifikator public öffentlich aufrufbar sein.

So weit bin ich schon komm aber bei den Instanzmethoden einfach nicht weiter und bräuchte einfach mal einen schubs in die richtige Richtung:

public class Flugzeug {
    private String Fluggesellschaft, Flugzeugtyp, istzeitLandung, istzeitStart, 
           sollzeitLandung, sollzeitStart, landebahn, parkstelle, startbahn;
    public enum State{
        Wartend, Landeanflug, Gelandet, Geparkt, Startvorbereitung, Gestartet
    }
    
    State status1 = State.Wartend;
    State status2 = State.Landeanflug;
    State status3 = State.Gelandet;
    State status4 = State.Geparkt;
    State status5 = State.Startvorbereitung;
    State status6 = State.Gestartet;
           
    private int Flugnummer;
    
    private static int anzahlFlugzeuge = 0;
    
    public Flugzeug (String typ, String gesellschaft, String zeitLandung){
        Flugzeugtyp = typ;
        Fluggesellschaft = gesellschaft;
        sollzeitLandung = zeitLandung;
    }
    
    vergebeLandebahn(){
        
    }
    
    
    }

 

[HimBromBeere]

deine vergieb-Methoden müssen alle etwa so ausehen:

public void set<Name der zu setzenden variablen>(int neuer_wert) {
    this.<Name der zu setzenden Variablen> = neuer_wert;
}

im Bsp. also:

public void setLandebahn(int landebahn) {this.landebahn = landebahn;}
public void setStatus(String status) {this.status = status;}

Da ich gestern schon eine Rüge bekommen habe, dass setter-Methoden keine Seiteneffekte haben dürfen, werde ich dir diesen Tipp mal weiterreichen, d.h. es wird nicht innerhalb des setLandebahn() der Status auf "landen" gesetzt, dass mussst du über eine eigene setter-Methoide machen

 

[hdi]

Guten Abend!

Erstmal der pauschale Newcomer-Anschiss: Variablen schreibt man in lowerCaseCamelCase. Also erster Buchstabe klein, und Teilwörter beginnen jeweils mit großem Buchstaben. Also zB:

Fluggesellschaft -> fluggesellschaft (oder flugGesellschaft)
istzeitLandung -> istZeitLandung

Nun zum Thema:
Die Sache mit den States hast du glaube ich nicht ganz richtig verstanden. Das enum ist korrekt, aber du brauchst nicht 6 Variablen die diese Konstanten einfach unter einem anderen Namen halten.. Sondern nur eine Variable, die den aktuellen Zustand speichert:

State aktuellerStatus; // evtl Initialisieren mit Anfangsstatus, zB Startvorbereitung oder Wartend

So weit bin ich schon komm aber bei den Instanzmethoden einfach nicht weiter und bräuchte einfach mal einen schubs in die richtige Richtung:

Was hier gefordert wird sind sogenannte "Setter". Das sind einfach Methoden um eine Instanz-Variable auf einen bestimmten wert zu setzen. Normalerweise folgen sie dem Muster:

setX(Y x){
   this.x = x;
}

Wobei das jetzt der Setter wäre für eine Instanz-Variable:

private Y x;

In deinem Fall heißen die Methoden halt nicht setXX, sondern haben einen ganz anderen Namen (nicht so tolles Design, aber anscheinend dooferweise so vorgegeben). Am Rest ändert sich aber nix. Die Methoden brauchen einen Parameter, und zwar von dem Typ, von dem die Variable ist der sie diesen Wert zuweisen sollen. zB:

vergebeLandebahn(String landebahn){
     this.landebahn = landebahn;
}

Die anderen Methoden analog.

 

[hdi]

Da ich gestern schon eine Rüge bekommen habe, dass setter-Methoden keine Seiteneffekte haben dürfen, werde ich dir diesen Tipp mal weiterreichen, d.h. es wird nicht innerhalb des setLandebahn() der Status auf "landen" gesetzt, dass mussst du über eine eigene setter-Methoide machen

Freut mich, dass du dir das gemerkt hast :toll: Allerdings wird genau dieser Seiteneffekt anscheinend explizit vom Aufgabensteller gefordert. Hab ich jetzt im Bsp auch vergessen noch als zweite Anweisung reinzunehmen. Aber es stimmt: Das ist eigentlich nicht so gut. Aber dass der Aufgabensteller hier selbst wohl nicht grad ein Java-Experte ist sieht man ja schon anhand der Namen, die für die Setter festgelegt wurden..

edit: @TO also um das nochmal klar zu machen - die Methode müsste so aussehen:

public void setLandebahn(String landebahn){
     this.landebahn = landebahn;
     this.aktuellerStatus = State.Landeanflug;
}

Analog die anderen. Also Wert setzen und Status ändern.

 

[platonderweise]

Hab gerade gesehen das eine ausführliche Antwort schon vorlag. Herzlichen Dank

 

[platonderweise]

Wie initialisiere ich einen Anfagnsstatus ??
Doofe Frage ich zieh sie wieder zurück. Sorry

 

[HimBromBeere]

vorzugsweise mit einem Standardwert, also noch über dem Konstruktor mit sowas:

public class MyClass {
    private State state = State.<default_status>;
...
}

EDIT: Gibt keine doofen Fragen, aber schön, dass du´s bereits rausbekommen hast...

 

[platonderweise]

Jetzt häng gleich nochmal bei der nächsten Teilaufgabe die da lautet:

Als letztes soll der Klasse Flugzeug eine Methode ausgabe() hinzugefügt werden. Diese Methode dient
dazu, den aktuellen Zustand eines Flugzeuges auf dem Bildschirm auszugeben.
Alle Methoden sollen durch den Modifikator public öffentlich aufrufbar sein.

Mein Code sieht bisher folgendermaßen aus

public class Flugzeug {
    private String Fluggesellschaft, Flugzeugtyp, istzeitLandung, istzeitStart, 
           sollzeitLandung, sollzeitStart, parkstelle, startbahn;
    private int landebahn;
    private State aktuellerStatus = State.Wartend;
    
    public enum State{
        Wartend, Landeanflug, Gelandet, Geparkt, Startvorbereitung, Gestartet
    }
    
    State status1 = State.Wartend;
    State status2 = State.Landeanflug;
    State status3 = State.Gelandet;
    State status4 = State.Geparkt;
    State status5 = State.Startvorbereitung;
    State status6 = State.Gestartet;
           
    private int Flugnummer;
    
    private static int anzahlFlugzeuge = 0;
    
    public Flugzeug (String typ, String gesellschaft, String zeitLandung){
        Flugzeugtyp = typ;
        Fluggesellschaft = gesellschaft;
        sollzeitLandung = zeitLandung;
    }
    
    public void vergebeLandebahn (int landebahn){
        this.landebahn = landebahn;
        this.aktuellerStatus = State.Landeanflug;
    }   
    
    public void vergebeParkstelle (String parkstelle){
        this.parkstelle = parkstelle;
        this.aktuellerStatus = State.Gelandet;
    }
    
    public void meldeGelandet (String istzeitLandung){
        this.istzeitLandung = istzeitLandung;
        this.aktuellerStatus = State.Geparkt;
    }
    
    public void vergebeStartbahn (String startbahn, String sollzeitStart){
        this.startbahn = startbahn;
        this.sollzeitStart = sollzeitStart;
        this.aktuellerStatus = State.Startvorbereitung;
    }
    
    public void meldeGestartet (String istzeitStart){
        this.istzeitStart = istzeitStart;
        this.aktuellerStatus = State.Gestartet;
    }
    
    public static void main (String [] args){
        
        public void ausgabe(){
            
        }
    }

 

[hdi]

Du kannst keine Methoden in anderen Methoden definineren. Die Methode ausgabe() muss also aus der main-Methode raus, und auf die Klassenebene. So, wie auch alle anderen Methoden. Und was die Logik angeht: Nun, wenn nichts genaueres verlangt wird, dann kannst du jetzt wohl selbst entscheiden was du alles zum Zustand des Flugzeugs zählst. Ich würd sagen auf jeden Fall dein state, also ob Wartend oder im Landeanflug usw, und von den restlichen Instanz-Variablen kannst du dir ja die wichtigsten raussuchen. Oder du gibst einfach alle Infos aus die es gibt.

 

[HimBromBeere]

Übrigens: verwende mal

Tags für deinen Code, da werden dann Schlüsselwörter auch noch hervorgehoben (in der Werkzeugleiste das Symbol zwischen oho und xml).

Deine Methode zum Ausgeben des Status sollte etwa so aussehen:

public <name der enum, deren Name ich grade vergessen hab> getState() {return this.state;}

Diese Funktion rufst du in deinem Hauptprogramm innerhalb eines System.out.println() auf, und schon haste deine Ausgabe

 

[platonderweise]

Jep, das hab ich selber gerade gemerkt
Aber wie gebe ich jetzt z.B. den aktuellen Status über die methode ausgabe wieder zurück?
Habs mit System.out.prinln versucht aber das möchte er ganz und gar nicht!!

 

[hdi]

Zurückgeben sollst du nix. Die Methode ist void. Sie gibt direkt etwas aus - und System.out.println() ist schon richtig. Zeig mal den Code.

 

[HimBromBeere]

Dann musst du den Wert deiner Enumeration irgendwie in einen String umbauen, also entweder über einen Switch(state), oder.... tja oder...?

 

[platonderweise]

 public static void ausgabe(){
        System.out.println(aktuellerStatus);

 

[hdi]

@HimBromBeere enums werden bei Konkatentation und/oder dem toString()-Aufruf automatisch zu einem String interpretiert, dessen Name genau der Name der Konstante ist.

@TO Du musst die Methode noch schließen mit einem }. Achso und ich seh grad: Die soll NICHT static sein, wie bist du darauf gekommen das einzufügen?

 

[platonderweise]

ich hab da ja noch einen Konstruktor mit Übergabeparametern

public Flugzeug (String typ, String gesellschaft, String zeitLandung){
        Flugzeugtyp = typ;
        Fluggesellschaft = gesellschaft;
        sollzeitLandung = zeitLandung;
    }

Ist das vielleicht des Rätsels Lösung?

 

[HimBromBeere]

Bin mir grad nicht sicher, ob man toString() in einer enum überschreiben kann, wahrscheinlich nicht. D.h. du musst dich um das Umwandeln von Status in Text irgendwie noch selbst kümmern...

EDIT: Oha, hat sich erledigt, danke HDI

 

[hdi]

Nein, das hat damit gar nix zu tun. Siehe mein edit im letzten Post.

 

[hdi]

Bin mir grad nicht sicher, ob man toString() in einer enum überschreiben kann, wahrscheinlich nicht. D.h. du musst dich um das Umwandeln von Status in Text irgendwie noch selbst kümmern...

Huhu, noch wach? Lies doch mal was ich geschrieben hab. Und überschreiben kannste die genauso wie auch bei ner Klasse.

edit: ja ja ok wir schreiben hier ständig gleichzeitig ^^

 

[platonderweise]

Erst einmal vielen vielen Dank für die krass gute Hilfe.
Eine Frage hätte ich allerdings noch.

Ich soll nun eine weiter Klasse anlegen um die Daten der ersten Klasse weiterzuverarbeiten.
Wie greif ich auf die andere Klasse zurück. Mit Befehl import ???

 

[hdi]

import brauchst du nur wenn die Klasse nicht im selben Package liegt. Zur Weiterverarbeitung der Daten musst du halt ein Objekt der einen Klasse in die andere Klasse reinreichen.

 

[platonderweise]

Als praktisches Beispiel. Wie gebe ich jetzt zum Beispiel den Satus in der neuen Klasse aus??

 

[platonderweise]

So weit bin ich schon

public class Kunde {
    private Flugzeug flugzeug;
    public static void main (String [] args){
        
      
    }
    
}

 

[HimBromBeere]

du musst in der Methode eine Referenz auf die Instanz der Klasse übergeben. Möchtest du z.B. bei der Erstellung eines Kunden festlegen, in welches Flugzeug er gehört, schreibst du das so:

public Kunde(Flugzeug f) {this.flugzeug = f;}

Aufruf sieht dann so aus:

Kunde kunde = new Kunde(flugzeug);

EDIT: hab gerade bemerkt, dass du ja eine stativ main() im Kunden hast, also ist das deine Eintrittsklasse. Dann kannst/ und brauchst du natrülich gar kein neues Objekt vom Typ Kunde erstellen (jdenfalls nicht ganz so einfach).

 

[platonderweise]

Das ist mir für diese Uhrzeit einfach zu viel
Ich danke Euch für die Hilfe :toll: