Question

class DSArray {

private:
    int size;
    string *dynamicArray;

public:
    DSArray();
    DSArray(int size, string []);
    ~DSArray();
    int getSize() const;
    void addEntry(string newString);
    bool deleteEntry(string find);
    string getEntry(const int index) const;
    void operator = (const DSArray& obj);
    string operator [] (int index);
    friend ostream& operator << (ostream &out, const DSArray& array);


};

DSArray::DSArray(){
    size = 0;
    dynamicArray = NULL;
}

DSArray::DSArray(int size, string []){
    dynamicArray = new string[size];
    for (int i = 0; i < size; i++){
        dynamicArray[i] = getEntry(i);
    }
    size = getSize();
}

DSArray::~DSArray(){
    delete[] dynamicArray;
}

int DSArray::getSize() const{
    return size;
}


void DSArray::addEntry(string newString){
    string *tempPtr = dynamicArray;
    dynamicArray = new string[size + 1];
    dynamicArray[size].assign(newString);
    for(int i = 0; i < size; i++){
        dynamicArray[i] = tempPtr[i];
    }
    if(size > 0){
        delete[] tempPtr;
    }
    size++;
}

ostream& operator << (ostream &out, const DSArray& array){
    for (int i = 0; i < array.getSize(); i++)
        out << array.dynamicArray[i];
    return out;
}



bool DSArray::deleteEntry(string toDelete)
{
    int index;
    for(index = 0; index < size; index++)
    {
        if(dynamicArray[index] == toDelete) break;
    }

    if(index == size) return false;

    string *newArray = new string[size--];

    int i;
    for(i = 0; i < index; i++) newArray[i] = dynamicArray[i];
    for(int k = index + 1; k <= size; k++, i++) newArray[i] = dynamicArray[k];

    delete [] dynamicArray;
    dynamicArray = newArray;

    return true;
}

string DSArray::getEntry(const int index) const{
    static string emptyStr = "";
    if(index > size){
        return emptyStr;
    }
    else{
        return dynamicArray[index];
    }

}


void DSArray::operator = (const DSArray& obj){
    DSArray temp(obj);
    if(this->size != 0)
        delete[] this->dynamicArray;
    this->size = obj.getSize();
    this->dynamicArray = new string [this->size];
    for(int i = 0; i < this->size; i++)
        this->dynamicArray[i] = obj.getEntry(i);
}

string DSArray::operator [] (int index){
    string emptyString = "";
    if (index >= size){
        return emptyString;
    }
    return dynamicArray[index];
}

int main()

{
    DSArray foods;
    string x[] = {"Burrito", "Sushi", "Pizza"};
    DSArray A(3, x);
    cout << A;
    A[3] = "Spaghetti";
    foods.addEntry("Steak");
    foods.deleteEntry("Sushi");




    return 0;

}

我试图在main中测试我的功能，但它实际上并没有在控制台中显示任何内容。我被告知我的构造函数没有正确设置大小，因为调用getSize会返回尚未初始化的对象大小。我究竟如何解决这个问题？也有人知道getEntry函数是否返回null而不是空字符串？我的任务说它应该返回一个null，但是，当我确实返回一个null时，它会给我一个运行时错误。任何帮助表示赞赏。谢谢！

Answer 1

您没有在构造函数中初始化成员变量size。使用：

DSArray::DSArray(int size, string input[]) : size(size) { ... }

此外，该行

    dynamicArray[i] = getEntry(i);

没有做任何有用的事情。它本质上是自我分配。

您可以通过为输入参数使用不同的名称来避免难以遵循的代码。另外，将输入字符串数组复制到成员变量。

DSArray::DSArray(int sizeIn, string input[]) : size(sizeIn) {
    dynamicArray = new string[size];
    for (int i = 0; i < size; i++){
        dynamicArray[i] = input[i];
    }
}

动态字符串数组构造函数和null返回

1 个答案: