minor fix & update readme

README.md

@@ -100,43 +100,44 @@ Every implementation should only be by using interface, there should be no direc
 
  - PlayerService -> implement IPlayerRepository, instead of direct PlayerRepository
 
-	    type PlayerService struct {
-		    PlayerRepository interfaces.IPlayerRepository
-	    }
 
-	    func (service *PlayerService) GetScores(player1Name string, player2Name string) (string, error) {
+    type PlayerService struct {
+      PlayerRepository interfaces.IPlayerRepository
+    }
 
-		    baseScore := [4]string{"Love", "Fifteen", "Thirty", "Forty"}
-		    var result string
+    func (service *PlayerService) GetScores(player1Name string, player2Name string) (string, error) {
 
-		    player1, err := service.PlayerRepository.GetPlayerByName(player1Name)
-		    if err != nil {
-		      //Handle error
-		    }
+      baseScore := [4]string{"Love", "Fifteen", "Thirty", "Forty"}
+      var result string
 
-		    player2, err := service.PlayerRepository.GetPlayerByName(player2Name)
-		    if err != nil {
-		      //Handle error
-		    }
+      player1, err := service.PlayerRepository.GetPlayerByName(player1Name)
+      if err != nil {
+        //Handle error
+      }
 
-		    if player1.Score < 4 && player2.Score < 4 && !(player1.Score+player2.Score == 6) {
+      player2, err := service.PlayerRepository.GetPlayerByName(player2Name)
+      if err != nil {
+        //Handle error
+      }
 
-		        s := baseScore[player1.Score]
+      if player1.Score < 4 && player2.Score < 4 && !(player1.Score+player2.Score == 6) {
 
-		        if player1.Score == player2.Score {
-		          result = s + "-All"
-		        } else {
-		          result = s + "-" + baseScore[player2.Score]
-		        }
-			}
+        s := baseScore[player1.Score]
 
-		    if player1.Score == player2.Score {
-		      result = "Deuce"
-		    }
+        if player1.Score == player2.Score {
+          result = s + "-All"
+        } else {
+          result = s + "-" + baseScore[player2.Score]
+        }
+      }
 
-	      return result, nil
-	    }
+      if player1.Score == player2.Score {
+        result = "Deuce"
+      }
 
+      return result, nil
+    }
+
 If you look into the implementation of these lines
 
     scores, err := controller.PlayerService.GetScores(player1Name, player2Name)

infrastructures/SQLiteHandler.go

@@ -14,7 +14,7 @@ func (handler *SQLiteHandler) Execute(statement string) {
 	handler.Conn.Exec(statement)
 }
 
-func (handler *SQLiteHandler) Query(statement string) (interfaces.Row, error) {
+func (handler *SQLiteHandler) Query(statement string) (interfaces.IRow, error) {
 	//fmt.Println(statement)
 	rows, err := handler.Conn.Query(statement)
 

interfaces/IDbHandler.go

@@ -1,11 +1,11 @@
 package interfaces
 
-type DbHandler interface {
+type IDbHandler interface {
 	Execute(statement string)
-	Query(statement string) (Row, error)
+	Query(statement string) (IRow, error)
 }
 
-type Row interface {
+type IRow interface {
 	Scan(dest ...interface{}) error
 	Next() bool
 }

repositories/PlayerRepository.go

@@ -2,7 +2,6 @@ package repositories
 
 import (
 	"github.com/afex/hystrix-go/hystrix"
-	"github.com/irahardianto/service-pattern-go/infrastructures"
 	"github.com/irahardianto/service-pattern-go/interfaces"
 	"github.com/irahardianto/service-pattern-go/models"
 
@@ -36,7 +35,7 @@ func (repository *PlayerRepositoryWithCircuitBreaker) GetPlayerByName(name strin
 }
 
 type PlayerRepository struct {
-	infrastructures.SQLiteHandler
+	interfaces.IDbHandler
 }
 
 func (repository *PlayerRepository) GetPlayerByName(name string) (models.PlayerModel, error) {

servicecontainer.go

@@ -22,7 +22,7 @@ func (k *kernel) InjectPlayerController() controllers.PlayerController {
 	sqliteHandler := &infrastructures.SQLiteHandler{}
 	sqliteHandler.Conn = sqlConn
 
-	playerRepository := &repositories.PlayerRepository{*sqliteHandler}
+	playerRepository := &repositories.PlayerRepository{sqliteHandler}
 
 	playerService := &services.PlayerService{}
 	playerService.PlayerRepository = &repositories.PlayerRepositoryWithCircuitBreaker{playerRepository}