Auf Thema antworten

Ein paar weitere Hinweise möchte ich mir auch noch erlauben:

a) Methoden sollten keine unerwarteten Ergebnisse haben und auch nur eine Sache machen: "containsMean(Array)" sollte also nur genau das machen: Prüfen, ob der Mittelwert in dem Array drin ist. Da sollte also keine Eingabe abgefragt werden und das Array verändert werden. (contains prüft etwas aber verändert nichts!) Hier wäre also die Eingabe heraus zu ziehen und ggf. eine neue Methode einzuführen.


b) Berechnung des Mittelwerts: Da Du den Mittelwert direkt in einem int speicherst hast Du bereits eine entsprechende ganze Zahl. Du musst also nichts mehr runden.


c) anzahl musst Du nicht durch hochzählen ermitteln. Du hast ja die Array Größe, die ja auch in der for Schleife verwendet wird. Daher ist klar: Anzahl wird die Größe des Arrays. Es spricht nichts gegen so eine Variable, aber da würde ich diese dann direkt setzen ([ICODE]int anzahl = array.length;[/ICODE] und dann nicht mehr verändern. Und dann würde ich statt array.length direkt anzahl verwenden.


d) Namensgebung: Du solltest Variablen sinnvoll benennen. array ist ein Array, aber was ist da drin? Das sind z.B. die Werte, mit denen du arbeitest, so dass "werte" schon besser wäre.


Edit: Ganz wichtig: Das ist "meckern auf sehr hohem Niveau" - Daher auch erst das späte erwähnen. Konzentriere dich auf die ersten Antworten bezüglich der eigentlichen Problematik. Die Punkte sind dann später ggf. ein "nice to have".



Oben