Gin: Прерывание против паники

Созданный на 14 июн. 2015 · 3Комментарии · Источник: gin-gonic/gin

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

if err != nil {
    c.AbortWithError(code, err)
    return
}

это уродливо с таким количеством return

Я думаю, что паника может быть лучшей практикой. Как Java throw Exception

if err != nil {
    panic(err) // maybe err is ErrUserNotFound
}

err определяется как

type HTTPError interface {
    HTTPStatus() int
}

используя промежуточное ПО catchError, чтобы перехватить ошибку и прервать выполнение с помощью HTTPStatus

func catchError() gin.HandlerFunc {
    return func(c *gin.Context) {
        defer func() {
            if err := recover(); err != nil {
                switch err.(type) {
                case HTTPError:
                    e := err.(*errors.Error)
                    c.JSON(e.HTTPStatus(), e)
                    c.Abort()
                default:
                    c.AbortWithStatus(http.StatusInternalServerError)
                }
            }
        }()
        c.Next()
    }
}

много некрасивых return удалены

bug wontfix

Источник

lilee

Самый полезный комментарий

Совет: реорганизуйте код на более мелкие функции, поэтому вместо повторения:

if err != nil {
    c.AbortWithStatus(code, error)
    return 
}

ты сделаешь:

func dosomething() error {
    err = dosomething1()
    if err != nil {
        return err
    }
    err = dosomething2()
    if err != nil {
        return err
    }
    err = dosomething3()
    if err != nil {
        return err
    }
}

func handler(c *gin.Context) {
    err := dosomething()
    if err != nil {
        c.AbortWithStatus(400, err)
        return;
    }
    // ...
}

manucorporat 14 июн. 2015

👍6

Все 3 Комментарий

@lilee в настоящее время лучше всего:

AbortWithError()
return;

а затем промежуточное программное обеспечение читает c.Errors , чтобы что-то сделать с ошибками.

Использование panic () - плохая идея:

Это не идиоматика Go. В Go panic () - строгое исключение (или утверждение). т.е. произошло что-то действительно плохое (вероятно, ошибка).
Вызов panic () выполняется медленно
Когда вызывается panic (), большинство промежуточного программного обеспечения не будет работать правильно, поскольку panic () обычно означает 500.
Отладка: вы не сможете отличить настоящие сбои (например, нет доступа к нулю) и ошибки.

Конечно, вы можете использовать все, что захотите, но использование panic () для управления логическим нормальным потоком - это анти-шаблон в Go.

manucorporat 14 июн. 2015

👍2

Совет: реорганизуйте код на более мелкие функции, поэтому вместо повторения:

if err != nil {
    c.AbortWithStatus(code, error)
    return 
}

ты сделаешь:

func dosomething() error {
    err = dosomething1()
    if err != nil {
        return err
    }
    err = dosomething2()
    if err != nil {
        return err
    }
    err = dosomething3()
    if err != nil {
        return err
    }
}

func handler(c *gin.Context) {
    err := dosomething()
    if err != nil {
        c.AbortWithStatus(400, err)
        return;
    }
    // ...
}

manucorporat 14 июн. 2015

👍6

Спасибо за объяснение

lilee 15 июн. 2015

Была ли эта страница полезной?

0 / 5 - 0 рейтинги