有关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