Skip to content

Commit

Permalink
refactor: crud handlers
Browse files Browse the repository at this point in the history
  • Loading branch information
ChristophBe committed Jun 3, 2021
1 parent 2cf8722 commit 451ed88
Show file tree
Hide file tree
Showing 8 changed files with 36 additions and 37 deletions.
5 changes: 0 additions & 5 deletions handlers/create.go
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,3 @@ func NewCreatHandler(service types.CreateService, responseWriter types.ResponseW
}
}
}

// Create is a http.Handler that handles the creation of a model
func (c crudHandlersImpl) Create(writer http.ResponseWriter, request *http.Request) {
NewCreatHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}
42 changes: 36 additions & 6 deletions handlers/crud-handlers.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ type CrudHandlers interface {
Delete(w http.ResponseWriter, r *http.Request)
}

type crudHandlersImpl struct {
service types.Service
responseWriter types.ResponseWriter
errorWriter types.ErrorResponseWriter
}

// NewCrudHandlers creates a instance of CrudHandlers.
func NewCrudHandlers(service types.Service, responseWriter types.ResponseWriter, errorWriter types.ErrorResponseWriter) CrudHandlers {
return crudHandlersImpl{
Expand All @@ -29,3 +23,39 @@ func NewCrudHandlers(service types.Service, responseWriter types.ResponseWriter,
errorWriter: errorWriter,
}
}

type crudHandlersImpl struct {
service types.Service
responseWriter types.ResponseWriter
errorWriter types.ErrorResponseWriter
}

// Create is a http.Handler that handles the creation of a model
func (c crudHandlersImpl) Create(writer http.ResponseWriter, request *http.Request) {
NewCreatHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}

// GetAll is a http.Handler for fetch a list of model.
func (c crudHandlersImpl) GetAll(writer http.ResponseWriter, request *http.Request) {
NewGetAllHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}

// GetOne returns a http handler for handling requests one specific model.
func (c crudHandlersImpl) GetOne(w http.ResponseWriter, r *http.Request) {
NewGetOneHandler(c.service, c.responseWriter, c.errorWriter)(w, r)
}

// Update is a http.Handler that handles partial updates for existing models.
func (c crudHandlersImpl) Update(writer http.ResponseWriter, request *http.Request) {
NewUpdateHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}

// Replace is a http.Handler that handles replacing an exing model.
func (c crudHandlersImpl) Replace(writer http.ResponseWriter, request *http.Request) {
NewReplaceHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}

// Delete is a http handler for handling the deletion of specific model.
func (c crudHandlersImpl) Delete(writer http.ResponseWriter, request *http.Request) {
NewDeleteHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}
5 changes: 0 additions & 5 deletions handlers/delete.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,3 @@ func NewDeleteHandler(service types.GetOneService, responseWriter types.Response
}
}
}

// Delete is a http handler for handling the deletion of specific model.
func (c crudHandlersImpl) Delete(writer http.ResponseWriter, request *http.Request) {
NewDeleteHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}
5 changes: 0 additions & 5 deletions handlers/get-all.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,3 @@ func NewGetAllHandler(service types.GetAllService, responseWriter types.Response
}
}
}

// GetAll is a http.Handler for fetch a list of model.
func (c crudHandlersImpl) GetAll(writer http.ResponseWriter, request *http.Request) {
NewGetAllHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}
5 changes: 0 additions & 5 deletions handlers/get-one.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,3 @@ func NewGetOneHandler(service types.GetOneService, responseWriter types.Response
}
}
}

// GetOne returns a http handler for handling requests one specific model.
func (c crudHandlersImpl) GetOne(w http.ResponseWriter, r *http.Request) {
NewGetOneHandler(c.service, c.responseWriter, c.errorWriter)(w, r)
}
5 changes: 0 additions & 5 deletions handlers/replace.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,3 @@ func NewReplaceHandler(service types.ReplaceService, responseWriter types.Respon
}
}
}

// Replace is a http.Handler that handles replacing an exing model.
func (c crudHandlersImpl) Replace(writer http.ResponseWriter, request *http.Request) {
NewReplaceHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}
1 change: 0 additions & 1 deletion handlers/test_helpers_test.go

This file was deleted.

5 changes: 0 additions & 5 deletions handlers/update.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,3 @@ func NewUpdateHandler(service types.UpdateService, responseWriter types.Response
}
}
}

// Update is a http.Handler that handles partial updates for existing models.
func (c crudHandlersImpl) Update(writer http.ResponseWriter, request *http.Request) {
NewUpdateHandler(c.service, c.responseWriter, c.errorWriter).ServeHTTP(writer, request)
}

0 comments on commit 451ed88

Please sign in to comment.