If..And If...And If 的 C# 结构

Question

我使用的是 Visual Studio 2013 Express，我是 Visual C# 的新手。我确信有更好的方法来做我正在尝试的事情，我将不胜感激任何建议。

我正在尝试编写的代码会评估一系列测试，并仅在所有测试 = TRUE 时设置一个标志。我目前正在使用六个嵌套的 if 结构来完成这项工作，虽然它有效，但我正在寻找更清洁、更专业的解决方案。这是示例(在这篇文章中缩短为三个级别):

private const string sDrive = "D:\\";
private const string sFolder = "FTP\\";
private const string sDivFolder = "ABC";
private static bool bSanity = false;

private static void SanityCheck()
{
    if (Directory.Exists(sDrive))
    {
        if (Directory.Exists(sDrive + sFolder))
        {
            if (Directory.Exists(sDrive + sFolder + sDivFolder))
            {
                bSanity = true;
            }
            else
            {
                Console.WriteLine("FATAL: Div folder doesn't exist.");
            }
        }
        else
        {
            Console.WriteLine("FATAL: Root folder doesn't exist.");
        }
    }
    else
    {
        Console.WriteLine("FATAL: Disk drive doesn't exist.");
    }
}  

Answer 1

主要问题是您是否需要保留与现在相同的错误报告。一般来说，如果需要的话，我认为通过反转案例来处理更简单。这将允许您使用 if/else if 移除嵌套:

private static void SanityCheck()
{ 
    if (!Directory.Exists(sDrive))
    {
       Console.WriteLine("FATAL: Disk drive doesn't exist.");
    }
    else if (!Directory.Exists(Path.Combine(sDrive, sFolder))
    {
       Console.WriteLine("FATAL: Root folder doesn't exist.");
    }
    else if (!Directory.Exists(Path.Combine(sDrive, sFolder, sDivFolder))
    {
        Console.WriteLine("FATAL: Div folder doesn't exist.");
    }
    else
    {
        bSanity = true;
    }
}  

如果不需要详细报错，直接查看最底层文件夹即可:

private static void SanityCheck()
{

    if (Directory.Exists(Path.Combine(sDrive, sFolder, sDivFolder))
         bSanity = true;
    else
        Console.WriteLine("FATAL: Drive or folder doesn't exist.");
}