myGully.com Boerse.SH - BOERSE.AM - BOERSE.IO - BOERSE.IM Boerse.BZ .TO Nachfolger
Zurück   myGully.com > Computer & Technik > Programmierung
Seite neu laden

[JAVA] Methodenaufruf und Integrationsproblem

Willkommen

myGully

Links

Forum

 
Antwort
 
Themen-Optionen Ansicht
Ungelesen 04.11.10, 18:42   #1
PornoPenner
Banned
 
Registriert seit: Aug 2010
Beiträge: 209
Bedankt: 70
PornoPenner ist noch neu hier! | 0 Respekt Punkte
Standard

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++;
PornoPenner ist offline   Mit Zitat antworten
Antwort


Forumregeln
Du kannst keine neue Themen eröffnen
Du kannst keine Antworten verfassen
Du kannst keine Anhänge posten
Du kannst nicht deine Beiträge editieren

BB code is An
Smileys sind An.
[IMG] Code ist An.
HTML-Code ist Aus.

Gehe zu


Alle Zeitangaben in WEZ +1. Es ist jetzt 20:51 Uhr.


Sitemap

().