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

И очевидное решение не всегда с точки зрения языка кошерное

01:52пожаловаться #1

inqfen

И очевидное решение не всегда с точки зрения языка кошерное

Это больше вопрос кодстайла. В данном случае я бы предпочел сделать так


vals := make(map[type]type)

val, err := getVal()
if err != nil { 
 ...
}

vals["key"] = val

Это прозрачно

02:01пожаловаться #2

Ну вот я к тому же, то ли err объявлять, то ли через промежуточную переменную делать

Это больше вопрос кодстайла. В данном случае я бы предпочел сделать так


vals := make(map[type]type)

val, err := getVal()
if err != nil { 
 ...
}

vals["key"] = val

Это прозрачно

можно сделать через if-else, тогда еще и пространство имен не будет засрано

02:03пожаловаться #4

Плюс в пользу решения выше: если функция вернет отшибку, то вы запишите в мапу невалидное значение, что может поломать вашу бизнес логику, если эта мапа используется где-то еще

можно сделать через if-else, тогда еще и пространство имен не будет засрано

vals := make(map[type]type)


if val, err := getVal(); err != nil { 
 ...
} else {
  vals["key"] = val
}

Если ошибка, exit(1)

Дальше имеет смысл только показать ошибку и выйти

02:06пожаловаться #8

inqfen

Если ошибка, exit(1)

А вот так в продакшн коде вообще делать не нужно ни при каких обстоятельствах)

02:06пожаловаться #9

А что неправильного в выходе с exit кодом не 0 при ошибке?

02:07пожаловаться #10

inqfen

А что неправильного в выходе с exit кодом не 0 при ошибке?

Ваше приложение падает при ошибке. Ошибки нужно обрабатывать, а не падать

Единственое, когда это действительно оправдано (исключая всякие паники) - при старте приложения, если никак не получается получить необходимые ресурсы (база лежит например), и делать это стоит только в main функции, чтобы не было неожиданностей для тех, кто будет работать с этим кодом после вас

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

Ну в данном случае - оно должно упасть, потому что у инстанса нет определенных вещей в метадате, либо метадата недоступна, причем должно упасть с читаемой ошибкой и exit code != 0, так как он должен быть обработан башем, это часть старта инстанса

02:13пожаловаться #12

  instanceParams["instance-id"], err = client.GetMetadata("instance-id")
  if err != nil {
    fmt.Println(err)
    os.Exit(1)
  }
  instanceParams["public-ip"], err = client.GetMetadata("public-ip")
  if err != nil {
    fmt.Println(err)
    os.Exit(1)
  }

При наличии любого из err дальшейшая работа не нужна, нужно выплюнуть ошибку и упасть

02:14пожаловаться #13

В дальнейшем exit code будет обработан уже башем, в разных сценариях падения возможно будут разные exit code

02:15пожаловаться #14

а кто это in Go-go!

все равно не очень красиво

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