3

Помогите отследить потерю оперативной памяти. При каждом запуске отчета она увеличивается на 2-3 мб. Вот как запускаю отчет

private void callLoading()
{
  loadingList f7 = new loadingList();
  f7.ShowDialog();
}

Код формы в которой открывается отчет

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using Microsoft.Reporting.WinForms;
using System.Data.SqlClient;
using System.IO;

namespace MSU
{
  public partial class loadingList : Form
  {
    SqlConnection cn = new SqlConnection(@"строка подключения");
    DandDDataSet r = new DandDDataSet();

    public loadingList()
    {
        InitializeComponent();
    }

    private void loadingList_Load(object sender, EventArgs e)
    {

        cn.Open();
        SqlDataAdapter da = new SqlDataAdapter("тут запрос", cn);
        da.Fill(r, r.Tables[0].TableName);

        ReportDataSource mds = new ReportDataSource("DataSet1", r.Tables[0]);
        this.reportViewer1.LocalReport.DataSources.Clear();
        this.reportViewer1.LocalReport.DataSources.Add(mds);
        this.reportViewer1.LocalReport.Refresh();

        this.reportViewer1.RefreshReport();
      }

      private void loadingList_FormClosing(object sender, FormClosingEventArgs e)
      {
        main m = this.Owner as main;
        m.dis();
        GC.SuppressFinalize(this);
        this.Dispose(true);
      }
  }
}
3
  • cn.Open(); - вижу, cn.Close(); - не вижу
    – user176262
    25 окт 2016 в 13:52
  • Для вставки кода в вопрос добавляйте перед каждой строчкой кода 4 пробела 25 окт 2016 в 13:53
  • @Igor, спасибо, действительно не закрыл, но на ситуацию это никак не повлияло. Запускаю отчет оперативка с потребления 22мб прыгает до 31мб, закрываю, падает до 29мб, открываю прыгает до 33мб, закрываю, падает до 31мб, открываю прыгает до 36, закрываю, падает до 33мб. Ну и так далее до упора можно открывать закрывать. 25 окт 2016 в 13:57

3 ответа 3

3

Закрытие модальной формы всего лишь прячет ее. А значит, форма остается в памяти. При использовании метода ShowDialog() нужно всегда вызывать Dispose(). Исправить легко:

private void callLoading()
{
    using (loadingList f7 = new loadingList())
    {
        f7.ShowDialog();
    }
}

Также стоит помнить, что тот факт, что вы с каждым открытием видите все увеличивающееся количество занятой памяти, вовсе не означает наличие утечки! CLR может и не собирать мусор некоторое время, если на то нет необходимости. Наличие утечки может показать только профайлер, в котором вы снимите и сравните два снэпшота ДО и ПОСЛЕ.


Бонус:

  • Уберите метод loadingList_FormClosing(). Он не нужен. Совсем.
  • SqlConnection является ограниченным ресурсом, поэтому его лучше всего закрывать и освобождать сразу, как вы закончили с ним работать. К счастью, для SqlConnection можно вызвать только Dispose(), коннекция при этом закроется автоматически:

    private void loadingList_Load(object sender, EventArgs e)
    {
        using (SqlConnection cn = new SqlConnection(@"строка подключения"))
        {
            cn.Open();
            //остальной код
        }
    }
    
9
  • 1
    Если верить этому - sql.ru/forum/1087139/…, то ето болезнь самой библиотеке reportviewer. Я сделал как вы порекомендовали но ничего не изменилось, где мне нужно использовать Dispose()? 25 окт 2016 в 14:44
  • @Winteriscoming тем не менее, даже если бы ReportViewer не тек, у вас все равно были бы утечки :). Смотрите тогда дальше по ссылкам, что делать с ReportViewer.
    – andreycha
    25 окт 2016 в 14:49
  • 1
    @Winteriscoming конструкция using уже вызывает Dispose().
    – andreycha
    25 окт 2016 в 14:49
  • диалог и должен только прятаться, т.к. может содержать заполненные свойства после работы. Вообще не стоит пересоздавать диалог перед каждым вызовом, поместите ссылку на него в поле родительской формы и вызывайте через это поле.
    – rdorn
    26 окт 2016 в 8:52
  • @rdorn пожалуйста по подробнее, как это сделать покажите пример, буду очень благодарен если поможет, я 2 день не могу найти решения как бороться с reportViewer. За один запуск -3мб оперативы, при закрытии высвобаждает максимум 1мб. 26 окт 2016 в 14:30
0

Большую часть уже описал @andreycha в своем ответе.

