Ausgabe problem in einer Hausaufgabe

[Kquadrator]

Ich weiß leider nicht was mein Fehler ist bei der Ausgabe
Kann mir da jemand vielleicht weiter helfen.

Also die Aufgabe ist, 3 Zahlen einlesen lassen und sortiert wieder ausgeben lassen (bubbelsort).
Das mit dem Sortieren war nicht mal so schwer, aber ich habe keine Ahnung wieso ich nichts zurück bekomme ?

Wichtig ist noch zu sagen, wegen der EinleseFunktion. wir haben in der Schule schon eine Klasse geschrieben wie man sehen kann (InteractivIO).

Ich hoffe jemand kann mir da helfen ?


Hier mein Quellcode:

class SortLong {
	//init
	  long number1 = 0;
	  long number2 = 1;
	  long number3 = 2;
	  
	  long []a = new long[3];
	  String test;

//-------------------------------------------------------------------------------------------------	
	public SortLong() throws Exception{
		number1 = InteractivIO.readLong("First number: ");
		number2 = InteractivIO.readLong("Secound number: ");
		number3 = InteractivIO.readLong("Third number: ");
	}

//-------------------------------------------------------------------------------------------------	
	  public long[] sorting(long []list) {           
		    long temp;                                  // helpvariablne for swapping
		    boolean swapping;                         	// merkt sich, ob getauscht
		    			 
			 a[0] = number1;
			 a[1] = number2;
			 a[2] = number3;
		    
		    
		    do {
		    	swapping = false;                     	// nimm an, dass nicht getauscht
		        for (int i=0; i<list.length-1; i++){    // durchlaufe Array
		            if (list[i] > list[i+1]) {          // falls Nachbarn falsch herum
		            	//swapping
		                temp = list[i];                 // bringe 
		                list[i] = list[i+1];            // beide Elemente
		                list[i+1] = temp;               // in die richtige Ordnung
		                //no swapping!
		                swapping = false;               // vermerke, dass getauscht
		            }
		        }
		    } while (!swapping);                        // while swapping
		    return list;
		    }


//-------------------------------------------------------------------------------------------------	
	public String toString(){
		output();
		return test;
	}
	
	public void output(){
		a = sorting(a);
		   for (int i = 0; i < a.length; i++) {
			      System.out.print (a[i] + " ");
			   }		
	}
	
//-------------------------------------------------------------------------------------------------	
	public static void main (String[]args) throws Exception {
		SortLong x;
		x = new SortLong();
		
		InteractivIO.write(x.toString());
	}
	
}//end

 

[SlaterB]

'while swapping' steht in deinem Kommentar,
'while !swapping' dagegen in deinen Code -> Endlosschleife

> swapping = false; // vermerke, dass getauscht

true?

--------

number1 = 4;
number2 = 5;
number3 = 2;

wurde bei mir nicht richtig sortiert,
Ausgabe: 4 2 5 null

(liegt vielleicht an obigen Fehlern)

---------

innerhalb der eigenen Klasse solltest du
Exemplatrvariablen wie a nicht als Parameter an andere eigene Operationen übergeben,
auch nicht als Rückgabewert zurück und dann überschrieben..
a = sorting(a);

sorting kennt doch a genauso, kann sorting alles selber machen

-------

das Sortieren sollte nicht in der toString()-Operation stattfinden!

welchen Sinn erfüllt test, welches bei toString() zurückgegeben wird?
die Ausgabe erfolgt bisher an System.out, nicht InteractiveIO

--------

wieso liest du die Nummern in einzelne Exemplarvariablen ein und fügst sie beim Sortieren in das Array ein?
das kann direkt im Konstruktor passieren, dann brauchst du

long number1 = 0;
long number2 = 1;
long number3 = 2;

nicht, zumindest nicht als Exemplarvariablen, nur lokal im Konstruktor

 

[Guest]

'while swapping' steht in deinem Kommentar,
'while !swapping' dagegen in deinen Code -> Endlosschleife

> swapping = false; // vermerke, dass getauscht

true?
[/quate]
ah stimmt... nun seh ich die endloss schleifen auch

number1 = 4;
number2 = 5;
number3 = 2;

wurde bei mir nicht richtig sortiert,
Ausgabe: 4 2 5 null

(liegt vielleicht an obigen Fehlern)

schon gelöst , dein Beispiel müsste nun auch gehen^.^

		    do {
		    	swapping = false;                     	// nimm an, dass nicht getauscht
		        for (int i=0; i<list.length-1; i++){    // durchlaufe Array
		        	for (int j=list.length-1; j > i; j=j-1) 
		            if (list[j-1] > list[j]) {          // falls Nachbarn falsch herum
		            	//swapping
		                temp = list[j-1];                 // bringe 
		                list[j-1] = list[j];            // beide Elemente
		                list[j] = temp;               // in die richtige Ordnung
		                //no swapping!
		                swapping = false;               // vermerke, dass getauscht
		            }
		        }
		    } while (swapping);                        // while swapping
		    return list;
		    }

---------

innerhalb der eigenen Klasse solltest du
Exemplatrvariablen wie a nicht als Parameter an andere eigene Operationen übergeben,
auch nicht als Rückgabewert zurück und dann überschrieben..
a = sorting(a);

sorting kennt doch a genauso, kann sorting alles selber machen

Du meinst also das ich den code teil in die Funktion "sorting" reinhauen soll?
bin grad etwas verwirrt wegen dem

