有关GetMemory()问题的思考

昨天笔试遇到这些个题目，只知道错在哪里，但不知道真正程序运行后的结果会是怎么样。回来查一下做个笔记。

例1.void GetMemory( char *p ){　p = (char *) malloc( 100 );}void Test( void ) {　char *str = NULL;　GetMemory( str ); 　strcpy( str, "hello world" );　printf( str );}//请问运行Test函数后会是什么样的结果？分析：上面程序在运行之后： 1，调用GetMemory( str )后， str并未产生变化，依然是NULL.只是改变的str的一个拷贝的内存的变化     2，strcpy( str, "hello world" );程序运行到这将产生错误。 //错误代码：segmentation fault 3，new的时候有可能内存出错，应该在*p = (char *) malloc( num ); 后判断内存是否申请成功，应加上：   if ( *p == NULL )   {　    ...//进行申请内存失败处理   }4，动态创建的内存没释放。GetMemory(str);             //把str传进去，str是一个指针，而他实际上是一个int        void   GetMemory(char   *p)     //   p是str的一个副本     {     p=(char   *)new   char[100];         //   p的值改变，但是str的值并没有改变。     }     而双重指针为什么就可以了呢：     GetMemory(&str);             //把str的地址传进去         void   GetMemory(char   **   p)     //   p是str地址的一个副本     {      *p   =   (char   *)new   char[100];         //   p指向的值改变，也就是str的值改变。     }

例2-1.char *GetMemory( void ){ 　char p[] = "hello world"; 　return p; }void Test( void ){ 　char *str = NULL; 　str = GetMemory(); 　printf( str ); }//请问运行Test函数后会是什么样的结果？分析：GetMemory中的p[]为函数内的局部自动变量，在函数返回后，内存已经被释放。这是很多程序员常犯的错误，其根源在于不理解变量的生存期。用调试器逐步跟踪Test，发现执行str=GetMemory语句后str不再是NULL指针，但是str的内容不是“hello world”，而是垃圾。例2-2.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;}//正确没有问题分析：虽然都是局部变量，例2-2用函数返回值来传递动态分配的内存，分配在“堆”中，声明周期为程序运行结束时消亡；而例2-1 return语句返回指向“栈”内存的指针，该内存在函数结束时自动消亡。

//例3：void GetMemory( char **p, int num ){　*p = (char *) malloc( num );}void Test( void ){　char *str = NULL;　GetMemory( &str, 100 );　strcpy( str, "hello" ); 　printf( str ); }//请问运行Test函数后会是什么样的结果？分析：【运行正确，但有内存泄露】题目三避免了题目一的问题，传入GetMemory的参数为字符串指针的指针，但是在GetMemory中执行申请及赋值语句*p = (char *) malloc( num );  后未判断内存是否申请成功，应加上if ( *p == NULL )  {  　...//进行申请内存失败处理  }  其次Test函数中也未对malloc的内存进行释放。也可以将指针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);  }  

例4：void Test( void )  {  　char *str = (char *) malloc( 100 );  　strcpy( str, "hello" );  　free( str );   　... //省略的其它语句  }  【运行正确，但有内存泄露】题目四与题目三存在同样的问题，在执行malloc后未进行内存是否申请成功的判断。if ( *str == NULL )  {  　...//进行申请内存失败处理  } 此外，在free(str)后未置str为空，导致可能变成一个“野指针”，应加上str = NULL;  

//例5：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");则程序会中断，并提示内存错误。

对内存操作的考查主要集中在：

（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;
}

http://blog.csdn.net/zhoubl668/article/details/6617130
http://blog.csdn.net/xiven/article/details/4345199