UML Klassen Diagramm zu Java Code Programmieren und ausführen

[Black Kong]

Hallo,
und zwar sollte Ich ein UML KLassen Diagramm in ein Code umformatieren. Indem es um Aufträge geht ( Um die Auftragsnummer, dem Besteller und ob diese Ausgeführt worden ist oder noch nicht).
Leider lässt es sich nicht ausführen wie gewünscht.

Bin für jede Hilfe dankbar !

package auftrag;

public class Auftrag {
private int auftragsnummer;
private boolean auftragFertiggestellt;
private String auftraggeber;

public Auftrag (int nummer,boolean fertigung,String name){
    auftragsnummer=nummer;
    auftragFertiggestellt=fertigung;
    auftraggeber=name;
    
}
public void setAuftragsnummer (int nummer){
    this.auftragsnummer=nummer;
    
}
public int getAuftragsnummer(){
    return auftragsnummer;
}
public void setAuftragFertiggestellt (boolean fertigung){
    this.auftragFertiggestellt=fertigung;
    boolean s =true;
    if(s){
        return fertigung= true;
    else{
        return fertigung=false;
    }
}

public boolean getAuftragFertiggestellt(){
    return auftragFertiggestellt;
}
public void setAuftraggeber (String name){
    this.auftraggeber=name;
}
public String getAuftraggeber(){
    return auftraggeber;
}
    public static void main(String[] args) {
      Auftrag erster = new Auftrag(2321;true,"Karl");
     System.out.println(erster);



    }
    
}

 

[mihe7]

Leider lässt es sich nicht ausführen wie gewünscht.

Was ist denn gewünscht?

 

[Robat]

public void setAuftragFertiggestellt (boolean fertigung){ 
    this.auftragFertiggestellt=fertigung; 
     boolean s =true; 
     if(s){ 
         return fertigung= true; 
     else{ 
         return fertigung=false; 
     }
 }

void und return passt in dem Fall nicht zusammen. Außerdem stimmen die geschweiften Klammern noch nicht. Denk da noch mal drüber nach

Auftrag erster = new Auftrag(2321;true,"Karl");

Hier vielleicht auch noch mal auf die Semikolons schauen

 

[Black Kong]

Was ist denn gewünscht?

Hallo,
Ich versuche eine Auftragsnummer,Auftragsgeber sowie ob die Bestellung fertiggestellt ist oder nicht auszugeben.
Mein Lehrer hat uns für diese Aufgabe halt wie oben schon erwähnt ein UML Klassendiagramm vorgegeben.

 

[Black Kong]

public void setAuftragFertiggestellt (boolean fertigung){
this.auftragFertiggestellt=fertigung;
boolean s =true;
if(s){
return fertigung= true;
else{
return fertigung=false;
}
}

void und return passt in dem Fall nicht zusammen. Außerdem stimmen die geschweiften Klammern noch nicht. Denk da noch mal drüber nach

Hier vielleicht auch noch mal auf die Semikolons schauen

Hallo,
hab den Code etwas berichtigt nach meinen Möglichkeiten. Leider bringt es immer noch nicht das Gewünschte Ergebnis.

public void setAuftragFertiggestellt (boolean fertigung){
    this.auftragFertiggestellt=fertigung;
    boolean s =true;
    if(s){
        System.out.println("Auftrag Fertig");
    else {
        System.out.println("Auftrag in Bearbeitung");
    }
}
}

 

[mihe7]

    public static void main(String[] args) {
      Auftrag erster = new Auftrag(2321, true, "Karl");
      System.out.println(erster.getAuftragsnummer());
      System.out.println(erster.getAuftraggeber());
      System.out.println(erster.getAuftragFertiggestellt());
    }

 

[mrBrown]

Hallo,
hab den Code etwas berichtigt nach meinen Möglichkeiten. Leider bringt es immer noch nicht das Gewünschte Ergebnis.

public void setAuftragFertiggestellt (boolean fertigung){
    this.auftragFertiggestellt=fertigung;
    boolean s =true;
    if(s){
        System.out.println("Auftrag Fertig");
    else {
        System.out.println("Auftrag in Bearbeitung");
    }
}
}

Das liegt daran, dass die Klammern immer noch falsch sind - guck noch mal genauer hin, wann du welchen Block öffnest und schließt

 

[Black Kong]

Das liegt daran, dass die Klammern immer noch falsch sind - guck noch mal genauer hin, wann du welchen Block öffnest und schließt

public void setAuftragFertiggestellt (boolean fertigung){
    this.auftragFertiggestellt=fertigung;
    boolean s =true;
    if(s){
        System.out.println("Auftrag Fertig");
    }
    else {
        System.out.println("Auftrag in Bearbeitung");
    }
}

Jetzt aber

    public static void main(String[] args) {
      Auftrag erster = new Auftrag(2321, true, "Karl");
      System.out.println(erster.getAuftragsnummer());
      System.out.println(erster.getAuftraggeber());
      System.out.println(erster.getAuftragFertiggestellt());
    }

Danke für die Antwort jetzt wird schonmal die Auftragsnummer sowie der Auftraggeber ausgegeben. Ob der Auftrag ausgeführt würde oder noch nicht erscheint jetzt "true".

 

[mihe7]

Ob der Auftrag ausgeführt würde oder noch nicht erscheint jetzt "true".

Ja, das kannst Du z. B. mit einem if auswerten.

 

[Black Kong]

Ja, das kannst Du z. B. mit einem if auswerten.

Habe ja eine if-Abfrage oben beim Setter initialisiert wird diese nicht beachtet wenn Ich die Sachen ausgebe ?

 

[Robat]

Im Setter wird immer nur was ausgegeben, wenn du den Setter auch aufrufst. Der Setter sollte auch nicht für Ausgaben zuständig sein. Er soll - wie der Name sagt - den boolean setzen.
Außerdem gibt dein Setter gerade *immer* "Auftrag fertiggestellt" aus

 

[Black Kong]

Im Setter wird immer nur was ausgegeben, wenn du den Setter auch aufrufst. Der Setter sollte auch nicht für Ausgaben zuständig sein. Er soll - wie der Name sagt - den boolean setzen.
Außerdem gibt dein Setter gerade *immer* "Auftrag fertiggestellt" aus

Hab mir ein paar Gedanken gemacht&ausprobiert,bräuchte aber einen kleinen Gedanken Anstoß wo ich jetzt meine if-Abfrage starten solle und mit was ich Sie verknüpfen solle.Da meine bisherigen Versuche zum scheitern verurteilt waren.

 

[Robat]

In der main Methode rufst du einfach den Getter auf. Wenn dieser true zurückgibt, gibst du "fertig.." aus ansonsten gibst du "nicht fertig.." aus.

 

[Black Kong]

In der main Methode rufst du einfach den Getter auf. Wenn dieser true zurückgibt, gibst du "fertig.." aus ansonsten gibst du "nicht fertig.." aus.

Das war sogar mein Erster Gedanke ,

 if(get.AuftragFertiggestellt=true){
          System.out.println("Auftrag Fertig");
      }
      else{
          System.out.println("In Bearbeitung");
      }

Hab es dann so ausprobiert was leider nicht funktioniert hat.

 

[Robat]

So ruft man auch keine Methoden auf. Außerdem vergleicht man primitive Datentypen mit ==. Nur ein = entspricht einer Zuweisung.
Zumal der Rückgabewert der Methode selbst bereits ein boolscher Ausdruck ist, daher musst du nicht noch mal auf true/false prüfen.

 

[mihe7]

@Black Kong wo kommt denn das Objekt "get" her? Bisher hatten wir nur "erster".

 

[Black Kong]

So ruft man auch keine Methoden auf. Außerdem vergleicht man primitive Datentypen mit ==. Nur ein = entspricht einer Zuweisung.
Zumal der Rückgabewert der Methode selbst bereits ein boolscher Ausdruck ist, daher musst du nicht noch mal auf true/false prüfen.

Danke hat geklappt bin mir nicht ganz sicher ob ich deine Antworten umgesetzt habe aber es wird ausgegeben.

@Black Kong wo kommt denn das Objekt "get" her? Bisher hatten wir nur "erster".

Ist mir auch gefallen habe es korrigiert

package auftrag;

public class Auftrag {
private int auftragsnummer;
private boolean auftragFertiggestellt;
private String auftraggeber;

public Auftrag (int nummer,boolean fertigung,String name){
    auftragsnummer=nummer;
    auftragFertiggestellt=fertigung;
    auftraggeber=name;
    
}
public void setAuftragsnummer (int nummer){
    this.auftragsnummer=nummer;
    
}
public int getAuftragsnummer(){
    return auftragsnummer;
}
public void setAuftragFertiggestellt (boolean fertigung){
    this.auftragFertiggestellt=fertigung;
   
}

public boolean getAuftragFertiggestellt(){
    return auftragFertiggestellt;
}
public void setAuftraggeber (String name){
    this.auftraggeber=name;
}
public String getAuftraggeber(){
    return auftraggeber;
}
    public static void main(String[] args) {
      
      Auftrag erster = new Auftrag(2321,true,"Heinrich");
     
        
      System.out.println(erster.getAuftragsnummer());
      System.out.println(erster.getAuftraggeber());
      System.out.println(erster.getAuftragFertiggestellt());
    boolean auftragFertiggestellt1 = erster.auftragFertiggestellt;
      if(erster.auftragFertiggestellt==true){
          System.out.println("Auftrag ist fertig");
      }
      else{
          System.out.println("In Bearbeitung");
      }
     
      
    
        
    }


    }

Vielen Dank nochmal an alle die mir behilflich bei der Problemlösung waren. Ich wünsche euch noch einen angenehmen Abend soweit es das Wetter zulässt.

 

[Robat]

Das funktioniert so zwar allerdings solltest du
a) schon die Methoden nutzen (momentan greifst du auf den Member auftragFertiggestellt direkt zu
b) wie oben beschrieben boolsche Werte nicht noch mal auf true/false prüfen

if(erster.getAuftragFertiggestellt()) {
    System.out.println("Auftrag ist fertig");
} else {
    System.out.println("In Bearbeitung");
}

 

[mihe7]

1. Die Variable auftragFertiggestellt1 wird nirgends benutzt.
2. Der Zugriff über erster.auftragFertiggestellt funktioniert hier nur "zufällig". Du solltest hier den Methodenaufruf verwenden.
3. == true ist überflüssig.

if (erster.getAuftragFertiggestellt()) {
    System.out.println("Auftrag ist fertig");
} else {
    System.out.println("In Bearbeitung");
}

reicht.

 

[Black Kong]

Das funktioniert so zwar allerdings solltest du
a) schon die Methoden nutzen (momentan greifst du auf den Member auftragFertiggestellt direkt zu
b) wie oben beschrieben boolsche Werte nicht noch mal auf true/false prüfen

if(erster.getAuftragFertiggestellt()) {
    System.out.println("Auftrag ist fertig");
} else {
    System.out.println("In Bearbeitung");
}

1. Die Variable auftragFertiggestellt1 wird nirgends benutzt.
2. Der Zugriff über erster.auftragFertiggestellt funktioniert hier nur "zufällig". Du solltest hier den Methodenaufruf verwenden.
3. == true ist überflüssig.

if (erster.getAuftragFertiggestellt()) {
    System.out.println("Auftrag ist fertig");
} else {
    System.out.println("In Bearbeitung");
}

reicht.

Danke an euch beide für die Mühe die ihr euch gibt. Bitte nehmt es mir nicht übel das Ich ab&zu noch grobe Fehler mache und eure Antworten nicht immer sofort umsetzten kann .

 

[Robat]

Bitte nehmt es mir nicht übel das Ich ab&zu noch grobe Fehler mache und eure Antworten nicht immer sofort umsetzten kann .

Ach quatsch .. nur durch Fehler lernt man. Und lieber einmal mehr nachfragen (und Interesse zeigen) als es einfach abschreiben

 

[mihe7]

Bitte nehmt es mir nicht übel das Ich ab&zu noch grobe Fehler mache und eure Antworten nicht immer sofort umsetzten kann

Du bist lustig, dafür ist so ein Forum da.

Den meisten, die hier antworten, geht es in erster Linie um Hilfe zur Selbsthilfe (zum Leidwesen mancher ) Gerade, wenn es um Grundlagen geht, werden die Antworten oft so formuliert, dass der Fragesteller nur in die richtige Richtung gelenkt wird und er die Möglichkeit bekommt, selbst auf die Lösung zu kommen. Der Aha-Effekt (und damit der Lerneffekt) ist dann einfach größer und es dürfte auch mehr Spaß machen als irgendetwas abzutippen, was man eh nicht kapiert.

Wenn Du was nicht verstehst, einfach nachfragen.