2012-10-24 93 views
0

有没有办法多线程这种方法?我可以看到的问题是,唯一的共享状态是克隆图像源,占用方法时间的一半左右。列表中约有5至10万名收件人。多线程任务

我已经尝试了一个ConcurrentQueue一个操作添加克隆和另一个TryDequeue并处理它们,但超过500个项目它只保存7秒。有没有办法让imgSrc回滚或撤消更改,以便它不必创建新的克隆?

public static void CreateImages(string fileSrc, string pathDestination, IEnumerable<Recipient> recipients) 
    { 
     var sfCenter = new StringFormat(); 
     sfCenter.Alignment = StringAlignment.Center; 
     sfCenter.LineAlignment = StringAlignment.Center; 

     using (var imgSrc = new System.Drawing.Bitmap(fileSrc)) 
     { 
      foreach (var rec in recipients) 
      { 
       using (var imgCopy = imgSrc.Clone() as Bitmap) 
       using (var graphicImage = Graphics.FromImage(imgCopy)) 
       { 
        graphicImage.SmoothingMode = SmoothingMode.AntiAlias; 
        graphicImage.DrawString(rec.Name, new Font("Arial", 16, FontStyle.Bold), Brushes.Black, new Rectangle(170, 105, 650, 50), sfCenter); 
        // plus other activity 
        var fileOut = pathDestination + rec.ID + ".jpg"; 
        imgCopy.Save(fileOut, ImageFormat.Jpeg); 
       } 
      } 
     } 
    } 
+1

你确定这段代码是CPU绑定的吗?我预计磁盘IO将成为瓶颈,在这种情况下添加更多线程可能会影响性能。 – rene

+0

将副本保存到磁盘至关重要吗?如果允许升级到.net 4.5,则可以使用异步,使用I/O完成端口,这比创建自己的线程更有效。如果没有,F#具有相应工作的异步计算表达式。 – Christian

+0

磁盘绑定;进一步的秒表检查显示克隆和图形部分只要保存即可。保存到磁盘;是。 .Net 4.5;不幸的是其中一台机器仍然运行XP。我只是认为只是我看不到如何扩展这个相当简单的算法。 – Cheval

回答

0

找不到更好的方法。这看起来很好。