Привет, насколько такой код корректный и нормальный? func (d *DB) queryBool(query

Там же один результат возвращается я имел в виду QueryRow или QueryRowContext. И rows тогда не надо закрывать, в первоначальном виде этого нет почему то

0

30.05.2023

Artur Karapetov

не хватает проверок на NotFound, UniqueViolation и соотвественно отдавать надо не pg ошибки, а свои из DAL, чтобы уже на слое с БЛ проверять эти ошибки и иметь возможность написать логику на них.

0

30.05.2023

Artur Karapetov

знаю что она кастрированная, но там вроде есть ошибки

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

Artur Karapetov
знаю что она кастрированная, но там вроде есть оши...

там так-то ошибок никаких и не может возникнуть, кроме как если диск посыпется, допустим у меня выше этой функции просто стоит паник, если err != nil ибо если вылетит какая то ошибка = дальше оно уже работу никак не может продолжить

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

Rostislav Teryaev
QueryContext(ctx, ...) наверное имел в виду

а какой тут толк мне от контекста? // rows.Close() я так понимаю в таком случае нужно добавить обязательно?

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

🦊 Fox 🌸 🌸
а какой тут толк мне от контекста? // rows.Close()...

https://go.dev/doc/database/querying а тут Rows не закрывается нигде, кроме последнего примера🤔

0

30.05.2023

Maxim Biryukov

🦊 Fox 🌸 🌸
https://go.dev/doc/database/querying а тут Rows не...

func albumsByArtist(artist string) ([]Album, error) { rows, err := db.Query("SELECT * FROM album WHERE artist = ?", artist) if err != nil { return nil, err } defer rows.Close()

0

30.05.2023

Maxim Biryukov

🦊 Fox 🌸 🌸
https://go.dev/doc/database/querying а тут Rows не...

там queryROW. его не нужно закрывать, его я и предложил.

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

Maxim Biryukov
там queryROW. его не нужно закрывать, его я и пред...

A

0

30.05.2023

Maxim Biryukov

🦊 Fox 🌸 🌸
а какой тут толк мне от контекста? // rows.Close()...

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

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

🦊 Fox 🌸 🌸
A

тоесть, вот так? func (d *DB) queryBool(query string, args ...any) (result bool, err error) { res := d.db.QueryRow(query, args...) err = res.Scan(&result) return } или лучше вот так? func (d *DB) queryBool(query string, args ...any) (bool, error) { var result bool res := d.db.QueryRow(query, args...) err := res.Scan(&result) return result, err }

0

30.05.2023

Maxim Biryukov

🦊 Fox 🌸 🌸
тоесть, вот так? func (d *DB) queryBool(query str...

мне первый больше нравится, но можно еще 1 и 2 строки в одной написать func (d *DB) queryBool(query string, args ...any) (result bool, err error) { err = d.db.QueryRow(query, args...).Scan(&result) return }

0

30.05.2023

🦊 Fox 🌸 🌸 Автор вопроса

Maxim Biryukov
мне первый больше нравится, но можно еще 1 и 2 стр...

окей, спасибо!

0

30.05.2023

Rostislav Teryaev

🦊 Fox 🌸 🌸
а какой тут толк мне от контекста? // rows.Close()...

контекст тут нужен для выставления таймаута. Чтобы приложение навечно не зависло на этой операции

0

30.05.2023