Desafio Tiago

[arxdsilva]

https://github.com/tcarreira/superhero

[arxdsilva]

documentacao: ok docker: ok envs:

• DB teria problema em nuvem caso DB_... nao existissem, por padrao (cloud) eh DATABASE_URL, o comportamento correto seria falhar inicializacao caso n existissem.

arquivos:

• poderiam ser melhor organizados, principalmente por dominios (ex: o setRoutes no arquivo da api, mas faria mais sentido ser um arquivo separado).

logica:

• panic's desnecessarios
• Nested Ifs (ex abaixo) geram alta complexidade

func (api *GroupAPI) GroupsGETHandler(c *gin.Context) {
    var group *models.Group
    var err error
    group, err = group.GetByName(api.DB, c.Param("name"))
    if err != nil {
        if _, ok := err.(*models.ErrorGroupNotFound); ok {
            c.JSON(http.StatusNotFound, errorResponseJSON{
                "Group not found", err.Error(),
            })
        } else {
            panic(err)
        }
    } else {
        c.JSON(http.StatusOK, group)
    }
}