Datum: 11/26/2001 17:25:17 
Von: Bewertung
An: 0004444

Betreff: Review zu Runde 1  
korrektheit: 10 
verwendbarkeit: 8 
programmierstil: 6 



Dokumentation: 

compilierbar 
startbar 


kleine anmerkung: ich verstehe den grund nicht, wieso alle variablen 
im klassenkopf mit static angenommen wurden. ist zwar einfacher, auf diese 
klasse von einer anderen klasse aus zuzugreifen, aber das war ja (noch) nicht nötig. 
diese anmerkung ist nichtmal ein verbesserungsvorschlag! 

anmerkung zur *.txt: es liegt ein logikfehler vor: in der *.txt steht, die elemente 
des arrays "NeueWB_Temp" werden ungefähr so wie (A+B)*(B+C) verbunden. es ist aber 
wichtig, dass die elemente genau folgend verbunden werden!!: AB AC BB BC im Programm 
stimmt die verknüpfung! 

die erläuterung der methoden .tochararray(), ... sind überflüssig. es ist wichtig, die 
*.txt so knapp wie möglich zu halten, um das interesse des lesers zu gewährleisten. auf 
stellen wie diese sollte daher verzichtet werden. 

wenn man die - public class erbgang extends EprogIO - erweitert, spart 
man sich die lästigen EprogIO.readWord() ... Eingaben. 
readWord() sieht einfach eleganter aus. 

die überprüfung mit der mendel-variable ist zwar richtig, aber etwas 
"unschön" und etwas schwierig verständlich. ich würde vorschlagen, 
eine einfache equals abfrage zu machen. 


es ist einfacher, die abfrage nach mendel1 oder mendel2 statt auf diese 
art, als vergleich zu machen: 
if ((WB1.charAt(0)==WB1.charAt(1)) && (WB2.charAt(0)==WB2.charAt(1))) 

der inhalt des arrays NeueWBArray[] ist etwas schwierig nachzuvollziehen. 



kritik von 
christian rohrauer 
matr.nr 0105884 
e-mail christian.rohrauer@rocketmail.com