Question

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

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

我在OOP中有点生疏，而且是C ++的新手。

如果我应该包含更多代码，请发布我将附加来自主要类和类的代码，我无法进行转换。该计划尚未完成。

Main.cpp的

#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;
};

Answer 1

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

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

Answer 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

Answer 3

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

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

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

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

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

Author author("John", "Doe");

如何正确地将变量传递给非默认构造函数？

3 个答案: