文号
838817
阅读
22452
回复
3
收藏
4
C语言多步错误处理最好还是使用do-while(0)-break
2017/08/27
IP:四川

使用嵌套的if块会嵌套过深以至于难以整理

使用平行的if块对C++/C99变量初始化不友好,【C++/C99变量初始化也是有效代码】,放if块里边限制生存期,放if块外面不受if块控制

使用【if (失败) goto 清理;】不美观,C++/C99变量初始化会有被跳过的警告

所以还是使用do-while(0)-break比较好

需要清理的资源变量声明;
do
{
    不需要清理的变量声明;
    可能失败的操作;
    if (失败) break;
    不需要清理的变量声明;
    可能失败的操作;
    if (失败) break;
    // ...
}while(0);
if (变量1需要清理)
    清理变量1;
if (变量2需要清理)
    清理变量2;
// ...
[ 修改于9年3个月前 - 2017/08/27 03:47:11 ]
acmilan作者
9年3个月前修改于 9年3个月前IP:四川
839294
3楼

不违背语言逻辑的做法都是可行的。。。

像是这种情况,就不能用goto。

#include <windows.h>
#include <tchar.h>
#include <string>
#include <iostream>
using namespace std;
int _tmain(int argc, TCHAR **argv)
{
	if (argc >= 1) goto b;
	int i = 1; // warning & assert fail
	//string s = "aaa"; // compile error
b:
	cout << i << endl;
	return 0;
}
</iostream></string></tchar.h></windows.h>

微软那个可以用goto是因为变量声明和初始化都写在函数开头附近,不存在被跳过的问题。

一般来说,如果你习惯于C89风格的那种变量声明全部写在开头的做法,完全可以随意使用goto。但是如果你习惯于ISO C++风格的那种变量声明写在程序中间的做法,就应该考虑一下其它写法。比如你如果觉得do-while(0)-break太拽,完全可以换一种写法。

	需要清理的资源变量声明;
	{
		不需要清理的变量声明;
		可能失败的操作;
		if (失败) goto 清理;
		// ...
	}
清理:
	if (变量1需要清理)
		清理变量1;
	// ...
引用
评论
笔记
{{note.content}}
{{n.user.username}}
{{fromNow(n.toc)}} {{n.status === noteStatus.disabled ? "已屏蔽" : ""}} {{n.status === noteStatus.unknown ? "正在审核" : ""}} {{n.status === noteStatus.deleted ? '已删除' : ''}}
  • 编辑
  • 删除
  • {{n.status === 'disabled' ? "解除屏蔽" : "屏蔽" }}
我也是有底线的