检查下面代码有什么问题?
void GetMemory( char *p )
{
p = (char *) malloc( 100 );
}
void Test( void )
{
char *str = NULL;
GetMemory( str );
strcpy( str, "hello world" );
printf( str );
}
参考答案
传入中GetMemory( char *p )函数的形参为字符串指针,在函数内部修改形参并不能真正的改变传入形参的实参值,执行完
char *str = NULL;
GetMemory( str );
后的str仍然为NULL;
1:传入形参并不能真正改变形参的值,执行完之后为空;
2:在函数GetMemory中和Test中没有malloc对应的free,造成内存泄露
纠错收藏
知识点:
malloc——向系统申请分配指定size个字节的内存空间,返回类型为void*类型。需强行转换为实际类型的指针。
eg:int *p;
p= (int *) malloc ( sizeof (int) );
存在问题:
函数的形参为字符串指针,在函数内部修改形参值并不能真正的改变实参值。在函数中malloc重新为形参申请了一段内存,使得形参指向新的内存,而实参str依然为NULL。
函数结束要对str进行free(),将内存块释放。
解决问题:
要为指针分配内存,应传入指针的地址,即 GetMemory( &str );
函数的参数应该用二级指针,即 GetMemory( char **p )
此时,p指向str,为*p分配内存即为str分配内存,即 *p = (char *) malloc( 100 );
#include <iostream>
using namespace std;
//传值调用
void GetMemory( char **p )
{
*p = (char *) malloc( 100 );
}
//引用调用
void GetMemory_1(char *&p)
{
p = (char *) malloc (100);
}
int main()
{
char *str = NULL;
char *str1 = NULL;
GetMemory( &str );
GetMemory_1( str1 );
strcpy( str, "hello world" );
strcpy( str1, "hello world1" );
cout<<str<<endl;
cout<<str1<<endl;
free(str);
free(str1);
return 0;
}
char p是声明一个字符类型变量p,char * p1是声明一个字符类型的指针变量p1,那么char ** p2就相当与给char类型的指针变量再声明一个指针.(2级指针)
举个例子,假如:
char p=‘a’;//假设声明了p后,p在内存的1000处,则&p=1000(&为取变量首地址)
char * p1=&p;//则p1=1000,*p1=p=‘a’,p1就和p关联了;假设p1被放在2000处
char ** p2=&p1;//则p2=2000,*p2=p1=&p=1000,**p2=&p1=p=‘a’
char *&p,char * &p,char*& p
几种写法都是同一个意思:声明p是能指向char型变量的指针的引用(引用是C++中的一种数据类型,实质是为已有的变量起一个别名)。把形式参数指针声明为引用,在函数中完成交换时就不必拷贝字符串本身,而只交换一下两个字符串的首指针就可以了。函数参数中的&表示C++的引用。char* &p表示p是一个char*类型的引用。所谓引用也就是外部变量的别名,修改形参p就是修改传入的实参p。所以在函数swap内部交换形参p和q,外部实参p和q的值也交换了。