fehler in der source

[morph]

Hallo wieder, kann mir jeder mit dieser Aufgabe helfen.

public class Assistent 
{
   
   public String name="unknown";
   public boolean abschluss=false; 
   public String abteilung;

   public void p(String s)
     {
	System.out.println(s);
     }
      
   
   public Assistent () 
     { 
     }
   
   public Assistent (String name, String abteilung)
     {
	String setName=name;
	String Abteilung=abteilung;
     }

   public void promovieren (abschluss) 
     {
	abschluss = true;
	p("Martin ist nun promoviert.");
     }
   
      	  
   public  istPromoviert (abschluss)
     {
	if (abschluss=true)
	  { 
	     p(Martin.name.substring(0,7) + "ist bereits promoviert.");
	     return true;
	  } else {
	     p(Martin.name.substring(0,7) + "ist noch nicht promoviert.");
	     return false;
	  }
     }
   
}

public static void main (String[] args)
{
   Assistent Martin = new Assistent();
   Martin.Assistent("Martin Muster", "Software Engineering");
   
   if (istPromoviert.abschluss=false) 
     	istPromoviert.abschluss=true;
   
}

Der Text zu der Übung ist:


Schreiben Sie ein kleines Programm, welches einen Assistenten promovieren lassen kann. Implementieren Sie dazu das rechtsstehende Klassendiagramm. Nach dem Deklarieren der Instanzvariablen, schreiben sie bitte Konstruktor und Methoden mit folgender Logik:

1. Der Konstruktor initialisiert alle Instanzvariablen der Klasse Assistent. Er bekommt dazu den Namen des Assistenten und den Namen der Abteilung übergeben. Der Default-Wert für die Instanzvariable "absch2.Die Methode promovieren() soll lediglich den Abschluss-Status auf "true" setzen und eine Meldung and der Standardausgabe erzeugen:

"Martin Muster ist nun promoviert"http://www.java-forum.org/de/posting.php?mode=newtopic&f=17

3. Die Methode istPromoviert() gibt einen Wahrheitswert zurück, der besagt, ob der betreffende Assistent bereits promoviert ist oder nicht. Wenn der Assistent promoviert ist, soll "true" zurück gegeben werden, sonst soll "false" zurückgegeben werden. An der Standardausgabe soll entweder

"Martin Muster ist noch nicht promoviert!"
oder
"Martin Muster ist bereits promoviert!"

ausgegeben werden.
4. Erzeugen sie in der main-Methode einen Assistenten. Schreiben sie eine geeignete Kontrollschleife, die mit der Methode istPromoviert() testet, ob der Assistent promoviert ist oder nicht. Wenn der Assistent noch nicht promoviert ist, soll er dies nun tun.
Nach Ablauf der Main-Methode könnten die Ausgaben ihres Programms so aussehen:

"Martin Muster ist noch nicht promoviert!"
"Martin Muster ist nun promoviert" .

Ich verstehe nicht die Fehlermeldungen die ich empfange.

Danke.

 

[lhein]

public class Assistent 
{ 
    
   private String name=; 
   private boolean abschluss=false; 
   private String abteilung; 
      
   public Assistent (String name, String abteilung) 
     { 
   this.name = name; 
   this.abteilung=abteilung; 
     } 

   private void p(String s) 
     { 
   System.out.println(s); 
     } 

   public void promovieren () 
     { 
   abschluss = true; 
   p(name + " ist nun promoviert."); 
     } 
    
   public boolean istPromoviert () 
     { 
   if (abschluss) 
     { 
        p(name + " ist bereits promoviert."); 
        return true; 
     } 
    else 
    { 
        p(name + " ist noch nicht promoviert."); 
        return false; 
     } 
  } 
} 

public static void main (String[] args) 
{ 
   Assistent Martin = new Assistent("Martin Muster", "Software Engineering"); 
    
   if (!Martin.istPromoviert()) 
{
        Martin.promovieren();
    
}
}

Das wars hoffentlich.
Code ist aber nicht wirklich schön.

Man sollte ein paar Grundsätze einhalten. Z.B. würde ich Strings immer mit s_VariablenName deklarieren, int-Werte immer mit i_Variablenname usw.
Objekte sollten auch sinnvoll benannt werden. Sprich Martin ist kein schönes Bezeichner. Besser wäre assistent_Martin statt Martin. Dann sieht man in grossen Klassen sofort, von welchem Typ es ist.

