Есть Var config = Config{} func main() {

Question

Есть Var config = Config{} func main() {

config, err := NewConfig(configFile)
}
парсится и заполняется
но в
func socketHandler(w http.ResponseWriter, r *http.Request) {}
config уже пустой
где косяк с областями видимости?

#backend #go #programming #russian

0

19.08.2021

10 ответов

13 просмотров

fenogentov Феногентов Автор вопроса

а почему? если эти данные используются во многих функциях

0

19.08.2021

Roman 🎄

Чтобы не было проблем с областями видимостями

0

19.08.2021

fenogentov Феногентов Автор вопроса

ок а чисто для развития - в чем может быть косяк?

0

19.08.2021

fenogentov Феногентов Автор вопроса

http.HandleFunc("/ws", socketHandler) как в socketHandler передать config?

0

19.08.2021

Roman 🎄

Косяк в том, что в config, err := NewConfig(configFile), переменная config - это локальная переменная, а не глобальная

0

19.08.2021

c

fenogentov Феногентов
http.HandleFunc("/ws", socketHandler) как в socket...

func socketHandler(w http.ResponseWriter, r *http.Request, cfg Config) {} ... cfg, err := NewConfig(configFile) ... http.HandleFunc("/ws", func(w http.ResponseWriter, r *http.Request) { return socketHandler(w, r, cfg) })

0

19.08.2021

Roman 🎄

То есть, чтобы пофиксить в данном случае надо писать var err error config,err = NewConfig(configFile)

0

19.08.2021

fenogentov Феногентов Автор вопроса

c
func socketHandler(w http.ResponseWriter, r *http....

прикольно, спасибо

0

19.08.2021

Andrei Yangabishev

по uber style имя должно быть _config, глобальные переменные должны выделяться. Хоть там про то, что случайно глобальную переменную перепишете, но ваш кейс тоже подходит. Я бы в линтер внес.

0

19.08.2021

Roman 🎄 · Accepted Answer

Roman 🎄

Лучше переписать код, чтобы там не было глобальных переменных

0

19.08.2021

170 похожих чатов

Есть Var config = Config{} func main() {

10 ответов

Похожие вопросы