c++ - 带核心转储的字符串数组(使用模板和运算符重载)

我正在尝试了解有关字符串数组和核心转储的信息。因此，我创建了一个带有一些运算符重载函数的类模板，这些函数可以添加和删除元素。当我在int，double和char特殊化上测试加法和减法重载方法时，该程序完美地添加和删除了元素(没有核心转储)；但是，当在字符串上尝试时，我得到了一个核心转储。我确保有#include <string>并使用std::string。谁能解释为什么这适用于int，double和char专用条件，但不适用于字符串？

template <class T> T Set<T>::iVal;
template <class T> T Set<T>::DELIM;

template <class T>

//
//default constructor
//
Set<T>::Set ( int s ){

        psize = ( s > 0 ? s: DEFAULTSIZE);
        //allocate an array of specified size
        set = new T[ psize ];

        if(!set) {
                //send an error is system cannot allocate memory
                cout << "Cannot Allocate Memory, exiting program... " << endl;
                exit (1);
        }

        for ( int i = 0; i < psize; i++){
                set[i] = iVal;
        }

        numOfElements = 0;
}

//
//custom constructor
//
template <class T>
Set<T>::Set( T array[] , int size, char name ){

        set = new T[size];

        psize = ( size > 0 ? size: DEFAULTSIZE);
        numOfElements = size;
        for ( int i = 0; i < psize; i++){
                set[i] = array[i];
        }

        if (!set){
                cout << "Cannot Allocate Memory, exiting program... " << endl;
                exit(1);
        }

        Set::name = name;
}

//
//subtraction operator that removes elements
//
template <class T>
bool Set<T>::operator-( T n ){
        bool status = false;
                for ( int i = 0; i < numOfElements; i++){
                        if ( set [i] == n ){
                                for ( int j = i; j < numOfElements; j++){
                                        set [j] = set[j + 1];
                                }

                                set[numOfElements - 1] = n;
                                --numOfElements;
                        }

                        status = true;
                }

                return (status);
}
//
//addition operator that adds elements
//
template <class T>
bool Set<T>::operator+( T n ){
        bool status = false;

        if ( !element(n) ){
                if ( psize == numOfElements ) {
                        T *ptr = new T[psize += 1];

                        for ( int i = 0; i < numOfElements; i++ )
                                ptr[i] = set[i];

                        delete [] set;

                        set = ptr;

                        psize += 1;

                        delete [] ptr;

                }

                set[numOfElements] = n;
                numOfElements += 1;

                status = true;
        }

        return status;

}
//In Driver
 //
        //Creating string arrays using the custom constructor to instantiate objects
        //
        string s1[4] = { "turtle" , "fish", "shark" , "lobster" };
        string s2[3] = { "turtle", "book", "pencil" };

        //
        //Testing string specialization
        //
        cout << "\nTesting strings.\n" << endl;
        Set <string> string1 ( s1, 4, 'A' );
        Set <string> string2 ( s2, 3, 'B' );
        //setting delimiter
        Set <string>::setDELIM ("stop");
        //
        //testing the addition and subtraction of elements
        //CORE DUMP HAPPENS HERE
        string1 - "shark";
        cout << "\nAfter removing an element from Set A, Set A = " << string1 << endl;
        string1 + "dolphin";
        cout << "\nAfter adding an element to Set A, Set A = " << string1 << endl;

最佳答案

我看到了两个都是不确定行为的问题。它可以与其他数据类型一起使用的事实可能只是由于运气好。

首先，让我们看一下-方法:

if ( set [i] == n ){
    for ( int j = i; j < numOfElements; j++){
        set [j] = set[j + 1];  // <-- Access past last element in array
    }

    set[numOfElements - 1] = n;
    --numOfElements;
}

上面的内循环应该测试j < numOfElements - 1。这样，set[j+1]不会查看数组末尾的元素(如果numOfElements恰好与分配的大小相同)。

现在让我们看一下+方法:

T *ptr = new T[psize += 1];
for ( int i = 0; i < numOfElements; i++ )
        ptr[i] = set[i];
delete [] set;
set = ptr;
psize += 1;
delete [] ptr;  // <-- BOOM!

在上面，您刚刚删除了set，然后将其替换为ptr指向的新内存。但是随后您删除了该内存。如果删除标记的行，这应该起作用。

关于c++ - 带核心转储的字符串数组(使用模板和运算符重载)，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/34012845/