Question

这是我的程序

static void Main(string[] args)
        {

            int[] arrayToSort = new int[] { 5,4,9};
            BubbleSort bubbleSort = new BubbleSort();
            int [] SortedArray = bubbleSort.SortArray(arrayToSort);
            foreach (int i in SortedArray)
                Console.Write(i.ToString() + "," );
            Console.WriteLine("Number of Iterations {0}",   
                               bubbleSort.IterationsCounter);
            Console.ReadLine();    

        }

 public class BubbleSort
    {
        public int IterationsCounter;
        public int[] SortArray(int[] arrayToSort)
        {
            for(int i = 0;i<arrayToSort.Length-1;i++)
            {
                if(arrayToSort[i]>arrayToSort[i+1])
                {
                    int temp=arrayToSort[i];
                    arrayToSort[i]=arrayToSort[i+1];
                    arrayToSort[i+1]=temp;
                    //IterationsCounter++;  Update:Moved this line out of if condition)
                    SortArray(arrayToSort);
                }
            IterationsCounter++; //Moved counter here:

            }
            return arrayToSort;
    }

输出：

4,5,9 Number of Iterations:1

怎么可能是对的？我的意思是数组已经排序，但肯定有多次迭代。我希望这有O（N ^ 2）的运行时间，但有些东西在这里。我不算正确的迭代吗？

修改

好的，我意识到3个项目还不够，根据建议，我将计数器移出 if ，如果现在我将输入更改为

 5,4,9,2,3,1,17

迭代次数变为78。这更好（在某种意义上它应该很高），但它不够高。那么这意味着算法有O（logn）时间？我以为bubblesort是O（n ^ 2）？

谢谢

Answer 1

放IterationsCounter ++;在if循环之外计算迭代次数。到目前为止，代码只计算交换次数，因为它只有在交换时才会增加。

Answer 2

您正在计算交换操作的数量，而不是迭代次数。冒泡排序的平均运行时间为O（n ^ 2），并不意味着每个冒泡排序必须进行如此多的迭代。例如，如果您对已排序的数组进行冒泡排序，并在整个数组传递后进行交换时设置标志。如果没有进行交换，那么应该清楚该数组已经按顺序排列，因为不需要交换两个元素。在这种情况下，冒泡排序应该结束。它似乎比quicksort更快，其平均时间复杂度为O（n log n），因为在这种情况下修改的冒泡排序的性能是O（N）。但是你必须考虑到一般情况。

我的BubbleSort类没有正确计算迭代次数

2 个答案: