c# - 发送电子邮件 MVC4 C#

Question

我正在尝试在操作中发送电子邮件。但是，该操作始终返回空白屏幕。

查看:

<% using(Html.BeginForm("Sendlink", "Home")) %>
    <% { %>
     <input type="text" id="toemail" value="" />
        <input type="submit" value="Send" />
    <% } %>

Controller :

public ActionResult Sendlink()
{
    return View();
}

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Sendlink(FormCollection formCollection)
{
    try
    {
        string message = Session["link"].ToString();
        string toemail = formCollection["toemail"];
        MailEngine.Send("mail@mail.com", toemail, "link", message);
        return RedirectToAction("CanvasShare");
    }
    catch
    {

    }
    return null;
}

邮件引擎类:

public static void Send(string from, string to, string subject, string body)
{
    try
    {
        MailMessage mail = new MailMessage(from, to, subject, body);
        SmtpClient client = new SmtpClient("smtp.mymail.com");
        client.DeliveryMethod = SmtpDeliveryMethod.Network;
        client.EnableSsl = false;
        client.Send(mail);
    }
    catch
    {

    }
}

Answer 1

您在应用程序中使用了空的 catch block 。这不是一个好主意，有关对此的更深入讨论，请参阅 Why are empty catch blocks a bad idea?及相关问题。

您似乎正在发生以下情况:

• 在 Sendlink(FormCollection formCollection) 方法的 try block 内某处抛出异常。此异常似乎源自 RedirectToAction("CanvasShare") 调用，因为该 try block 内的所有其他调用都不会生成异常。 (特别是因为您抑制了 MailEngine.Send 方法抛出的异常。)
• 调用 Sendlink(FormCollection formCollection) 中的空 catch block 。这是您应该生成错误消息并将其显示给用户的地方。但是，您已决定将其留空，这样就没人知道出了什么问题以及问题出在哪里。
• 控制流到达 Sendlink(FormCollection formCollection) 方法中的 return null; 语句。我的猜测是你把它放在那里是因为编译器提示缺少返回值。现在返回此 null 并导致呈现空 View 。

明显的解决方法是检查 RedirectToAction 并找出它抛出异常的原因。此异常可能表示您的代码或应用程序存在问题，您需要采取措施防止它发生。

下一个修复是在您的应用程序中实际实现错误处理。删除所有空的 catch block 并考虑是否要抛出异常或者是否要立即处理它。然而，忽略它几乎不是一个好主意。

为了说明您的应用程序中的问题:如果未抛出 RedirectToAction 中的异常，您的电子邮件发送可能仍然会失败。但是，您的 UI 没有任何方法可以发现出现问题，因为您忽略了 MailClient.Send 方法中的异常。如果您将其交付生产，电子邮件发送将无声地失败，您的客户会想知道为什么他们从未收到过电子邮件。这样一来，您就很难找出实际问题是什么以及发生在哪里。