专栏首页TIGERB的技术博客go新手容易犯的三个致命错误

go新手容易犯的三个致命错误

前言

最近因为以前一些重要且古老的go项目基本没有人专职维护了,所以被安排去熟悉这些项目的代码,所以看了大量go的代码。历史原因,这些代码中或多或少有一些刚刚从PHPer转过来的Gopher去设计和开发的,自然有不少是在php(fpm模式下)码代码思路下埋藏的一些坑。今天我就来和大家一起分享一下最近发现的出现比率比较高的三个致命错误

三个致命错误

致命错误一: defer的错误使用

  • 现象:死循环代码块中直接使用defer(非函数内部的defer)
  • 问题:defer代码一直不会执行
  • 例如:下面的示例,正常情况下defer redisConn.Close()一直不会执行,所以redis的连接数会持续增长得不到释放,搞不好redis直接被打挂。
  • 经验:监测服务资源发现socket(redis/mysql等)连接持续增长,就需要我们去找代码里出现的类似的代码了

监测redis连接数会持续增长命令: watch -n 2 "redis-cli -h 127.0.0.1 -p 6379 info | grep 'connected_clients'" 下面的代码会导致connected_clients持续增长

package main

import (
    "fmt"
    "time"

    "github.com/gomodule/redigo/redis"
)

var RedisPool *redis.Pool

func init() {
    RedisPool = NewRedisPool()
    fmt.Println("RedisPool.Stats: ", RedisPool.Stats())
}

func main() {
    for {
        redisConn := RedisPool.Get()
        // 下意识的defer 但是忘了是在for循环了 除了进程挂了基本是不会执行这个defer了 资源得不到释放
        defer redisConn.Close()

        // 一堆业务逻辑
        _, err := redisConn.Do("set", "demo_key", "666")
        if err != nil {
            fmt.Println("redis set err: ", err.Error())
            continue
        }
        res, _ := redis.String(redisConn.Do("get", "demo_key"))
        fmt.Println("get demo_key: ", res)
        time.Sleep(1 * time.Second)
    }
}

func NewRedisPool() *redis.Pool {
    return &redis.Pool{
        MaxIdle:     6,
        IdleTimeout: 240 * time.Second,
        Dial: func() (redis.Conn, error) {
            c, err := redis.Dial("tcp", "127.0.0.1:6379")
            if err != nil {
                return nil, err
            }
            return c, nil
        },
        TestOnBorrow: func(c redis.Conn, t time.Time) error {
            if time.Since(t) < time.Minute {
                return nil
            }
            _, err := c.Do("PING")
            return err
        },
    }
}

致命错误二: 死循环中一直持有一个连接

  • 现象:死循环外面获取的连接,在死循环中使用,所以直到进程挂掉为止,这个goroutine一直持有该连接
  • 问题:如果资源服务端因为种种原因主动挂掉了这个连接(比如服务端超时),这个循环的代码之后就永远连接服务,代码逻辑就不用说了基本无法正常执行
  • 例如:下面的示例,redis因为redis proxy超时主动关闭了连接,就会报EOF
  • 经验:如果服务大范围报EOF错误,就需要我们去排查类似的代码了
package main

import (
    "fmt"
    "time"

    "github.com/gomodule/redigo/redis"
)

var RedisPool *redis.Pool

func init() {
    RedisPool = NewRedisPool()
    fmt.Println("RedisPool.Stats: ", RedisPool.Stats())
}

func main() {
    // 死循环外面获取的连接 所以直到进程挂掉这个goroutine一直持有是这个连接
    redisConn := RedisPool.Get()
    defer redisConn.Close()
    
    for {
        // 一堆业务逻辑
        _, err := redisConn.Do("set", "demo_key", "666")
        if err != nil {
            fmt.Println("redis set err: ", err.Error())
            continue
        }
        res, _ := redis.String(redisConn.Do("get", "demo_key"))
        fmt.Println("get demo_key: ", res)
        time.Sleep(1 * time.Second)
    }
}

func NewRedisPool() *redis.Pool {
    return &redis.Pool{
        MaxIdle:     6,
        IdleTimeout: 240 * time.Second,
        Dial: func() (redis.Conn, error) {
            c, err := redis.Dial("tcp", "127.0.0.1:6379")
            if err != nil {
                return nil, err
            }
            return c, nil
        },
        TestOnBorrow: func(c redis.Conn, t time.Time) error {
            if time.Since(t) < time.Minute {
                return nil
            }
            _, err := c.Do("PING")
            return err
        },
    }
}

