Go语言HTTP请求中的空指针解引用：原因、避免与最佳实践

// 原始代码片段（存在问题）
func fetchPic(picUrl []string) {
    for key, value := range picUrl {
        // ...
        httpRequest, err := http.Get(string(value)) // 此处可能返回nil的httpRequest和非nil的err
        fmt.Print("load ok \n") // 错误：未在此处检查err
        httpRequest.Body.Close() // 如果httpRequest为nil，这里会引发panic
        result, readErr := ioutil.ReadAll(httpRequest.Body) // 同上，也会引发panic
        // ...
    }
}

resp, err := http.Get(url) // 使用resp替代httpRequest，更符合Go惯例
if err != nil {
    // 处理错误，例如打印日志，跳过当前URL，或者返回错误
    fmt.Printf("Error fetching URL %s: %v\n", url, err)
    return // 或者 continue
}
// 只有当err为nil时，resp才保证是非nil的，可以安全地进行后续操作

resp, err := http.Get(url)
if err != nil {
    fmt.Printf("Error fetching URL %s: %v\n", url, err)
    return
}
// 确保在函数退出前关闭响应体，无论后续操作是否成功
defer resp.Body.Close()

// 现在可以安全地读取响应体
bodyBytes, readErr := ioutil.ReadAll(resp.Body)
if readErr != nil {
    fmt.Printf("Error reading response body from %s: %v\n", url, readErr)
    return
}
// ... 处理 bodyBytes

package main

import (
    "bufio"
    "fmt"
    "io"
    "io/ioutil"
    "net/http"
    "os"
    "path/filepath" // 引入filepath处理路径
    "strconv"       // 引入strconv用于数字转字符串
)

func main() {
    fileName := "meinv.txt"
    file, err := os.Open(fileName)
    if err != nil {
        fmt.Printf("Error opening file %s: %v\n", fileName, err)
        return
    }
    defer file.Close() // 确保文件在main函数退出前关闭

    // 预估容量，避免频繁扩容，如果知道大致行数
    picUrls := make([]string, 0, 2000)
    reader := bufio.NewReader(file)
    for {
        lineBytes, _, err := reader.ReadLine()
        if err == io.EOF {
            break // 文件读取完毕
        }
        if err != nil {
            fmt.Printf("Error reading line from file: %v\n", err)
            break // 其他读取错误，中断循环
        }
        line := string(lineBytes)
        if line != "" { // 避免添加空行
            fmt.Printf("File loaded: %s \n", line)
            picUrls = append(picUrls, line)
        }
    }
    fmt.Printf("File loaded, ready to download %d URLs.\n", len(picUrls))
    fetchPic(picUrls)
}

func fetchPic(picUrls []string) {
    // 确保图片保存目录存在
    outputDir := "pics"
    if _, err := os.Stat(outputDir); os.IsNotExist(err) {
        err := os.Mkdir(outputDir, 0755) // 0755是常见的目录权限
        if err != nil {
            fmt.Printf("Error creating directory '%s': %v\n", outputDir, err)
            return // 目录创建失败，无法继续
        }
    }

    for key, value := range picUrls {
        fmt.Printf("Downloading %d: %s \n", key, value)

        // 1. 发起HTTP GET请求并立即检查错误
        resp, err := http.Get(value)
        if err != nil {
            fmt.Printf("Error fetching URL %s: %v\n", value, err)
            continue // 跳过当前URL，处理下一个
        }
        // 2. 使用defer确保响应体关闭，释放资源。必须在确认resp非nil后立即放置。
        defer resp.Body.Close()

        // 3. 读取响应体内容并检查错误
        bodyBytes, readErr := ioutil.ReadAll(resp.Body)
        if readErr != nil {
            fmt.Printf("Error reading response body from %s: %v\n", value, readErr)
            continue // 读取失败，跳过
        }

        // 4. 构建文件名并保存文件
        // 使用strconv.Itoa将key转换为字符串，避免与内置len函数冲突
        // 使用filepath.Join构建跨平台兼容的路径
        filePath := filepath.Join(outputDir, strconv.Itoa(key)+".jpg")
        writeErr := ioutil.WriteFile(filePath, bodyBytes, 0644) // 0644是常见的文件权限
        if writeErr != nil {
            fmt.Printf("Error writing file %s: %v\n", filePath, writeErr)
            continue // 写入失败，跳过
        }

        fmt.Printf("Successfully downloaded and saved %s (size: %d bytes)\n", filePath, len(bodyBytes))
    }
}