Question

在this question获得帮助之后，我被引导做了更好的调试。在那个过程中，我发现我的问题是：

在C ++中工作时，尝试将类的成员变量设置为值有效，但不能在循环时使用。我已经将我的代码（跟随）减少到我认为最简单的代码仍然会产生错误。

调用类Mover的一个函数来修改变量pMovXPOS，然后可以在相同的范围内（在该函数内）和它被调用的位置（在循环内）更新。但是，在循环时，似乎变量被重置为其原始值。

我在这里发布了整个测试代码。问题出在Main-test.cpp文件的RunWorld（）函数中。如果编译并运行，您应该看到显示变量的输出，然后被重置。

这是范围问题吗？建筑/破坏问题？指针/参考问题？我不知道从哪里开始（除了更好的调试）。

（因为我是C ++的新手，我确信我使用过的风格和/或方法存在一些明显的问题。如果有任何重要的no-nos，请自由指出这些问题。）

提前感谢您的帮助！

//Main-Test.cpp
#include "Globals-Test.h"
#include "Mover-Test.h"

std::vector < Mover > AllMovers;

long SysCounter;

Mover CreateNewMover() {
Mover TempMover;

    TempMover.setXPOS(5);
    TempMover.setYPOS(10);

    return TempMover;

}

void RunWorld() {
Mover TempMover;
unsigned int iMoverLoop;

    srand ( time(NULL) );

    AllMovers.push_back(CreateNewMover());

    for (SysCounter = 0; SysCounter <= 50; SysCounter++) {
        for (iMoverLoop = 0; iMoverLoop < AllMovers.size(); iMoverLoop++) {
            std::cout << "Loop #:" << SysCounter << std::endl;
            TempMover = AllMovers.at(iMoverLoop);
            std::cout << "Is: " << TempMover.getXPOS() << std::endl;
            TempMover.DoMove();             
            std::cout << "Is: " << TempMover.getXPOS() << std::endl;
        }
    }
}



int main() {
    RunWorld();
    return 0;
}

//Globals-Test.h
#include <stdlib.h>
#include <stdio.h>
#include <iostream>
#include <sstream>
#include <unistd.h>
#include <ctype.h>
#include <math.h>
#include <string>
#include <vector>
#include <time.h>
#include <fstream>

//Mover-Test.h
extern long MoverIndex;

class Mover {

private:

    int pMovXPOS;
    int pMovYPOS;

public:

    int getXPOS();
    void setXPOS(int newXPOS);
    int getYPOS();
    void setYPOS(int newYPOS);

    Mover();
    ~Mover();

    void DoMove();

};

//Mover-Test.cpp
#include "Globals-Test.h"
#include "Mover-Test.h"

Mover::Mover() {    

}

Mover::~Mover() {

}

int Mover::getXPOS() {
    return pMovXPOS;
}

void Mover::setXPOS(int newXPOS) {
    pMovXPOS = newXPOS;
}

int Mover::getYPOS() {
    return pMovYPOS;
}

void Mover::setYPOS(int newYPOS) {
    pMovYPOS = newYPOS;
}

void Mover::DoMove() {
pMovXPOS = pMovXPOS + 1;
pMovYPOS = pMovYPOS + 1;

}

//Compiled with:
g++ -Wall -lm -c Main-Test.cpp
g++ -Wall -lm -c Mover-Test.cpp
g++ -Wall Mover-Test.o Main-Test.o -o world-test.exe -lm

Answer 1

你的问题就在这一行：

TempMover = AllMovers.at(iMoverLoop);

您正在创建索引为Mover的{{1}}副本，然后修改该副本。向量中的对象永远不会被修改，并且在下一次迭代中，您的更改将丢失，因为iMoverLoop中的下一个副本会覆盖TempMover

解决此问题的一种方法是使用AllMovers的引用。例如：

TempMover

C ++类成员变量重置循环后？

1 个答案: