函数中局部变量的返回

案例:

char* TEST::test01()
{
    int fd = -1,fr = -1,wf = -1;
    char status[20] = FLASH_COMPLETED;
    wf = waitfor("/usr/nfs_share/test",5000,100);
    if (wf == -1)
    {
        strcpy(status,FLASH_COMPLETED);
        MW_LOGE("mount /usr/nfs_share/test failed!!!");
    }
    else
    {
        fd = open("/usr/nfs_share/test", O_RDONLY);
        if (fd != -1)
        {
            fr = read(fd, status, 20);
            if (fr > 0)
            {
                MW_LOGW("read success !!!");
            }
            else
            {
                MW_LOGE("read error !!!,fr = [%d], error code = [%d]", fr, errno);
            }
            close(fd);
            MW_LOGW("last flash status = [%s]\n", status);
        }
        else
        {
            strcpy(status,FLASH_COMPLETED);
            MW_LOGE("open /usr/nfs_share/test failed, error code = [%d]", errno);
        }
    }
    return status;
}

在另一处调用

strncmp(FLASH_NOT_COMPLETED,mTEST->test01().c_str(),20))

运行strncmp()时发生coredump,原因分析:
strncmp()中的mTEST->test01()为空指针

因为test01()函数返回的status为局部变量status[20]的指针,在这个函数运行完后,会释放掉这个局部变量,因此返回值为空指针

修改方法:
1、声明一个全局变量,返回这个全局变量
2、使用string代替

string TEST::test01()
{
    int fd = -1,fr = -1,wf = -1;
    char status[20] = FLASH_COMPLETED;
    wf = waitfor("/usr/nfs_share/test",5000,100);
    if (wf == -1)
    {
        strcpy(status,FLASH_COMPLETED);
        MW_LOGE("mount /usr/nfs_share/test failed!!!");
    }
    else
    {
        fd = open("/usr/nfs_share/test", O_RDONLY);
        if (fd != -1)
        {
            fr = read(fd, status, 20);
            if (fr > 0)
            {
                MW_LOGW("read success !!!");
            }
            else
            {
                MW_LOGE("read error !!!,fr = [%d], error code = [%d]", fr, errno);
            }
            close(fd);
            MW_LOGW("last flash status = [%s]\n", status);
        }
        else
        {
            strcpy(status,FLASH_COMPLETED);
            MW_LOGE("open /usr/nfs_share/test failed, error code = [%d]", errno);
        }
    }
    return string(status);
}

百度查找相关知识点后,总结如下:

一般说来,函数中是可以进行局部变量的返回的,不然岂不是全部要用全局变量,如果使用了全局变量,那还有 必要进行返回吗?那函数就没有它存在的意义了!但是要注意了,这里所谓的局部变量的返回很有内涵,什么样的值才可以进行返回而不出错?
其实,只要遵守一句话即可:函数不能返回指向栈内存的指针!

为什么?因为返回的都是值拷贝!
我们知道,局部变量的作用域是函数内部,函数一旦执行结束,栈上的局部变量会进行销毁,内存得到释放。因此,此时函数返回的是该局部变量的值拷贝,这是没有问题的。但是如果返回的是局部变量的地址,那么返回的只是该局部变量指针的拷贝,而随着函数运行结束,该拷贝指针所指向的栈内存已经被释放,那么指向一个未知区域就会导致调用的错误。

那如果返回的指针指向的是堆内存,又会怎么样?
这样的使用是没有问题的,在函数内new空间,在函数外delete空间。但是这样并不是一种好的编程风格,尽量在同一个作用域内进行new和delete操作,否则还要调用者手动进行内存的释放,试问这样的接口是不是很烂。如果确实需要这样做,那就传指针进去吧!
好吧,通过几个典型的例子看一下,返回局部变量要注意的地方。

1.正确。最normal的情况。

int returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<returnValue();
	return 0;
}
 
char returnValue()
{
	int value=3;
	return value;
}

2.错误。最normal错误。虽然value被释放,但是它的值不一定会被清除,所以有时候你这么用看起来结果好像也是对的,但是隐患无穷。

int* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<*(returnValue());
	return 0;
}
 
int* returnValue()
{
	int value=3;
	return &value;
}

3.正确。不用奇怪,“HelloJacky”是一个字符串常量,储存在只读数据段,return str只是返回了该字符串在只读数据段所在的首地址,当函数退出后,该字符串所在的内存不会被回收,所以是正常的。

char* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<returnValue();
	return 0;
}
 
char* returnValue()
{
	char* str="HelloJacky";
	return str;
}

4.错误。这一回“HelloJacky”是栈内的局部变量,函数退出时内存被释放,因此返回栈内局部变量的地址是错误的。

char* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<returnValue();
	return 0;
}
 
char* returnValue()
{
	char str[]="HelloJacky";
	return str;
}

5.正确。如果你非要返回一个局部变量的地址,那么加上static吧。

char* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<returnValue();
	return 0;
}
 
char* returnValue()
{
	static char str[]="HelloJacky";
	return str;
}

6.错误,一样的,数组也不能作为函数的返回值,因为数组名其实是局部变量的首地址。

int* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<*(returnValue());
	return 0;
}
 
int* returnValue()
{
	int value[3]={1,2,3};
	return value;
}

7.正确。加上static修饰符吧,那数组也可以返回了。

int* returnValue();
 
int _tmain(int argc, _TCHAR* argv[])
{
	std::cout<<*(returnValue());
	return 0;
}
 
int* returnValue()
{
	static int value[3]={1,2,3};
	return value;
}

8.正确。函数内申请空间,调用后释放空间,只是这样做的坏处就如上面所说接口不灵活。

char* newMemory(int size);
 
int _tmain(int argc, _TCHAR* argv[])
{
	char* p=newMemory(2);
	if(p!=NULL)
	{
		*p='a';
	}
	std::cout<<*p;
	delete [] p;
	return 0;
}
 
char* newMemory(int size)
{
	char* p=NULL;
	p=new char[size];
	return p;
}

文中引用的文章
https://blog.csdn.net/jackystudio/article/details/11523353

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值