Отправка больших электронных писем с путями вложений в базе данных

Question

Мне просто нужна помощь. В основном я создаю приложение Windows, которое отправляет массовые письма нашим клиентам. поле «электронная почта» и «вложение» - из базы данных. поле вложения содержит только путь, в котором находится файл. Код работает, но вместо получения 5 сообщений электронной почты я получаю 15 писем.

Примечание: моя база данных содержит только 5 записей, поэтому я должен получить только 5 писем с вложениями:

Можете ли вы помочь мне, пожалуйста, спасибо!

Вот мой код:

 string email; 
     string attachment; 
     ArrayList emailList = new ArrayList(); 
     ArrayList attachList = new ArrayList(); 
     private static readonly Logger log = new _EventLogger(); 

     private void btnSend_Click(object sender, EventArgs e) 
     { 
      conn.Open(); 
      SqlCommand cmdgetEmail = new SqlCommand("Select EMAIL, PATH from MEMBERREQUIREMENTS WHERE STATUS=0", conn); 
      SqlDataReader getEmail = cmdgetEmail.ExecuteReader(); 
      //count = 0; 
      while (getEmail.Read()) 
      { 
       //count++; 
       //email = getEmail.GetValue(i).ToString(); 
       //emailList.Add(email); 
       //i = i + 1 - 1; 
       email = getEmail.GetString(0); 
       emailList.Add(email); 
       attachment = getEmail.GetString(1); 
       attachList.Add(attachment); 
      } 
      getEmail.Close(); 
      conn.Close(); 
      sendMail(); 
     } 
      private void sendMail() 
      { 

       string from="myemail@email.com"; 
       foreach (string sendTo in emailList) 
       { 

        foreach (string sendAttachments in attachList) 
        { 
        MailMessage mail = new MailMessage(); 
        mail.To.Add(sendTo); 
        mail.From = new MailAddress(from, "Company Name'", Encoding.UTF8); 
        mail.Subject = subject; 
        mail.Body = msgBodyHead + msgBodyHead2 + msgDate + msgGreet + msgBody + msgAdobe + msgAssistance + msgCompliment + msgfooter; 
        mail.IsBodyHtml = true; 
        mail.Priority = MailPriority.High;  
        mail.Attachments.Add(new Attachment(sendAttachments)); 

         SmtpClient client = new SmtpClient(); 
         client.Credentials = new System.Net.NetworkCredential(from, "password"); 
         client.Host = "192.167.89.0"; 
         client.EnableSsl = false; 
         try 
         { 

          progress(); 
          client.Send(mail); 

         } 

         catch (Exception ex) 
         { 
          ProgressBar1.Visible = false; 
          timer1.Enabled = false; 
          Exception excpt = ex; 
          string errorMessage = string.Empty; 

          while (excpt != null) 
          { 

           errorMessage += excpt.ToString(); excpt = excpt.InnerException; 
           log.Error("Email - LMS Application Error", ex); 
           lblError.Text = "There was an error occured while processing your request.\n Please see Event Viewer for more details."; 
           lblError.ForeColor = System.Drawing.Color.Red; 
          } 
         } 

        } 
       } 
      } 

Answer 1

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

Затем вы выполняете итерацию по этой отправке, отправив электронное письмо для каждой комбинации адреса и вложения. Я предполагаю, что это не то, что вы хотите сделать. Я полагаю, что либо:

• Каждый получатель должен получить только прикрепленное приложение, указанное в их записи. В этом случае вы не хотите перебирать вложения для каждого адреса, или, возможно,
• каждый получатель должен получать каждое вложение, но вложения должны быть перечислены в отдельной таблице и не сохраняться в записи для каждого адреса в ваш MEMBERREQUIREMENTS таблица.

UPDATE:

Ниже приведен код, чтобы сделать первый. Обратите внимание, что это лишь минимальное редактирование Опубликованная кода, и не устанавливает имена переменных и т.д., или предоставить любую дополнительную проверку ошибок (которые вы, вероятно, хотите сделать):

private static readonly Logger log = new _EventLogger(); 


private void btnSend_Click(object sender, EventArgs e) 
{ 
    conn.Open(); 
    SqlCommand cmdgetEmail = new SqlCommand("Select EMAIL, PATH from MEMBERREQUIREMENTS WHERE STATUS=0", conn); 
    SqlDataReader getEmail = cmdgetEmail.ExecuteReader(); 
    while (getEmail.Read()) 
    { 
     email = getEmail.GetString(0); 
     attachment = getEmail.GetString(1); 
     this.sendMail(email, attachment) 
    } 
    getEmail.Close(); 
    conn.Close(); 
} 

