Seg Fault - 将变量传递给方法更改全局值

时间:2012-04-17 13:44:47

标签: c++ pointers dynamic-memory-allocation

我是c ++的新手,我正在尝试使用动态内存创建一些基本对象。 我正在向方法传递一个int参数,它正在改变全局变量的值。我认为它与我为新对象分配内存的方式有关,我不能让它以任何其他方式编译。

int main () {
    int inp;
    CRectangle rectb (2,2);
    cout << "enter number of items to add" << endl;
    cin >> inp; // let's say inp = 7
    rectb.addItemsArray(inp);
    cout << "inp after adding items: " << inp << endl; // inp is now 1.
}

标题文件:

class CRectangle {
    int width;
    int height;
    item *items[]; // SOLUTION: change this line to "item *items"
    int input;

public:
        CRectangle (int,int);
        int addItemsArray(int);
        int area () { return (width*height); }
        int get_items(int);

};

- 和 -

class item {
    int foo;
    char bar;
public:
    //SOLUTION add "item ();" here (a default constructor declaration without arguments)
    item (int, char);
    int get_foo();
    char get_bar();
};

方法:

int CRectangle::addItemsArray(int in) {
    cout << "value of in at begginning:" << in << endl; //in = 7
    int i;
    i = 0;
    //SOLUTION: add "items = new item[in];" on this line.
    while (i < in) {
        items[i] = new item(1, 'z'); //SOLUTION: change this line to "items[i] = item(1, 'z');"
        i++;
    }
    cout << "value of in at end " << in << endl; //in = 7
    return 1;
}

有时我会收到总线错误或seg错误。有时它会像预期的那样以较低的数字(如2或3)工作,但并非总是如此。

非常感谢任何帮助。

编辑(CRectangle的构造函数):

CRectangle::CRectangle (int a, int b) {
    width = a;
    height = b;
} 

(item的构造函数):

/* SOLUTION add default item constuctor
item::item() {
    foo = 0;
    bar = 'a';
}
*/

item::item(int arg, char arg2) {
    foo = arg;
    bar = arg2;
}

2 个答案:

答案 0 :(得分:4)

问题是您没有为放入items的指针分配任何存储空间。我建议改变:

item *items[];

std::vector<item*> items;

然后使用以下内容添加项目:

items.push_back(new item(1, 'z'));

答案 1 :(得分:1)

看起来您忘了创建项目数组...

您定义了一个动态分配的数组(不是item * items [100],而是item * items [])。在使用阵列之前,必须分配内存来保存项目:

items = new item[100];

并且不要忘记用

删除它
delete [] items; 

最后。 ;)

而不是

int i;
i = 0;
while (i < in) {
   items[i] = new item(1, 'z');
   i++;
}

我会用

for (int i=0; i<in; i++)
{
   items[i] = new item(1, 'z');
}