Question

我是C ++编程的新手，最近写了一个mergesort方法来排序一些数组。对于我的个人测试，它适用于整数和双精度。但是当我尝试对字符串进行排序时，它会给我一个“语义问题”错误，我很困惑。完整的代码是：

#include <iostream>
#include <string>
using namespace std;

template<typename T>
class Sorting{
public:
static void merge(T* a, int left, int mid, int right){
    int i=left; int j=mid+1; int k=0;
    T t[right-left+1]; //****************ERROR LINE

    for(;i<=mid && j<=right;k++){
        if(*(a+i)<=*(a+j)){
            t[k]=a[i];
            i++;
        }
        else{
            t[k]=a[j];
            j++;
        }
    }

    for(;i<=mid;i++,k++) t[k]=a[i];

    for(;j<=right;j++,k++) t[k]=a[j];

    for(i=0;i<k;i++) a[left+i]=t[i];
}

//Mergesort top-level function. Left is starting index, right is ending index
static void mergesort(T* a, int left, int right){
    if(left>=right) return;
    int mid=left+((right-left)>>1);
    mergesort(a, left, mid);
    mergesort(a, mid+1, right);
    merge(a, left, mid, right);
}
};


int main(){
const int len=5; 
string ss[len]={
    "Yep",
    "Nope",
    "5",
    "2.5",
    "Stackoverflow"
};
double ar[len]={4.2, 3, 5.6, -15, 0};

Sorting<double>::mergesort(ar, 0, 4); for(int i=0; i<len;i++) cout<<ar[i]<<endl;
Sorting<string>::mergesort(ss, 0, 4); for(int i=0; i<len;i++) cout<<ss[i]<<endl;
return 0;
}

我在“// * * ERROR LINE”中遇到语义错误，如：

Variable length array of non-POD element type 'std::__1::basic_string<char>'

这个错误在谈论什么？我该如何修改我的代码？

Answer 1

在错误消息中， POD 指的是plain old data type

您可以使用std::vector个，即

   std::vector<T> t;
   t.resize (right-left+1);

您还可以使t指针数组（即T* t[right-left+1];并相应地更新代码）。

^{BTW，您使用的是可变长度数组，这是其他一些编译器无法提供的GCC extension。}

但是C ++标准库中提供了排序。您需要#include<algorithm>并在标准C ++容器上使用std::sort。

Answer 2

你有一个可变长度数组：

T t[right-left+1];

这是您的特定编译器支持的扩展，而不是C ++标准的一部分。它不适用于像std::string这样的复杂对象类型 - 因此出现错误消息。您可以将其替换为vector：

std::vector<T> t(right - left + 1);

Basile使用指针的想法虽然更好 - 复制std::string对象非常重要（即内存密集，速度慢）...你只想跟踪要移动的a[]个元素，而不是整理它们的副本然后将它们复制回来。

用于字符串的Mergesort

2 个答案: