1

我正在做一个combsort algorithim作为班级作业，每当我运行它时它都会循环。我不确定我做错了什么（我在 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());
    }
}
4

1 回答 1

4

随着循环内的每次迭代，您的gap价值会越来越大while，因此它是无限循环的。

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

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

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

于 2013-03-05T02:49:46.657 回答