4

我有一个非常奇怪的问题,只发生在生产环境中。异常有消息

“委托给实例方法不能有 null 'this'”。

抛出异常的方法很简单,而且工作了很长时间,所以问题一定是环境中的一个模糊的依赖,或者类似的东西......

我正在使用托管在 Azure 中的 ASP.NET Web API,并且控制器的操作方法是通过 AJAX 执行的。

这是引发异常的代码:

public class BlacklistService : IBlacklistService
{
    public bool Verify(string blacklist, string message)
    {
        if (string.IsNullOrEmpty(blacklist)) return true;
        var split = blacklist.ToLower().Split(';'); // exception is thrown here
        return !split.Any(message.Contains);
    }
}

这是堆栈跟踪的相关部分:

at System.MulticastDelegate.ThrowNullThisInDelegateToInstance() 
at System.MulticastDelegate.CtorClosed(Object target, IntPtr methodPtr) 
at MyApp.Business.Services.BlacklistService.Verify(String blacklist, String message)
at MyApp.Business.Services.ContactMessageFactory.GetVerifiedStatus(String mensagem)
at MyApp.Business.Services.ContactMessageFactory.GetMailMessage(ContactForm contactForm)
at MyApp.Business.ContactEmailService.Send(ContactForm contactForm)

有人可以找出这个异常的可能原因吗?提前致谢。

4

3 回答 3

4

问题在于message事实上null。你可以很容易地重现这个:

void Main()
{
    Verify("hello", null);
}

public bool Verify(string blacklist, string message)
{
    if (string.IsNullOrEmpty(blacklist)) return true;
    var split = blacklist.ToLower().Split(';'); // exception is thrown here
    return !split.Any(message.Contains);
}

发生的事情是通过方法组转换message.Contains传递给Func<string, bool>构造函数,它看起来像这样:

Func<string, bool> func = ((string)null).Contains;
return !split.Any(func);

这就是MulticastDelegate香蕉的原因。您还可以在生成的 IL 中看到:

IL_0028:  ldftn       System.String.Contains
IL_002E:  newobj      System.Func<System.String,System.Boolean>..ctor
IL_0033:  call        System.Linq.Enumerable.Any

为了避免这种情况发生,请确保您也将检查消息设为空:

public bool Verify(string blacklist, string message)
{
    if (string.IsNullOrEmpty(blacklist)) return true;
    if (string.IsNullOrEmpty(message)) return false;

    var split = blacklist.ToLower().Split(';'); // exception is thrown here
    return !split.Any(message.Contains);
}
于 2015-09-15T18:47:04.513 回答
2

具有 null 的委托this是最后使用的方法string.Contains(),它使用您的message变量作为this指针。换句话说,在message为空的地方进行了调用。

于 2015-09-15T18:47:27.973 回答
1

当消息为空时失败。可以用这个

return !split.Any(part => (message != null && message.Contains(part)));
于 2015-09-15T18:41:49.343 回答