我正在编写一个函数,它将缓冲区的内容复制到另一个缓冲区。我使用了一个空指针,这样函数就不是特定类型的了。我有一个可测试的版本,看起来这个功能运行正常。但我不知道我所做的是否合法,所以我的问题是,如果有,我所做的工作的陷阱是什么。
#include <stdio.h>
#include <stdlib.h>
void* voidcpy(void *void_ptr, size_t nbytes)
{
char *char_ptr = void_ptr;
char *cpy_char_ptr = NULL;
size_t i = 0;
if((cpy_char_ptr = malloc(nbytes)) == NULL) return NULL;
for(; i < nbytes; i++) cpy_char_ptr[i] = char_ptr[i];
return cpy_char_ptr;
}
int main()
{
short int *intp = NULL;
short int *cpy_intp = NULL;
size_t siz = 5;
int i = 0;
if((intp = malloc(siz * sizeof(short int))) == NULL)
{
perror("(malloc)");
return -1;
}
intp[0] = 0;
intp[1] = 14;
intp[2] = 187;
intp[3] = 12678;
intp[4] = -234;
if((cpy_intp = voidcpy(intp, siz * sizeof(short int))) == NULL)
return -2;
printf("intp = %p\ncpy_intp = %p\n\n", (void*)intp, (void*)cpy_intp);
for(; i < siz; i++) printf("cpy_intp = %i\n", cpy_intp[i]);
free(intp);
free(cpy_intp);
return 0;
}发布于 2014-04-05 17:37:28
是的,这是完全合法的,在C中,您可以合法地为任何其他指针类型指定一个空指针,也可以将任何指针类型分配给一个空指针。
在C++中,这是不允许的。在C++中,您必须使用reinterpret_cast来转换成不同的指针类型,因为C中允许的空闲空指针转换被认为是一个很容易出错的“循环孔”。
当然,这个想法是有道理的,如果你不小心,你可能做错了事情,例如,你可以很容易地把指向这个函数的指针的指针错误地传递给你,然后你的函数就会很高兴地覆盖堆栈上的指针。然而,这并不是实现的错误,它只是函数的使用方式,而且memcpy的行为也没有什么不同。
尽管如此,您最好还是使用memcpy,因为这很可能会得到更好的优化,尽管现在编译器可能也会从您的代码中得到一个相当不错的版本。
再加几个指点;
1)不需要malloc原始数组,您可以像这样静态地初始化它
short int int_arr[] = {
0,
14,
187,
12678,
-234,
};2)然后可以以下列方式调用您的函数:
cpy_int_arr = voidcpy(int_arr, sizeof(int_arr));3)如果您不想静态地定义数组,那么使用指针获取元素大小,这样就可以更改数组类型,而无需在代码中的其他地方更改数组类型,从而减少“循环孔”空抛的潜在危险:
cpy_intp = voidcpy(intp, siz * sizeof(*intp));4)不需要在void*调用中转换为printf
5)尝试立即赋值变量,不要将赋值放在if语句中:
char *cpy_char_ptr = malloc(nbytes)
if (cpy_char_ptr == NULL)
return NULL;6)类似地,可以在循环子句中定义一个迭代变量:
for(size_t i = 0; i < nbytes; i++) cpy_char_ptr[i] = char_ptr[i];尽可能晚定义变量并立即初始化它们的原因是,您尽可能地将变量的范围保持在较小的范围内,并且在变量初始化之前不能错误地使用它。
7) (个人偏好)不要在标识符(intp,voidcpy)中使用类型名称,如果标识符声明的类型与变量的实际情况不同(例如,您的类型实际上是一个short int,而不是变量名所建议的int ),那么您的代码将变得很难阅读/理解,或者当您更改类型时,可能会出错,您就需要在整个代码中更改标识符。
https://stackoverflow.com/questions/22882916
复制相似问题