jtlimo / unnatended-test Goto Github PK

É comum que serviços sejam encerrados de uma forma elegante. Recomendo a leitura deste artigo.

Para ser honesto, acho que todas essas funções não precisam estar dentro da struct do server, isso apenas está deixando seu objeto demasiadamente grande, o que vai ocupar mais recurso de memória.

Você pode facilmente ter algo assim:

/// server.go
type Server struct {
	Router *mux.Router
	Db     *database.Database
}

E fazer a inicialização do restante no main:

// main.go
func main() {
    router := mux.NewRouter()
    db := database.New()
    
    // register services
    cards.Register(router)
    
    srv := Sever{
        db: db,
        router: router,
    }
    
    // all the other cool stuffs

}

// cards/service.go
func Register(r *mux.Router) {
    r.HandleFunc("/deck", createDeck).Methods("POST").Queries("cards", "{cards}")
}

Isso aqui pode ser problemático.
Saca só... Você está falando card.NewCard, certo?

Eu lendo isso entendo que você está pegando o package card e nele chamando a função NewCard, mas não é o que está acontecendo.

Você está criando um "deck de cartas" ao invés disso, e colocando o construtor dentro da struct sem motivo algum. Ou seja, cada Card sua dentro do seu []Cards tem um NewCard, que é usado para gerar um deck de cartas.

Minha recomendação é que você tire esse NewCard de da struct Card e deixa uma função normal...

func NewCards(cardCodes []string) ([]Card, error) {
    // continua igual
}