函数完成后类变量重置

时间:2013-06-27 00:35:46

标签: c++ class instance

我是java / python / etc背景下的C ++新手,我想在下学期上课之前自学OO编程。

我正在尝试使用SFML创建一个动画系统,但我的一个类变量遇到了一些麻烦;我增加后它会一直重置为0。我将从代码开始,然后使用我正在使用的日志输出来帮助弄清楚发生了什么。

解决方案:对C ++不熟悉,我是一个白痴,并在我的getter函数中返回了我的类的新实例;使用[class]& func()...而不是[class] func()解决了这个问题,但现在我有一些重构要做。

代码(标题):

...

typedef std::vector<Frame> frameVect; // (Frame defined above)
typedef std::vector<double> dubVect;

...

class limbAnim
{
private:
    int limbNum;
    int numFrames;
    int curFrame;
    frameVect frames;

public:
    limbAnim(int limb, int nFrames, frameVect F);
    <getters/setters>
    void incCurFrame();

    dubVect incrementAnimation(dubVect curPos, double curRot);
}

代码(cpp):

... (include vector, ofstream, etc)

std::ofstream AnimLog("log.log")

typedef std::vector<Frame> frameVect; // (Frame defined above)
typedef std::vector<double> dubVect;

...

limbAnim::limbAnim(int limb, int nFrames, frameVect F)
{
    limbNum = limb;
    curFrame = 0;
    numFrames = nFrames;
    frames = F;
}

void limbAnim::incCurFrame()
{
    curFrame=curFrame+1;
    if (curFrame >= numFrames)
    {
        curFrame = 0;
        AnimLog << "Greater than." << std::endl;
    }
}

dubVect limbAnim::incrementAnimation(dubVect curPos, double curRot)
{
    AnimLog << limbNum << ", " << numFrames << std::endl;

    if (numFrames > 0)
    {
        AnimLog << curFrame << std::endl;
        dubVect curStepP = frames[curFrame].getStepPos();
        double curStepR = frames[curFrame].getStepRot();

        curPos[0] = curPos[0] + curStepP[0];
        curPos[1] = curPos[1] + curStepP[1];

        curRot = curRot + curStepR;

        incCurFrame();
        AnimLog << "Incremented: " << curFrame << std::endl;
    }

    dubVect retV = curPos;
    retV.push_back(curRot);
    return retV;
}

所以,我的日志输出看起来不错,因为我正在测试4个肢体6和6的帧。 8,除了那些肢体'curFrame似乎在递增后重置为0:

...
5, 0
6, 2
0
Incremented: 1
7, 0
8, 2
0
Incremented: 1
9, 0
...
5, 0
6, 2
0
Incremented: 1
7, 0
8, 2
0
Incremented: 1
9, 0
...(ad nauseam)

编辑:调用增量函数的代码。 (main.cpp中)

// (Outside main loop.)
Animation walk_anim(12, "assets/anim/walk.dat");

// (Inside main loop.)
for (int i=0; i<12; i++)
{
    dubVect animDat = walk_anim.getLimbFrame(i).incrementAnimation(limbPos[i], curDegs[i]);
    dubVect newPos = getDVect(animDat[0], animDat[1]);
    double newRot = animDat[2];
    curDegs[i] = newRot;
    if (curDegs[i] >= 360)
        curDegs[i] -=360;
    limbPos[i] = newPos;
}

getLimbFrame

Animation::Animation(int lNum, string fName)
{
    numLimbs = lNum;
    fileName = fName;

    // Fill up limbVect with correct # of empty frames.
    for (int i=0; i<numLimbs; i++)
    {
        frameVect emptyFVect;
        limbAnim LA(i, 0, emptyFVect);
        limbFrames.push_back(LA);
    }

    // Boring .dat parsing, populates the 'frames' var of each limbAnim.
    loadAnim();
}

limbAnim Animation::getLimbFrame(int index)
{
    if (index < numLimbs)
    {
        return limbFrames[index];
    }
}

2 个答案:

答案 0 :(得分:4)

希望您知道您的函数按值进行参数化,因此它们可以处理某些内容的副本

您小心地避免在您调用incrementAnimation时显示真正有趣的代码部分,很可能它遵循与其他函数相同的错误模式。

我建议阅读如何通过引用和const引用传递对象 - 以及函数参数如何在C ++中工作。

答案 1 :(得分:1)

我认为您需要使用static关键字声明该成员变量,然后您可以说它是一个类变量,它将是您的类的每个实例的共享。像这样:

static int curFrame;

然后你需要从课堂外面初始化它。请记住,声明与初始化有很大不同。

您可以阅读herehere