Java comb排序陷入无限循环

Question

我正在做一个组合排序算法作为一个类赋值，当我运行它的时候它就会循环。我不确定我做错了什么(我让它在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);