Bitte kritisieren: rekursive Sortier-Methode

[PieceX]

/*
*Diese Methode bekommt ein 2d StringArray, welches zeilenweise
*nach doubleWerten sortiert wird
*Ich freue mich immer über Hinweise, die zur Verbesserung meines
*rekursiven Verständnisses führen...
*/

public String[][] sortByDouble(String[][] toSort,int i){
		String val1="",val2="";
		if(i-1>=0 && toSort.length>1){
			val1 = toSort[i][0];
			val2 = toSort[i-1][0];	
		}
		if(!(val1!=null 
				&& val2.equals(""))){
			if(Double.parseDouble(val1)
					>=Double.parseDouble(val2))return sortByDouble(toSort,i-1);
			else {
				String[][] tmpSort = toSort;
				for(int c=0;c<5;c++){
					String t2 = toSort[i-1][c];
					String t1 = toSort[i][c];
					tmpSort[i][c] = t2;
					tmpSort[i-1][c]= t1;
				}
				return sortByDouble(tmpSort,toSort.length-1);
			}
		}
		else return toSort;
	}

 

[André Uhres]

Funktioniert das Ding denn bei dir?
Ich hab Probleme damit umzugehen.
Es ist nicht dokumentiert was man als "int i" übergeben muss.

 

[byte]

1. Es ist sinnvoll, eine zusätzliche Methode zu definieren, die dann den (ersten) rekursiven Aufruf mit dem korrekten i startet.

2. Warum arbeitest Du mit einem String-Array, wenn Du Double-Werte sortieren willst? Deine Methode funktioniert spätestens dann nicht, wenn ich ein Array mit beliebigen Strings übergebe.