C语言多步错误处理最好还是使用do-while(0)-break
acmilan2017/08/27软件综合 IP:四川

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

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

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

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

<code class="language-cpp">需要清理的资源变量声明;
do
{
    不需要清理的变量声明;
    可能失败的操作;
    if (失败) break;
    不需要清理的变量声明;
    可能失败的操作;
    if (失败) break;
    // ...
}while(0);
if (变量1需要清理)
    清理变量1;
if (变量2需要清理)
    清理变量2;
// ...
</code>

[修改于 7年4个月前 - 2017/08/27 03:47:11]

来自:计算机科学 / 软件综合
3
已屏蔽 原因:{{ notice.reason }}已屏蔽
{{notice.noticeContent}}
~~空空如也
celeron533
7年4个月前 IP:江苏
838971
强行套一层啊。。。
引用
评论
加载评论中,请稍候...
200字以内,仅用于支线交流,主线讨论请采用回复功能。
折叠评论
TBsoft
7年3个月前 修改于 7年3个月前 IP:湖北
839288
这是Visual Studio 2010(Visual C++ 10.0)中MFC源程序的WINMAIN.CPP文件代码(渲染代码我目前做不好,如果有经验者能帮助一下最好):

int AFXAPI AfxWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
_In_ LPTSTR lpCmdLine, int nCmdShow)
{
ASSERT(hPrevInstance == NULL);

int nReturnCode = -1;
CWinThread* pThread = AfxGetThread();
CWinApp* pApp = AfxGetApp();

// AFX internal initialization
if (!AfxWinInit(hInstance, hPrevInstance, lpCmdLine, nCmdShow))
goto InitFailure;

// App global initializations (rare)
if (pApp != NULL && !pApp->InitApplication())
goto InitFailure;

// Perform specific initializations
if (!pThread->InitInstance())
{
if (pThread->m_pMainWnd != NULL)
{
TRACE(traceAppMsg, 0, "Warning: Destroying non-NULL m_pMainWnd\n");
pThread->m_pMainWnd->DestroyWindow();
}
nReturnCode = pThread->ExitInstance();
goto InitFailure;
}
nReturnCode = pThread->Run();

InitFailure:
#ifdef _DEBUG
// Check for missing AfxLockTempMap calls
if (AfxGetModuleThreadState()->m_nTempMapLock != 0)
{
TRACE(traceAppMsg, 0, "Warning: Temp map lock count non-zero (%ld).\n",
AfxGetModuleThreadState()->m_nTempMapLock);
}
AfxLockTempMaps();
AfxUnlockTempMaps(-1);
#endif

AfxWinTerm();
return nReturnCode;
}

微软自己都是goto用得不亦乐乎,用goto也没什么不可以,特别是在要做错误后集中清理的情况下,如果没有try...catch...finally这样的控制结构,goto反而可能是一种很清晰简单的做法。

笔者从非结构化的BASIC语言(就是那种老式带行号的)开始编程,一直保留着用goto的习惯,goto并不是编程中的绝对禁忌。

任何时候,都是人选择合适的编程语言和语言特性使用,不是人被编程语言牵着鼻子走。
引用
评论
加载评论中,请稍候...
200字以内,仅用于支线交流,主线讨论请采用回复功能。
折叠评论
acmilan作者
7年3个月前 修改于 7年3个月前 IP:四川
839294

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

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

<code class="language-cpp">#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></code>

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

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

<code class="language-cpp">	需要清理的资源变量声明;
	{
		不需要清理的变量声明;
		可能失败的操作;
		if (失败) goto 清理;
		// ...
	}
清理:
	if (变量1需要清理)
		清理变量1;
	// ...
</code>
引用
评论
加载评论中,请稍候...
200字以内,仅用于支线交流,主线讨论请采用回复功能。
折叠评论

想参与大家的讨论?现在就 登录 或者 注册

所属专业
所属分类
上级专业
同级专业
acmilan
进士 学者 笔友
文章
461
回复
2934
学术分
4
2009/05/30注册,5年10个月前活动
暂无简介
主体类型:个人
所属领域:无
认证方式:邮箱
IP归属地:未同步
文件下载
加载中...
{{errorInfo}}
{{downloadWarning}}
你在 {{downloadTime}} 下载过当前文件。
文件名称:{{resource.defaultFile.name}}
下载次数:{{resource.hits}}
上传用户:{{uploader.username}}
所需积分:{{costScores}},{{holdScores}}下载当前附件免费{{description}}
积分不足,去充值
文件已丢失

当前账号的附件下载数量限制如下:
时段 个数
{{f.startingTime}}点 - {{f.endTime}}点 {{f.fileCount}}
视频暂不能访问,请登录试试
仅供内部学术交流或培训使用,请先保存到本地。本内容不代表科创观点,未经原作者同意,请勿转载。
音频暂不能访问,请登录试试
支持的图片格式:jpg, jpeg, png
插入公式
评论控制
加载中...
文号:{{pid}}
投诉或举报
加载中...
{{tip}}
请选择违规类型:
{{reason.type}}

空空如也

加载中...
详情
详情
推送到专栏从专栏移除
设为匿名取消匿名
查看作者
回复
只看作者
加入收藏取消收藏
收藏
取消收藏
折叠回复
置顶取消置顶
评学术分
鼓励
设为精选取消精选
管理提醒
编辑
通过审核
评论控制
退修或删除
历史版本
违规记录
投诉或举报
加入黑名单移除黑名单
查看IP
{{format('YYYY/MM/DD HH:mm:ss', toc)}}