- go fmt your code, make everyone happier
- multiple if statements can be collapsed into switch
- use
chan struct{}
to pass signalchan bool
makes it less clear, btwstruct{}
is more optimal
- prefer
30 * time.Seconds
instead oftime.Duration(30) * time.Seconds
- always wrap for-select idiom to a function
- group
const
declarations by type andvar
by logic and/or type - every blocking or IO function call should be cancelable or at least timeoutable
- implement
Stringer
interface for integers const values
- run
go format
on CI and compare diff- this will ensure that everything was generated and commited
- to run Travis-CI with the latest Go use
travis 1
- check if there are mistakes in code formatting
diff -u <(echo -n) <(gofmt -d .)
- best candidate to make something once in a thread-safe way is
sync.Once
- don't user flags, mutexes, channels or atomics
- do not omit
defer
- 200ns speedup is neglecatble in a most cases
- always close http body aka
defer r.Body.Close()
- unless you need leaked goroutine
- filtering without allocating
b := a[:0] for _, x := range a { if f(x) { b = append(b, x) } }
-
time.Time
has pointer fieldtime.Location
and this is bad go GC- it's relevant only for big number of
time.Time
, use timestamp instead
- it's relevant only for big number of
- prefer
regexp.MustCompile
instead ofregexp.Compile
- in most cases your regex is immutable, so init it in
func init
- in most cases your regex is immutable, so init it in
- do not overuse
fmt.Sprintf
in your hot path. It is costly due to maintaining the buffer pool and dynamic dispatches for interfaces.- if you are doing
fmt.Sprintf("%s%s", var1, var2)
, consider simple string concatenation. - if you are doing
fmt.Sprintf("%x", var)
, consider usinghex.EncodeToString
orstrconv.FormatInt(var, 16)
- if you are doing
- always discard body e.g.
io.Copy(ioutil.Discard, resp.Body)
if you don't use it- HTTP client's Transport will not reuse connections unless the body is read to completion and closed
res, _ := client.Do(req) io.Copy(ioutil.Discard, res.Body) defer res.Body.Close()
- strip your binaries with this command
go build -ldflags="-s -w" ...
- easy way to split test into different builds
- use
// +build integration
and run them withgo test -v --tags integration .
- use
-
go test -short
allows to reduce set of tests to be runnedfunc TestSomething(t *testing.T) { if testing.Short() { t.Skip("skipping test in short mode.") } }
- skip test deppending on architecture
if runtime.GOARM == "arm" { t.Skip("this doesn't work under ARM") }
- quick replace
gofmt -w -l -r "panic(err) -> log.Error(err)" .
- dump goroutines https://stackoverflow.com/a/27398062/433041
go func() { sigs := make(chan os.Signal, 1) signal.Notify(sigs, syscall.SIGQUIT) buf := make([]byte, 1<<20) for { <-sigs stacklen := runtime.Stack(buf, true) log.Printf("=== received SIGQUIT ===\n*** goroutine dump...\n%s\n*** end\n", buf[:stacklen]) } }()
- check interface implementation during compilation
var _ io.Reader = (*MyFastReader)(nil)
- if a param of len is nil then it's zero
- anonymous structs are cool
var hits struct { sync.Mutex n int } hits.Lock() hits.n++ hits.Unlock()
-
httputil.DumpRequest
is very useful thing, don't create your own