通过循环数组来归档每个字符串

Question

我目前正在制作一个软件，它允许用户输入最多 6 个目录，每个目录都保存为一个字符串（在一个数组中），然后循环意味着检查数组和任何不为空的目录，即实际上分配了一个目录意味着将其压缩到自己的存档中。这是我到目前为止的代码。

        private void ZipIt()
        {
        int nxtFileNum = 0;
        string Destination = @"C:\tmpZip" + nxtFileNum + ".zip";
        // Check all fields, check if empty, if not save to Selection array
        // Seems a inefficient - Possibly loop through Text box control type and collect?
        if (String.IsNullOrEmpty(tboxSelect1.Text) == false) { BckupArray[0] = tboxSelect1.Text; };
        if (String.IsNullOrEmpty(tboxSelect2.Text) == false) { BckupArray[1] = tboxSelect2.Text; };
        if (String.IsNullOrEmpty(tboxSelect3.Text) == false) { BckupArray[2] = tboxSelect3.Text; };
        if (String.IsNullOrEmpty(tboxSelect4.Text) == false) { BckupArray[3] = tboxSelect4.Text; };
        if (String.IsNullOrEmpty(tboxSelect5.Text) == false) { BckupArray[4] = tboxSelect5.Text; };
        if (String.IsNullOrEmpty(tboxSelect6.Text) == false) { BckupArray[5] = tboxSelect6.Text; };

        // Create a new ZipFile entity and then loop through each array member, checking if
        // it has an assigned value, if so compress it, if not, skip it.
        using (ZipFile ZipIt = new ZipFile())
        {
            nxtFileNum++;
            foreach (String q in BckupArray)
            {
                if (q != null)
                {
                    ZipIt.AddDirectory(q);
                    ZipIt.Comment = "This archive was created at " + System.DateTime.Now.ToString("G");
                    ZipIt.Save(Destination);
                }
            }
        }        
    }

我想做的是将第一个用户给定位置保存到 tmpZip0.7z，第二个用户给定位置保存到 tmpZip1.7z，依此类推，但目前它所做的只是将每个目录添加到 tmpZip0.zip。

---

另外，作为旁注，我如何让它在选择要存档的目录之后命名每个存档？

我目前正在使用 DotNetZip (Ionic.Zip) dll。

我希望我给了大家足够的信息。

Answer 1

您需要切换一些内容：

foreach (String q in BckupArray)
{
    nxtFileNum++;
    if (q != null)
    {
        using (ZipFile ZipIt = new ZipFile())
        {
            string Destination = @"C:\tmpZip" + nxtFileNum + ".zip";
            ZipIt.AddDirectory(q);
            ZipIt.Comment = "This archive was created at " + 
                            System.DateTime.Now.ToString("G");
            ZipIt.Save(Destination);
        }
    }
}     

原因：

1. 字符串 Destination 在创建后就被固定了。它不会改变，只是因为您增加了 nxtFileNum。
2. 您只创建了一个 ZipFile 并且仅递增了 nxtFileNum 一次，因为它们位于 foreach 循环之外
3. 将创建 ZipFile 的部分放入if 确保只有在真正使用实例时才会创建实例。

Answer 2

好吧，您可以这样做：

var strings = Controls.OfType<TextBox>()
                      .Select(x => x.Text)
                      .Where(text => !string.IsNullOrEmpty(text))
                      .ToList();

using (ZipFile ZipIt = new ZipFile())
{
    nxtFileNum++;
    string comment = string.Format("This archive was created at {0:G}",
                                   DateTime.Now);
    foreach (string directory in strings)
    {
        ZipIt.AddDirectory(directory);
        ZipIt.Comment = comment;
        ZipIt.Save(Destination + "." + nxtFileNum);
    }
}   

不过，这显然会拉动所有文本框。另一种方法是使用 List 类型的集合或类似的集合来代替六个不同的变量。

请注意，即使用户没有指定前三个名称，始终会创建 .1、.2、.3 等。如果您想绝对忠实于用户给出的定位，请告诉我。

顺便说一句，我不清楚您是否真的应该重用相同的 ZipFile 对象。我希望这更合适：（

string comment = string.Format("This archive was created at {0:G}",
                               DateTime.Now);
int fileIndex = 0;
foreach (string directory in strings)
{
    fileIndex++;
    using (ZipFile zipFile = new ZipFile())
    {
        zipFile.AddDirectory(directory);
        zipFile.Comment = comment;
        zipFile.Save(Destination + "." + fileIndex);
    }
}

顺便说一下，请注意我如何将变量重命名为更常规 - 变量通常以小写字母开头。）