我将char ** input
从main()
传递到processInExp()
函数,然后再次将它从processInExp()
函数传递到getInput()
函数,以便在读取文件时动态分配它。
选中时,getInput()
函数input
内部的内存分配正确,但在in processInExp()
中使用时遇到运行时错误。有什么问题吗?
以下是我的代码:
int getInput(char ** input, const char * fileName)
{
int numInput = 0;
int i, j;
char c;
char tempInput[100];
FILE * pFile;
if((pFile = fopen(fileName, "r")) == NULL)
{
printf("Cannot read file %s\n", fileName);
system("PAUSE");
exit(1);
}
while(!feof(pFile))
{
c = fgetc(pFile);
if(c == '\n') ++numInput;
}
/* printf("%d\n", numInput); */
input = (char**)malloc(numInput * sizeof(char*)); /* #2 MALLOC input */
rewind(pFile);
for(i = 0; !feof(pFile); ++i)
{
fscanf(pFile, "%[^\n]%*c", tempInput);
/* printf("%s\n", tempInput); */
input[i] = (char*)malloc((strlen(tempInput) + 1) * sizeof(char)); /* #3 MALLOC input[] */
strcpy(input[i], tempInput);
/* printf("%s\n", input[i]); */ /* #4 PRINT OUT PERFECTLY */
memset(tempInput, 0, sizeof(tempInput));
}
fclose(pFile);
return numInput;
}
void processInExp(char ** input, char ** output, const char * fileName)
{
int numFormula;
int i;
numFormula = getInput(input, fileName); /* #1 PASSING input */
/* printf("%s\n", input[0]); */ /* #5 RUNTIME ERROR */
output = (char**)malloc(numFormula * sizeof(char*));
system("PAUSE");
for(i = 0; i < numFormula; ++i)
{
convertIntoPost(input[i], output[i]);
printf("%d. %s -> %s", (i + 1), input[i], output[i]);
}
}
最佳答案
C
使用传递值传递函数参数。因此,从函数getInput()
内部,您不能更改变量input
并期望该更改反映回传递给函数的实际参数中。为此,您需要一个指向要传递的变量的指针,在本例中,您需要这样做
int getInput(char *** input, const char * fileName) { //notice the extra *
需要这样称呼
char ** inp = NULL;
getInput(&inp, ..........);
然后,
getInput()
将能够将内存分配给函数内部的*input
,该函数将反映到inp
中。否则,从
getInput()
返回后,实际参数仍将未初始化,进一步使用该参数(在您的情况下,在for
函数的processInExp()
循环中)将导致undefined behaviour。也就是说,还有两件事需要注意,
请see why not to cast返回值
malloc()
和C
中的族。检查Why is
while ( !feof (file) )
always wrong?