Как правильно обрабатывать ошибки pgx?

Question

[loljapanes]

Подскажите пожалуйста, как правильно обрабатывать sql ошибки? Использую pgx, сейчас обрабатываю ошибки таким образом:

if err != nil {
		var pgErr *pgconn.PgError
		if errors.As(err, &pgErr) {
			if pgErr.Code == "23505" {
				w.Write([]byte("Email is already exists"))
				return
			}
		}
	}

но честно говоря думаю что это не совсем верный способ.

Answer 1

[FedorWK]

if pgErr, ok := err.(pgconn.PgError); ok {
        errorHandler(pgErr)
}

Т.е. через обычное приведение типа.

Comments

[loljapanes]

Подскажите пожалуйста, сейчас выводит ошибку:

Impossible type assertion: 'pgconn.PgError' does not implement 'error'

и не совсем понял errorHandler будет отправлять на клиент такую ошибку:

ERROR: duplicate key value violates unique constraint "user_email_key" (SQLSTATE 23505)

ведь это не совсем удобно читать обычному пользователю?

[FedorWK]

loljapanes,
errorHandler просто для примера написал. Можно сразу после if обрабатывать ошибку, а можно вынести обработку ошибок в отдельную функцию и там формировать вывод для пользователя.

Что касается первого вопроса, действительно, советуют обрабатывать через "As".

Тогда возвращаясь к первоначальному вопросу, основная проблема тут в magic number "23505". Если в библиотеке нет списка человекочитаемых констант с номерами ошибок (я не нашел), можно завести свой. Например:

package pgerrcode

const (
        UniqueConstrintViolation = "23505"
        AnotherError = "another code"
        ...
)

И далее в программе сравнивать по названию констант:

if err != nil {
    var pgErr *pgconn.PgError
    if errors.As(err, &pgErr) {
      if pgErr.Code == pgerrorcode.UniqueConstrintViolation {
        w.Write([]byte("Email is already exists"))
        return
      }
    }
  }

[loljapanes]

FedorWK, Благодарю вас!