Ansonsten immer Klassenvariablen geschützt halten, es sei denn es ist nicht anders machbar (warum auch immer). Sprich public ist keine gute Lösung. Dann sollte man die sogenannten Getter und Setter Methoden gut benennen.
Bei Wahrheitsprüfungen z.B. isPromoviert() oder bei get-Methoden z.B. getName() oder getAbteilung(). Selbiges gilt auch für Setter.

Die Methode p ist hier mal Humbug, da ich das System.out.println auch getrost immer dahin schreiben kann wo ich es brauche. Macht in meinen Augen also keinen wirklichen Sinn.

Ansonsten haben andere hier sicher auch noch ein paar Tipps für Dich, wie Du Deinen Programmierstil verbessern kannst.

LR

 

[Beni]

Auch wenn LR schneller war: *grrr*

Huii, Java-Syntax lernen, wäre eine Lösung...

1. Jede Methode muss in einer Klasse sein, auch das public static void main.
2. Dieser Syntax public void promovieren (abschluss) existiert ganz einfach nicht, schreib einfach public void promovieren()
3.

Assistent Martin = new Assistent();
   Martin.Assistent("Martin Muster", "Software Engineering");

"Assistent" ist der Konstruktor. Ein Konstruktor kann nur einmal aufgerufen werden, wenn das Objekt initialisiert wird (mit dem Keyword "new").
Du musst schreiben:

Assistent Martin = new Assistent("Martin Muster", "Software Engineering");

4.if (istPromoviert.abschluss=false) "=" ist der Zuweisungsoperator. Er weisst etwas zu, und hat mit "=" der Mathematik keine Verwandtschaft.
Du benötigst "==", den Vergleichoperator.
Noch einfach ist "!", die Negation: if ( ! condition )
5. Diese Zeile if (istPromoviert.abschluss=false) ist aber noch anderweitig falsch:
istPromoviert ist eine Methode. Der Zusatz ".abschluss" ist kein Java-Syntax. Einfach weglassen!
Und du musst noch sagen wer promoviert ist: Martin.istPromoviert();
6. Wenn wir bei "istPromoviert" sind: Du musst angeben, was die Methode zurückgibt (ein boolean): public boolean istPromoviert ()
7. p(Martin.name.substring(0,7) + "ist bereits promoviert."); ist dafür im Objekt drin (naja, eine Variable Martin existiert dafür an dieser Stelle nicht). Streich den Zusatz Martin!

Und jetzt mal die Klasse, wie sie funktionieren würde:

public class Assistent{
    // Die sind private, niemand ausser der Assistent kann diese Daten verändern!
   private String name="unknown";
   private boolean abschluss=false;
   private String abteilung;

   public Assistent (){ // Codingconventions von Sun: die beginnende Klammer kommt auf dieselbe Zeile.
   }
 
   public Assistent (String name, String abteilung){
       // Die Instanzvariablen werden auf den Wert der Argumente gesetzt
       this.name = name; 
       this.abteilung = abteilung;
   }
   
   public void p(String s){
       System.out.println(s);
   }

   public void promovieren (){
       abschluss = true;
       p("Martin ist nun promoviert.");
   }
   
   // Gibt einen Boolean zurück
   public boolean istPromoviert (){
       if ( abschluss ){ // nicht if( abschlus = true ), 1. weil x = true == x ist (Logik !!!), 2. weil damit abschlus true zugewiesen wird.
           p(name + " ist bereits promoviert.");
           return true;
       }
       else {
           p(name + " ist noch nicht promoviert.");
           return false;
       }
     }
   public static void main (String[] args){
       // Neues Objekt erzeugen, Variablennamen werden IMMER klein geschrieben.
      Assistent martin = new Assistent("Martin Muster", "Software Engineering");
      
      // martin Fragen, wie es geht, und vielleicht promovieren
      if( ! martin.istPromoviert() )
          martin.promovieren();
   }   
}

mfg Beni

 

[lhein]

Das mit dem main ist mir beim Durchlesen garnet aufgefallen. Aber nu isses ja raus

