c# 定位内存快速增长_C#Parallel.ForEach()内存使用量不断增长

public string SavePath { get; set; } = @"I:\files\";

public void DownloadList(List list)

{

var rest = ExcludeDownloaded(list);

var result = Parallel.ForEach(rest, link=>

{

Download(link);

});

}

private void Download(string link)

{

using(var net = new System.Net.WebClient())

{

var data = net.DownloadData(link);

var fileName = code to generate unique fileName;

if (File.Exists(fileName))

return;

File.WriteAllBytes(fileName, data);

}

}

var downloader = new DownloaderService();

var links = downloader.GetLinks();

downloader.DownloadList(links);

I observed the usage of RAM for the project keeps growing

I guess there is something wrong on the Parallel.ForEach(), but I cannot figure it out.

Is there the memory leak, or what is happening?

Update 1

After changed to the new code

private void Download(string link)

{

using(var net = new System.Net.WebClient())

{

var fileName = code to generate unique fileName;

if (File.Exists(fileName))

return;

var data = net.DownloadFile(link, fileName);

Track theTrack = new Track(fileName);

theTrack.Title = GetCDName();

theTrack.Save();

}

}

I still observed increasing memory use after keeping running for 9 hours, it is much slowly growing usage though.

Just wondering, is it because that I didn't free the memory use of theTrack file?

Btw, I use ALT package for update file metadata, unfortunately, it doesn't implement IDisposable interface.

解决方案

Use WebClient.DownloadFile() to download directly to a file so you don't have the whole file in memory.

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值