1

我是线程新手，在这个问题上停留在这里。我有一个使用多线程的应用程序。我有一个在 ftp 服务器上上传数千张图像的功能。对于每个图像，我正在创建一个新线程。该线程调用一个函数来连接ftp服务器，上传文件，如果上传成功返回布尔值。

我的问题是，由于我上传了数千张图像并且每个图像都在创建自己的线程，因此在一段时间后我得到内存不足异常并且应用程序冻结。

我的代码如下：

public Int16 UploadFiles(string[] files)
{
     foreach (var fileName in files)
        {
            if (UploadFile(fileName))
            {
                strLogText += "\r\n\tFile: " + fileName + " downloaded.";
            }
        }
}

private bool UploadFile(string fileName)
    {
        var blnDownload = false;
        var thread = new Thread(() => DownLoadFileNow(fileName, out blnDownload)) {IsBackground = true};
        thread.Start();
        return blnDownload;
    }

    private void DownLoadFileNow(string fileName, out bool blnDownload)
    {
        //Get file path and name on source ftp server
        var srcFolder = GetSrcFolderName(fileName);

        //Get Local Folder Name for downloaded files
        var trgFolder = GetLocalFolder(fileName, "D");

        var reqFtp =
            (FtpWebRequest) WebRequest.Create(new Uri("ftp://" + _strSourceFtpurl + srcFolder + "/" + fileName));
        reqFtp.Method = WebRequestMethods.Ftp.DownloadFile;
        reqFtp.UseBinary = true;
        reqFtp.Credentials = new NetworkCredential(_strSourceFtpUser, _strSourceFtpPassword);
        var outputStream = new FileStream(trgFolder + "\\" + fileName, FileMode.Create);

        try
        {
            var response = (FtpWebResponse) reqFtp.GetResponse();
            var ftpStream = response.GetResponseStream();
            const int bufferSize = 2048;
            var buffer = new byte[bufferSize];

            if (ftpStream != null)
            {
                int readCount = ftpStream.Read(buffer, 0, bufferSize);
                while (readCount > 0)
                {
                    outputStream.Write(buffer, 0, readCount);
                    readCount = ftpStream.Read(buffer, 0, bufferSize);
                }

                ftpStream.Close();
            }
            response.Close();
            blnDownload = true;
        }
        catch (WebException ex)
        {
            _log.WriteLog("Error in Downloading File (" + fileName + "):\r\n\t" + ex.Message, "");
            //Delete newly created file from local system
            outputStream.Close();
            if (File.Exists(trgFolder + "/" + fileName))
                File.Delete(trgFolder + "/" + fileName);
        }
        catch (Exception ex)
        {
            _log.WriteLog("Error in Downloading File (" + fileName + "):\r\n\t" + ex.Message, "");
        }
        finally
        {
            outputStream.Close();
            outputStream.Dispose();
        }
        blnDownload = false;
    }

请帮助并让我知道如何限制创建的线程数，以便一次运行的线程不超过 10-20 个。

4

3 回答 3

3

你不能创建这么多线程。一种替代方法是使用 parrelle 扩展。

public void UploadFiles(string[] files)
{
    files.AsParallel().ForAll(fileName =>
    {
        if (UploadFile(fileName))
        {
            strLogText += "\r\n\tFile: " + fileName + " downloaded.";
        }
    });
}
于 2013-07-25T15:37:58.017 回答
2

尝试用调用 DownloadFileNow 而不是 UploadFile(String file) 方法的 Parallel.ForEach() 替换 UploadFiles(string[] files) 中的 foreach。

Parallel.Foreach 将从线程池中提取线程，这是您想要的，并将简化您的代码。

 Parallel.ForEach(files, fileName =>
        DownloadFileNow(fileName);
        strLogText += "\r\n\tFile: " + fileName + " downloaded.";
 );
于 2013-07-25T15:37:38.140 回答
1

正如其他人指出的那样，您不应该创建那么多线程。现在......这Parallel.ForEach()会给你很好的语法糖，你应该走那条路！

只是想指出，您需要将线程池作为一个概念来查找。看，让大量线程并行运行是没有意义的。对于每个这样的任务，都有一个最佳线程数，超过这个数，整个线程开销实际上会开始减慢你的速度。或者，在你的情况下，用尽你所有的记忆。

如果你把任务想象成一堆放在桌子（文件夹）上的照片，把线程想象成为你跑腿的员工，那么让一名员工从桌子上拿一张照片，把它放在一个信封里，然后把它带到邮局，对于每张照片，都将永远花费。因此，您雇用了另一名员工。还有一个。但是，一旦你接触到一定数量的照片盖章者，他们就会开始互相干扰。他们在办公桌前排队。他们在邮局前排队。整个信封的情况对负责办公用品的可怜的玛丽来说是痛苦的——他们也在她面前排队！因此，找出最佳员工数量（您可以运行测试或只是猜测一下）并分配他们重复该任务，直到桌子空无一人......

这是一个经常出现的主/从模式。

于 2013-07-25T15:47:03.080 回答