2016-04-01 11 views
-1

Я написал код для наиболее значимой сортировки radix, но я продолжаю получать ошибку IndexBoundsException в строке: arr [i] = arr1.get (i) ;. И теперь я не знаю, что не так, потому что логика кажется мне правдой.Самая значимая цифра Radix Sort в Java

public static int[] msdRadixSort(int[] arr) { 
    if (arr == null) { 
     throw new IllegalArgumentException("The array is null!"); 
    } 
    //search through to find msd 
    int temp = arr[0]; 
    for (int i = 0; i < arr.length; i++) { 

     if (Math.abs(temp) <= Math.abs(arr[i])) { 
      temp = arr[i]; 
     } 
    } 
    //find the length of the msd 
    int k = 1; 
    while (!(temp/10 == 0)) { 
     temp = temp/10; 
     k++; 
    } 
    ArrayList<Integer> arr1 = new ArrayList<Integer>(arr.length); 
    for (int i = 0; i < arr.length; i++) { 
     arr1.add(arr[i]); 
    } 
    msdRadixSort(arr1, k); 
    System.out.println(arr1); 
    for (int i = 0; i < arr.length; i++) { 
     arr[i] = arr1.get(i); 
    } 
    return arr; 
} 
+0

Где вы получаете эту ошибку? Покажите трассировку стека и покажите нам строку, вызывающую ее (так как мы не можем видеть номера строк). Кроме того, вы пробовали * отлаживать * код? – Andreas

+0

У меня есть. Я думаю, что ошибка возникает во вложенном цикле: for (int j = 0; j user1979029

ответ

1

вложенная петля неверна она должна быть

for (int j = 0; j < bucket.length; j++) { 
    for (int i = 0; i < bucket[j].size(); i++) { 
     arr.add(bucket[j].get(i)); 
    } 
} 
+0

Это не устранило ошибку. Я не могу понять, что не так с моим кодом. Я думаю, что ведра попадают в правильное ведро. – user1979029

+0

Какая ошибка вы получаете, потому что когда я запускал ее с моими тестовыми значениями, я не получал IndexOutOfBoundsException – minecraftwarlock

+0

Я получаю ошибку в рекурсивном методе в bucket [temp] .add (i) ;, msdRadixSort (bucket [i ], k - 1); и на нереверсивном: msdRadixSort (arr1, k); – user1979029