Wo liegt hier der Fehler?

[Enriko]

Hallo wo liegt der fehler hier warum funzt es nicht?

class Sort {
public static void main(String args[]) {
int tausch;
int max = 0;
int x = 3, y = 2, z = 1;
int[] arr = new int[5];

arry[0] = x;
arry[1] = y;
arry[2] = z;

for (int i = arry.length; i > 1; i--) {

for (int j = 0; j > arry.length; j++)

if (arry[max] < arry[j]) { //max = j;
tausch = arr[max]; //hier werden
arry[max] = arry[j]; //die werte des
aryr[j] = tausch; //arrays getauscht
}

System.out.println(arry[max]);
}
}
}

 

[SamHotte]

Erster Fehler: keine Code-Tags.
Zweiter Fehler: max und tausch sind gleich 0

 

[HAOSK]

Dritter Fehler: arr[3] und arr[4] sind nicht initialisiert...

 

[Enriko]

Erster Fehler: keine Code-Tags.
Zweiter Fehler: max und tausch sind gleich 0

sorry blöde frage aber was meinst du mit Code-Tags?
und max muss doch 0 sein damit die zweite schleife von vorn anfangen kann!

 

[Enriko]

Dritter Fehler: arr[3] und arr[4] sind nicht initialisiert...

warum muss ich arry[3] und arry[4] initialisiern ? Ich hab doch nur drei Zahlen die ich in den Array speichern muss mehr brauch ich doch nicht oder?

 

[HAOSK]

hast auch Recht... die 2. FOR-Schleife

for (int j = 0; j > arry.length; j++)

wird ja niemals ausgeführt.

sonst hättets du ein Exception bei If-Abfrage

if (arry[max] < arry[j])

 

[SamHotte]

Erster Fehler: keine Code-Tags.
Zweiter Fehler: max und tausch sind gleich 0

sorry blöde frage aber was meinst du mit Code-Tags?
und max muss doch 0 sein damit die zweite schleife von vorn anfangen kann!

Um deinen Code herum machst du Tags, damit er auch hier im Forum als Code dargestellt wird. Den Anfangs-Tag findest du, wenn du auf den Knopf "Code" drückst (oberhalb des Eingabefelds für ein Foren-Post). Das Ende-Tag sieht genauso aus, hat aber noch ein "/" vor dem "code".
Die Beschreibung ist etwas mühsam, da es diesen Text vergrießknaddeln würde, wenn ich die Tags reinschreibe, wie sie sind ...

max = 0 sorgt dafür, dass dein Algorithmus nicht funktioniert, denn du überschreibst damit einen der "echten" Werte.

 

[Enriko]

Erster Fehler: keine Code-Tags.
Zweiter Fehler: max und tausch sind gleich 0

sorry blöde frage aber was meinst du mit Code-Tags?
und max muss doch 0 sein damit die zweite schleife von vorn anfangen kann!

Um deinen Code herum machst du Tags, damit er auch hier im Forum als Code dargestellt wird. Den Anfangs-Tag findest du, wenn du auf den Knopf "Code" drückst (oberhalb des Eingabefelds für ein Foren-Post). Das Ende-Tag sieht genauso aus, hat aber noch ein "/" vor dem "code".
Die Beschreibung ist etwas mühsam, da es diesen Text vergrießknaddeln würde, wenn ich die Tags reinschreibe, wie sie sind ...

max = 0 sorgt dafür, dass dein Algorithmus nicht funktioniert, denn du überschreibst damit einen der "echten" Werte.

Ich bin die ganze zeit am rumraten,weiss gar nicht wie da ran gehen soll?!

 

[SamHotte]

Zur Klarstellung: die Code-Tags sind nur hier für's Forum. Sie helfen zu erkennen, was Code ist und was sonstiger Text.

In deinem Programm hast du folgendes (dein allererster Post):

class Sort { 
public static void main(String args[]) { 
int tausch; 
int max = 0; 
int x = 3, y = 2, z = 1; 
int[] arr = new int[5]; 

arry[0] = x; 
arry[1] = y; 
arry[2] = z; 

for (int i = arry.length; i > 1; i--) { 

for (int j = 0; j > arry.length; j++) 

if (arry[max] < arry[j]) { //max = j; 
tausch = arr[max]; //hier werden 
arry[max] = arry[j]; //die werte des 
aryr[j] = tausch; //arrays getauscht 
} 

System.out.println(arry[max]); 
} 
} 
}

 

[SamHotte]

So, jetzt sieht man die Zeilennummern

In Zeile 18 hat max den Wert 0 (hätte man mit debuggen auch herausfinden können), deswegen funktioniert dein Sortieren nicht.

Außerdem sind diverse Syntaxfehler drin, wie kompilierst du eigentlich?

 

[Enriko]

So, jetzt sieht man die Zeilennummern

In Zeile 18 hat max den Wert 0 (hätte man mit debuggen auch herausfinden können), deswegen funktioniert dein Sortieren nicht.

Außerdem sind diverse Syntaxfehler drin, wie kompilierst du eigentlich?

Und was muss sonst in zeile 18 eintragen? Könntest du mir vielleicht bitte den richtigen Code posten ?

 

[PyroPi]

...
for (int j = 0; j > arry.length; j++)
...

Du benutzt die for-Schleife schon völlig falsch.

for (initialisierung; weitermachBedingung; zwischenschritt) anweisung;

durchläuft die Schleife, solange die weitermachBedinung erfüllt ist. Das entspricht dem hier:

initialisierung;
repeat
    anweisung;
    zwischenschritt;
until (NOT weitermachBedinung);

weitermachBedingung ist also nicht die Abbruchbedingung für die for-Schleife!

Willst du eine Anweisung n-mal ausführen, schreibst du das z.B. so:

for (int i = 0; i < n; i++) anweisung;

oder

for (int i = 1; i <= n; i++) anweisung;

 

[SamHotte]

Fertigen Code gibt's net, denn ich weiß nicht wirklich, was gedanklich hinter deinem Programm für ein Algorithmus steht. Aber den Tipp mit max müsstest du umsetzen können, wenn du erstmal syntaktisch fehlerfreien Code aus dem Zeug gemacht hast. Schau dir mal die Indizes deiner zu vertauschenden Werte an. Mit debuggen solltest du herausfinden können, was wie falsch läuft.

 

[Enriko]

Kann mir irgend jemand den Richtigen code Posten ?