1

我不知道为什么我会收到这个错误。看起来很初级。无论如何,我有一个名为 EmailSender 的单例类。下面的代码简短易懂。问题是我不能在 MainWindow 类中使用 sender。我尝试的任何操作,例如 sender.Send(),都被视为我已经完成了 asdafsafas.Send()。它被视为随机字符串。不知道为什么会这样。

using System;
using System.Net.Mail;
using System.Windows.Forms;

namespace SendMail
{
    public partial class MainWindow : Form
    {
        #region Private variables
        private MailMessage msg = new MailMessage();
        private EmailSender sender = EmailSender.GetInstance();
        #endregion

        public MainWindow()
        {
            InitializeComponent();

        }

        private MailMessage PrepareMailMessage()
        {

            return msg;
        }

        private void btnSend_Click(object sender, EventArgs e)
        {

        }
    }
}

这是 GetInstance 方法:

public static EmailSender GetInstance()
{
    return _instance ?? (_instance = new EmailSender());
}
4

4 回答 4

6

这是因为您定义此方法的方式(sender 是一个参数)。它首先找到方法参数,而不是您的类级别变量。您可以对此进行限定:

private void btnSend_Click(object sender, EventArgs e)
{
    // sender here is the "(object sender, " paramater, so it's defined
    // as system object.

    // use this instead:
    this.sender.Send(); // The "this" will make the class find the instance level variable instead of using the "object sender" argument
}
于 2011-11-04T19:21:21.380 回答
1

那是因为sender不是邮件对象,而是触发事件的按钮。您需要SmtpClient发送电子邮件:

private void btnSend_Click(object sender, EventArgs e)  
{  
    SmtpClient client = new SmtpClient("192.0.0.1", 25); //host, port
    client.Send(msg);
}

此外,MailMessage该类实现了IDisposable,因此您需要一些代码来处理完成后的消息。

我创建了一个包装器,其中包含您发送电子邮件所需的一切,包括处置:

/// <summary>
/// Wrapper class for the System.Net.Mail objects
/// </summary>
public class SmtpMailMessage : IDisposable
{
    #region declarations

    MailMessage Message;
    SmtpClient SmtpMailClient;

    #endregion

    #region constructors

    /// <summary>
    /// Default constructor for the SmtpMailMessage class
    /// </summary>
    public SmtpMailMessage()
    {
        //initialize the mail message
        Message = new MailMessage();
        Message.Priority = MailPriority.Normal;
        Message.DeliveryNotificationOptions = DeliveryNotificationOptions.OnFailure;            
        Message.From = new MailAddress("xxx@abc.com");           

        //initialize the smtp client
        SmtpMailClient = new SmtpClient();
        SmtpMailClient.DeliveryMethod = SmtpDeliveryMethod.Network;
        SmtpMailClient.Host = "192.168.0.1";
        SmtpMailClient.Port = 25;
    }

    /// <summary>
    /// Parameterized constructor for the SmtpMailMessage class. Allows for override of the default
    /// SMTP host and port number
    /// </summary>
    /// <param name="HostIP">The IP address of the exchange server</param>
    /// <param name="PortNumber">The port number for ingoing and outgoing SMTP messages</param>
    public SmtpMailMessage(string HostIP, int PortNumber) : this()
    {
        //override the smtp host value
        SmtpMailClient.Host = HostIP;

        //override the smtp port value
        SmtpMailClient.Port = PortNumber;
    }

    #endregion

    #region subject / body

    /// <summary>
    /// The body content of the mail message
    /// </summary>
    public string Body
    {
        get
        {
            return Message.Body;
        }
        set
        {
            Message.Body = value;
        }
    }

    /// <summary>
    /// the subject of the mail message
    /// </summary>
    public string Subject
    {
        get
        {
            return Message.Subject;
        }
        set
        {
            Message.Subject = value;
        }
    }

    #endregion

    #region mail type

    /// <summary>
    /// Gets or sets a value that determines whether the mail message
    /// should be formatted as HTML or text
    /// </summary>
    public bool IsHtmlMessage
    {
        get
        {
            return Message.IsBodyHtml;
        }
        set
        {
            Message.IsBodyHtml = value;
        }
    }

    #endregion

    #region sender

    /// <summary>
    /// Gets or sets the from address of this message
    /// </summary>
    public string From
    {
        get
        {
            return Message.From.Address;
        }
        set
        {
            Message.From = new MailAddress(value);
        }
    }

    #endregion

    #region recipients

    /// <summary>
    /// Gets the collection of recipients
    /// </summary>
    public MailAddressCollection To
    {
        get
        {
            return Message.To;

        }
    }

    /// <summary>
    /// Gets the collection of CC recipients 
    /// </summary>
    public MailAddressCollection CC
    {
        get
        {
            return Message.CC;
        }
    }

    /// <summary>
    /// Gets the collection of Bcc recipients
    /// </summary>
    public MailAddressCollection Bcc
    {
        get
        {
            return Message.Bcc;
        }
    }

    #endregion

    #region delivery notification

    /// <summary>
    /// Gets or sets the delivery notification settings for this message
    /// </summary>
    public DeliveryNotificationOptions DeliveryNotifications
    {
        get
        {
            return Message.DeliveryNotificationOptions;
        }
        set
        {
            Message.DeliveryNotificationOptions = value;
        }
    }

    #endregion

    #region priority

    /// <summary>
    /// Gets or sets the Priority of this message
    /// </summary>
    public MailPriority PriorityLevel
    {
        get
        {
            return Message.Priority;
        }
        set
        {
            Message.Priority = value;
        }
    }

    #endregion

    #region send methods

    /// <summary>
    /// Sends the message anonymously (without credentials)
    /// </summary>
    public void Send()
    {
        SmtpMailClient.Send(Message);
    }

    /// <summary>
    /// Sends the message with authorization from a network account   
    /// </summary>
    /// <param name="Username">The Windows username of the authorizing user</param>
    /// <param name="Password">The Windows password of the authorizing user</param>
    /// <param name="Domain">The domain name of the network to which the authorizing user belongs</param>
    public void Send(string Username, string Password, string Domain)
    {
        //attach a network credential to this message using the information passed into the method
        SmtpMailClient.Credentials = new NetworkCredential(Username, Password, Domain);

        //send the message
        SmtpMailClient.Send(Message);
    }

    #endregion

    #region IDisposable implementation

    ~SmtpMailMessage()
    {
        Dispose(false);
    }

    public void Dispose()
    {
        Dispose(true);            
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposing)
        {
            if (Message != null)
                Message.Dispose();
            Message = null;                
            SmtpMailClient = null;
        }
    }

    #endregion        
}

执行:

using (SmtpMailMessage mail = new SmtpMailMessage("192.168.0.1", 25))
{
    //set the to address to the primary email
    mail.To.Add("xxx@abc.com");

    //set the message type and subject and body
    mail.IsHtmlMessage = true;
    mail.Subject = "Foo";
    mail.Body = "Hello world!";

    //send the email
    mail.Send();
}
于 2011-11-04T19:19:34.750 回答
0

如果您在btnSend_Click方法范围内执行此操作,则参数object sender优先于全局EmailSender sender.

您应该重命名全局变量,例如:EmailSender m_sender或准确指定您想要的发件人:this.sender.Send()

于 2011-11-04T19:20:59.877 回答
0

我猜你在 btnSend_Click 函数中调用了 sender.Send 。

在该函数中有一个参数,也称为发送者(对象发送者)。现在你的代码很困惑使用哪一个。所以重命名您的私人 var 发件人。

于 2011-11-04T19:22:07.623 回答