联网驱动器列表答案

【问题标题】：List of Networked drives联网驱动器列表
【发布时间】：2013-10-23 03:16:27
【问题描述】：

我使用了一个很好的例子here 来获取我的驱动器列表。它似乎正在工作，但我很确定我有一个逻辑错误，因为它只列出了我最后一个“本地”和我最后一个“网络”驱动器。如果有人可以提供建议，那就太好了。

这是我的代码：

protected void Page_Load(object sender, EventArgs e)
{
    DriveInfo[] drives = DriveInfo.GetDrives();
    foreach (DriveInfo drive in drives)
    {
        bool isLocal = IsLocalDrive(drive.Name);
        if (isLocal)
        {
            loc = drive.Name;
        }
        else
        {
            net = drive.Name;
        }            
    }

    local = loc + " ~ ";
    network = net + " ~ ";
}

和

   protected void Button1_Click(object sender, EventArgs e)
   {
      Label1.Text = "Local drives: " + local;
      Label2.Text = "Network drives: " + network;
   }

这只会产生：

本地驱动器：D:\ ~

网络驱动器：Z:\ ~

正如我所料：

本地驱动器：A:\ ~ C:\ ~ D:\ ~

网络驱动器：H:\~I:\~J:\~P:\~U:\~V:\~W:\~X:\~Z:\~

【问题讨论】：

我已经编辑了你的标题。请参阅“Should questions include “tags” in their titles?”，其中的共识是“不，他们不应该”。
您知道这只会使驱动器位于服务器上而不是客户端计算机上，对吗？只要这是您想要的，那很好，但是您标记了 ASP.Net，所以我假设有人正在通过网络访问您的服务器。不知道知道哪些本地服务器驱动器可用会对访问者做什么......
@JohnSaunders 谢谢，我会记住的。
@MichaelTodd 我一定误解了这个例子。我会继续研究，因为最终结果将是为用户提供一个保存数据的地方，但应该只选择他们的网络驱动器（而不是本地驱动器），以便整个“团队”都可以访问。
@MichaelTodd 的意思是您将永远能够通过使用 Web 应用程序获取客户端计算机上的驱动器列表。这将是严重的安全漏洞。

标签： c# asp.net

【解决方案1】：

您只看到最后一个字母，因为您在 foreach 循环的每次迭代中完全覆盖了字符串。相反，您应该附加到值：

local += string.Format("{0} ~ ", loc);
network += string.Format("{0} ~ ", net);

【讨论】：

谢谢，我知道它必须是这样的。我只是没有看到它。
在将信息累积到字符串中时，还可以考虑使用 StringBuilder。

【解决方案2】：

你的直接问题是这个

改用这个：

 loc += drive.Name + " ";

 net += drive.Name + " ";

建议：

使用 StringBuilder 创建字符串。

            StringBuilder localSB = new StringBuilder();
            StringBuilder netSB = new StringBuilder();

            DriveInfo[] drives = DriveInfo.GetDrives();
            foreach (DriveInfo drive in drives)
            {
                string loc = string.Empty;
                string net = string.Empty;

                bool isLocal = IsLocalDrive(drive.Name);
                if (isLocal)
                {
                    loc = drive.Name;

                }
                else
                {
                    net = drive.Name;

                }

                if (!String.IsNullOrEmpty(loc))
                {
                    localSB.Append(string.Format("{0} ~ ", loc));
                }
                if (!String.IsNullOrEmpty(net))
                {
                    netSB.Append(string.Format("{0} ~ ", net));
                }
            }

            string localFinal = localSB.ToString();
            string netFinal = netSB.ToString();

【讨论】：