BTW: Codingconventions von Sun: die beginnende Klammer kommt auf dieselbe Zeile

Davon halt ich nicht sehr viel....Leserlichkeit ist da echt mies in meinen Augen.

LR

 

[biker126]

dito!

is vor allem bei if-else-for-while konstrukten saudoof die öffnende klammer in die gleich zeile wie die anweisung zu schreiben...

 

[Beni]

Ich würde das mal "kompakt" nennen!

Zwar arten solche Diskussionen regelmässig aus, aber mal ganz ehrlich: findet ihr eine Klammer pro Zeile nicht eine grausame Verschwendung?

Naja, das ist Gewöhnungssache, ich kanns sehr schnell lesen.

mfg Beni

 

[morph]

Na ja ich lerne seit Zwei wochen.

Vielen Dank für die Ratschlage und die Hilfe .

GRüsse

 

[morph]

Benny erklär mir bitte wieso soll ich immer den Default-Konstruktor schreiben, falls ich ihn in meinem Source nicht benutze.

danke :?:

 

[lhein]

Die Klammer in extra Zeilen macht den Code übersichtlicher, da wird glaube ich niemand widersprechen...wahrscheinlich nichtmal Du, beni.

Wenn man mal in fremdem Code nach Fehlern gesucht hat oder ihn einfach nur nachvollziehen wollte, und die LOC eine gewisse Zahl überschreiten, dann wirst Du Dich an meine Worte zurückerinnern. Und das beziehe ich nicht nur auf Java, sondern weite die Aussage auf andere Programmiersprachen aus. Ich denke da nur an 100e Zeilen umfassende C Funktionen, die noch durch etliche Loops und Bedingungen läuft.
Da kann man manchmal schon verzweifeln....denn Sinn eines lesbaren Codes ist es nicht, dass ich damit zurechtkomme, sondern dass andere Leute auch damit umgehen können....zumindest wenn man beruflich programmiert...für daheim mags mal nicht so relevant sein.

LR

 

[Illuvatar]

@LR: Ich finde Klammern in extra Zeilen auch übersichtlicher.

@morph: Ich bin zwar nicht Beni, aber ich antworte trotzdem:
Der Defaultkonstruktor ist nicht unbedingt nötig. In diesem Fall würde er sogar zu sehr seltsamen Ergebnissen führen, da die Variablen nicht initialisiert würden. Wenn du ihn weglassen würdest, könntest du keinen Konstruktor ohne Parameter benutzen - siehe auch hier.
Er ist eben nicht falsch, aber theoretisch unnötig, und wenn dort die Variablen eine Standard-Initialisierung bekommen, sogar nützlich.

 

[Beni]

@LR
Ich hab durchaus schon 100zeilige C-Funktionen studiert (mühsam...). Und ich finde beide Schreibarten *lesbar*. Aber die eine ist nun einmal kompakter als die andere, darum bevorzug ich die eine...

Die Codingconventions sind eigentlich dazu gedacht, dass alle gleich schreiben, sollten also eine Hilfe sein. Leider halten sich nicht alle dran :cry:

Aber gute Kommentare, schöne Variablennamen, etc., sind sowieso wichtiger als Klammern, also lassen wir diese Diskussion.

mfg Beni


@morph
Gute Frage, (gibts dazu eine offizielle Begründung :?: ).
Also persönlich würde ich sie schreiben, damit du schnell ein Test-Objekt herstellen kannst, ohne dass du lange nachdenken musst, welches Argument jetzt wo wie warum wann hinkommt.

Aber man muss sie nicht schreiben, manchmal ist es auch nicht so sinnvoll (und eigentlich wäre es auch beim Assistenten überflüssig, die haben ja immer einen Namen).

mfg Beni

 

[lhein]

Naja, in allen Firmen die ich kenne gab es eigene Coding-Conventions...von daher nehm ich mal an, dass man es nie auf einen Nenner bringt.

LR

 

[bygones]

Ich schreibe auch alle Klammern in die gleiche Zeile !! Ich finde das persönlich besser...

Aber dank "Code Formatter" von Eclipse (anderer IDEs) ist das ja kein Problem mehr für unterschiedl. Vorlieben :roll: