c++ - (C++) 在构造函数中初始化的变量被放在一个无关数组的末尾。

Question

我是 C++ 的新手，我花了几个小时试图确定为什么 main.cpp 中的“execute.ListAllBooks()”函数似乎包含分配给 libraryName 变量的字符串(声明为在 Library.h 中并在 Library.cpp 中的构造函数中初始化)在名为“BookList”的数组中。数组的最后一个元素列为“MyLibrary”，而我希望它是一个空字符串。我还有一个问题是为什么数组中似乎有 11 个元素。我以前用 Java 编写代码，考虑到我用这一行“public:std::string BookList[10]”初始化数组，我希望有十个元素。任何帮助，将不胜感激。谢谢!

主要.cpp

#include <iostream>
#include "Library.h"   

int main() {
    Library execute("MyLibrary");
    execute.AddBook("Book 1");
    execute.AddBook("Book 2");
    execute.AddBook("Book 3");
    execute.AddBook("Book 4");
    execute.AddBook("Book 5");
    execute.AddBook("Book 6");
    execute.AddBook("Book 7");
    execute.AddBook("Book 8");
    execute.AddBook("Book 9");
    execute.AddBook("Book 10");
    execute.ListAllBooks();
    return 0;  
}

库.cpp

#include <iostream>
#include "Library.h"

using namespace std;

Library::Library(const string &name) {
    libraryName = name;
    length = 0;
}

bool Library::AddBook(const string &name) {
    counter = 0;
    int arraySize = sizeof(BookList)/sizeof(*BookList);
    while(counter < arraySize) {
        if(BookList[counter] == name) {
            return false;
        }
        counter++;
    }
    if(length >= arraySize) {
        return false;
    }
    else {
        length++;
        BookList[length - 1] = name;
        return true;
    }
}

bool Library::RemoveBook(const std::string &name) {
    counter = 0;
    while(counter < sizeof(BookList)/sizeof(*BookList)) {
        if(BookList[counter] == name) {
            BookList[counter] = "";
            length--;
            while(counter < length - 1) {
                BookList[counter] = BookList[counter + 1];
                BookList[counter + 1] = "";
                counter++;
            }
            return true;
        }
        counter++;
    }
    return false;
}

void Library::ListAllBooks() const {
    int length = sizeof(BookList);
    int counter = 0;
    while(counter < length) {
        cout << BookList[counter] + "," << endl;
        counter++;
    }
}

库.h

#include <iostream>

#ifndef ASS1_LIBRARY_H
#define ASS1_LIBRARY_H

class Library {

    public: std::string BookList[10];
    private: std::string libraryName;
    private: int length;
    private: int counter;

    public: explicit Library(const std::string &name);

    // Add a new book,
    // return true for success, false if book already in library
    bool AddBook(const std::string &name);

    // Remove a book
    // return true for success, false if book not in library
    bool RemoveBook(const std::string &name);

    // List all books in library
    public: void ListAllBooks() const;

    // Return true if book in library, false otherwise
    bool IsInLibrary(const std::string &name) const;

};

// friend function
std::ostream &operator<<(std::ostream &out, const Library &lib);

#endif //ASS1_LIBRARY_H

Answer 1

行 int length = sizeof(BookList); 不正确。

sizeof返回对象存储在内存中所需的字节数。 Libary 的大小恰好是 11。当访问 BookList[11] 时，您正在溢出数组并读取内存中的下一个字符串 libraryName 因为它们在您的类中定义的顺序。

您可以将类定义更改为:

 class Library {
      public: std::string BookList[10];
      public: std::string hello "Hello";
      ...

并且访问 BookList[11] 将导致 Hello。

看起来您已经在 AddBook 中计算了正确的 length，所以一起删除 int length = sizeof(BookList); 行应该可以解决您的问题。

在某种程度上相关的注释中，您可能会考虑使用 std::array以帮助防止将来出现此类错误。