重载构造函数时如何避免NullReferenceException?

时间:2016-09-19 13:01:27

标签: c# nullreferenceexception constructor-overloading

我做了一个例子,以便更好地解释我的情况

void Main()
{
    var a = new Lol(null);
}

public class Lol
{
    public Lol(string a, string b)
    {
        if(a == null || b == null)
        {
            throw new Exception();
        }
    }

    public Lol(Tuple<string, string> k)
        : this(k.Item1, k.Item2)
    {
    }
}

在这种情况下,我在第二个构造函数中得到NullReferenceException。有没有办法从方法内部处理它,保持相同的结构,或者我应该创建一个私有方法并让两个构造函数都调用此方法?

6 个答案:

答案 0 :(得分:1)

您可以将逻辑抽象为辅助方法,并让两个构造函数都调用帮助程序。

public class Lol
{
    public Lol(string a, string b)
    {
        LolHelper(a, b);
    }

    public Lol(Tuple<string, string> k)
    {
        (k!=null)
            ?LolHelper(k.Item1, k.Item2)
            :LolHelper(null, null);
    }

    private void LolHelper(string a, string b)
    {
        if(a == null || b == null)
        {
            throw new Exception();
        }
    }
}

答案 1 :(得分:1)

不改变任何逻辑,你可以这样做:

public class Lol
{
    public Lol(string a, string b)
    {
        if(a == null || b == null)
        {
            throw new Exception();
        }
    }

    public Lol(Tuple<string, string> k)
    : this(k != null ? k.Item1 : null, k != null ? k.Item2 : null)
    {
    }
}

然而,在更复杂的情况下,这可能不起作用(尽管你不应该在构造函数链中放置任何复杂的逻辑)。

答案 2 :(得分:1)

这应该适用于带有C#6的VS2015:

this(k?.Item1, k?.Item2)

最后:

void Main()
{
    var a = new Lol(null);
}

public class Lol
{
    public Lol(string a, string b)
    {
        if(a == null || b == null)
            throw new Exception();
    }

    public Lol(Tuple<string, string> k)
        : this(k?.Item1, k?.Item2)
    {
    }
}

答案 3 :(得分:1)

首先不在构造函数中传递null。如果你想要对应于a和b的字段或属性为null,只需将它放在构造函数中,如下所示:

private string a;
private string b;
public Lol()
        {
            a= null;
            b= null;
        }

在Main()中使用:

var a = new Lol();

如果要传递值,而不是null,请使用适当的构造函数。

答案 4 :(得分:0)

这是设计使然,你需要在第二个构造函数中检查

public Lol(Tuple<string, string> k)

{
    if(k == null || k.Item1 == null || k.Item2 == null)
    {
        throw new Exception();
    }
}

答案 5 :(得分:0)

我的方法是扩展方法:

selectedItemsLabel

用法:

public static class Requirements
{
    public static T NotNull<T>(this T value, string parameterName, string message)
        where T : class =>
        value ?? throw new ArgumentNullException(parameterName, message);

    public static T NotNull<T>(this T value, string parameterName)
        where T : class =>
        value ?? throw new ArgumentNullException(parameterName);
}