我的重构不正确和清晰

本文关键字：清晰 不正确 重构 我的 | 更新日期: 2023-09-27 18:00:58

这是原始源代码，其中包含一个非常常见的代码模式，我认为这是一种反模式，或者充其量是不必要的冗长：

private bool SymbolDevice;
. . .
    if((oemInfo.IndexOf("SYMBOL") > -1) || (oemInfo.IndexOf("MOTOROLA") > -1))
                            SymbolDevice = true;
                        else
                        {
                            SymbolDevice = false;
                        }

我会这样重构它：

SymbolDevice = ((oemInfo.IndexOf("SYMBOL") > -1) || (oemInfo.IndexOf("MOTOROLA") > -1));

Resharper(版本2.0，Visual Studio 2003/.NET 1.1可用的最新版本，该项目是(以这种方式重构它：

SymbolDevice = (oemInfo.IndexOf("SYMBOL") > -1) || (oemInfo.IndexOf("MOTOROLA") > -1) ? true : false;

我同意 Resharper 的重构改进了旧代码，但有什么理由我会选择它而不是我的版本吗？

我的重构不正确和清晰

我肯定会选择你的版本。 x ? true : false相当于x。

相关文章：