Три смертельные ошибки, которые часто совершают новички

Go

предисловие

В последнее время из-за того, что некоторые важные и древние го-проекты принципиально не поддерживались штатно, их устраивали для ознакомления с кодом этих проектов, поэтому я много читал го-кода. По историческим причинам некоторые из этих кодов более или менее спроектированы и разработаны Гофером, который только что перешел из PHPer.Естественно, многие из них представляют собой какие-то ямы, погребенные под кодовой идеей php (режим fpm). Сегодня я поделюсь с вами недавно обнаруженными, которые не так просто найти и имеют высокую скорость появления.три фатальные ошибки.

три фатальные ошибки

Фатальная ошибка номер один: неправильное использование defer

  • Явление: defer используется непосредственно в блоке кода бесконечного цикла (defer не внутри функции)
  • Проблема: код отсрочки никогда не выполняется
  • Например: следующий пример, при нормальных обстоятельствахdefer redisConn.Close()Он не будет выполняться постоянно, поэтому количество соединений redis будет продолжать расти и не освобождаться.
  • Опыт: Мониторинг сервисных ресурсов и обнаружение соединений сокетов (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
		},
	}
}

Фатальная ошибка 2: соединение удерживается в бесконечном цикле

  • Явление: соединение, полученное вне бесконечного цикла, используется в бесконечном цикле, поэтому, пока процесс не зависнет, горутина удерживает соединение.
  • Проблема: Если ресурсный сервер по разным причинам активно вешает соединение (например, тайм-аут сервера), то код этого цикла будет подключен к сервису навсегда, и логика кода не сможет нормально выполняться.
  • Например: в следующем примере Redis активно закрывает соединение из-за истечения времени ожидания прокси-сервера Redis, и он сообщит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 () используется в неправильном месте

  • Феномен: иногда, когда я попадаю в бизнес-журнал, я получаю сообщение об ошибке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
}

Код компилирует и проходит, но при работе к блоку кода тамнулевой указательпаники, смоделируйте следующим образом:

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

Эпилог

Наконец, такой Gopher, как мы, который переведен с PHPer (fmp), должен учитывать, что мы программируем в сценарии резидентной памяти, при кодировании, например, он не ограничивается следующими тремя пунктами:

  • глобальная переменная
  • потокобезопасность
  • Рециркулировать

Категории