1

我正在用 C++ 编写一个家庭作业程序。我在将值从一个初始化的构造函数传递到另一个时遇到了一点问题，它说：

error C2664: 'Book::Book(std::string,Author *,Publisher *,double)' : cannot convert parameter 2 from 'Author' to 'Author *'

我对 OOP 和 C++ 新手有点生疏。

如果我应该包含更多代码，请发布我将附加来自主类和我无法进行转换的类的代码。该程序还没有完成。

主文件

#include <iostream>
using namespace std;

#include "Book.h"

void main()
{
    cout << "Book 1" << endl;

    Author *pAuthor = new Author("John", "Doe");
    Publisher *pPublisher = new Publisher("Wrox", "10475 Crosspoint Blvd.", "Indianapolis");
    Book *book = new Book("Memory Management", *pAuthor, *pPublisher, 49.99);

    cout << "Book 2" << endl;
    int i;
    cin >> i;
};
4

3 回答 3

7

您不应该取消引用 Author 和 Publisher 指针，因为 Book 构造函数需要指针，而不是传入的值。

但是，在处理所有这些指针时，您可能会遇到很多内存管理问题。这对于一个小程序来说是可以的，因为一旦程序退出，所有内存都会被释放，但是养成正确管理内存的习惯很重要。如果您想避免按值传递，也许您应该阅读 C++ 中的引用。

于 2012-09-02T18:22:52.177 回答
2

Book 的构造函数需要指针，但你给它取消引用的值。

                                   wants pointer       wants pointer
                                    ^                  ^                                        
                                    |                  |
                                    |                  |
Book::Book(string title, Author *pAuthor, Publisher *pPublisher, double price)
{
    title = title;
    price = price;
}

但在 main() 中，

Book *book = new Book("Memory Management", *pAuthor, *pPublisher, 49.99);
                                             |          |
                                             |          |
                                            \/         \/
                                        Dereferenced, so its a value copied
于 2012-09-02T18:23:44.370 回答
2

如果您只是设置值，您可能应该将参数作为 const 引用接收：

Author::Author(const string& first, const string& last)
{
    firstName = first;
    lastName = last;
}

然后以正常方式传递它们：

Author *pAuthor = new Author("John", "Doe");

或者，如果您仅在该方法中使用 author 对象，则实际上不需要使用new，只需将其放在堆栈上即可，这将避免程序中潜在的内存泄漏：

Author author("John", "Doe");
于 2012-09-02T18:23:54.133 回答