转帖地址:http://blog.csdn.net/xiven/archive/2009/07/13/4345199.aspx
错误程序:
void GetMemory( char *p ){ p = (char *) malloc( 100 );}void Test( void ) { char *str = NULL; GetMemory( str ); strcpy( str, "hello world" ); printf( “%s”,str );}
这个一个考验对指针理解的题目,上面程序在运行之后:
1,调用GetMemory( str )后, str并未产生变化,依然是NULL.只是改变的str的一个拷贝的内存的变化
2,strcpy( str, "hello world" );程序运行到这将产生错误。
3,new的时候有可能内存出错,应该在*p = (char *) malloc( num ); 后判断内存是否申请成功,应加上: if ( *p == NULL ) { ...//进行申请内存失败处理 }
4,动态创建的内存没释放。
错误分析:
而双重指针为什么就可以了呢: GetMemory(&str); //把str的地址传进去 void GetMemory(char ** p) // p是str地址的一个副本 {
*p = (char *)new char[100]; // p指向的值改变,也就是str的值改变。 }
修改方法1:(推荐使用这种方法)
void GetMemory2(char **p)变为二级指针. void GetMemory2(char **p, int num) { *p = (char *)malloc(sizeof(char) * num); } void Test(void){ char *str=NULL; GetMemory=(&str); strcpy(str,"hello world"); printf(str); }
修改方法2:
char *GetMemory(){ char *p=(char *)malloc(100); return p; } void Test(void){ char *str=NULL; str=GetMemory(); strcpy(str,"hello world"); printf(str); }
附录A(相关资料)
试题5:char *GetMemory( void ){ char p[] = "hello world"; return p; }
void Test( void ){ char *str = NULL; str = GetMemory(); printf( str ); } 试题6:void GetMemory( char **p, int num ){ *p = (char *) malloc( num );}
void Test( void ){ char *str = NULL; GetMemory( &str, 100 ); strcpy( str, "hello" ); printf( str ); } 试题7:
void Test( void ){ char *str = (char *) malloc( 100 ); strcpy( str, "hello" ); free( str ); ... //省略的其它语句}
解答:
试题5中char p[] = "hello world"; return p; 的p[]数组为函数内的局部自动变量,在函数返回后,内存已经被释放。这是许多程序员常犯的错误,其根源在于不理解变量的生存期。
试题6中 1、GetMemory避免了试题4的问题,传入GetMemory的参数为字符串指针的指针,但是在GetMemory中执行申请内存及赋值语句*p = (char *) malloc( num ); 后未判断内存是否申请成功,应加上:if ( *p == NULL ){ ...//进行申请内存失败处理} 2、试题6的Test函数中也未对malloc的内存进行释放。
试题7中 存在与试题6同样的问题,在执行char *str = (char *) malloc(100); 后未进行内存是否申请成功的判断;另外,在free(str)后未置str为空,导致可能变成一个“野”指针,应加上: str = NULL;
剖析:
试题4~7考查面试者对内存操作的理解程度,基本功扎实的面试者一般都能正确的回答其中50~60的错误。但是要完全解答正确,却也绝非易事。
对内存操作的考查主要集中在:
(1)指针的理解;
(2)变量的生存期及作用范围;
(3)良好的动态内存申请和释放习惯。
再看看下面的一段程序有什么错误:
swap( int* p1,int* p2 ){ int *p; *p = *p1; *p1 = *p2; *p2 = *p;}
在swap函数中,p是一个“野”指针,有可能指向系统区,导致程序运行的崩溃。在VC++中DEBUG运行时提示错误“Access Violation”。该程序应该改为:
swap( int* p1,int* p2 ){ int p; p = *p1; *p1 = *p2; *p2 = p;}