private void sendMail(string sendTo, string sendAttachments) 
{ 
    MailMessage mail = new MailMessage(); 
    mail.To.Add(sendTo); 
    mail.From = new MailAddress(from, "Company Name'", Encoding.UTF8); 
    mail.Subject = subject; 
    mail.Body = msgBodyHead + msgBodyHead2 + msgDate + msgGreet + msgBody + msgAdobe + msgAssistance + msgCompliment + msgfooter; 
    mail.IsBodyHtml = true; 
    mail.Priority = MailPriority.High; 
    mail.Attachments.Add(new Attachment(sendAttachments)); 

    SmtpClient client = new SmtpClient(); 
    client.Credentials = new System.Net.NetworkCredential(from, "password"); 
    client.Host = "192.167.89.0"; 
    client.EnableSsl = false; 
    try 
    { 

     progress(); 
     client.Send(mail); 

    } 

    catch (Exception ex) 
    { 
     ProgressBar1.Visible = false; 
     timer1.Enabled = false; 
     Exception excpt = ex; 
     string errorMessage = string.Empty; 

     while (excpt != null) 
     { 

      errorMessage += excpt.ToString(); excpt = excpt.InnerException; 
      log.Error("Email - LMS Application Error", ex); 
      lblError.Text = "There was an error occured while processing your request.\n Please see Event Viewer for more details."; 
      lblError.ForeColor = System.Drawing.Color.Red; 
     } 
    } 
} 

Answer 2

Это потому, что у вас есть вызов client.Send(mail); внутри ваших привязок для цикла foreach. Он должен находиться за пределами этого цикла.

попробуйте изменить метод sendMail к этому:

private void sendMail() 
{ 
    string from = "myemail@email.com"; 
    foreach (string sendTo in emailList) 
    { 
     MailMessage mail = new MailMessage(); 
     mail.To.Add(sendTo); 
     mail.From = new MailAddress(from, "Company Name'", Encoding.UTF8); 
     mail.Subject = subject; 
     mail.Body = msgBodyHead + msgBodyHead2 + msgDate + msgGreet + msgBody + msgAdobe + msgAssistance + msgCompliment + msgfooter; 
     mail.IsBodyHtml = true; 
     mail.Priority = MailPriority.High; 
     foreach (string sendAttachments in attachList) 
     { 
      mail.Attachments.Add(new Attachment(sendAttachments)); 
     } 

     SmtpClient client = new SmtpClient(); 
     client.Credentials = new System.Net.NetworkCredential(from, "password"); 
     client.Host = "192.167.89.0"; 
     client.EnableSsl = false; 
     try 
     { 

      progress(); 
      client.Send(mail); 

     } 

     catch (Exception ex) 
     { 
      ProgressBar1.Visible = false; 
      timer1.Enabled = false; 
      Exception excpt = ex; 
      string errorMessage = string.Empty; 

      while (excpt != null) 
      { 

       errorMessage += excpt.ToString(); excpt = excpt.InnerException; 
       log.Error("Email - LMS Application Error", ex); 
       lblError.Text = "There was an error occured while processing your request.\n Please see Event Viewer for more details."; 
       lblError.ForeColor = System.Drawing.Color.Red; 
      } 
     } 
    } 
} 

Answer 3

У вас есть две петли, одна для списка отправителей (5) и один для списка вложений (вероятно, 3), что в общей сложности 15 выполнение кода для отправки электронной почты. Я бы реорганизовать метод sendmail к чему-то вроде:

foreach (string sendTo in emailList) 
{ 
    // construct email fields ... 
    // ... 

    foreach (string sendAttachments in attachList) 
    { 
     mail.Attachments.Add(new Attachment(sendAttachments)); 
    } 

    // send email ... 
    // ... 
} 

Answer 4

 private void sendMail() 
     { 

      string from="myemail@email.com"; 
      foreach (string sendTo in emailList) 
      { 

       foreach (string sendAttachments in attachList) 
       { 

Вы итерация attachList, но я ничего не вижу в этом цикле, чтобы ограничить attachList для вложений, предназначенных только для получателя sendTo.

Answer 5

Не знаю, зачем вам это нужно, что списки массивов и циклов в методе Sendmail, возможно, что-то вроде этого:

/// ...snip 
    email = getEmail.GetString(0); 
    attachment = getEmail.GetString(1); 
    SendOneMail(email, attachment); 
    /// ... snip 


    private void SendOneMail(string email, string attach) 
    { 
    /// send yust one mail 
    }