我遇到了一个问题,我的代码在尝试使用列表的 size() 函数时出现段错误。根据 stackoverflow 的建议:-)我构建了一个发生段错误的最小情况(在下面的调用 inventory.size() 中)。这是:
#include <list>
class Thing {};
class Player {
private:
int xpCalcArray[99];
std::list<Thing*> inventory;
public:
Player();
int addToInv(Thing& t); // return 1 on success, 0 on failure
};
Player::Player() {
// set up XP calculation array
for (int i=1; i<100; i++) {
if (i<=10) {
xpCalcArray[i] = i*100;
}
if (i>10 && i<=50) {
xpCalcArray[i] = i*1000;
}
if (i>50 && i<=99) {
xpCalcArray[i] = i*5000;
}
}
}
int Player::addToInv(Thing& t) {
if (inventory.size() == 52) {
return 0;
} else {
inventory.push_back(&t);
}
return 1;
}
int main(int argc, char *argv[]) {
Thing t;
Player pc;
pc.addToInv(t);
return 1;
}
我注意到,当我删除 Player 构造函数中数组的设置时,它工作正常,所以这看起来是问题所在。我究竟做错了什么?
您正在越界访问数组,这会导致未定义的行为。该数组的有效索引范围
int xpCalcArray[99];
是 0 到 98。您正在此处访问索引 99:
if (i>50 && i<=99) {
xpCalcArray[i] = i*5000;
}
你的外循环应该是
for (int i=0; i<99; i++) { ... }
请注意,我从 0 开始,尽管这是假设您实际上想要访问第一个元素。
那么你的最终条件可以简化为
if (i>50) {
xpCalcArray[i] = i*5000;
}
如果您打算使用大小为 100 的数组,那么您需要
int xpCalcArray[100];
然后在之间循环int i=0; i<100;
.
本文内容由网友自发贡献,版权归原作者所有,本站不承担相应法律责任。如您发现有涉嫌抄袭侵权的内容,请联系:hwhale#tublm.com(使用前将#替换为@)