我在为copyconstructor
课程创建doublependulum
时遇到问题。我在doubelpendulum.h
文件中创建了一个文件,但它似乎无法在doublependulum.cpp
中识别出来。我一直收到这个错误:
错误C2512:'Pendulum':没有合适的默认构造函数
我不明白为什么我必须添加一个合适的构造函数或为什么我定义的那个(规则103-104)不正确。如果是这样,有人可以告诉我为什么它是必要的或者我的有什么问题?
#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
#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;
}
#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
#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;
}
#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':递归所有控制路径,函数会导致运行时堆栈溢出
我一直在寻找好几个小时,希望有人可以向我解释我的构造函数有什么问题。
答案 0 :(得分:3)
你定义了一个构造函数Pendulum :: Pendulum(const double,const double,double,double),因此编译器不会定义默认构造函数。
答案 1 :(得分:3)
第一个问题是,您正在尝试默认构建类PendUp
中类型为PendDown
的成员Pendulum
和DoublePendulum
,而您的班级Pendulum
没有默认构造函数。你可以默认构造一个没有默认构造函数的类。为类Pendulum
提供默认构造函数或停止尝试在DoublePendulum
中默认构造它。哪个是正确的解决方案取决于您的意图。只有你知道你的意图。
正如在其他答案中已经注意到的那样,你的类Pendulum
没有默认构造函数的原因是在C ++中为类提供任何用户定义的构造函数,编译器立即停止为该类提供隐式默认构造函数。在您的情况下,您在类Pendulum
中显式声明了两个构造函数,它们会自动禁用其默认构造函数。要重新启用它,您必须明确声明它。
第二个问题源于类Pendulum
中的复制构造函数和复制赋值运算符的设计。您将复制构造函数实现为对复制赋值运算符的简单调用。但是,您的复制赋值运算符接受其值的值。这意味着为了准备参数,必须通过调用copy-constructor来复制构造实际参数。这是你的无限递归。即为了完成复制构造函数,您必须调用复制赋值,但为了调用复制赋值,您必须调用复制构造函数。
首先,为什么要在课程Pendulum
中进行复制作业,以便按值接受其参数?实际上,复制赋值运算符的当前实现似乎没有任何意义:它没有向左侧赋值。使你的副本赋值通过const引用接受它的参数,并且无限递归问题应该消失。另外,让您的副本分配实际将事物分配到其左侧(即*this
对象)。
其次,通过调用复制赋值运算符来实现复制构造函数的想法被打破了。赋值期望一个对象处于完全构造状态,而在构造函数内部,该对象尚未构造。更好的想法是以相反的方式做到这一点:通过使用着名的 copy-and-swap 习语来通过复制构造实现复制分配。
答案 2 :(得分:2)
这两个函数是相互递归的:
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)
答案 3 :(得分:0)
赋值运算符和复制构造函数存在问题。看来你的班级不需要这些,所以只需删除它们就可以解决问题,而且似乎是最干净的解决方案。