If 语句的更好语法

Question

我有一个这样的条件：

if (string.IsNullOrEmpty(filename) || size != "Large" || size != "Medium" || size != "Small")

将来我可能必须在 if 语句中管理更多的 size 。

我想知道是否存在一种更易于管理和可读的方式来编写此条件。

请提供一个真实的例子，感谢您抽出宝贵的时间。

Answer 1

您可以保留单词的哈希集并检查：

HashSet<string> filterWords = new HashSet<string>();
// Put all words in the hash set


if (filterWords.contains(size))
{
    // Do what ever you need
}

Answer 2

// you could externalize and manage this list somewhere else 
var sizes = new[] { "Large", "Medium", "Small" }; 

if (string.IsNullOrEmpty(filename) || !sizes.Contains(size))
{
    ...
}

Answer 3

将尺寸放入某种集合中，并使用“包含”。有关示例，请参阅 MSDN： http://msdn.microsoft.com/en-us /library/bb352880.aspx

Answer 4

如果它们要改变，也许静态列表更好：

private static List<string> Sizes = new List<string> { "large", "medium", "small" };

if (string.IsNullOrEmpty(filename) || Sizes.Contains(size.ToLower()))
{

}

对于更干净的代码，将大小检查封装到它自己的方法中，并在需要时修改该方法：

if (MeetsSizeRequirementsOrIsNull(filename, size))
{        
}

private static bool MeetsSizeRequirementsOrIsNull(string filename, string size)
{
    List<string> sizes = new List<string>() { "..." };

    return string.IsNullOrEmpty(filename) || sizes.Contains(size.ToLower())
}

Answer 5

如果测试结果你要做不止一件事，将其写成 switch/case 可能会更具可读性。

否则，如果您有大量这些值，最好将列表保存在某个字典中 - 比如说 notHandledSizes。分配“大”=>正确，“中”=> true，...只需检查该字典中的 size 是否存在且为 true。

Answer 6

选项 1：

编写一个返回 bool 值且仅包含大小测试的小函数，并在 if 中使用该函数。

if (string.IsNullOrEmpty(filename) || GoodSize(size))
{
 //...
}

private bool GoodSize(string size)
{
   return size != "Large" || size != "Medium" || size != "Small";
}

选项 2：

创建要测试的尺寸列表并使用 Contains：

var goodSizes = new[] { "Large", "Medium", "Small" };
if (string.IsNullOrEmpty(filename) || !goodSizes.Contains(size))
{
    //...
}

您可以组合这两个选项以获得更好的清晰度和封装性。

Answer 7

下面的换行符使其更具可读性。

   if (string.IsNullOrEmpty(filename) || 
    size != "Large" || 
    size != "Medium" || 
    size != "Small")

设计提示

如果许多对象涉及长 if 条件，那么最好在这些类中编写返回 true/false 的小属性/方法。

if (string.IsNullOrEmpty(filename) || object.IsProperSize)

有时枚举标志属性也有助于这种情况。请参阅此处以获得详细解释。

Answer 8

您所做的看起来是最直接的方法。任何修改都只会把混乱转移到其他地方。如果您不需要在其他地方重用此代码，我会保持原样。