Ubuntu中文论坛

由于昨天升级不慎，系统崩溃，重新安装之后，相同的程序出现段错误……
#include <stdlib.h>
#include <string.h>
#define M 1000000
#define LEN 100
void transfer(char *in)
{

char * word[M];
int times[M];
float frequency[M];
char * flag = NULL;
//gdb调试到这里出错……
char * source = (char *)malloc(LEN);
char * label = (char *)malloc(LEN);
char * token = (char *)malloc(LEN);
int i = 0;
int j;
int sum = 0;
strcpy(source, strtok(in, " "));
strcpy(label, strtok(NULL, " "));
printf("%s",label);
while(1)
{
flag = strtok(NULL, " ");
if(flag == NULL)
break;
strcpy(token, flag);
word = (char *)malloc(LEN);
strcpy(word, token);
flag = strtok(NULL, " ");
if(flag == NULL)
break;
strcpy(token, flag);
times = atoi(token);
sum += times;
i++;
}
for(j=0;j<i;j++)
{
frequency[j] = times[j]*1.0/sum;
printf(" %s:%f",word[j],frequency[j]);
}
printf("\n");
for(j=0;j<i;j++)
free(word[j]);
free(label);
free(source);
free(token);
}
int main()
{
char * in = (char *)malloc(M);
char * fileName = (char *)malloc(LEN);
freopen("output.txt", "w",stdout);
scanf("%s",fileName);
FILE *fp = fopen(fileName, "r");
while(fgets(in, M-1, fp) != NULL)
{
transfer(in);
}
fclose(fp);
free (in);
free (fileName);
return 0;
}

new呢？

没有用过New,原来写malloc都没有问题……

不知道为什么你不喜欢 char source[12];
段错误，基本都是操作不存在的内存定位。你为什么不这样简单的实际分配到堆栈呢？又不需要数据在堆里面共享。

因为要处理的字符串都很长……
不过现在已经解决了，就是将M的值改小一点就可以了。
对内存的操作还需要好好学习，多谢了~

sizeof(char) * 100 + sizeof(int) * 1000000 + sizeof(float) * 1000000
= 1 * 100 + 4 * 1000000 + 4 * 1000000
= 8000100 (B) = 8000 (K) = 8 M
才8M就把内存耗光了 ？

注意不要声明几M大小的临时变量，可以用 malloc() 动态分配。
因为在系统中，栈的大小是有限制的，不像堆内存那样可以随便用。
char * word[M];
int times[M];
float frequency[M];

这里消耗了好几M栈内存，建议这里用 malloc 分配。在C/C++ 这种语言中，需要特别注意这种问题。要不然函数多调用几次，一旦栈比较深了以后就出问题