0

クラスの の作成に問題がありcopyconstructorます。doublependulumファイルに作成しましたdoubelpendulum.hが、 で認識されないようですdoublependulum.cpp。このエラーが発生し続けます:

エラー C2512: '振り子': 適切な既定のコンストラクターがありません

適切なコンストラクターを追加する必要がある理由、または定義したコンストラクター (ルール 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;
}

doublependulum.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

doublependulum.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;
}

メイン.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': すべての制御パスで再帰的です。ランタイム スタック オーバーフロー

私は何時間も探していましたが、コンストラクターの何が問題なのか誰かが私に説明してくれることを願っています。

4

4 に答える 4

3

コンストラクター Pendulum::Pendulum(const double,const double,double,double) を定義したため、コンパイラーはデフォルトのコンストラクターを定義しません。

于 2012-05-08T17:19:47.863 に答える
3

最初の問題は、クラスにデフォルトのコンストラクターがないのに、クラスのメンバーPendUpPendDown型をデフォルト構築しようとしているということです。デフォルトのコンストラクターを持たないクラスをデフォルトで構築できます。クラスのデフォルト コンストラクターを提供するか、. どちらが適切な解決策であるかは、意図によって異なります。あなたの意図が何であったかは、あなただけが知っています。PendulumDoublePendulumPendulumPendulumDoublePendulum

他の回答で指摘されているように、クラスPendulumにデフォルトのコンストラクターがない理由は、C++ では、クラスにユーザー定義のコンストラクターを提供すると、コンパイラーがそのクラスの暗黙的なデフォルト コンストラクターの提供をすぐに停止するためです。あなたの場合、クラスPendulumで2つのコンストラクターを明示的に宣言しました。これにより、デフォルトのコンストラクターが自動的に無効になりました。再度有効にするには、明示的に宣言する必要があります。


2 番目の問題は、クラスのコピー コンストラクターとコピー代入演算子の設計に根ざしていますPendulum。コピー コンストラクターは、コピー代入演算子の単純な呼び出しとして実装しました。ただし、コピー代入演算子はそのパラメーターを値で受け入れます。これは、パラメータを準備するために、実際の引数をコピー コンストラクタの呼び出しによってコピー構築する必要があることを意味します。これが無限再帰です。つまり、コピー コンストラクターを完了するには、コピー代入を呼び出す必要がありますが、コピー代入を呼び出すには、コピー コンストラクターを呼び出す必要があります。

まず、なぜPendulum引数を値で受け入れるようにクラスでコピー割り当てを行ったのですか? 実際、コピー代入演算子の現在の実装はまったく意味がないようです。左辺には何も代入しません。copy-assignment がその引数を const 参照によって受け入れるようにすると、無限再帰の問題は解決するはずです。また、実際にその左側 (つまりオブジェクト)に物を割り当てるようにコピー割り当てを行います。*this

第 2 に、コピー代入演算子を呼び出してコピー コンストラクターを実装するというアイデア自体が壊れています。代入は完全に構築された状態のオブジェクトを想定していますが、コンストラクター内ではオブジェクトはまだ構築されていません。よく知られたコピー アンド スワップイディオムを使用して、コピー構築を通じてコピー代入を実装することです。

于 2012-05-08T17:29:06.020 に答える
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、呼び出しはコピー コンストラクターを呼び出します。次に、コピー コンストラクターが inokesoperator=などを実行します。

試す:

Pendulum& Pendulum::operator=(const Pendulum& param)
于 2012-05-08T17:19:56.450 に答える
0

代入演算子とコピーコンストラクターには問題があります。あなたのクラスはこれらを必要としないので、それらを削除するだけで問題が解決し、最もクリーンな解決策のようです。

于 2012-05-08T17:27:29.717 に答える