linkedlist insertion

懵懂的女人 提交于 2020-01-06 18:37:50

问题


Can anyone tell me why Iam running off heap space when I try to run this code on a list of 2000 elements?

public static <T extends Comparable <? super T>> void insertionSort2(List<T> portion){
    int i = 0;
    int j = 0;
    T value; 
    //List <T> sorted = new LinkedList<T>();

    // goes through the list
    for (i = 1; i < portion.size(); i++) {

        // takes each value of the list
        value = (T) portion.remove(i);

        // the index j takes the value of I and checks the rest of the array
        // from the point i
        j = i - 1;

        while (j >= 0 && (portion.get(j).compareTo(value) >= 0)) {
            portion.add(j + 1, portion.get(j));

            j--;

        }
        // put the value in the correct location.
        portion.add(j + 1, value);
    }
}

回答1:


This seems to fix your method:

while (j >= 0 && (portion.get(j).compareTo(value) >= 0)) {
            portion.add(j, portion.remove(j));

Your code keeps adding elements instead of moving them.



来源:https://stackoverflow.com/questions/15375690/linkedlist-insertion

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!