das Sortieren sollte nicht in der toString()-Operation stattfinden!

welchen Sinn erfüllt test, welches bei toString() zurückgegeben wird?
die Ausgabe erfolgt bisher an System.out, nicht InteractiveIO

Problem ist hier, das meine funktion "write" nur strings nehmen kann
da hab ich mir mit dem Test etwas weiter geholfen ^^;

wieso liest du die Nummern in einzelne Exemplarvariablen ein und fügst sie beim Sortieren in das Array ein?
das kann direkt im Konstruktor passieren, dann brauchst du

long number1 = 0;
long number2 = 1;
long number3 = 2;

nicht, zumindest nicht als Exemplarvariablen, nur lokal im Konstruktor

ok... also soll das wieder in den konstruktor rein... wie ich das jetzt verstehe oO?

 

[Guest]

number1 = 4;
number2 = 5;
number3 = 2;

wurde bei mir nicht richtig sortiert,
Ausgabe: 4 2 5 null

(liegt vielleicht an obigen Fehlern)

schon gelöst , dein Beispiel müsste nun auch gehen^.^

		    do {
		    	swapping = false;                     	// nimm an, dass nicht getauscht
		        for (int i=0; i<list.length-1; i++){    // durchlaufe Array
		        	for (int j=list.length-1; j > i; j=j-1) 
		            if (list[j-1] > list[j]) {          // falls Nachbarn falsch herum
		            	//swapping
		                temp = list[j-1];                 // bringe 
		                list[j-1] = list[j];            // beide Elemente
		                list[j] = temp;               // in die richtige Ordnung
		                //no swapping!
		                swapping = false;               // vermerke, dass getauscht
		            }
		        }
		    } while (swapping);                        // while swapping
		    return list;
		    }

---------

innerhalb der eigenen Klasse solltest du
Exemplatrvariablen wie a nicht als Parameter an andere eigene Operationen übergeben,
auch nicht als Rückgabewert zurück und dann überschrieben..
a = sorting(a);

sorting kennt doch a genauso, kann sorting alles selber machen

Du meinst also das ich den code teil in die Funktion "sorting" reinhauen soll?
bin grad etwas verwirrt wegen dem

das Sortieren sollte nicht in der toString()-Operation stattfinden!

welchen Sinn erfüllt test, welches bei toString() zurückgegeben wird?
die Ausgabe erfolgt bisher an System.out, nicht InteractiveIO

Problem ist hier, das meine funktion "write" nur strings nehmen kann
da hab ich mir mit dem Test etwas weiter geholfen ^^;

wieso liest du die Nummern in einzelne Exemplarvariablen ein und fügst sie beim Sortieren in das Array ein?
das kann direkt im Konstruktor passieren, dann brauchst du

long number1 = 0;
long number2 = 1;
long number3 = 2;

nicht, zumindest nicht als Exemplarvariablen, nur lokal im Konstruktor

ok... also soll das wieder in den konstruktor rein... wie ich das jetzt verstehe oO?

 

[SlaterB]

> schon gelöst , dein Beispiel müsste nun auch gehen^.^

solange du nirgendwo swapping = true hast,
kann es nicht gehen,

nicht selber getestet?

> Du meinst also das ich den code teil in die Funktion "sorting" reinhauen soll?

siehe weiter unten

dein Code ist auch noch besonders prekär, als dass du den Parameter long[] list und a selber gleichzeitig benutzt,
list ist anfangs leer,
wenn du in a was reinschreibst, dann auch in list, denn list ist ja gerade a

nachdem du in a etwas eingefügt hast kannst du list erst benutzen, verrückt


> ok... also soll das wieder in den konstruktor rein... wie ich das jetzt verstehe oO?

ist natürlich Geschmackssache, aber ich finde ja

public class SortLong
{
    private long[] a = new long[3];

    public SortLong()
        throws Exception
    {
        a[0] = 4;
        a[1] = 5;
        a[2] = 2;
        sorting();
    }

    private void sorting()
    {
        long temp; // helpvariablne for swapping
        boolean swapping; // merkt sich, ob getauscht

        do
        {
            swapping = false; // nimm an, dass nicht getauscht
            for (int i = 0; i < a.length - 1; i++)
            { // durchlaufe Array
                if (a[i] > a[i + 1])
                {
                    temp = a[i]; // bringe
                    a[i] = a[i + 1]; // beide Elemente
                    a[i + 1] = temp; // in die richtige Ordnung
                    swapping = true; // vermerke, dass getauscht
                }
            }
        }
        while (swapping); // while swapping
    }

    public String toString()
    {
        String st = "";
        for (int i = 0; i < a.length; i++)
        {
            st += a[i] + " ";
        }
        return st;
    }

    public static void main(String[] args)
        throws Exception
    {
        SortLong x = new SortLong();
        System.out.println(x.toString());
    }

}// end

 

[Kquadrato]

Also das seltsame ist das Programm geht nun oO
"First number:
4
Secound number:
5
Third number:
2
2 4 5 null"

Aber sobald ich deinen Ratschlag befolge geht es nicht mehr ...
soll sich mein Lehrer über diese seltsame Lösung freuen +hust+
(wenn der auch immer so seltsame Sachen verlangt)

1000Dank für die schnelle Hilfe, ich glaub ich wär da hunder jahre gesessen und hätte nichts gesehen