C＃运算符和可读性

Question

我只是在处理一些代码并发现自己犯了这个错误

if (stringName == "firstName" || "lastName")
   // Do code 

显然这是错误的，应该是

if (stringName == "firstName" || stringName == "lastName")
   // Do code 

但它只是让我思考可读性会更容易吗？ 也许有一些逻辑可以说除非指定一个新的stringName，否则使用第一个？

真的不是一个问题，我只是好奇，如果有一些我不完全理解编译这样的声明背后的逻辑。

Answer 1

我认为你的提议会混淆表达式解析的规则 - 现在，'=='变成了一个quadreny（？）运算符，而不是二进制运算符。 我发现自己错过了SQL的'IN'运算符，并且使用了类似这样的东西：

if (stringName.In("foo", "bar", "baz"))
{

}

// in an extension method class
public static bool In<T>(this T value, params T[] values)
{
    return values.Contains(value);
}

Answer 2

问题是如果对布尔有效。

stringName == "firstName"返回一个布尔值。
"lastName"是一个字符串文字。

|| 是一个短路的布尔值或运算符，两侧都有布尔值。

换句话说，您想要更改||的定义 这通常是一个坏主意。

从理论上讲，你可以让解析器推断出你的意思......但这很快就会变得含糊不清。

if (stringName == firstName || lastName)

看起来不错吧？ 但究竟什么是lastName？

如果我这样做怎么办？

const bool lastName = false;

此外， &&与||相反 ，但stringName == firstName && lastName与上述逻辑不同，实际上没有意义。

Answer 3

当代码明显错误以便修复它时编译器猜测程序员的意图是一个非常非常糟糕的主意。

Answer 4

即使有括号，也没有意义。 stringName == ("firstName" || "lastName")看起来好像要测试两个字符串的真实性，并且这些字符串总是为真，然后将该布尔结果与字符串stringName进行比较。

如果你添加这样的括号(stringName == "firstName") || "lastName" (stringName == "firstName") || "lastName" ，条件也总是如此，因为无论stringName是否等于"firstName" ， "lastName"始终为true。

我喜欢Ruby的做法：

["firstName", "lastName"].include? stringName

您可以像其他人一样建议使用Contains或者将String扩展方法写入您可以执行的操作：

stringName.EqualsOneOf(new[] {"firstName", "lastName"})

Answer 5

他们不允许这种语法的原因很可能是因为可读性。 如果您是第一次查看代码，并且您并未完全处于最佳状态，则可能无法立即看到您将stringName与"firstName"和"lastName" 。 它只是让你的意图更加明确。

然后，括号可能会解决这个问题。

Answer 6

这只有在operator||(string,string)返回时才会起作用..一种字符串集合，你有一个Equals重载，它接受一个字符串和字符串集合，并验证字符串是否在集合中。 对于很少使用的构造，在幕后做了很多工作。

特别是因为你已经可以做以下事情：

if(new string[]{"firstName","lastName"}.Contains(stringName))
    // code

Answer 7

我发现Contains（）函数解决了这个问题，即：

string[] ValidNames = new string[] { "firstName", "lastName"};

if(ValidNames.Contains(stringName))
{
    //Do Code
}

Answer 8

我不介意SQL语法如下：

if(stringName in ("firsName", "lastName"))
{
}

Answer 9

这就是为什么，作为一种习惯，我总是这样做：

if ((stringName == "firstName") || (stringName == "lastName"))
   // Do code

一段时间后，它成为第二天性。

Answer 10

当FCL包含丰富的内容以创建在此主题中看到的各种各样的答案时，您不需要具有更灵活的C＃语法，因为可读性很快就会成为您如何在所有丰富内容中创建答案的功能。 它很大程度上归结为方法和对象调用之间的选择。

这是一个示例（只是众多中的另一个），它能够在一个字符串数组中同时定位一个或多个字符串，或者将您认为合适的任何其他条件应用于该组字符串。 与任何代码一样，缩进，间距和代码注释对于理解此代码示例起着重要作用。

        bool found = Array.Exists(
            // array of strings to search
            new[] { "c#", ".net", "programming", "design patterns", "work", "play", "bits", "bytes", "break" },
            // criteria - can even satisfy multiple conditions simultaneously if desired
            str => (str == ".NET" || str == "work") //look for ".NET" or "work"
            );