1

Из-за следующих строк происходит утечка памяти:

MailMessage msg = new MailMessage(mail, mail, "?", "");
msg.Attachments.Add(new Attachment(filePath, MediaTypeNames.Text.Plain));
var client = new SmtpClient("smtp.mail.ru", 25);
client.Credentials = new NetworkCredential(mail, "password");
client.EnableSsl = true;
client.Send(msg);
client.Dispose();
msg.Attachments.Dispose();
msg.Dispose();

Можно ли решить эту проблему? Вроде все dispose вызвал.

0

2 ответа 2

4

Dispose не освобождает память. Он просто просит класс уничтожить неуправляемые ресурсы(всякие там handles и прочее, что там внути client используется).

Память в .net освобождается недетерминированно, сборщиком мусора. Течет - это когда потребление памяти растет со временем, при каждом вызове вашего кода. А у вас - просто не освобождается мгновенно, так и задумано.

2
  • Эта часть кода вызывается один раз, но в диспетчере задач постоянно увеличивается потребление ОЗУ программой
    – KnowNamed
    18 авг 2018 в 7:33
  • @KnowNamed тогда вам стоит взять профайлер памяти и поискать реальную утечку
    – user177221
    18 авг 2018 в 12:26
1

Вместо явного вызова Dispose() лучше использовать using

using(var client = new SmtpClient("smtp.mail.ru", 25))    
using(var msg = new MailMessage(mail, mail, "?", ""))
{
    msg.Attachments.Add(new Attachment(filePath, MediaTypeNames.Text.Plain));
    client.Credentials = new NetworkCredential(mail, "password");
    client.EnableSsl = true;
    client.Send(msg);    
}

Уничтожать Attachment не имеет смысла, он сам уничтожится при уничтожении сообщения

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.