VB.NET - Создание объектов на каждом цикле

Question

Пожалуйста, обратите внимание на код ниже:

Public Sub MassDelete() 
     Dim objCommand As SqlCommand 
     Dim objCon As SqlConnection 
     Dim objDR As SqlDataReader 
     Try 
      Dim _ConString As String = "Data Source=IANSCOMPUTER;Initial Catalog=Test5;Integrated Security=True;MultipleActiveResultSets=true" 
      objCon = New SqlConnection(_ConString) 
      Using objCon 
        objCommand = New SqlCommand("SELECT * FROM Person WHERE StartDate <= dateadd(-6,year," & Now & ")") 
       objCommand.Connection = objCon 
       Using objCommand 
        objCon.Open() 
        objDR = objCommand.ExecuteReader 
        Do While objDR.Read 
         'Send ID over web service 
         'Send email to person 
         'Add log entry to log file 
        Loop 
        objDR.Close() 
       End Using 
      End Using 
     Catch ex As Exception 
      Throw 
     Finally 

     End Try 

Я вижу много кода, как это от предыдущего разработчика то есть одна функция, которая петлями через таблицу, например, лицом, транспортным средством, учетной записью и т. д. и совершает действия над тысячами или в некоторых случаях миллионами записей. В случае вышесказанного, лица удаляются через шесть лет после даты начала. Для каждой итерации цикла while существует три действия: отправьте идентификатор по веб-службе, отправьте электронное письмо человеку и добавьте запись журнала в файл журнала. Я пытаюсь реорганизовать этот код, поскольку я продолжаю разработку рассматриваемой системы.

Я считаю, что должны быть разные классы для Email, WebService и Log и вместо выполнения всей работы в функции MassDelete; работа будет распространена среди классов, например. письмо будет отправлено из класса электронной почты. Существуют ли какие-либо ограничения на создание объектов для WebService, Email и Log для каждой итерации цикла и использования их переменных экземпляра (в Person может быть миллион записей)?

Answer 1

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

В этом случае я защищаю хранение услуги отдельно от данных (например, Person записи), которая просто передается в определенные вызовы служб. Я действительно не знаю VB.NET (я использую C#), поэтому при необходимости настройте синтаксис.

' Note that the services are bound to some things - but NOT people 
' I recommend using DI/IoC (Dependency Injection) 
' Also the Logger is generally a larger application-wide service 
Var mailer = New EmailService(SMTPServerString, SMTPTimeout) 
Var ws = New WebService(WSEndpoint) 
Var log = New LoggerService() 
Var objCon = OpenConnection() 

Using objCon 
    Var objCommand = objCon.CreateCommand(..); 
    Using objCommand 
     Var objDR = objCommand.ExecuteReader 
     Do While objDR.Read 
      ' Person different per loop 
      Var person = New Person(objDR) 
      ' But services are the same and are not "bound" to a person 
      ws.RecordDelete(person.Id) 
      mailer.Send(person.Email, Subject, Message) 
      log.Log(Log.Information, "Deleted person: " & person.Id) 
     Loop 
    End Using 
End Using 

Конечно, этот код по-прежнему не является очень отказоустойчивым или распространяемым. Очереди сообщений (например, AMPQ), возможно, стоит изучить.