c++使用HRESULT返回S_OK
C++ use of HRESULT to return S_OK
在我正在处理的遗留c++代码中,我注意到许多返回HRESULT的方法遵循将局部变量设置为S_OK的做法,然后在方法结束时返回它而不更改它。
HRESULT function()
{
HRESULT hr = S_OK;
// do some stuff
// none of which changes hr
...
return hr;
}
以上代码有什么好处吗?或者我可以安全地将它重构为
void function()
{
// do some stuff
// none of which changes hr
...
}
您建议的重构应该可以正常工作。只要注意,如果代码中有任何地方使用了函数的返回值,就会出现编译错误,并且必须修复这些地方。
还要注意,在代码的未来版本中,将更难向该函数添加错误情况,因为您将不得不重新引入返回值并更改所有调用者。最初的开发人员返回S_OK
可能是出于一致性的考虑,或者是预期该函数有一天可能会产生错误。
如果你的函数总是成功(或者换句话说永远不会执行失败的东西),你可以摆脱它。这取决于内容和函数的作用。如果您切换到void
并调用您的函数,您将永远不会知道调用是否"成功"。
请记住,函数体在将来可能会更改,当您突然需要返回错误时,您必须更新调用以正确处理错误。
相关文章:
- 来自 std::list 的迭代器 .end() 按预期返回"0xcdcdcdcdcdcdcdcd"但 .begin()
- 什么时候在C++中返回常量引用是个好主意
- 你能重载对象变量名本身返回的内容吗
- 为什么 Serial.println(<char[]>);返回随机字符?
- C++映射:具有自定义类的运算符[]不起作用(总是返回0)
- 如何获取std::result_of函数的返回类型
- QueryWorkingSet总是返回false
- (C++)分析树以计算返回错误值的简单算术表达式
- 访问者访问变体并返回不同类型时出错
- 如何返回一个类的两个对象相加的结果
- OpenInventor从9.8升级到10.4.2后,GLSL纹理返回零
- lower_bound()返回最后一个元素
- "throw expression code" 1e7 >返回 d 是什么?投掷标准::overflow_error( "too big" ) : d;意味 着?
- 奇怪的(对我来说)返回声明 - 在谷歌上找不到任何关于它的信息
- 如何取消对nullptr的屏蔽,返回正确的对象
- 何时按值返回OK
- 当坏ip提供给CURLOPT_DNS_SERVERS时,Curl返回all ok
- 只写对象的链接操作符= - OK返回rhs而不是*this
- 从内存效率的角度来看,在c++ 11中返回一个字符串OK
- 为什么getpeername()返回OK,结果却不是预期的