在包含该方法的对象的构造函数中调用实例方法是一个好习惯吗?

时间:2012-06-10 10:13:04

标签: c# .net asp.net-mvc

示例:

public class EmailBusinessLogic
{
    #region Fields and Constructors
    SmtpClient smtp;
    Parameter prm;

    public EmailBusinessLogic()
    {
        prm = CostHelper.GetParameter();
        smtp = new SmtpClient(prm.EmailHost, prm.EmailPort);
        smtp.UseDefaultCredentials = prm.EmailUseDefaultCredentials;
        smtp.DeliveryMethod = GetDeliveryMethod(prm.EmailDeliveryMethod); //CALL TO METHOD DOWN BELOW, IS THIS A GOOD PRACTICE?
        smtp.EnableSsl = prm.EmailEnableSSL;
        smtp.Credentials = new NetworkCredential(prm.AppUserName, prm.AppPass, prm.AppNetworkDomain);

    }
    #endregion

    #region Instance Methods
    public SmtpDeliveryMethod GetDeliveryMethod(string name)
    {
        switch (name)
        {
            case "Network": return SmtpDeliveryMethod.Network;
            case "IISDirectory": return SmtpDeliveryMethod.PickupDirectoryFromIis;
            case "OtherDirectory": return SmtpDeliveryMethod.SpecifiedPickupDirectory;
            default: throw new NonExistentObjectException();
        }
    } 

我问这是因为它有一个小悖论,我知道在将这个类实例化为一个新对象时,总是首先调用构造函数。我不想让它成为静态方法,因为这将创建一个不会被垃圾收集的静态实例。

如果我错了,请纠正我,非常欢迎来自经验丰富的程序员的明确答复。谢谢。

1 个答案:

答案 0 :(得分:8)

在构造函数中调用非静态方法可以正常 - 但是你应该避免使用虚方法,因为如果方法被覆盖,那么实际的方法将是调用尚未调用类型级构造函数的类型 - 可能会导致问题。

调用静态方法会更可靠。我认为想要静态方法的原因有点混乱,并且不适用。绝对没有理由避免使用静态方法。静态方法未实现为“未收集垃圾的静态实例”上的实例方法。 对于静态方法没有实例