对二维数组进行排序...发生异常 (C++)

Question

在解决问题时，需要对二维数组进行排序。我使用了归并排序，并设置了

#include <iostream>
#include <stack>
#include <queue>
#include <vector>

using namespace std;
int*sorted[1000]; 
void merge(int** list, int left, int mid, int right); 
void merge_sort(int** list, int left, int right);
int main(void) {
    int N;
    vector <int> L, H; 
    int l, h, max = 0, max_position, highest, highest_position = 0, area = 0;   cin >> N;
    int** info = (int**)malloc(sizeof(int*) * N); 

    for (int i = 0; i < N; i++) {
        info[i] = (int*)malloc(sizeof(int) * 2); 
        }
    }
    merge_sort(info, 0, N - 1); 


}
void merge(int** list, int left, int mid, int right) {
    int i, j, k, l; 
    i = left; j = mid + 1; k = left; 

    for (int s = 0; s < 7; s++) {
        for (int v = 0; v < 2; v++) {
            cout << list[s][v] << ' ';
        }
        cout << '\n'; 
    }


    while (i <= mid && j <= right) {
        if (list[i][1] <= list[j][1])   // exception occured 
        {
            sorted[k++] = list[i++]; 
        }
        else { 
            sorted[k++] = list[j++];
        }
    }
    if (i > mid) // 1번째 array를 다 쓴 경우 
    {
        for (l = j; l <= right; l++) {
            sorted[k++] = list[l];
        }
    }
    else {
        for (l = i; l <= mid; l++) {
            sorted[k++] = list[l]; 
        }
    }
    for (l = left; l <= right; l++) {
        list[l] = sorted[k]; //  I think problem is related with this line 
    }

}
void merge_sort(int** list, int left, int right) {
    int mid; 
    if (left < right) {
        mid = (left + right) / 2;
        merge_sort(list, left, mid);
        merge_sort(list, mid + 1, right);
        merge(list, left, mid, right); 
    }
}

我分配了排序时需要的排序数组作为整数指针数组，因为我认为我需要分配二维数组每一行的收件人。

但问题似乎表明我与指针有关的基本想法是错误的。有人可以帮我解决这个问题吗？

Answer 1

在您的情况下，它实际上并不是对二维数组进行排序，而只是对整数对进行排序。 您可以通过第二个元素来比较它们。另一件事，在这种情况下，您真的不想使用 malloc 或 new 。所以，让我们来看看你的解决方案，慢慢改进它。

1. 删除不必要的 malloc。

malloc 是一个剩余的 C 函数，（几乎）不应该在 C++ 代码中使用。在 C++ 中，您有 new 做类似的事情，但您也不应该这样做！只需使用std::vector，它会在后台分配，负责内存管理并为您释放它。 （注意，你忘了自己释放内存）。

// in main:
std::vector<std::vector<int>> info;
// set the first dimension, how many "rows" you need
info.resize(N);
// for each "row" change "column" size to 2
for (std::vector<int>& v : info) v.resize(2);

记住，通过引用传递向量以避免复制！

1. 用std::pair/struct替换第二维

如果您总是知道其中一个维度，很可能您只是在对数字对/元组或一些有意义的“对象”进行排序。

所以你把上面的代码替换为：

// in main:
std::vector<std::pair<int, int>> info;
// only one dimension, how many pairs of ints this vector holds
info.resize(N);

如果这些是代表某物的数字，例如一本书的 ID 和年龄，您可以将它们保存在一个结构中。

// below includes, in global scope:

struct BookInfo {
  int age;
  int id;
};

// in main
std::vector<BookInfo> info;
// only one dimension, how many book infos this vector holds
info.resize(N);

1. 使用标准库！

在标准库中，您有一个 std::sort 函数。它可以作为一个参数，你想如何比较元素 - 在你的情况下，通过对的第二个值。

因此，您可以通过首先创建一个比较两个元素的函数来对所有内容进行排序：

bool compareBookInfos(const BookInfo& b1, const BookInfo& b2) {
  // return true, if b1 should be before b2 in sorted array
  return b1.id < b2.id;
}

或对相同：

bool compareIntPairBy2nd(const std::pair<int, int>& p1, const std::pair<int, int>& p2) {
  // return true, if p1 should be before p2 in sorted array
  return p1.second < p2.second;
}

现在，调用std::sort 方法：

  std::sort(info.begin(), info.end(), compareBookInfos);

几行代码就搞定了！

如果是编写自己的排序函数的任务，std::vector of pairs/structs 仍应改进您的代码并帮助您调试问题，可能提供更好的诊断。