C++面试之GetMemory问题

题目一：

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。编译器总是要为每个参数制作临时副本，指针参数p的副本是_p，编译器使_p=p。如果函数体内的程序修改了_p的内容，就导致参数p的内容作相应的修改，这就是指针可以用作输出参数的原因。在本例中，_p申请了新的内存，只是把_p所指的内存地址改变了，但是p丝毫未变。所以GetMemory并不能输出任何东西。事实上，每执行一次GetMemory就会泄露一块内存，因为没有用free释放内存。

题目二：

char *GetMemory( void )
{ 
　char p[] = "hello world"; 
　return p; 
}

void Test( void )
{ 
　char *str = NULL; 
　str = GetMemory(); 
　printf( str ); 
}

【运行错误】GetMemory中的p[]为函数内的局部自动变量，在函数返回后，内存已经被释放。这是很多程序员常犯的错误，其根源在于不理解变量的生存期。用调试器逐步跟踪Test，发现执行str=GetMemory语句后str不再是NULL指针，但是str的内容不是“hello world”，而是垃圾。

题目三：

void GetMemory( char **p, int num )
{
　*p = (char *) malloc( num );
}

void Test( void )
{
　char *str = NULL;
　GetMemory( &str, 100 );
　strcpy( str, "hello" ); 
　printf( str ); 
}

【运行正确，但有内存泄露】题目三避免了题目一的问题，传入GetMemory的参数为字符串指针的指针，但是在GetMemory中执行申请及赋值语句

*p = (char *) malloc( num );

后未判断内存是否申请成功，应加上

if ( *p == NULL )
{
　...//进行申请内存失败处理
}

也可以将指针str的引用传给指针p，这样GetMemory函数内部对指针p的操作就等价于对指针str的操作：

void GetMemory( char *&p)     //对指针的引用，函数内部对指针p的修改就等价于对指针str的修改
{
	p = (char *) malloc( 100 );
}

void Test(void)
{
	char *str=NULL;
	GetMemory(str);
	strcpy( str, "hello world" );
	puts(str);
}

题目四：

void Test( void )
{
　char *str = (char *) malloc( 100 );
　strcpy( str, "hello" );
　free( str ); 
　... //省略的其它语句
}

【运行正确，但有内存泄露】题目四与题目三存在同样的问题，在执行malloc后未进行内存是否申请成功的判断。此外，在free(str)后未置str为空，导致可能变成一个“野指针”，应加上

str = NULL;

题目三的Test函数中也未对malloc的内存进行释放。

题目五：

char* GetMemory(int num)
{
	char* p = (char*)malloc(100);
	return p;
}

void Test(void)
{
	char* str = NULL;
	str = GetMemory(100);
	strcpy(str, "hello");
	cout<<str<<endl;
}

【运行正确】注意题目五和题目二的区别。虽然都是局部变量，但题目五用函数返回值来传递动态内存；而题目二return语句返回指向“栈”内存的指针，因为该内存在函数结束时自动消亡。
题目六：

char* GetMemory(void)
{
	char* p = "hello world";
	return p;
}

void Test(void)
{
	char* str = NULL;
	str = GetMemory();
	cout<<str<<endl;
}	

【运行正确，但不合理】虽然Test运行不会出错，但是函数GetMemory的设计概念却是错误的。因为GetMemory内的“hello world”是常量字符串，位于静态存储区，它在程序生命期内恒定不变。无论什么时候调用GetMemory，它返回的始终是同一个“只读”的内存块。例如，如想执行

strcpy(str, "hello test");

则程序会中断，并提示内存错误。

题目七：

int* GetMemory(int* ptr)
{
	ptr = new int(999);
	return ptr;
}

int main()
{
	int *ptr1 = 0, *ptr2 = 0;
	ptr1 = GetMemory(ptr2);
	if(ptr1) { cout<<*ptr1<<'\n'; } else { cout<<"ptr1 == NULL\n"; }
	if(ptr2) { cout<<*ptr2<<'\n'; } else { cout<<"ptr2 == NULL\n"; }
	
	system("pause");
	return 0;
}

程序输出：

999

ptr2 == NULL

参考链接：

1. http://www.cppblog.com/mydriverc/articles/35389.html

2. http://blog.csdn.net/zhoubl668/article/details/6617130