致命错误三:err.Error()使用位置不对

  • 现象:有时候打业务log的时候,获取错误信息err.Error()的代码忘了写在err !=nil
  • 问题:代码可以编译通过,但是运行到该处代码块时空指针panic
  • 问题:例下面的示例,模拟业务中某些情况才会执行下面的代码块
  • 经验:养成强类型语言下严谨的逻辑习惯
package main

import (
    "fmt"
    "log"
    "time"
)

func main() {
    var i int
    ticker := time.NewTicker(1 * time.Second)
    for v := range ticker.C {
        fmt.Println(v, i)
        i = i + 1
        // 模拟业务中某些情况才会执行下面的代码块
        if i == 6 {
            res, err := Simulate(i)
            // 有时候打业务log的时候 获取错误信息 err.Error() 的代码忘了写在err != nil里 导致空指针
            log.Println(fmt.Sprintf("res:%t i:%d err:%s", res, i, err.Error()))
            if err != nil {
                return
            }
        }
    }
}

func Simulate(i int) (b bool, err error) {
    return true, nil
}

代码可以编译通过,但是运行到该处代码块时空指针panic,如下模拟:

2019-01-19 23:56:48.044504 +0800 CST m=+1.005583125 0
2019-01-19 23:56:49.039491 +0800 CST m=+2.000557249 1
2019-01-19 23:56:50.03956 +0800 CST m=+3.000614086 2
2019-01-19 23:56:51.043367 +0800 CST m=+4.004408337 3
2019-01-19 23:56:52.040469 +0800 CST m=+5.001497207 4
2019-01-19 23:56:53.039643 +0800 CST m=+6.000658300 5
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x1097a7f]

goroutine 1 [running]:
main.main()
        /Users/tigerb/github/easy-tips/go/src/go-learn/main.go:19 +0x1df

结语

最后说一句,像我们这样从PHPer(fmp)转过来的Gopher,码代码的时候一定要考虑到我们是在常驻内存的场景下编程,例如并不限于下面三点:

  • 全局变量
  • 线程安全
  • 资源回收

本文分享自微信公众号 - TIGERB的技术博客(tigerbcode),作者:TIGERB

原文出处及转载信息见文内详细说明,如有侵权,请联系 yunjia_community@tencent.com 删除。

原始发表时间:2019-01-20

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

我来说两句

0 条评论
登录 后参与评论

相关文章

  • 面向对象的设计过程

    再接着,一个业务逻辑(拆成多个类方法)从头写到尾,也许、可能、貌似、猜测大多数人停留到了这个阶段。

    用户1093396
  • 客户决策 | 我的代码没有else

    「策略模式」比较简单,大家平常工作中应该经常使用到,所以本文作为复习,帮助大家温故知新。我们先来看下定义:

    用户1093396
  • [Skr-Shop]通用抽奖工具之需求分析

    还不更新❓❓❓来啦~ Skr Shop年底第二弹《营销体系》第二篇《通用抽奖工具之需求分析》~

    用户1093396
  • Redis 学习笔记9 - 使用 redigo 访问 redis

    Redis 的 API 非常简单和易于编程。处理使用命令行工具来连接,也可以用你喜欢的语言,本文描述使用 Go 语言 通过 redigo 库连接。

    zhangyunfeiVir
  • 【Go 语言社区】Golang语言操作redis连接池的方法

    func newPool(server, password string) *redis.Pool { return &redis.Pool{ ...

    李海彬
  • go语言操作redis连接池的方法

    unc newPool(server, password string) *redis.Pool { return &redis.Pool{ ...

    李海彬
  • 520|解锁Python表白新姿势

    刘早起
  • JavaWeb三层结构---课设02

    收获总结 1三层架构模式 区分层次的目的即为了“高内聚,低耦合”的思想 ? 分层介绍: Javaweb设计分为三层:数据访问层,业务逻辑层和表示层。 数据访问层...

    用户1631856
  • UI设计师axure工具入门指南

    编程软文
  • Good Feature Matching: Towards Accurate, Robust VO/VSLAM with Low Latency 良好的特征匹配:实现准确、鲁棒的低延迟VO/VSLA

    —在VO或VSLAM系统中保持性能(精确度和鲁棒性)和效率(延迟)的取舍是一个重要的课题。基于特征的系统展现了良好的性能,但由于显式的数据关联有更高的时延;直接...

    用户1150922

扫码关注云+社区

领取腾讯云代金券