我如何使它只发送一次电子邮件？ [关闭]

Question

private void timer4_Tick(object sender, EventArgs e)
{
     se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");

     if (se.photossendended == true)
     {
           se.photossendended = false;
           timer4.Enabled = false;
           timer5.Enabled = true;
     }
}

直到se.photossendended == true，它将继续生成se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");

但我希望它只执行一次，并继续检查 se.photossendended 是否为真。 所以我试着做while(true)

private void timer4_Tick(object sender, EventArgs e)
{
     se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");

     while(true)
     {
          if (se.photossendended == true)
          {
               se.photossendended = false;
               timer4.Enabled = false;
               timer5.Enabled = true;
          }
     }
}

但是它会保存所有的程序并且永远不会使它成为真的，因为程序没有继续并且它都卡在这个循环中。 所以它永远不会是真的，循环将永远保持下去。

编辑**

这是 se 类 SendEmail

public void SendPhotos(string fileNameToSend) 
        {
            try
            {
                MailAddress from = new MailAddress("username", "User " + (char)0xD8 + " Name",
                System.Text.Encoding.UTF8);
                MailAddress to = new MailAddress("myrmail");
                photosmessage = new MailMessage(from, to);
                photosmessage.Body = "Please check the log file attachment i have some bugs.";
                string someArrows = new string(new char[] { '\u2190', '\u2191', '\u2192', '\u2193' });
                photosmessage.Body += Environment.NewLine + someArrows;
                photosmessage.BodyEncoding = System.Text.Encoding.UTF8;
                photosmessage.Subject = "Log File For Checking Bugs" + someArrows;
                photosmessage.SubjectEncoding = System.Text.Encoding.UTF8;
                Attachment myAttachment = new Attachment(fileNameToSend, MediaTypeNames.Application.Octet);
                photosmessage.Attachments.Add(myAttachment);
                SmtpClient photossend = new SmtpClient("smtp.gmail.com", 587);
                photossend.SendCompleted += new SendCompletedEventHandler(photossend_SendCompleted);
                photossend.EnableSsl = true;
                photossend.Timeout = 10000;
                photossend.DeliveryMethod = SmtpDeliveryMethod.Network;
                photossend.UseDefaultCredentials = false;
                photossend.Credentials = new NetworkCredential("user", "pass");
                string userState = "test message1";
                photossend.SendAsync(photosmessage, userState);
                SendLogFile.Enabled = false;
            }

            catch (Exception errors)
            {
                Logger.Write("Error sending message :" + errors);
            }
        }

        private void photossend_SendCompleted(object sender, AsyncCompletedEventArgs e)
        {
            photosmessage.Dispose();
            photossendended = true;
        }

我想确保发送的电子邮件是真的：photosendended = true; 然后在 Timer4 滴答事件的 Form1 中，如果它真的停止计时器激活计时器 5，我想发送一次电子邮件，然后再发送第二封电子邮件并一遍又一遍。

我有 4 个计时器滴答事件，我可以禁用并一一启用它们。 原因是我只想在前一封邮件发送完毕后才发送每封邮件。

Answer 1

我猜您正在尝试在不阻塞 UI 的情况下异步发送邮件，但还想等到发送完成后再继续下一封邮件。

如果您使用的是 c#5/.Net 4.5，您可以在 async 方法中使用 SendMailAsync：

async void SendMails()
{
    await server.SendMailAsync(mailMessage1);
    await server.SendMailAsync(mailMessage2);
}

所以，你的方法可以是这样的

public Task SendPhotos(string fileNameToSend)
{
    try
    {
        MailAddress from = new MailAddress("username", "User " + (char)0xD8 + " Name", System.Text.Encoding.UTF8);
        MailAddress to = new MailAddress("myrmail");
        var photosmessage = new MailMessage(from, to);
        photosmessage.Body = "Please check the log file attachment i have some bugs.";
        string someArrows = new string(new char[] { '\u2190', '\u2191', '\u2192', '\u2193' });
        photosmessage.Body += Environment.NewLine + someArrows;
        photosmessage.BodyEncoding = System.Text.Encoding.UTF8;
        photosmessage.Subject = "Log File For Checking Bugs" + someArrows;
        photosmessage.SubjectEncoding = System.Text.Encoding.UTF8;
        Attachment myAttachment = new Attachment(fileNameToSend, MediaTypeNames.Application.Octet);
        photosmessage.Attachments.Add(myAttachment);
        SmtpClient photossend = new SmtpClient("smtp.gmail.com", 587);
        photossend.EnableSsl = true;
        photossend.Timeout = 10000;
        photossend.DeliveryMethod = SmtpDeliveryMethod.Network;
        photossend.UseDefaultCredentials = false;
        photossend.Credentials = new NetworkCredential("user", "pass");
        SendLogFile.Enabled = false;
        return photossend.SendMailAsync(photosmessage);
    }
    catch (Exception errors)
    {
        Logger.Write("Error sending message :" + errors);
        return Task.FromResult<object>(null);
    }
}

你可以使用它

await se.SendPhotos(photofilesDir1 + "\\" + "photofiles.zip");
await se.SendPhotos(photofilesDir2 + "\\" + "photofiles.zip");

PS：现在为您的方法取一个更好的名称是SendPhotosAsync

Answer 2

private void timer4_Tick(object sender, EventArgs e)
{
    if(!se.photossendended)
    {
        se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");
        se.photossendended = true;
        timer4.Enabled = false;
        timer5.Enabled = true;
    }            
}

您似乎想要异步发送电子邮件。您几乎已经完成了photossend_SendCompleted 中的代码。剩下的代码应该如下：

bool sendingStarted;
private void timer4_Tick(object sender, EventArgs e)
{
     if(!sendingStarted) {
         se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");
         sendingStarted = true;
     }
     if(photossended){
       timer4.Enabled = false;
       timer5.Enabled = true;   
    }            
}

我认为您应该公开SendEmail 类的事件SendCompleted，以便我们可以这样做：

se.SendCompleted += (s,e) => {
    timer4.Enabled = false;
    timer5.Enabled = true;//Of course we still need the flag sendingStarted.
};

Answer 3

我不知道se.SendPhotos 是什么样子，但你可以这样做

private void timer4_Tick(object sender, EventArgs e)
{
     se.SendPhotos(photofilesDir + "\\" + "photofiles.zip");

     while(true)
     {
          if (se.photossendended == true)
          {
               se.photossendended = false;
               timer4.Enabled = false;
               timer5.Enabled = true;
               break;
          }
     }
}

但这更像是一种破解而不是解决方案：/