client/hud: Added a flag to know if we already checked the path viabi…

…lity

So we do not check it every refresh tick when we know it does not change

action/action.go

@@ -25,6 +25,7 @@ type Action struct {
 	NavigateTo           *NavigateToPayload           `json:"navigate_to, omitempty"`
 	StartGame            *StartGamePayload            `json:"start_game, omitempty"`
 	GoHome               *GoHomePayload               `json:"go_home, omitempty"`
+	CheckedPath          *CheckedPathPayload          `json:"checked_path,omitempty"`
 
 	OpenTowerMenu  *OpenTowerMenuPayload  `json:"open_tower_menu, omitempty"`
 	CloseTowerMenu *CloseTowerMenuPayload `json:"close_tower_menu, omitempty"`
@@ -356,6 +357,19 @@ func NewGoHome() *Action {
 	}
 }
 
+type CheckedPathPayload struct {
+	Checked bool
+}
+
+func NewCheckedPath(cp bool) *Action {
+	return &Action{
+		Type: CheckedPath,
+		CheckedPath: &CheckedPathPayload{
+			Checked: cp,
+		},
+	}
+}
+
 type UpdateStatePayload struct {
 	Players *UpdateStatePlayersPayload
 	Towers  *UpdateStateTowersPayload

action/type.go

@@ -27,6 +27,7 @@ const (
 	OpenTowerMenu
 	CloseTowerMenu
 	GoHome
+	CheckedPath
 
 	// Specific to WS
 	JoinRoom

client/action.go

@@ -170,3 +170,9 @@ func (ac *ActionDispatcher) GoHome() {
 	gha := action.NewGoHome()
 	ac.dispatcher.Dispatch(gha)
 }
+
+// CheckedPath will set the value of the path checked
+func (ac *ActionDispatcher) CheckedPath(cp bool) {
+	cpa := action.NewCheckedPath(cp)
+	ac.dispatcher.Dispatch(cpa)
+}

client/hud.go

@@ -41,6 +41,7 @@ type HUDState struct {
 	TowerOpenMenuID string
 
 	LastCursorPosition utils.Object
+	CheckedPath        bool
 }
 
 type SelectedTower struct {
@@ -169,7 +170,6 @@ func (hs *HUDStore) Update() error {
 			return nil
 		}
 		for _, t := range tws {
-
 			if clickAbsolute.IsColliding(t.Object) && cp.ID == t.PlayerID {
 				if hst.TowerOpenMenuID != "" {
 					// When the user clicks 2 times on the same tower we remove it
@@ -236,7 +236,7 @@ func (hs *HUDStore) Update() error {
 			// Only check if the line is blocked when is still valid position and it has not moved.
 			// TODO: We can improve this by storing this result (if blocking or not) so we only validate
 			// this once and not when the mouse is static with a selected tower
-			if !invalid && (hst.LastCursorPosition.X == float64(x) && hst.LastCursorPosition.Y == float64(y)) {
+			if !invalid && (hst.LastCursorPosition.X == float64(x) && hst.LastCursorPosition.Y == float64(y) && !hst.CheckedPath) {
 				var fakex, fakey float64 = hs.game.Store.Map.GetRandomSpawnCoordinatesForLineID(cp.LineID)
 				utws = append(utws, utils.Object{
 					X: hst.SelectedTower.X + cs.X,
@@ -253,6 +253,7 @@ func (hs *HUDStore) Update() error {
 				if len(steps) == 0 {
 					invalid = true
 				}
+				actionDispatcher.CheckedPath(true)
 			}
 			if invalid != hst.SelectedTower.Invalid {
 				actionDispatcher.SelectedTowerInvalid(invalid)
@@ -399,6 +400,9 @@ func (hs *HUDStore) Reduce(state, a interface{}) interface{} {
 				hstate.SelectedTower.Y = float64(closestMultiple(act.CursorMove.Y, multiple)) - (hstate.SoldierButton.H / 2)
 			}
 		}
+		// If it has moved we set the CheckedPath as not checked as it's only checked
+		// when the Cursor has not moved
+		hstate.CheckedPath = false
 	case action.PlaceTower, action.DeselectTower:
 		hstate.SelectedTower = nil
 	case action.SelectedTowerInvalid:
@@ -409,6 +413,8 @@ func (hs *HUDStore) Reduce(state, a interface{}) interface{} {
 		hstate.TowerOpenMenuID = act.OpenTowerMenu.TowerID
 	case action.CloseTowerMenu:
 		hstate.TowerOpenMenuID = ""
+	case action.CheckedPath:
+		hstate.CheckedPath = act.CheckedPath.Checked
 	default:
 	}
 

client/router.go

@@ -29,8 +29,8 @@ func NewRouterStore(d *flux.Dispatcher, g *Game, l *Lobby) *RouterStore {
 	}
 
 	rs.ReduceStore = flux.NewReduceStore(d, rs.Reduce, RouterState{
-		//Route: LobbyRoute,
-		Route: GameRoute,
+		Route: LobbyRoute,
+		//Route: GameRoute,
 	})
 
 	return rs