Question

我正在做一个combsort算法作为类赋值，每当我运行它时它就会循环。我不确定我做错了什么（我在C ++中使用它，但那是一段时间以前，这些技能并没有像我希望的那样翻译）。我一直在研究它一个半小时，并通过电子邮件发送给一些朋友，但遗憾的是没有人有任何想法。我想我只需要有更多经验的人来告诉我我搞砸了什么。谢谢！

import java.util.ArrayList;

public class CombSort {
    public CombSort()
    {
        super();
    }

    public ArrayList<Integer> combSort(ArrayList<Integer> sortMe)
    {
        int swap;
        int size = sortMe.size();
        int gap = size;
        boolean swapped = false;

        while ((gap > 1) || swapped)
        {
            if (gap > 1)
            {
                gap = (int) ((size)*((double)gap / 1.247330950103979));
            }

            swapped = false;

            for (int i = 0; gap + i < size; ++i)
            {
                if (sortMe.get(i) - sortMe.get(i + gap) > 0)
                {
                    swap = sortMe.get(i);
                    sortMe.set(i, sortMe.get(i + gap));
                    sortMe.set(i + gap, swap);
                    swapped = true;
                }
            }
        }
        return sortMe;
    }

    public static void main(String[] args)
    {
        ArrayList<Integer> randomArrayList = new ArrayList<Integer>(7);
        randomArrayList.add(5);
        randomArrayList.add(7);
        randomArrayList.add(2);
        randomArrayList.add(6);
        randomArrayList.add(8);
        randomArrayList.add(2);
        randomArrayList.add(9);
        CombSort combSorter = new CombSort();
        System.out.println(combSorter.combSort(randomArrayList).toString());
    }
}

Answer 1

gap循环中的每次迭代都会使while值变大，因此它无限循环的原因。

您应该只将间距乘以梳子因子，而不是大小。

(int) ((size)*((double)gap / 1.247330950103979));应该是

(int) ((double)gap / 1.247330950103979);

Java梳子排序陷入无限循环

1 个答案: