Go REST日志统计接口的异常处理与无响应问题咨询

现存核心故障点 #

• 自定义的recoverInvalidAccess仅做控制台日志打印，没有向HTTP响应写入任何错误信息，触发panic后连接直接中断，客户端拿不到任何返回内容。
• 字符串切片操作无边界判断：不管findDate返回的字符串长度是否满足切片要求，直接执行date[len(date)-8:]、date[:len(date)-9]的截取操作，只要遇到格式不匹配的日志行，直接触发切片越界panic。
• 文件打开失败后的流程中断缺失：os.Open返回错误后，没有直接向客户端返回错误、终止执行，反而继续用nil的文件句柄创建读取器，后续操作直接触发空指针panic；且defer file.Close()写在错误判断逻辑外，打开失败时对nil值执行Close操作同样会触发panic。
• 错误返回逻辑无效：定义了res model.ResponseResult结构体，但仅在出错时给Error字段赋值，从未把res序列化写入HTTP响应，所有赋值的错误信息根本到不了客户端。
• 逐行读取的循环退出逻辑混乱：遇到非EOF的读取错误时，仅把错误赋值给变量就跳出内层循环，没有中断整体流程，导致读取出错后还在继续处理空内容行。
• JSON编码返回时未校验错误：没有判断json.NewEncoder(w).Encode()的返回值，编码失败时也没有做兜底响应。

正确的异常处理改造逻辑 #

1. 调整panic兜底逻辑，返回标准化错误响应 #

改造原来仅打印日志的recover函数，捕获panic后设置正确的HTTP状态码，返回结构化错误信息，避免请求直接断连。注意defer注册函数时要把ResponseWriter传入，确保兜底逻辑能写入响应：

func recoverInvalidAccess(w http.ResponseWriter) {
    if r := recover(); r != nil {
        // 打印panic堆栈方便内部排查，生产环境不要把内部错误细节返回给客户端
        fmt.Printf("recover panic: %v\n", r)
        w.WriteHeader(http.StatusInternalServerError)
        json.NewEncoder(w).Encode(map[string]string{
            "error": "internal server error",
        })
    }
}

在接口函数开头注册时传入w参数：

func IPStatistics(w http.ResponseWriter, r *http.Request) {
    defer recoverInvalidAccess(w)
    // ... 后续业务逻辑
}

2. 对可预判错误做显式判断，不要全靠recover兜底 #

recover仅用来兜底完全没预料到的运行时异常，所有可提前识别的错误必须通过if判断处理，避免无意义的栈展开开销：

• 正则提取类函数（findIP、findDate、findRequstMethod）的返回值要加长度校验，提取到的字符串长度不足以完成切片操作时，直接跳过当前异常日志行，不要硬切：

date := findDate(line)
// 校验日期字符串长度，满足截取要求才做切片
if len(date) < 9 {
    // 格式异常的行打日志记录后跳过即可
    fmt.Printf("invalid log line, date extract failed: %s\n", line)
    continue
}
time := date[len(date)-8:]
date = date[:len(date)-9]

• 文件打开操作拿到错误后第一时间中断流程，不要继续向下执行：

file, err := os.Open(fn)
if err != nil {
    w.WriteHeader(http.StatusInternalServerError)
    json.NewEncoder(w).Encode(map[string]string{
        "error": "log file open failed",
    })
    return // 必须加return终止流程
}
defer file.Close() // 确保file非nil之后再加defer，避免空指针panic

3. 规范读取循环的错误处理 #

逐行读取日志时，遇到非EOF的读取错误时，直接终止循环并返回错误给客户端，不要带着错误状态继续处理数据：

for {
    var buffer bytes.Buffer
    var l []byte
    var isPrefix bool
    for {
        l, isPrefix, err = reader.ReadLine()
        buffer.Write(l)
        if !isPrefix {
            break
        }
        if err != nil {
            if err != io.EOF {
                // 读取出错直接返回错误，不要仅给res.Error赋值
                w.WriteHeader(http.StatusInternalServerError)
                json.NewEncoder(w).Encode(map[string]string{
                    "error": "read log file failed",
                })
                return
            }
            break
        }
    }
    line := buffer.String()
    // 逐行做字段提取、长度校验...
    if err == io.EOF {
        break
    }
}

4. JSON返回前做错误校验 #

if err := json.NewEncoder(w).Encode(IPDataL1); err != nil {
    fmt.Printf("encode response failed: %v\n", err)
}

生产环境注意事项 #

• 切片操作前一定要校验源字符串长度：日志文件中经常会存在格式错误、异常断行的内容，直接硬切截取字符串是生产环境最容易触发线上panic的场景。
• 不要把recover作为常规错误处理手段：可预判的错误必须用显式的错误判断流程处理，recover仅作为最后一层兜底，避免所有流程都靠panic驱动。
• defer注册的函数如果依赖外部非nil的对象，一定要确保对象初始化成功之后再注册defer，否则对nil值执行方法调用会直接触发panic。

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