解决SonarQube检测到的C#中HttpResponseMessage空引用Bug

咱们先拆解下SonarQube为什么会报这个Bug：虽然你在代码开头就初始化了response，但静态分析工具检测到存在潜在的执行路径，会导致进入else分支时response为null。具体来说可能的场景包括：

1. 极端情况下SendHttpRequest返回null（虽然HttpClient.SendAsync正常不会返回null，但代码层面没有禁止返回null的约束）；
2. 静态分析工具可能没完全识别到你在try块外的初始化逻辑，误以为某些异常路径下response未被赋值。

下面给你两种靠谱的解决方式：

方式一：在else分支中先判空再访问属性 #

直接在日志输出前检查response是否为null，避免空引用异常，同时让SonarQube的分析逻辑满意：

else {
    string errorMsg = response != null 
        ? $"Http response failure with Status Code : {response.StatusCode}"
        : "Http request failed: response is null";
    log.Error(errorMsg);
}

或者更直白的分支判断：

else {
    if (response == null) {
        log.Error("Http request failed: response is null");
    } else {
        log.Error("Http response failure with Status Code : " + response.StatusCode);
    }
}

方式二：确保SendHttpRequest永远不返回null #

给SendHttpRequest加一层兜底，保证无论什么情况都返回一个有效的HttpResponseMessage实例，从根源上消除null的可能性：

public virtual async Task<HttpResponseMessage> SendHttpRequest(HttpRequestMessage httpRequest) {
    using (HttpClient client = new HttpClient()) {
        HttpResponseMessage response = await client.SendAsync(httpRequest).ConfigureAwait(false);
        // 极端情况兜底，避免返回null
        return response ?? new HttpResponseMessage(HttpStatusCode.InternalServerError);
    }
}

额外小提醒 #

你的代码里每次调用SendHttpRequest都创建新的HttpClient实例，这其实不太推荐——频繁创建HttpClient会导致系统socket资源耗尽。建议把HttpClient作为类的静态成员或者通过依赖注入复用，这是个优化点，但和当前Bug无关。

内容的提问来源于stack exchange，提问作者CrazyCoder