1

我正在拍摄图像并处理它们。然后我试图将位图（这些图像）保存在一个文件中。

例如

  • 我拍摄“沙漠”和“花卉”图像，并异步处理它们 4 次。

  • 然后我调用(await) 8 次 saveAsync() 方法来保存它们。

    • 发生的情况是desert_modified 和flower_modified、flower_modified(1)、flower_modified(2)、flower_modified(3) 被保存。
    • 没有图像 Desert_modified(1)、desert_modified(2)、desert_modified(3)。

将要拯救他们的方法永远不会完成。有什么建议吗？

async private Task saveAsync(Bitmap bitmap, String path, String fileName, int requestNo)
{
    int temp = 1;
    String pth = path;
    String fileName1 = fileName;
    try
    {
        await Task.Run(() =>
            {

                if (File.Exists(pth))
                {

                    String[] array = fileName1.Split('.');
                    String modifiedFileName = array[0] + "_modified" + "(" + temp + ")." + array[1];
                    pth = fullPath.Replace(fileName1, modifiedFileName);

                    while (File.Exists(pth))
                    {
                        temp++;
                        //array = fileName.Split('.');
                        modifiedFileName = array[0] + "_modified" + "(" + temp + ")." + array[1];
                        pth = fullPath.Replace(fileName1, modifiedFileName);

                    }

                    bitmap.Save(@pth);

                }
                else
                {
                    bitmap.Save(@pth);
                }
                bitmap.Dispose();
            });

    }
    catch (Exception ex)
    {
        outputTextBox2.Text += ex.InnerException.Message;
    }
}
4

2 回答 2

2

您使用String.Replace，这实际上不适合这里。我认为您的代码中的主要错误是您试图在 while 循环中替换fileName1in ，尽管fileName1没有出现，因为它在 while 循环之前被修改。因此,在 while 循环中永远不会改变，这就是 while 循环永远不会结束的原因。pthpthpth

要解决此问题，请使用pth = fullPath.Replace(pth, modifiedFileName) OR，我会推荐：pth = modifiedFileName.

无论如何，使用 不会节省任何内存Replace，因为 C# 以及许多其他语言中的字符串是不可变的！所以即使通过Replace，也会生成一个新的字符串。

编辑： ......嗯..也许这个答案并不完全正确，因为你正在研究fullPath 但另一方面，fullPath没有在这个函数中定义:) ..这甚至来自哪里？

于 2013-08-09T00:24:55.590 回答
2

如果您有兴趣改进我在评论中暗示的代码，这里是一个开始：

如您所见，这

  • 删除（名称令人困惑的）局部变量，如pth,fileName1和modifiedFileName
  • 不再滥用 Split（错误地）剖析路径（如果目录有扩展名怎么办？如果文件名包含.自己怎么办？）
  • 用于PathCombine获取正确的路径分隔符
  • 使用固定格式字符串轻松生成候选文件名
  • 使用单个do-while循环而不是复制生成文件名的行

总的来说，我认为它要简单得多。

await Task.Run(() =>
  {
      var fmt = Path.Combine(path, Path.GetFileNameWithoutExtension(fileName)) + "_modified{0}" + Path.GetExtension(fileName);

      int counter = 1;
      string newName;
      do {
          newName = string.Format(fmt, counter++);
      }
      while (File.Exists(newName));

      using (bitmap)
          bitmap.Save(newName);
  });

我在本地 PC 上把它做成了一个简单的独立测试程序，这样我就可以实际测试它了：

using System.IO;
using System;

public class X
{
    public class Bitmap : IDisposable { 
        public void Dispose() { } 
        public void Save(string location)
        {
            Console.WriteLine("Saving: {0}", location);
            using (var s = new StreamWriter(location))
                s.WriteLine("hello world");
        }
    }

    private static void saveAsync(Bitmap bitmap, String path, String fileName, int requestNo)
    {
        Action a = () =>
          {
              var fmt = Path.Combine(path, Path.GetFileNameWithoutExtension(fileName)) + "_modified{0}" + Path.GetExtension(fileName);

              int counter = 1;
              string newName;
              do {
                  newName = string.Format(fmt, counter++);
              }
              while (File.Exists(newName));

              using (bitmap)
                  bitmap.Save(newName);
          };

        a();
        a();
        a();
        a();
    }

    public static void Main()
    {
        saveAsync(new Bitmap(), "/tmp", "foo.png", 3);
    }
}

这打印

Saving: /tmp/foo_modified1.png
Saving: /tmp/foo_modified2.png
Saving: /tmp/foo_modified3.png
Saving: /tmp/foo_modified4.png
于 2013-08-09T00:57:42.433 回答