关于 if 语句的更短代码

Question

我想尝试以下代码：

//all arrays are List<T> type.
if (m.terms[0] != null && m.terms[0].labels != null && m.terms[0].labels[0].title == "Part-of-speech")
{
    result = true; 
}

但在以下情况下偶尔会出现运行时错误

i. m.terms == null

ii. m.terms != null，但 m.terms[0] 未初始化。

iii. m.terms != null, 并且 m.terms[0] 已经存在但是 m.terms[0].label 未初始化。

...

所以我确实将其修改为这样：

if (m.terms[0] != null)
{
    if (m.terms[0].labels != null)
    {
        if (m.terms[0].labels[0].title == "Part-of-speech") { result = true; }
    }
}

这是最好的方法吗？

Answer 1

int index = 0;
int labelIndex = 0;
string titleToCheck = "Part-of-speech";

if (m != null && m.terms != null && m.terms.Count > index)// or m.Length...
{
    if (m.terms[index] != null && m.terms[index].labels != null &&
        m.terms[index].labels.Count > labelIndex)
    {
        if (m.terms[index].labels[labelIndex].title == titleToCheck)
        {
            result = true; 
        }
    }
}

Answer 2

这都是关于可读性的。 C# 使用短路评估，因此在功能上没有区别。

Answer 3

试试这个

if (m!=null && m.terms!= null && m.terms[0].labels!=null && m.terms[0].labels[0].title!=null && m.terms[0].labels[0].title == "Part-of-speech")

Answer 4

是的，最好将每个 null 检查分成单独的 if 语句。

原因是第二个和第三个条件要求第一个条件不为空。如果第一个条件为空，那么第二个和第三个条件将依次抛出错误，因为它们的父条件为空但正在尝试访问。

Answer 5

&& 是一个短路运算符，因此您编写的第一种方式和第二种方式在功能上是等效的。

if (a && b && c)
{
    // work 
}

仅当 a 返回 true 时才会评估 b。 （c 也是如此）。

在您的代码中，检查 m.terms[0].labels 不会出现问题，因为 if m.terms[0] 会导致表达式短路已经为空。

但是，为了完全覆盖自己，您可能需要添加对 m 和 m.terms 的检查。

m != null && m.terms != null && m.terms.Count > 0 && m.terms[0] != null ...

当它从左到右评估时，它将在第一个不通过的条件下中断，其余的将不受检查。