Если диалог используется достаточно часто в приложении, то можно не создавать его заново каждый раз, как он потребовался. Для этого разместим нужный диалог в форме которая его будет использовать. В этом нет ничего сложного, но раз уж попросили, пусть будет в виде ответа.

public partial MyForm : Form
{
  private loadingList _loadingList;
  ...
  public loadingList()
  {
    InitializeComponent();
    _loadingList = new loadingList();
    //настраиваем параметры диалога если надо
    ...
    private void callLoading()
    {
      loadingList.ShowDialog();
    }
    ...
  }
}

Важно! Если утечка памяти в коде самого диалога или компонента внутри него, то такой подход может только ухудшить ситуацию, т.к. "подтекающий" компонент будет в памяти постоянно и, соответственно, ресурсы тоже будут течь постоянно.


Если так уж важно освободить память, можно вызвать GC.Collect(), но делать это надо осторожно, т.к. GC приостанавливает все потоки приложения на время сборки мусора и реальная необходимость в прямом обращении к GC бывает не так часто, например при работе с очень большими объектами, от которых нужно избавиться заранее чтобы не поймать OutOfMemoryException (из личной практики, другого варианта просто не было). Ради пары-тройки мегабайт явно не стоит, и от реальной утечки памяти это тоже не поможет.

8
  • Утечка внутри окна, перед вызовом окна потребление памяти 23мб, вызвали окно память не изменилась, но как только нажимаем кнопку просмотреть отчет, reportviewer вытягивает инфу и сходу пожирает 10мб после закрытия не освобождает, последующее открытие отчета поджирает 1-3мб и не освобождает. GC.Collect() - не помогает, возможно я не в том участке ставил? Я ставил тут private void loadingList_FormClosing(object sender, FormClosingEventArgs e) { } 27 окт 2016 в 7:20
  • @Winteriscoming если с английским дружите, посмотрите тут stackoverflow.com/a/6221590/5796587, перевод могу сделать но скорее всего только в выходные, если больше ни кто не соберется.
    – rdorn
    27 окт 2016 в 7:36
  • я пробовал описанные способы, мне не помогло, как впрочем и многим людям, которые оставляли комментарии под этим решением. Я мягко говоря в отчаянии. 27 окт 2016 в 8:23
  • правильно ли я сделал? это в файле конфига - <?xml version="1.0" encoding="utf-8" ?> <configuration> <configSections> </configSections> <connectionStrings> <add name="тут строка подключения" providerName="System.Data.SqlClient" /> </connectionStrings> <runtime> <NetFx40_LegacySecurityPolicy enabled="true"/> </runtime> <startup> <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.5" /> </startup> </configuration> 27 окт 2016 в 17:12
  • Это в форме генерации отчета - da.Fill(r, r.Tables[0].TableName); ReportDataSource mds = new ReportDataSource("DataSet1", r.Tables[0]); this.reportViewer1.LocalReport.DataSources.Clear(); this.reportViewer1.LocalReport.DataSources.Add(mds); this.reportViewer1.LocalReport.Refresh(); this.reportViewer1.RefreshReport(); this.reportViewer1.LocalReport.ExecuteReportInCurrentAppDomain(AppDomain.CurrentDomain.Evidence); 27 окт 2016 в 17:12
0

Победил пожирание оперативной памяти вот так :

        using (SqlConnection cn = new SqlConnection(@"строка"))
          {
            cn.Open();
            using (SqlCommand cmd = new SqlCommand())
            {
               using (DandDDataSet m = new DandDDataSet())
                {
                  using (SqlDataAdapter da = new SqlDataAdapter("тут запрос", cn))
                   {
                    da.Fill(r, r.Tables[0].TableName);

                    ReportDataSource mds = new ReportDataSource("DataSet1",r.Tables[0]);
                    this.reportViewer1.LocalReport.DataSources.Clear();
                    this.reportViewer1.LocalReport.DataSources.Add(mds);
                    this.reportViewer1.LocalReport.Refresh();

                    this.reportViewer1.RefreshReport();  
                   }
               }  
           }  
         }  

Мне писали что не нужен метод при закрытии формы, но как раз он и подчищает

  private void loadingList_FormClosing(object sender, FormClosingEventArgs e)
  {            
     this.reportViewer1.Reset();
     this.reportViewer1.Dispose();
     GC.SuppressFinalize(reportViewer1);
     GC.Collect();  
  }
1
  • 1. В вопросе у вас другой код. 2. Этот код (если он действительно нужен) должен находиться в Dispose() формы, а не в FormClosing.
    – andreycha
    3 ноя 2016 в 13:01

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

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