Goland提示Go for循环内变量未使用但外部已调用如何解决

问题根因 #

• Go语言中短变量声明符:=会在当前代码块作用域内创建全新的局部变量，不会复用外层作用域的同名变量
• 你代码中循环内的response, err := client.Do(request)实际创建了两个循环块内的局部变量：
  • 外层声明的response变量从未被实际赋值，因此被判定为未使用
  • err仅存在于循环内部，循环外的if err != nil访问的是完全不存在的变量，因此报未引用错误

修复步骤 #

1. 提前在函数作用域声明err变量，和response放在同一层级
2. 循环内替换短变量声明符:=为普通赋值符=，直接给外层变量赋值
3. 额外修正原代码中fmt.Printf传参错误：输出到stderr应该用fmt.Fprintf，fmt.Printf第一个参数要求是格式字符串，不能传os.Stderr

修复后完整代码 #

var backoffSchedule = []time.Duration{
    1 * time.Second,
    3 * time.Second,
    10 * time.Second,
}
// 提前在函数作用域声明两个变量
var response *http.Response
var err error

for _, backoff := range backoffSchedule {
    // 用=赋值，不创建新局部变量
    response, err = client.Do(request)
    if err == nil {
        break
    }
    // 替换为Fprintf输出到stderr
    fmt.Fprintf(os.Stderr, "Request error: %s\n", err)
    fmt.Fprintf(os.Stderr, "Retrying in %s\n", backoff)
    time.Sleep(backoff)

}

if err != nil {
    fmt.Printf("CRITICAL: API request failed: %s\n", err)
    os.Exit(2)
}

if response.StatusCode <= 199 || response.StatusCode >= 300 {
    // Non-successful response, alert
    fmt.Printf("CRITICAL: API request returned not ok: %s\n", response.Status)
    os.Exit(2)
}

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