Java梳排序陷入无限循环

Question

我正在做一个组合排序算法作为一个类作业，每当我运行它时它都会循环。我不确定我做错了什么(我让它在 C++ 中工作，但那是不久前的事，而且这些技能并没有像我希望的那样转化)。我已经仔细研究了一个半小时，并给一些 friend 发了电子邮件，但不幸的是，没有人有任何想法。我想我只需要有更多经验的人来告诉我我搞砸了什么。谢谢!

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);