Java-Forum.org  

Zurück   Java-Forum.org > Andere Sprachen (NEU!) > Softwareentwicklung

Softwareentwicklung: Entwicklungsvorgaben

Allgemeine Softwareentwicklung - Andere Programmiersprachen, Regex, OOP, Design Patterns

Antwort    
Themen-Optionen Thema durchsuchen Ansicht
Alt 19.02.2012, 08:16   #1
cousa
 
Entwicklungsvorgaben - Standard Entwicklungsvorgaben

Hallo Forum,
inspiriert durch den "schockierenden" Eintrag über den VB Entwickler in der Plauderecke habe ich mir einige Gedanken über meinen eigenen Programmierstil gemacht. Insbesondere gibt es bei uns zwei "feindliche Guppen" Entwickler was einige Stille bzw. Vorgehensweisen angeht.

Wie macht Ihr das bzw. was findet ihr sauberer?

Methode früh raus oder if else bis zum Ende?
Java Code: Quelltext in neuem Fenster öffnen
  1. public boolean berechneStatus(Objekt basis,Objekt erweiterung){
  2.   if (basis== null||erweiterung == null){
  3.     return false;
  4.   }
  5.   if(basis.getWert()== null){
  6.      throw new Exception("msg");
  7.   }
  8.   //hier findet die lange Berechnung statt
  9.   return true;
  10. }
Java Code: Quelltext in neuem Fenster öffnen
  1. public boolean berechneStatus(Objekt basis,Objekt erweiterung){
  2.   if (basis!= null&&erweiterung != null){
  3.     //viele viele Zeilen
  4.    if(basis.getWert==null){
  5.      throw new Exception("msg");
  6.    }
  7.  }else{
  8.   return false;
  9.  }
  10. return true;
  11. }


Wird klar was ich versuche auszudrücken?

Entweder möglichst schnell raus aus der Methode und so eine oder sogar mehrere If Ebenen zu sparen oder ist der Weg übersichtlicher eine IF Else und dann nach teilweise 40 Zeilen ein [JAVA] }else{ return false; }. Ist es nicht schöner einfach prüfen und raus? Aber wenn irgendwann im else Zweig doch noch Berechnungen hinzugefügt werden wird es schnell unübersichtlich.Aber auch hier versuche ich so schnell wie es geht wieder aus der Methode rauszukommen.

Habt Ihr noch solche "Streitpunkte" oder was sind eure kuriosesten Vorgaben/Eigenheiten bei der Entwicklung?

Schönen Sonntag!

cousa
 

 
Anzeige
 

AW: Entwicklungsvorgaben



 
Alt 19.02.2012, 10:05   #2
Xandaros
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Ich würde in dem Fall das return nach oben setzen, da es meiner Meinung nach übersichtlicher ist.

Irgendwie kommt es mir komisch vor so wenig zu posten, aber ich habe dem nichts mehr hinzuzufügen...
 
Alt 19.02.2012, 10:44   #3
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Ich finde beide Varianten nicht unbedingt schön. Soll der boolean angeben ob die Berechnung möglich war oder nicht? Dann lieber ne Exception werfen für den Fall dass die Parameter nicht korrekt übergeben wurden.
Generell halte ich es auch für ok die Methode am Anfang oder mitten drin mit nem return zu verlassen, da Methoden eh kurz und übersichtlich sein sollten. Ob das für deine 40-50 Zeilen lange Methode auch gilt kann man schlecht sagen ohne die mal zu sehen, aber die kann man vermutlich auch noch in kleinere Methoden aufteilen, gerade wenn du größere if-else blöcke hast.
 
Alt 19.02.2012, 11:34   #4
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

wie schon gesagt wurde, da Methoden so kurz wie moeglich bzw noetig gehalten werden sollen, ist es eher egal, da man die Methode so und so mit einem Blick ueberschauen kann.
Dein 2. Code ist aber so und so komisch. Wenn die Bedingung nicht true ist, geht er in den else zweig und returned false, danach kommt noch ein return true ?! Wann soll das eintreten?
 
Alt 19.02.2012, 11:35   #5
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Das tritt ein wenn die if Bedingung zutrifft
 
Alt 19.02.2012, 11:57   #6
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Variante 1. Ich finde zu viele verschachtelungen unschön und es ist imho einfacher zu lesen. Varianate 1 scheint sogar (eine stille) konvention bei uns im Betrieb zu sein.
 
Alt 19.02.2012, 12:01   #7
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Zitat:
Entweder möglichst schnell raus aus der Methode und so eine oder sogar mehrere If Ebenen zu sparen oder ist der Weg übersichtlicher eine IF Else und dann nach teilweise 40 Zeilen ein [JAVA] }else{ return false; }.
Keine der Varianten ist "sauber" wenn die Methode 40 Zeilen hat...

Wie schon gesagt wurde, halte die Methoden kurz, sehr kurz.
 
Alt 19.02.2012, 12:59   #8
cousa
Themenstarter
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Zitat: maki
Beitrag anzeigen
Keine der Varianten ist "sauber" wenn die Methode 40 Zeilen hat...

Wie schon gesagt wurde, halte die Methoden kurz, sehr kurz.
Das ist bei meinem code auch Standard. Ist leider ein "Alt System" das nicht sehr robust entwickelt worden ist.

cu cousa
 
Alt 19.02.2012, 13:18   #9
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Zitat: cousa
Beitrag anzeigen
Das ist bei meinem code auch Standard. Ist leider ein "Alt System" das nicht sehr robust entwickelt worden ist.
Dann verstehe ich die Frage nicht...
 
Alt 19.02.2012, 15:03   #10
MasterK
 
Entwicklungsvorgaben - Standard AW: Entwicklungsvorgaben

Variante 1. Natürlich sollte eine methode immer so kurz sein, dass sich die frage gar nicht stellt, aber das ist in der realität nunmal nicht immer sinnvoll machbar. Manchmal ist eine methode eben 80 zeilen lang, und es macht nicht immer sinn diese dann künstlich aufzusplitten. Und in dem fall ist es schön, wenn man sozusagen "vorbedinungen" gleich zu beginn der methode sieht und nicht irgendwo mittendrin aus der methode gesprungen oder abgebrochen wird.

Dennoch ist variante 2 auch objektiv gesehen nicht "falsch" und ich würde solchen code problemlos akzeptieren.

Übel wirds erst, wenn spezies kommen die sowas hinrotzen:
Java Code: Quelltext in neuem Fenster öffnen
  1. public boolean machwas(){
  2.  
  3.   boolean ret = true;
  4.   do {
  5.     //...
  6.     if (...) {
  7.       ret = false;
  8.       break;
  9.     }
  10.     //... mehr logik
  11.     if (...) break;
  12.  
  13.     //usw, am besten über 500 zeilen
  14.  
  15.   } while(false);
  16.   // blabla
  17.   return ret;
  18. }
Sowas habe ich das erste mal bei einem bei mir auf arbeit gesehen (der wirklich nur so programmiert mit hunderte zeilen langen methoden), aber danach auch mal hier im forum. Wer sowas schreibt, gehört geschlagen, pausenlos.
 
Antwort    

Zurück   Java-Forum.org > Andere Sprachen (NEU!) > Softwareentwicklung

Themen-Optionen Thema durchsuchen
Thema durchsuchen:

Erweiterte Suche
Ansicht




Sie betrachten gerade Entwicklungsvorgaben