关于C语言指针传递参数的问题

关于C语言的函数传参，有时候由于数据量比较大，比如参数是一个结构体，我们会考虑用结构体指针来传参，以避免在函数调用时发生大量数据的拷贝 。有两种形式，一种是用在参数列表里面，而当仅仅是获取某些数据时，可以考虑使用不带参数的函数调用，通过函数返回值来获取数据。

下面的例子是采用指针传递参数、获取数据时，可能会踩到的坑：

#include <stdlib.h>
#include <stdio.h>

typedef struct TEST_STRUCT
{
	int a[4];
	char b[20];
} sParam, *p_sParam;

sParam *get_param2()
{
	sParam data = {0};
	for(int i=0; i<4; i++)
	{
		data.a[i] = 2;
	}
	strcpy(data.b, "22222");
	
	//能够打印出data的数据
	for(int i=0; i<4; i++){
		printf("data.a[%d]=%d\n",i,data.a[i]);
	}
	printf("data.b=%s\n",data.b);
	
	return &data;//函数返回data的地址,但同时这个地址的内存空间被释放(局部变量),所以实际返回的是一个野指针
}

int get_param1(sParam *mydata)
{
    if(NULL == mydata)
        return -1;

	sParam data = {0};
	for(int i=0; i<4; i++)
	{
		data.a[i] = 1;
	}
	strcpy(data.b, "hhhh");
	
	mydata = &data;//mydata指向的是一个局部变量data的地址,函数返回后,data的数据被释放
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,mydata->a[i]);
	}
	printf("mydata->b=%s\n",mydata->b);
	
	return mydata;//函数返回后,mydata实际指向的是一个野指针
}

int main()
{
	sParam mydata = {0};
	get_param1(&mydata);
	
	//不能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata.a[%d]=%d\n",i,mydata.a[i]);
	}
	printf("mydata.b=%s\n",mydata.b);
	
	sParam *p_mydata = {0};
	p_mydata = get_param2();
	
	//不能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,p_mydata->a[i]);
	}
	printf("mydata->b=%s\n",p_mydata->b);
	
    while (1)
        sleep(5);
}

如果说一定要用指针传递数据，那么可以稍微修改即可拿到正确的数据：

#include <stdlib.h>
#include <stdio.h>

typedef struct TEST_STRUCT
{
	int a[4];
	char b[20];
} sParam, *p_sParam;

sParam g_data = {0};
sParam *get_param2()
{
	sParam data = {0};
	for(int i=0; i<4; i++)
	{
		data.a[i] = 2;
	}
	strcpy(data.b, "22222");
	
	//能够打印出data的数据
	for(int i=0; i<4; i++){
		printf("data.a[%d]=%d\n",i,data.a[i]);
	}
	printf("data.b=%s\n",data.b);
	
	memcpy(&g_data, &data, sizeof(data));//将局部变量拷贝给全局变量
	return &g_data;//返回全局变量的地址
}

int get_param1(sParam *mydata)
{
    if(NULL == mydata)
        return -1;
	sParam data = {0};
	for(int i=0; i<4; i++)
	{
		data.a[i] = 1;
	}
	strcpy(data.b, "hhhh");
	
	//mydata = &data;//mydata指向的是一个局部变量data的地址,函数返回后,data的数据被释放
	memcpy(mydata, &data, sizeof(data));
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,mydata->a[i]);
	}
	printf("mydata->b=%s\n",mydata->b);
	
	return mydata;//函数返回后,mydata已经拿到数据
}

int main()
{
	sParam mydata = {0};
	get_param1(&mydata);
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata.a[%d]=%d\n",i,mydata.a[i]);
	}
	printf("mydata.b=%s\n",mydata.b);
	
	sParam *p_mydata = {0};
	p_mydata = get_param2();
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,p_mydata->a[i]);
	}
	printf("mydata->b=%s\n",p_mydata->b);
	
    while (1)
        sleep(5);
}

当然，使用指针设置或者获取数据，也需要注意避免破坏源数据，如果拿捏不好，则使用非指针变量。

也许我们觉得一个函数调用而已，get_param2()就用上了全局变量，这样不好，而且get_param1(sParam *mydata)中使用了memcpy，也会发生数据拷贝，用指针的优势就被抵消了，反而还增加了误操作的风险，考虑到这些，可以修改成：

#include <stdlib.h>
#include <stdio.h>

typedef struct TEST_STRUCT
{
	int a[4];
	char b[20];
} sParam, *p_sParam;

sParam get_param2()
{
	sParam data = {0};
	for(int i=0; i<4; i++)
	{
		data.a[i] = 2;
	}
	strcpy(data.b, "22222");
	
	//能够打印出data的数据
	for(int i=0; i<4; i++){
		printf("data.a[%d]=%d\n",i,data.a[i]);
	}
	printf("data.b=%s\n",data.b);

	return data;//返回data数据本身,随后局部变量data被释放,但数据已经返回回去了
}

int get_param1(sParam *mydata)
{
    if(NULL == mydata)
        return -1;
	//sParam data = {0};
	for(int i=0; i<4; i++)
	{
		mydata->a[i] = 1;
	}
	strcpy(mydata->b, "hhhh");
	
	//mydata = &data;//mydata指向的是一个局部变量data的地址,函数返回后,data的数据被释放
	//memcpy(mydata, &data, sizeof(data));
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,mydata->a[i]);
	}
	printf("mydata->b=%s\n",mydata->b);
	
	return mydata;//函数返回后,mydata已经拿到数据
}

int main()
{
	sParam mydata = {0};
	get_param1(&mydata);
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata.a[%d]=%d\n",i,mydata.a[i]);
	}
	printf("mydata.b=%s\n",mydata.b);
	
	sParam mydata2 = {0};
	mydata2 = get_param2();
	
	//能够打印出mydata的数据
	for(int i=0; i<4; i++){
		printf("mydata->a[%d]=%d\n",i,mydata2.a[i]);
	}
	printf("mydata->b=%s\n",mydata2.b);
	
    while (1)
        sleep(5);
}

......