JavaScript try-catch是否忽略预期的偶然错误错误做法?
时间:2020-03-06 14:48:10 来源:igfitidea点击:
在JavaScript中,使用try-catch块并忽略错误而不是在块中测试许多属性是否为null是错误的吗?
try{ if(myInfo.person.name == newInfo.person.name && myInfo.person.address.street == newInfo.person.address.street && myInfo.person.address.zip == newInfo.person.address.zip) { this.setAddress(newInfo); } } catch(e) {} // ignore missing args
解决方案
对于给出的示例,我会说这是不好的做法。但是,在某些情况下,简单地捕获预期的错误可能会更有效。在将字符串转换为GUID之前验证字符串的格式将是一个很好的示例。
我认为,如果我们要捕获异常,请对其进行处理。否则,让它冒出气泡,以便更高级别可以某种方式处理它(即使只是浏览器向我们报告错误)。
是的。一方面,除了缺少参数外,还可能由于多种原因引发异常。包罗万象将隐藏那些可能不希望的情况。
如果我们期望某个特定条件,则如果我们明确测试它,则代码将更易于维护。我将上面的内容写成
if( myInfo && newInfo && myInfo.person && newInfo.person && myInfo.person.address && newInfo.person.address && ( myInfo.person.name == newInfo.person.name && myInfo.person.address.street == newInfo.person.address.street && myInfo.person.address.zip == newInfo.person.address.zip ) ) { this.setAddress(newInfo); }
例如,这使效果更加清晰,假设newInfo已全部填写,但是myInfo的一部分缺失了吗?也许我们确实希望在这种情况下调用setAddress()?如果是这样,则需要更改该逻辑!
值得一提的是,在IE中,即使规格说明可以,也不能使用try / finally组合。为了执行"最终",即使已为空,也必须定义一个catch块。
//this will [NOT] do the reset in Internet Explorer try{ doErrorProneAction(); } finally { //clean up this.reset(); } //this [WILL] do the reset in Internet Explorer try{ doErrorProneAction(); } catch(ex){ //do nothing } finally { //clean up this.reset(); }
我们总是可以编写一个辅助函数来为我们做检查:
function pathEquals(obj1, obj2, path) { var properties = path.split("."); for (var i = 0, l = properties.length; i < l; i++) { var property = properties[i]; if (obj1 === null || typeof obj1[property] == "undefined" || obj2 === null || typeof obj2[property] == "undefined") { return false; } obj1 = obj1[property]; obj2 = obj2[property]; } return (obj1 === obj2); } if (pathEquals(myInfo, newInfo, "person.name") && pathEquals(myInfo, newInfo, "person.address.street") && pathEquals(myInfo, newInfo, "person.address.zip")) { this.setAddress(newInfo); }