Question

这段c++代码总是出现segmentation fault,希望大家指出错误

这段c++代码总是出现segmentation fault,希望大家指出错误#include<iostream>
#include<vector>
#include<ctime>
#include<cstdlib>
using namespace std;
class virus
{
public:
double reproductionrate;
double reproduce;
double resistance;
virus()
{
resistance=rand()/(RAND_MAX+1.0);
reproductionrate=0.2;
}
void Reproduce()
{
reproduce=rand()/(RAND_MAX+1.0);
}
};
class patient
{
public:
double immunity;
int numVirusCell;
virus* vnow[3000];
vector<virus*> v;
patient(double i,int n)
{
immunity=i;
numVirusCell=n;
}
void takedrug()
{
immunity+=0.1;
}
void see()
{
virus* p[3000];
int i,j,add=0;
for(i=0;i<numVirusCell;i++)
{
p[i]=new virus;
}
for(i=1;i<=100;i++)
{
for(j=0;j<numVirusCell;j++)
{
p[j]->Reproduce();
if(p[j]->resistance<=immunity)
{
numVirusCell--;
v.push_back(p[j]);
delete p[j];
p[j]=NULL;
}
else{
if(p[j]->reproduce>immunity&&p[j]->reproduce>p[j]->reproductionrate)
{
numVirusCell++;
vnow[j+add]=p[j];
vnow[j+add+1]=new virus;
v.push_back(vnow[j+add+1]);
add++;
}
else
{
vnow[j+add]=p[j];
}
}
}
add=0;
if(numVirusCell>1000)cout<<"第"<<i<<"小时，达到1000"<<endl;
else
for(j=0;j<numVirusCell;j++)p[j]=vnow[j];
}
}
};
int main()
{
srand(time(NULL));
patient p1(0.1,10);
cout<<p1.immunity<<endl;
p1.see();
return 0;
}

Answer 1

numVirusCell++; 之后你没有给p[numVirusCell-1]赋值，导致再次执行
for (j = 0; j<numVirusCell; j++)
{
p[j]->Reproduce(); ----》这里会访问p[10]所指向的对象，而p[10]并未初始化，所以会发生非法访问，造成程序崩溃

追问

我new一个对象的空间赋给该类指针后，这个对象空间会自己调用构造函数吗

追答

会

追问

也就是说，是我循环过程中numviruscell改变导致循环终止条件改变，使得访问了一个不存在的p[j]对象，导致错误
我这样改了之后
(把循环条件中的numviruscell用另一个变量n代替，循环结束再更新n)
可还是出现了segmentationfault....

追答

因为你的代码还有其他漏洞，一步一步修正吧
这次的问题出在delete p[j];上，
由于for(j=0;j<n;j++)整个大循环会被执行100次，也就是j为0的情况会走100次，这100次中会多次调用到delete p[j]; 而p[0]早在第一次调用delete p[j];就被释放掉了，第二次再调用它，就会发生程序崩溃。

追问

我在for(j=o;j<n;j++)循环下面重新对p[j]赋了值，这个不行么

追答

你用vnow[j]给p[j]赋值，但vnow里的元素并不是每一个都有值的，因为你的代码里给vnow进行赋值的索引并不连续。所以发生了某些vnow[j]未曾初始化的情况，造成p[j]依然是非法内存指针。

追问

改好了，谢谢啦

Answer 2

错误代码
#include "string.h"#include <stdlib.h>#include <stdio.h>void func1(char ** dest,char * src,int n) {
(*dest) = (char*)malloc(sizeof(char)*n); strcpy(*dest,src);

}int main(int argc,char** args) { char ** p = NULL; char str[] = "foreach_break"; int len = sizeof(str); printf("%d\n",len);
func1(p,str,len);
printf("%s\n",*p); free(p);
p = NULL;
}1234567891011121314151617181920

正确代码
#include "stdio.h"#include "string.h"#include "stdlib.h"void func1(char ** dest,char * src,int n) {
(*dest) = (char*)malloc(sizeof(char)*n); strcpy(*dest,src);

}int main(int argc,char** args) { char * p = NULL; char str[] = "foreach_break"; int len = sizeof(str); printf("%d\n",len);
func1(&p,str,len); printf("%s\n",p); free(p); //p = NULL;}12345678910111213141516171819202