我的 quickSort() 程序不能正常工作，为什么？ [关闭]答案

【问题标题】：My quickSort() program cannot work well, why? [closed]我的 quickSort() 程序不能正常工作，为什么？ [关闭]
【发布时间】：2015-11-15 07:20:18
【问题描述】：

如果你能帮助我，真的很感谢！下面是我对数字向量（双精度型）进行排序的代码。 vec中的元素个数是200,000。

程序在几秒钟后停在绿线处。错误信息是

“线程 1：EXC_BAD_ACCESS（代码 = 2，地址=0x7fff5f3fffd8）”。

就像图片一样：

/*This is block to call quickSort*/
void VectorDoubleSort:: sort(std::vector<double>& vec){
      int start = 0;
      int end = vec.size() - 1;
      quickSort(vec, start, end);
}




/*This block to fulfill the quickSort method*/
    void VectorDoubleSort:: quickSort(std::vector<double>& vec, int start, int end)
    {
        int pivot = (start + end) / 2;
        int smallIndex = start;

        if (start < end) {
            double tempA = vec[start];
            vec[pivot] = tempA;

            for (int i = start + 1; i <= end; i++) {
                if (vec[i] <= vec[start]) {
                    smallIndex += 1;
                    double tempB = vec[i];
                    vec[i] = vec[smallIndex];
                    vec[smallIndex] = tempB;
                }
            }

            double tempC = vec[start];
            vec[start] = vec[smallIndex];
            vec[smallIndex] = tempC;

            pivot = smallIndex;

            quickSort(vec, start, pivot - 1);
            quickSort(vec, pivot + 1, end);
        }
    }

【问题讨论】：

不能正常工作是什么意思？在问题中提供信息，而不是期望我们从图片中收集信息。
进程停止在绿线，我找不到原因。真的很郁闷。
通常您应该在问题中包含文本错误消息，而不是显示屏幕截图。
错误信息是“线程 1：EXC_BAD_ACCESS (code = 2, address=0x7fff5f3fffd8)”。程序停在绿线处。
@WBLee 使用Insturments中的Zombie工具找出问题的原因，更多细节：stackoverflow.com/a/11717981/2128327

标签： c++ algorithm sorting debugging quicksort

【解决方案1】：

首先，这里是错误的：

int pivot = (start + end) / 2;
double tempA = vec[start];
vec[pivot] = tempA;

你丢失了vec[pivot] 的内容，之后在这里if (vec[i] <= vec[start]) 你按vec[start] 的值对你的向量排序，然后pivot 是开始，那么到底为什么，您将枢轴设置为(start + end) / 2;？当你在比较中使用 NO 保存值，但从vec[start]重新读取，可以通过向量中的swap操作改变。毕竟你订购了start，但是你的索引在订购周期for (int i = start + 1; i <= end; i++)中增加了，所以在迭代之后你有（当然，如果以前的错误已修复）： {pivot, sub array <= pivot, sub array > pivot}，所以你需要而不是只是 swap 如果从 end 到 start + 1 你需要移动 sub array <= pivot 到 start 并插入 pivot , 因此，如果尽可能多地重用您的代码，那么正确的解决方案是：

#include <algorithm>
#include <cassert>
#include <iostream>
#include <iterator>
#include <vector>

namespace VectorDoubleSort {
void sort(std::vector<double> &vec);
void quickSort(std::vector<double> &vec, int start, int end);
}

void VectorDoubleSort::sort(std::vector<double> &vec) {
  if (vec.empty())
    return;
  int start = 0;
  int end = vec.size() - 1;
  quickSort(vec, start, end);
}

/*This block to fulfill the quickSort method*/
void VectorDoubleSort::quickSort(std::vector<double> &vec, int start, int end) {
  if (start < end) {
    const double pivotVal = vec[start];
    int smallIndex = end;

    for (int i = end; i > start; --i) {
      if (vec[i] >= pivotVal) {
        std::swap(vec[i], vec[smallIndex]);
        --smallIndex;
      }
    }
    std::swap(vec[start], vec[smallIndex]);

    const int pivot = smallIndex;
    quickSort(vec, start, pivot - 1);
    quickSort(vec, pivot + 1, end);
  }
}

bool test(const std::vector<double> &arr) {
  std::vector<double> good = arr;
  std::sort(good.begin(), good.end());
  std::vector<double> my = arr;
  VectorDoubleSort::sort(my);
  std::copy(my.begin(), my.end(),
            std::ostream_iterator<double>(std::cout, ", "));
  std::cout << "\n";
  return my == good;
}

int main() {
  assert(test({}));
  assert(test({3., 1., 2., 17., 18., -1., -5.}));
  assert(test({3., 1., 2.}));
  assert(test({3., 2.}));
  assert(test({3.}));
  assert(test({3., 532523, 5235, 05325, 535, 5738157, 535, 501, 9780, 14}));
}

注意：我使用c++11(initializer_list)，但仅用于测试，因此无需测试您可以重用c++98或c++03

【讨论】：