11

编辑：在底部发布了固定代码。感谢大家的帮助！

我只是在学习 c++ 并且在继承方面遇到了麻烦。我已经搜索和搜索并尝试了任何我能做的事情，但我无法在保留我希望它拥有的功能的同时编译这段代码。

我觉得我犯了一个愚蠢的错误，或者我只是错过了一些重要的概念，但如果有人可以看看它，我会非常感激！

如果我注释掉 StarSystem 构造函数中创建对象的 3 行，那么它会编译，所以我知道这与问题有关。

    #include <iostream> 
    #include <vector>
    #include <string>
    #include <stdlib.h>
    #include <time.h>

    using namespace std;

    class SystemBody
    {
        public:
            SystemBody();
            int systembodyindex;
            int starsystemindex;

            SystemBody(int systembodyindex, int starsystemindex)
            {
                cout << "StarSystem " << starsystemindex << ": creating empty SystemBody " << systembodyindex << endl;
            }
    };


    class Star : public SystemBody
    {
        public:
            Star();
            string startype;

            Star(int systembodyindex, int starsystemindex)
            {
                cout << "StarSystem " << starsystemindex << ": converting empty SystemBody into Star " << systembodyindex << endl;
            }
    };

    class Planet : public SystemBody
    {
        public:
            Planet();
            string planettype;

            Planet(int systembodyindex, int starsystemindex)
            {
                cout << "StarSystem " << starsystemindex << ": converting empty SystemBody into Planet " << systembodyindex << endl;
            }

    };

    class ExitNode : public SystemBody
    {
        public:
            ExitNode();
            vector<int> connectedindexlist;
            ExitNode(int systembodyindex, int starsystemindex)
            {
                cout << "StarSystem " << starsystemindex << ": converting empty SystemBody into Exit Node " << systembodyindex << endl;
            }


    };


    class StarSystem
    {
        public:
            StarSystem();
            int starsystemindex;
            vector<StarSystem> connectedlist;
            vector<Planet> planetlist;

            StarSystem(int index)
            {
                starsystemindex = index;
                cout << "--Creating StarSystem: " << starsystemindex << endl;
                int numberofbodies = (rand() % 4) + 2;
                    for ( int i = 0; i < numberofbodies; i +=1 )
                    {
                        if ( i == 0 )
                        {
                            Star body(i, starsystemindex);
                        }
                        else if ( i == numberofbodies )
                        {
                            ExitNode body(i, starsystemindex);
                        }
                        else
                        {
                            Planet body(i, starsystemindex);
                        }

                    }

            }

            void addConnection(StarSystem connectedstarsystem)
            {
                cout << "--StarSystem " << starsystemindex << ": Adding connection to StarSystem " << connectedstarsystem.starsystemindex << endl;
                connectedlist.push_back(connectedstarsystem);
            }

    };



    int main()
    {
        srand(time(0));
        StarSystem starsystem0(0);
        return 0;
    }

编辑：

感谢大家的帮助！只需在此处发布固定代码，以防将来有人会发现这很有用。

#include <iostream> 
#include <vector>
#include <string>
#include <stdlib.h>
#include <time.h>

using namespace std;

class SystemBody
{
    public:
        int systembodyindex;
        int starsystemindex;
        SystemBody ( )
        {
            cout << "----SystemBody BEING CREATED WITH NO PARAMETERS" << endl;
        }
        SystemBody ( int bodyindex, int systemindex )
        {
            systembodyindex = bodyindex;
            starsystemindex = systemindex;
            cout << "----StarSystem " << starsystemindex << ": creating empty SystemBody " << systembodyindex << endl;
        }

};


class Star : public SystemBody
{
    public:
        Star ( int bodyindex, int systemindex ) : SystemBody ( bodyindex, systemindex )
        {
            cout << "----StarSystem " << starsystemindex << ": converting empty SystemBody into Star " << systembodyindex << endl;
        }
};


class Planet : public SystemBody
{
    public:
        Planet ( int bodyindex, int systemindex ) : SystemBody ( bodyindex, systemindex )
        {
            cout << "----StarSystem " << starsystemindex << ": converting empty SystemBody into Planet " << systembodyindex << endl;
        }
};

class ExitNode : public SystemBody
{
    public:
        ExitNode ( int bodyindex, int systemindex ) : SystemBody ( bodyindex, systemindex )
        {
            cout << "----StarSystem " << starsystemindex << ": converting empty SystemBody into ExitNode " << systembodyindex << endl;
        }
};


class StarSystem
{
    public:
        int starsystemindex;
        vector<StarSystem> connectedlist;
        vector<Planet> planetlist;

        StarSystem ( int index )
        {
            starsystemindex = index;
            cout << "--Creating StarSystem: " << starsystemindex << endl;
            int numberofbodies = (rand() % 4 ) + 2;
            for ( int i = 0; i <= numberofbodies; i +=1 )
            {
                if ( i == 0)
                {
                    Star body(i, starsystemindex);
                }
                else if ( i == numberofbodies )
                {
                    ExitNode body(i, starsystemindex);
                }
                else
                {
                    Planet body(i, starsystemindex);
                }
            }
        }
};

int main()
{

    srand(time(0));
    StarSystem starsystem0(0);
    return 0;
}
4

3 回答 3

22

也许只有我一个人，但这里的构造函数是一个尚未定义的简单声明：

class StarSystem
    {
        public:
            StarSystem(); // <--- Undefined!

你声明了一个构造函数，但没有定义这个构造函数中实际发生的事情。

如果它是一个什么都不做的构造函数，做

StarSystem () {} // Defined.
// Nothing happens inside, but everything gets default-constructed!

附带说明一下，在发布此类内容时，发布错误编号并添加注释或某种指示错误发生位置的指示符会有所帮助（因此我们可以在您的巨大代码块中看到它）。

编辑： 作为重要说明，如果您根本不使用该构造函数，只需将其删除。

于 2013-04-14T23:00:33.770 回答
3

你认为因为你没有打电话，所以SystemBody()你不需要定义它。但是，您是在间接调用它。

不要做 

SystemBody() {};

按照建议。这不是你想要的。如果您不使用它，请将其完全删除。

您的 Star 类继承自SystemBody. 这意味着当一个 newStar被构造时，构造函数SystemBody被调用。

这条线

Star(int systembodyindex, int starsystemindex)
    {

实际上编译为

Star(int systembodyindex, int starsystemindex) :
    SystemBody()   // Here
    {

SystemBody如果您自己不调用，编译器会调用默认构造函数。

如果您考虑一下，您需要SystemBody在创建新的Star. 你可以像这样明确地做到这一点

Star(int systembodyindex, int starsystemindex) :
    SystemBody(systembodyindex)   // Here
    {
于 2013-04-14T23:04:49.553 回答
1

您定义了许多默认构造函数，但没有实现它们。代替

StarSystem(); // <- it is OK if you implement this somewhere but you didn't

写

StarSystem(){}
            ^^ this is empty implementation
于 2013-04-14T23:00:54.717 回答