Телеграмм чат группы gogolang страница 20199

07:48пожаловаться #1

есть какой-нить линтер, который проверяет, что в канал не посылается адресс локальной переменной?

08:24пожаловаться #2

Vladyslav Yarema in Go-go!

Ivan Remen

Ну или с экспоненциальной задержкой в идеале)

backoff не подходит? https://github.com/cenkalti/backoff

08:38пожаловаться #3

koder

есть какой-нить линтер, который проверяет, что в канал не посылается адресс локальной переменной?

Нет никаких локальных переменных

09:12пожаловаться #4

Daniel Podolsky

Нет никаких локальных переменных

ага, и ложки нет, и слоника

09:25пожаловаться #5

Тем не менее.

09:27пожаловаться #6

var x []Struct = ...
for _, y := range x {
    ch <- &y
}

это сслылка на локальную переменную и в этом коде бага

09:30пожаловаться #7

Да, но дело не в локальности

09:30пожаловаться #8

var x []Struct = ...
for _, y := range x {
    z := y
    ch <- &z
}

z - такая же локальная, а баги нет

09:32пожаловаться #9

ок, не будем прояснять термины
есть ли линтер, который умеет ловить передачу ссылки с алиасингом

09:32пожаловаться #10

Daniel Podolsky

var x []Struct = ...
for _, y := range x {
    z := y
    ch <- &z
}

z - такая же локальная, а баги нет

да, эта бага может проявится массой способов, но отловить другие врядли можно так просто

09:33пожаловаться #11

koder

да, эта бага может проявится массой способов, но отловить другие врядли можно так просто

именно эта - только одним, на самом деле.

09:34пожаловаться #12

писать любую переменную, которая передивает итерацию цикла

09:35пожаловаться #13

переменная вне цикла

09:35пожаловаться #14

переменная на уровне модуля

09:35пожаловаться #15

поле в структуре

09:35пожаловаться #16

ну, хорошо, двумя

1. передать куда-то указатель на переменную цикла
2. использовать переменную цикла в замыкании

все остальное приведет к копированию переменной

09:36пожаловаться #17

Fast Go linters runner golangci-lint.

https://golangci-lint.run/usage/linters/ - вот этот список надо изучить

golangci-lint

Linters | golangci-lint

09:37пожаловаться #18

спасибо

09:37пожаловаться #19

оказывается - go vet это ловит :)