BubbleSort

[AlphaMinus]

Hallo Leute,

Ich habe für den BubbleSort ein kleines Programm geschrieben und verstehe nicht was falsch sein sollte bzw. warum Eclipse ein error ausgibt.

public class BubbleSort {

	public static void führeaus(int[] x) {
		boolean unsortiert=true;
				int temp;
		
		while (unsortiert) (unsortiert)=false;
		for (int i=0; i<x.length-1; i++)
			if (x[i] > x[i+1]) {
				temp        = x[i];
				x[i]        = x[i+1];
				x[i+1]      = temp;
				unsortiert= true;
			}
		}
	


	
public static void main (String[] args) {
	int[] liste = { 20,1000,234,534,4363,234,1,534,3244,5,245325,12,343,14,124,5514,153,25324,1243,14,3,134,1,341,4,134,131};
	sortiere(liste);
	for (int i=0; i<liste.length; i++)
		System.out.print(liste[i]+" ");
}

}

Eclipse zeigt den Fehler bei sortiere aber wieso?

Danke für eure Antworten hoffe ich könnt mir helfen ^^

Grüße AlphaMinus

 

[HimBromBeere]

while (unsortiert) (unsortiert)=false;

Was bitte soll das heißen?!

EDIT: Wenn du schon einen Fehler beschreibst, warum schickst du dann nicht gleich die Fehlermeldung mit? Das wäre um einiges einfacher für uns, denn so muss man erstmal die Glaskugel befragen...

Übrigens: deine FühreAus-Methode wird nie aufgerufen (ist wahrscheinlich auch besser, denn der Name ist mehr als unglücklich...).

 

[nillehammer]

Fehlermeldungen lesen/kopieren/posten schafft sehr schnell Klarheit. In diesem Fall ist es aber offensichtlich, es gibt überhaupt keine Methode namens

sortiere

. Du hast nur eine namens

führeaus

. Solch ein Name ist aus mehreren Gründen schlecht:
- Enthält Sonderzeichen
- Beschreibt nicht, was die Methode macht (sortiere ist da besser).

Das wird aber nicht der einzige Fehler bleiben. Deine while-Schleife sieht auch komisch aus, wie HimBromBeere schon bemerkt hat.

 

[AlphaMinus]

Sorry für dir Umständlichkeiten hier der Fehler:

Exception in thread "main" java.lang.Error: Unresolved compilation problem:
The method sortiere(int[]) is undefined for the type BubbleSort

at BubbleSort.main(BubbleSort.java:25)

Wie bereits gesagt den rest müsst ihr garnicht überprüfen, denn Eclipse setzt den fehler nur auf sortiere irgendetwas müsste an dieser Stelle falsch sein

 

[nillehammer]

The method sortiere(int[]) is undefined for the type BubbleSort

Das ist doch recht verständlich, was da falsch ist, oder? Falls nicht, siehe mein letzter Post.

 

[AlphaMinus]

EDIT: Es hat jetzt funktioniert, es hat tatsächlich an der Methode gelegen die ich auch in sortiere unbennen musste, anscheinend sind diese beiden miteinander verknüpft. Danke trotzdem hier nochmal das ganze "richtig".

public class BubbleSort {

	public static void sortiere (int[] x) {
		boolean unsortiert=true;
				int temp;
		
		while (unsortiert) {
			
			(unsortiert)=false;
		for (int i=0; i<x.length-1; i++)
			if (x[i] > x[i+1]) {
				temp        = x[i];
				x[i]        = x[i+1];
				x[i+1]      = temp;
				unsortiert= true;
			
		}
	
		}
	}
	
public static void main (String[] args) {
	int[] liste = { 20,1000,234,534,4363,234,1,534,3244,5,245325,12,343,14,124,5514,153,25324,1243,14,3,134,1,341,4,134,131};
	sortiere(liste);
	for (int i=0; i<liste.length; i++)
		System.out.print(liste[i]+" ");
}

}

 

[EasyEagle]

Hallo AlphaMinus und willkommen im Forum,

also:

1) Zeile 8

 while (unsortiert) (unsortiert)=false;

macht wirklich wenig Sinn.
2) Du setzt

 unsortet = true

sobald der erste Wert ausgetauscht wurde. Was zum momentan Entwicklungsstand nicht viel macht, weil deine while-Schleife nichts ausrichtet.
3) Und das beantwortet deine Frage: Es gibt keine Methode

 sortiere(int[] _intArray)

, die heißt bei dir

 führeaus(int[] x)

EDIT: ich war zu langsam

 

[EasyEagle]

OK, jetzt siehts besser aus

Jetzt noch sauber einrücken, geschweifte Klammern setzten und bei

 (unsorted) = false

die Klammern löschen.
Dann ist es sauber programmiert

lg