Funktional mag es ok sein.
Ich weiß nicht inwiefern die Code-Qualität eine Rolle spielt, aber die ist nicht so dicke.
An ner Uni würde die Annahme schon scheitern.
1. Ein paar sinnlose Kommentare ...
2. ... und fehlende . Ehrlich gesagt sind mehr sinnlose Kommentare vorhanden als notwendige.
Dass Du mit Sysout ne Ausgabe machst oder ne Methode aufrufst ist vollkommen klar und braucht nicht
kommentiert werden. Was aber eine Methode bewerkstelligt ist gar nicht kommentiert.
An der Uni könntest Du damit schon durchfallen.
z.B. " // Erzeugen der Methode "While"
// Wahrheitswert auf "true" setzen
//Array mit Werten
???
3. void init(int laenge, int X[])
--> sollte nur das Array als Parameter haben: init(int array[])
4. Unnötige Klammern
{System.out.println("Array nach Bubblesort");}
5. Die Methode display() hat in der statischen Klasse BubbleSort nichts zu suchen
6. Ineffizienter Code: Du initialisiert eine Variable auf "true" und setzt diese danach auf "false"
Außerdem setzt Du unsortiert bei jedem Schleifendurchgang erneut.
Sollte genau andersrum sein (also while(!sortiert) ...)
boolean unsortiert = true;// Boolscher-Wert für die Abbruchbedingung
while (unsortiert) { // Während unsortiert
unsortiert = false;// Boolscher-Wert ist "False"
7. Was macht die Methode "while1"? Wieso "while1"? Weil while-Schleifen vorhanden sind?
8. Packagenamen entsprechen nicht der Konvention
9. Fehlende private-Deklarationen
10. Statt "for (int i = 0; i < F.length - 1; i++)" verwende "for (int i: F)"
11. swap und inner werden in zwei verschiedenen Methoden initialisiert
11. Keine direkte Initialisierung:
int i;
i = 1;
12. Statt i = i + 1
verwende i++;
|