2

我有一个重载的=运算符，它正确地传递了particle类中的一些值，但没有传递其他值。我在网上进行了故障排除，找不到与此问题直接相关的任何内容。我在 C++ 方面相当胜任的同行无法提供帮助，并建议我在这里发帖。任何帮助将不胜感激。

width、height、wrap和particlen的所有值array都完美通过。但是，xpos、ypos、xvel和yvel的值传递的值不正确。对于xposand ypos，每 11 个元素都正确通过，但所有其他元素都等于 0；我希望这些值都不为零。在main中，在立即进行的操作中gen1 = gen0;，所有取消引用的 gen0 值都是正确的。我估计他们没有正确通过？如果需要，我会很乐意发布更多代码/信息。同样，任何见解都将不胜感激。谢谢你。

相关代码：

class particle{             
private:
    int* xpos;  
    int* ypos; 
    int* xvel;
    int* yvel;
    int* array;             
    int width;
    int height; 
    int wrap;
    int particlen;
public:
    void operator=(const particle&);
}

重载 = 运算符的相关部分：

void particle::operator=(const particle &current){
    int a,b,i,j;
    width = current.width;
    height = current.height;
    wrap = current.wrap;
    particlen = current.particlen;
    array = new int[width*height];
    xpos = new int[particlen];
    ypos = new int[particlen];
    xvel = new int[particlen];
    yvel = new int[particlen];

    for(a=0; a<height; a++){
        for(b=0; b< width; b++){
        this->array[a*width + b] = current.array[a*width + b];
        }
    } 

for(i = 0; i < particlen; i++){
    this->xpos[i] = current.xpos[i];
    this->ypos[i] = current.ypos[i];
    this->xvel[i] = current.xvel[i];
    this->yvel[i] = current.yvel[i];
}

main的相关部分：

int main(){
    particle gen0,gen1;
    gen1 = gen0;
}

在 Sehe 的建议下，我编辑了我的代码。但是，我现在使用最简单的命令遇到了内存分配问题。<< 运算符，在我的 main 中调用的第一个操作不允许将我的数组变量设置为等于读入文件中的整数。我绝对知道该文件是可读的，因为第一个输出行打印。但是，然后我在array[i*width + j] = k;. GDB 打印输出：程序收到信号 EXC_BAD_ACCESS，无法访问内存。原因：KERN_INVALID_ADDRESS at address: 0x0000000000000000 0x000000010001c4f4 inparticle::operator<< (this=0x7fff5fbfcb08, file=0x7fff5fbffa90 "100100") at Before typedef 1 vector.cpp:686 686 array[i*width]

我在网上查找了向量库的正确术语，并认为我的语法是正确的。有任何想法吗？

更新（美国东部标准时间 8 月 2 日 9:00）

我仍然在使用 NULL 指针时遇到问题。使用 gdb 进行调试时，我收到以下消息：

KERN_INVALID_ADDRESS 地址：0x0000000000000000 0x000000010001d807 inparticle::operator<< (this=0x7fff5fbfcb08, file=0x7fff5fbffa90 "5040") at before typedef 1 vector.cpp:688 688 array[(i*width + j)]

我对 NULL 指针没有什么经验；关于为什么会发生这种情况以及如何解决的任何建议？下面是相关代码（总代码约900行）

class particle{             
private:
    std::vector<int> xpos;  
    std::vector<int> ypos; 
    std::vector<int> xvel; 
    std::vector<int> yvel; 
    std::vector<int> array; 
    int width;      
    int height;     
    int wrap;       
    int particlen; // number of particles read in
public:
    void operator<<(particle);
    void Collision(int, int, particle); 
    void operator>>(char*);
    void operator<<(char*);
};

void particle::operator<<(char* file) // Reads initial input file
{
ifstream in_file;
in_file.open( file);    // open the file
int i,j,k;
}
in_file >> width >> height >> wrap >> particlen;
    for(i=0; i<height; i++){
        for(j=0; j< width; j++){
            in_file >> k;
            array[i*width + j] = k; // This is line 688 which GDB references
        }
    }
}

void particle::operator<<(particle current){
int i,k,j,l;

for(k = 0; k < height*width; k++)
{
    array[k] = 0;
}

k = 0;

for(i=0; i < particlen; i++)
{
    cout << "Current x pos is" << current.xpos[i] << endl;
}

for(i=0; i < particlen; i++)
{

    if(array[ (current.ypos[i]-1)*width + (current.xpos[i] - 1) ] == 0 )
    {
    //cout << "Current X position[" << i << "] is" << current.xpos[i] << endl;
        xpos[i] = current.xpos[i] + (current.xvel[i])*timeinc;
        if(xpos[i] > width) xpos[i] = xpos[i] - width; 
        ypos[i] = current.ypos[i] + (current.yvel[i])*timeinc;
        if(ypos[i] > width) ypos[i] = ypos[i] - height;

    //cout << "Next X position[" << i << "] is" << xpos[i] << endl;
    }

    if(array[ (current.ypos[i]-1)*width + (current.xpos[i] - 1) ] > 1 && current.JC[i] != 1)
    {
        for(l=i+1; l < particlen; l++)
        {
            if(current.xpos[l] == current.xpos[i] && current.ypos[l] == current.ypos[i] && current.JC[l] != 1)
            {
            Collision(i,l,current);
            }
        }
    }
}

for(i=0; i < particlen; i++)
{
    array[ (xpos[i]-1)*width + (ypos[i]-1) ] = 1;
}
Display();}

int main()
{
    char filename[256];
    particle gen0, gen1;
    gen0 << filename;
    gen1 = gen0;
    gen0 << gen1; // Calculates the next state of gen0, based on the values in gen1.
}

我知道这很长，但我认为这是我可以提供的最少信息。我认为可能问题出在gen0 << gen1操作员上，所以我将方法包括在内。

4

1 回答 1

2

就像我在评论中暗示的那样，99% 的人说你违反了三法则

值得注意的是，您有赋值运算符，但看不到复制构造函数。这可能意味着如果复制，数组指针最终将“非法”共享。我还假设您在某处有一个析构函数删除这些数组（在一个实例中），从而使复制到另一个实例中的数组无效。

首先通过避免手动内存管理来修复它：

#include <vector>

class particle{             
    std::vector<int> xpos;  
    std::vector<int> ypos; 
    std::vector<int> xvel;
    std::vector<int> yvel;
    std::vector<int> array;             
    int width;
    int height; 
    int wrap;
    int particlen; // or use `xpos.size()` e.g.
};

int main()
{
    particle gen0,gen1;
    gen1 = gen0;
}

PS。您可能需要考虑使用v.at(i)而不是v[i]在需要边界检查时使用。这对调试问题有很大帮助。

于 2013-08-02T00:10:46.877 回答