因此,我正在用C编写一个简单的切片函数,它接受字符串数组,标记切片开始的字符串和切片的大小。在函数中,我malloc了一个新数组,然后继续将切片中的每个字符串复制到新数组中。然而,我在第一个字符串上得到一个段错误,即使我已经为结果数组错置了空间。
代码是这样的:
char** slice(char** args, char* start, int size){
int i = 0;
// need to find start first
char* cursor = args[0];
int j = 0;
while(cursor != NULL){
if(strcmp(cursor, start) == 0){
break;
}
j++;
cursor = args[j];
}
char** result = malloc(MAX_INPUT * size);
while(i < size){
strcpy(result[i], args[j+i]);
i++;
}
return result;
}
导致段错误的行是——
strcpy(result[i], args[j+i]);
我已经使用GDB来窥视结果和args中的值,结果[i]是0x0,这是NULL,但结果本身是一个地址,但我不确定为什么malloc不工作。堆栈空间用完了吗?这意味着我完蛋了吗?
result[i]
为未初始化指针。你犯了和下面相同的错误:
char *ptr;
strcpy(ptr, args[j+i]);
必须使result[i]
指向某个已分配的空间,然后才能将字符复制到其中。此外,MAX_INPUT * size
是分配给指针数组的错误空间量。
另一个问题是,如果size
大于start
之后数组中剩余的字符串数量,那么您将读取数组的末尾。
那么你的函数永远不会在新数组的末尾放置一个NULL
,所以调用者无法知道你返回的切片有多大。
cursor
也是多余的,你可以直接写args[j]
。这个函数基本上是一团糟。
代码可以是(警告:未测试):
char** slice(char** args, char const *start, int slice_size)
{
// Find index of "start"
int start_index;
for (start_index = 0; args[start_index]; ++start_index)
if ( !strcmp(args[start_index], start) )
break;
// Abort if "start" was not present (remove this line if you want to
// instead return an empty terminated list)
if ( !args[start_index] )
return NULL;
// Allocate array of pointers to new strings, allowing space for terminator
char **result = malloc((slice_size + 1) * sizeof *result);
if ( !result )
return NULL;
// Copy strings in, allocating space for each string, stopping if no more args
int i;
for (i = 0; i < slice_size && args[start_index + i]; ++i)
result[i] = strdup(args[start_index + i]);
// Terminate the list
result[i] = NULL;
return result;
}
这一行:
char** result = malloc(MAX_INPUT * size);
mallocs MAX_INPUT字符乘以大小。size的内容是什么意思
总的来说,需要的是一个malloc,用于char *
的数量,我在代码中没有看到。
然后,在malloc参数正确后,
代码需要为每个字符串使用strdup()
而不是strcpy()
或malloc空间,然后使用strcpy()
,可能在包含两个函数调用的循环中