Как изящно обрабатывать ошибки в веб-сервисе

Вы можете сделать его немного более СУХИМ с помощью помощника:

func handleError(c *gin.Context, err error) bool {
    if err != nil {
        c.JSON(400, gin.H{"error": err.Error()})
        return true
    }
    return false
}

Используется в качестве:

err = tx.Commit()
if handleError(c,err) {
    return
}

Это только сокращает обработку ошибок количество строк с 4 строк до 3, но при этом абстрагируется от повторяющейся логики, позволяя вам изменить обработку повторяющихся ошибок в одном месте вместо везде, где обрабатывается ошибка (например, если вы хотите добавить журнал ошибок или изменить ответ об ошибке и т. д.)

Мой обычный подход - использовать функцию обертывания. У этого есть преимущество (по сравнению с ответом Адриана - который также является хорошим, кстати) в том, что обработка ошибок остается в более идиоматической форме (return result, err, в отличие от засорения вашего кода вызовами типа handleError(err)), при этом все еще консолидируя это в одно место.

func DeleteAPI(c *gin.Context) {
    num, err := deleteAPI(c)
    if err != nil {
        c.JSON(400, gin.H{"error": err.Error()})
        return
    }
    c.JSON(200, gin.H{"deleted": num})
}

func deleteAPI(c *gin.Context) (int, error) {
    var db = c.MustGet("db").(*sql.DB)
    query := "DELETE FROM table WHERE some condition"
    tx, err := db.Begin()
    if err != nil {
        return 0, err
    }
    defer tx.Rollback()
    result, err := tx.Exec(query)
    if err != nil {
        return 0, err
    }
    num, err := result.RowsAffected()
    if err != nil {
        return 0, err
    }
    err = tx.Commit()
    if err != nil {
        return 0, err
    }
    return num, nil
}

Для меня (и вообще для кодеров Go) приоритетом является читаемость кода над DRY. И из трех вариантов (ваш оригинал, Адриан и мой), на мой взгляд, моя версия более читабельна по той простой причине, что ошибки обрабатываются совершенно идиоматическим способом, и они переходят в верхний обработчик. Этот же подход работает одинаково хорошо, если ваш контроллер в конечном итоге вызывает другие функции, возвращающие ошибки. Перенося всю обработку ошибок в самую верхнюю функцию, вы избавляетесь от беспорядка, связанного с обработкой ошибок (кроме простой конструкции if err! = Nil {return err} `) во всем остальном коде.

Также стоит отметить, что этот подход можно эффективно комбинировать с подходом Адриана, особенно для использования с несколькими обработчиками, путем изменения функции "обертывания" следующим образом:

func DeleteAPI(c *gin.Context) {
    result, err := deleteAPI(c)
    if handleError(c, err) {
        return
    }
    c.JSON(200, gin.H{"deleted": num})
}