0

我不确定这是否是无效的做法；或良好的做法。我不知所措的原因是我应该使用属性而不是局部变量吗？

我的推理和目标；是对本地磁盘驱动器的非常基本的检测。

我想指出一些事情：

  • 我没有选择a boolean value，因为我希望能够调用这个类来返回驱动器路径。因此从方法中检索到的名称；能够Path Combined在某些派生类中。

我的例子：

    public class Drive
    {
        // Variable:
        public string nameOfDrive;

        public Drive()
        {
            // Call Method.
            DriveName();
        }

        public string DriveName()
        {
            DriveInfo [] drives = DriveInfo.GetDrives();
            foreach (DriveInfo d in drives)
            {
                // Verify Valid 'C:' is Present.
                if (d.Name == @"C:")
                {
                    // Set Name:
                    nameOfDrive = d.Name;
                    // Return Result.
                    return d.Name;
                }
            }
            // Exception:
            throw new Exception("Unable to locate the C: Drive... Please map the correct drive.");
        }

    }
    /*
     * The above method and class contains a verification
     * for the 'C:' Drive.  Once the items are validated;
     * it will create a return variable for the 'C:'.  
     * Otherwise it will throw an Exception.
    */

现在这是我不确定什么是更好的做法的地方。我应该使用属性而不是public string nameOfDrive. 还是我真的很遥远-这不是返回可在其他类中使用的值的最佳方法吗？还是直接引用成员变量是不好的做法？

第二个例子：

    public class Drive
    {
        private string nameOfDrive;
        public string NameOfDrive
        {
            get { return nameOfDrive; }
        }
        public Drive()
        {
            // Call Method.
            DriveName();
        }
        public string DriveName()
        {
            // Obtain Drive Information:
            DriveInfo [] drives = DriveInfo.GetDrives();
            foreach (DriveInfo d in drives)
            {
                // Verify Valid 'C:' is Present.
                if (d.Name == @"C:")
                {
                    // Set Name:
                    nameOfDrive = d.Name;
                    // Return Result.
                    return d.Name;
                }
            }
            // Exception:
            throw new Exception("Unable to locate the C: Drive... Please map the correct drive.");
        }
    }
    /*
     * The above method and class contains a verification
     * for the 'C:' Drive.  Once the items are validated;
     * it will create a return variable for the 'C:'.  
     * Otherwise it will throw an Exception.
    */

这样它就被标记为只读并确保它从方法中读取正确的值？

更新：

我很感激答案；但为什么它是更好的做法？

  • 对安全有益吗？
  • 更整洁？
  • 更灵活

是什么使它成为更好的解决方案；这就是我试图理解的。

4

5 回答 5

4

您可以使用Lazy该类来执行此操作。它专门用于解决延迟初始化可能需要一些时间来计算的值的确切问题。您可以为Lazy对象提供一个用于计算该值的方法，第一次请求该值时，它使用该函数生成该值，并且所有后续调用仅返回该第一个值。它还具有线程安全的优点（该函数只会被调用一次，无论有多少人在生成该值之前请求该值，并且他们都等到它被计算返回）。

public class Drive
{
    private Lazy<string> nameOfDrive = new Lazy<string>(DriveName);

    public string NameOfDrive
    {
        get { return nameOfDrive.Value; }
    }

    private static string DriveName()
    {
        DriveInfo[] drives = DriveInfo.GetDrives();

        foreach (DriveInfo d in drives)
        {
            if (d.Name == @"C:")
                return d.Name;
        }

        throw new Exception("Unable to locate the C: Drive... Please map the correct drive.");
    }
}
于 2012-12-28T20:22:13.913 回答
3

我会使用带有私有成员变量的只读属性。这样，如果您在不破坏调用代码的情况下更改查找驱动器号的方式，更新类就更容易了。

为什么你的DriveName方法返回任何东西？它是公开使用还是仅用于填充您的NameOfDrive财产？如果仅在课堂内使用，我会将其设为私有且无效。

编辑：考虑一下，这似乎也是一种奇怪的方式，不仅可以检查驱动器的存在，还可以检查开头的字母。为什么要求用户的驱动器号是C:？用户设置机器的方式无关紧要。如果他们愿意，他们应该能够拥有他们的操作系统驱动器Q:，并且它不应该破坏您的代码。

于 2012-12-28T19:44:16.210 回答
2

虽然，这不一定是坏习惯，但也不好。

在大多数情况下，当您有一个简单的数据类时应该使用字段（通常不涉及实际代码，只是一种存储一些值的方法）。如果您超出了该级别的复杂性，您通常应该让您的类使用属性。几个原因：

  1. 稍后从字段转换为属性将破坏依赖关系并要求重新编译使用您的类的所有代码
  2. 属性具有更细粒度的控制。快速浏览一下您的用例，您似乎应该有一个 getter，它会自动填充和缓存驱动器号，并使默认设置器私有，因此它是只读的
  3. 属性可以是虚拟的。这意味着人们可以更容易地将您的课程扩展到您最初想象的之外。
于 2012-12-28T20:47:18.040 回答
1

我会创建一个类（甚至是扩展）来提取 cdrive。让消费者根据他们的需要抛出错误。

通过创建一个通用方法，允许根据情况重用流程，并坚持使用面向对象的原则，将概念隔离为唯一的对象。

void Main()
{

  if (Drive.AcquireCDrive() == null)
      throw new Exception("Unable to locate the C: Drive... Please map the correct drive.");

}

public class Drive
{
    public static DriveInfo AcquireCDrive()
    {
       return DriveInfo.GetDrives()
                       .OfType<DriveInfo>()
                       .Where (drive => drive.IsReady)
                       .FirstOrDefault( drive => drive.Name.Contains(@"C:"));
    }
}
于 2012-12-28T20:16:30.550 回答
0

可能更好的做法是使用只读属性，但延迟加载它。

注意下面，我将DriveName()方法设为私有，并且没有DriveName()在构造函数中调用该方法。

public class Drive
{
    private string nameOfDrive = null;

    public string NameOfDrive
    {
        get 
        {
            if (nameOfDrive == null)
                nameOfDrive = DriveName();
            return nameOfDrive; 
        }
    }

    public Drive()
    {    }

    private string DriveName()
    {
        DriveInfo[] drives = DriveInfo.GetDrives();

        foreach (DriveInfo d in drives)
        {
            if (d.Name == @"C:")            
                return d.Name;
        }

        throw new Exception("Unable to locate the C: Drive... Please map the correct drive.");
    }
}
于 2012-12-28T19:54:02.060 回答