-1

我不知道为什么这段代码会崩溃,它看起来对我来说是正确的,但它只是崩溃了一个SIGSEV. 我来自 Java,习惯于“有用”的错误消息......

主文件

#include <cstdlib>
#include <stdio.h>

#include "DuckV.h"
#include "PenguinV.h" 

bool tryFlyOOP(IBird* birdy)
{
    return birdy->canFly();
}
int main(int argc, char** argv)
{
   DuckV* duckV;
   PenguinV* penguinV;

    printf("My OOP duck %s fly\n", tryFlyOOP(duckV) ? "can" : "can't");
    printf("OOP Penguins %s fly\n", tryFlyOOP(penguinV) ? "can" : "can't");

    return 0;
}

鸟:

#ifndef IBIRD_H
#define IBIRD_H

class IBird
{
public:
    IBird () {}
    virtual bool canFly() {return true;};
};

#endif  /* IBIRD_H */

DuckV/PenguinV 除了名字和返回值都一样

#ifndef DUCKV_H
#define DUCKV_H

#include "IBird.h"

class DuckV : public IBird
{
public:
    DuckV(){}
    virtual bool canFly() {return true;}
};

#endif  /* DUCKV_H */

我试过改变周围的东西,但我就是不明白。任何帮助将不胜感激 :)。

4

3 回答 3

4

你还没有初始化你的指针:

DuckV* duckV; // points to a random location. No DuckV object exists.

我建议您放弃指针并执行以下操作:

bool tryFlyOOP(const IBird& birdy)
{
    return birdy.canFly();
}

然后

DuckV duckV;
std::cout << "My OOP duck " << (tryFlyOOP(duckV) ? "can" : "can't") << " fly\n";

这将要求您创建您的canFly()成员函数const

class IBird
{
public:
    IBird () {}
    virtual bool canFly() const {return true;};
};
于 2013-06-26T15:08:40.283 回答
2

这是因为您的对象没有被创建。

你应该做 :

DuckV* duckV = new DuckV();
PenguinV* penguinV = new PenguinV();

在您的代码中,您刚刚声明了指针。

或者你可以这样做:

DuckV duckV;
PenguinV penguinV;

bool tryFlyOOP(const IBird& birdy)
{
    return birdy.canFly();
}

在最后一个示例中,您使用参考,谁也是这样做的好方法。

于 2013-06-26T15:10:31.210 回答
0

您尚未初始化变量(duckV 和 penguinV)。由于您想将它们用作指针,因此请使用“new”运算符创建新对象并将其分配给这些变量。或者 - 更好的是 - 在堆栈上创建它们并传递引用。

于 2013-06-26T15:10:50.140 回答