ReSharper告诉我在IF语句冗余内的Boolean.TryParse（）？

Question

我想对以下IF语句做一些说明。我知道这种说法可以用许多不同的方式来写......这不是我所要求的。我很好奇为什么ReSharper 7告诉我比较canceled == true是多余的。

bool canceled; 
if (Boolean.TryParse(Request.QueryString["cancel"], out canceled) && 
    canceled == true) 
{ 
    // Transaction canceled... 
} 

这是我的理解是Boolean.TryParse()将基于转换的成功，而不是实际结果输出参数的返回真/假。为什么然后比较canceled == true是多余的？这一点很可能是错误的，对吧？

Answer 1

只需使用

if (Boolean.TryParse(Request.QueryString["cancel"], out canceled) && canceled) 
{ 
    // Transaction canceled... 
} 

由于canceled是不是空的，你不需要明确与true布尔比较，为(canceled == true) == canceled。