утечка памяти в C# чтение данных из базы данных sql

Question

Я только новичок в C# и взял на себя проект на работе, который включает в себя код, внесенный другими разработчиками. При чтении данных из sql происходит утечка памяти. В основном код считывает результат ping из базы данных и возвращает его в искровую линию, которую я затем показываю на приборной панели. Надеюсь, ты поможешь.

[ActionName("get-response-times")] 
    public JsonResult GetResponseTime() 
    { 
     try 
     { 
      //todo...get list of sites we we want to check from database 
      var entities = new Entities(); 

      var sites = entities.Sites.ToList(); 

      var status = new List<ResponseDataModel>(); 

      foreach (var site in sites) 
      { 
       var response = Ping(site.URL); 

       site.SiteResponseHistories.Add(new SiteResponseHistory 
       { 
        CreateDate = DateTime.UtcNow, 
        ResponseTime = (int)response, 
        Site = site 
       }); 

       status.Add(new ResponseDataModel 
       { 

        ResponseTime = (int)response, 
        Name = site.Name, 
        ID = site.Id, 
        History = site.SiteResponseHistories.OrderByDescending(a => a.CreateDate).Select(a => a.ResponseTime.GetValueOrDefault(0)).Take(100).Reverse().ToArray() 
       }); 
      } 

      entities.SaveChanges(); 

      return Json(status); 
     } 
     catch (Exception) 
     { 
      // handle if necessary 
     } 

     return Json(""); 
    } 

    protected long Ping(string url) 
    { 
     try 
     { 
      HttpWebRequest request = (HttpWebRequest)HttpWebRequest.Create(url); 
      request.Timeout = 3000; 
      request.AllowAutoRedirect = false; // find out if this site is up and don't follow a redirector 
      request.Method = "HEAD"; 

      Stopwatch watch = new Stopwatch(); 

      watch.Start(); 

      using (var response = request.GetResponse()) 
      { 
       return watch.ElapsedMilliseconds; 
      } 
     } 
     catch 
     { 
      return -1; 
     } 
    } 

Answer 1

Я подозреваю, что Entities это класс, порожденный Entity Framework и унаследовал от DbContext. Если это так попробуйте использовать using блок:

using(var entities = new Entities()) 
{ 
    ... 
    entities.SaveChanges(); 
} 

Пожалуйста, обратите внимание, что в целом это не идеальное решение, чтобы начать новый экземпляр DbContext для каждого запроса. Подробнее см. this отличный ответ.

Answer 2

Вы можете попробовать сделать это:

using (var context = new Context()) 
{  
    // Perform data access using the context 
} 

и ваш следующий код:

 [ActionName("get-response-times")] 
    public JsonResult GetResponseTime() 
    { 
     try 
     { 
      var status = new List<ResponseDataModel>(); 
      //todo...get list of sites we we want to check from database 
      using (var entities = new Entities()) 
      { 
       var sites = entities.Sites.ToList(); 

       foreach (var site in sites) 
       { 
        var response = Ping(site.URL); 

        site.SiteResponseHistories.Add(new SiteResponseHistory 
        { 
         CreateDate = DateTime.UtcNow, 
         ResponseTime = (int)response, 
         Site = site 
        }); 

        status.Add(new ResponseDataModel 
        { 

         ResponseTime = (int)response, 
         Name = site.Name, 
         ID = site.Id, 
         History = site.SiteResponseHistories.OrderByDescending(a => a.CreateDate).Select(a => a.ResponseTime.GetValueOrDefault(0)).Take(100).Reverse().ToArray() 
        }); 
       } 

       entities.SaveChanges(); 
      } 
      return Json(status); 
     } 
     catch (Exception) 
     { 
      // handle if necessary 
     } 

     return Json(""); 
    }