复制构造函数没有合适的默认构造函数
Posted
技术标签:
【中文标题】复制构造函数没有合适的默认构造函数【英文标题】:Copy Constructor has no appropriate default constructor 【发布时间】:2012-05-08 17:10:58 【问题描述】:我在为我的 doublependulum
班级创建 copyconstructor
时遇到问题。我在我的doubelpendulum.h
文件中创建了一个,但它似乎在doublependulum.cpp
中无法识别它。我不断收到此错误:
错误 C2512: 'Pendulum' : 没有合适的默认构造函数可用
我不明白为什么我必须添加适当的构造函数,或者为什么我定义的构造函数(规则 103-104)不正确。如果是这样,谁能告诉我为什么它是必要的或我的有什么问题?
钟摆.h
#include<string>
using std::string;
#ifndef pendulum_H
#define pendulum_H
class Pendulum
public:
Pendulum(const double,const double,double,double);
//check function
const double check(const double, const string) const; //used for (L,M)
// getfuncties
double getL() const;
double getM() const;
double getTheta();
double getOmega();
//overload operator
Pendulum operator+ (Pendulum);
Pendulum operator*(const double a);
Pendulum operator=(Pendulum);
//copy constructor
Pendulum(const Pendulum& );
private:
double L_,M_,Theta_,Omega_;
;
#endif
钟摆.cpp
#include "pendulum.h"
#include <iostream>
#include <string>
using namespace::std;
//constructor
Pendulum::Pendulum(const double L, const double M, double Theta, double Omega)
:L_(check(L,"L")),M_(check(M,"M"))
//check functie
const double Pendulum::check(const double d, const string str) const
if (d<0.)
cout << "ERROR: " << str << " (" << d << ") has to be positive" << endl;
exit(EXIT_FAILURE);
return d;
//getfuncties
double Pendulum::getM() const
return M_;
double Pendulum::getL() const
return L_;
double Pendulum::getTheta()
return Theta_;
double Pendulum::getOmega()
return Omega_;
//overloading operators
Pendulum Pendulum::operator+ (Pendulum param)
return Pendulum(L_, M_, Theta_+param.Theta_, Omega_+param.Omega_);
Pendulum Pendulum:: operator* (const double param)
return Pendulum(L_, M_, param*Theta_, param*Omega_);
Pendulum Pendulum::operator=(Pendulum param)
return Pendulum(L_, M_, param.Theta_, param.Omega_);
//copy constructor
Pendulum::Pendulum(const Pendulum& Object)
*this=Object;
双摆.h
#ifndef doublependulum_H
#define doublependulum_H
#include "pendulum.h"
class DoublePendulum
public:
//constructor
DoublePendulum(Pendulum ,Pendulum );
//getfuncties
Pendulum getUp();
Pendulum getDown();
//Overload operators
DoublePendulum operator+ (DoublePendulum);
DoublePendulum operator*(const double a);
DoublePendulum &operator=(const DoublePendulum &);
//copy constructor
DoublePendulum(const DoublePendulum& );
private:
Pendulum PendUp;
Pendulum PendDown;
;
#endif
双摆.cpp
#include "doublependulum.h"
//constructor
DoublePendulum::DoublePendulum(Pendulum Up,Pendulum Down)
:PendUp(Up),PendDown(Down)
//getfunctions
Pendulum DoublePendulum::getUp()
return PendUp;
Pendulum DoublePendulum::getDown()
return PendDown;
//Overload operators
DoublePendulum DoublePendulum::operator+ (DoublePendulum param)
return DoublePendulum(PendUp + param.PendUp,PendDown + param.PendDown);
DoublePendulum DoublePendulum::operator* (const double param)
return DoublePendulum(PendUp*param,PendDown*param);
DoublePendulum& DoublePendulum::operator= (const DoublePendulum& param)
return *this; // assign to members of this object
//Copy constructor
DoublePendulum::DoublePendulum(const DoublePendulum& Object)
*this=Object;
Main.cpp
#include "pendulum.h"
#include "doublependulum.h"
int main()
return 0;
在我添加规则之前,一切都会编译:143-147 我收到此错误:
error C2512: 'Pendulum' : no appropriate default constructor available
error C2512: 'Pendulum' : no appropriate default constructor available
还有一个令人担忧的警告
正在生成代码... c:\users\niels\documents\visual studio 2010\projects\examenopdracht\examenopdracht\pendulum.cpp(55): 警告 C4717: 'Pendulum::Pendulum' : 在所有控制路径上递归,函数会导致运行时堆栈溢出
我一直在寻找几个小时,希望有人能向我解释我的构造函数出了什么问题。
【问题讨论】:
对不起,链接错误:pastebin.com/B4UdXw53 请在问题中发布简化代码以说明问题。您收到的错误意味着您在未定义默认构造函数时默认在某处初始化Pendulum
。
请不要使用 pastebin 链接。当该链接消失时,您的问题对未来的访问者变得毫无用处。正如 AJG85 所说,请包含简化代码来演示该问题。我已插入链接的内容,但请将其编辑为具有代表性的最小示例。
【参考方案1】:
第一个问题是您试图在您的类DoublePendulum
中默认构造类型为Pendulum
的成员PendUp
和PendDown
,而您的类Pendulum
没有默认构造函数。您可以默认构造一个没有默认构造函数的类。为类Pendulum
提供默认构造函数,或者停止尝试在DoublePendulum
中默认构造它。哪个是正确的解决方案取决于您的意图。只有你知道你的意图是什么。
正如在其他答案中所指出的,您的类 Pendulum
没有默认构造函数的原因是,在 C++ 中,一旦您为类提供 any 用户定义的构造函数,编译器会立即停止为该类提供隐式默认构造函数。在您的情况下,您在类Pendulum
中显式声明了两个构造函数,这会自动禁用其默认构造函数。要重新启用它,您必须明确声明它。
第二个问题根源于Pendulum
类中copy-constructor和copy-assignment操作符的设计。您将复制构造函数实现为对复制赋值运算符的简单调用。但是,您的复制赋值运算符接受其参数按值。这意味着为了准备参数,必须通过调用复制构造函数来复制构造实际参数。这是你的无限递归。 IE。为了完成复制构造函数,您必须调用复制分配,但为了调用复制分配,您必须调用复制构造函数。
首先,您为什么要在Pendulum
类中进行复制分配以接受其参数按值?事实上,复制赋值运算符的当前实现似乎根本没有任何意义:它没有将任何东西分配给它的左侧。通过 const 引用使您的复制分配接受其参数,并且无限递归问题应该消失。此外,将您的复制分配实际分配 东西到它的左侧(即*this
对象)。
其次,通过调用复制赋值运算符来实现复制构造函数的想法被打破了。赋值期望对象处于完全构造状态,而在构造函数内部,对象尚未构造。一个更好的想法是反过来做:通过使用众所周知的 copy-and-swap 成语通过复制构造来实现复制分配。
【讨论】:
您好,首先,非常感谢您的澄清。几个月前我还没有听说过 c++,所以我的知识相当少。但据我了解您在说什么,我做了以下事情: 仅在复制构造函数中调用复制赋值运算符令人困惑,因此导致无限递归。作为该问题的解决方案,我执行了以下操作:我将 pendulum 类中的复制构造函数更改为 Pendulum::Pendulum(const Pendulum& Object) : M_(Object.M_), L_(Object.L_), Theta_(Object.Theta_ ), Omega_(Object.Omega_) @ComputerSaysNo:太好了。这将使您的复制构造函数工作,这将消除无限递归。但是,这并不能解决复制赋值运算符的问题。原始形式的复制赋值运算符不会将任何内容分配到其左侧。为什么?这是你的意图吗? 我想我找到了解决这个问题的方法。但是,我对使用“this 指针”有一些疑问 'Pendulum& Pendulum::operator=( const Pendulum & Pendulum2) (*this).Theta_=Pendulum2.Theta_; (*this).Omega_=Pendulum2.Omega_;返回*这个; ' 现在我真的在左侧分配了一些东西。每个 Omega_ 和 Theta_ 都将被替换。并且 M_ 和 L_ 将是相同的。这就是我想要的。【参考方案2】:你定义了一个构造函数 Pendulum::Pendulum(const double,const double,double,double) 所以编译器不会定义默认构造函数。
【讨论】:
【参考方案3】:这两个函数是相互递归的:
Pendulum Pendulum::operator=(Pendulum param)
return Pendulum(L_, M_, param.Theta_, param.Omega_);
//copy constructor
Pendulum::Pendulum(const Pendulum& Object)
*this=Object;
*this=Object
行调用operator=(Pendulum)
。但是,为了创建局部变量param
,调用会调用复制构造函数。然后复制构造函数调用operator=
等。
试试:
Pendulum& Pendulum::operator=(const Pendulum& param)
【讨论】:
Pendulum& Pendulum::operator=(const Pendulum& param)
会是一个更好的选择,这样他就可以在这里 retrn *this 。它将在a=b=c=d
等对象的链接过程中很有用
非常感谢,我改了:)。我明白为什么我的编程方式出错了。【参考方案4】:
赋值运算符和复制构造函数存在问题。您的班级似乎不需要这些,因此只需删除它们即可解决问题,并且似乎是最干净的解决方案。
【讨论】:
以上是关于复制构造函数没有合适的默认构造函数的主要内容,如果未能解决你的问题,请参考以下文章