From 02a5f418008ef11509114ae32f4a765e27067c27 Mon Sep 17 00:00:00 2001
From: efim <efim.nefedov@nordigy.ru>
Date: Sun, 2 Jul 2023 05:26:23 +0000
Subject: [PATCH] refactor: make choices object, move out style

this should allow to write 'isBeating' function over choices.
and maybe move to enums
---
 .../src/main/resources/templates/index.html   |  6 +-
 .../main/scala/rockpaperscissors/Models.scala | 85 +++++++++++++------
 2 files changed, 61 insertions(+), 30 deletions(-)

diff --git a/14-rock-paper-scissors/src/main/resources/templates/index.html b/14-rock-paper-scissors/src/main/resources/templates/index.html
index aefb2fc..6701159 100644
--- a/14-rock-paper-scissors/src/main/resources/templates/index.html
+++ b/14-rock-paper-scissors/src/main/resources/templates/index.html
@@ -86,9 +86,9 @@
               --translation: 50%;
               position: absolute;
             "
-            th:style="| --diameter: ${choiceBadgeData.c.diameter};
-                         --bg-dark: ${choiceBadgeData.c.bgDark};
-                         --bg-bright: ${choiceBadgeData.c.bgBright};
+            th:style="| --diameter: ${choiceBadgeData.s.diameter};
+                         --bg-dark: ${choiceBadgeData.s.bgDark};
+                         --bg-bright: ${choiceBadgeData.s.bgBright};
                          ${choiceBadgeData.p.toStyle}; |"
           >
             <div
diff --git a/14-rock-paper-scissors/src/main/scala/rockpaperscissors/Models.scala b/14-rock-paper-scissors/src/main/scala/rockpaperscissors/Models.scala
index ebef0bd..a574288 100644
--- a/14-rock-paper-scissors/src/main/scala/rockpaperscissors/Models.scala
+++ b/14-rock-paper-scissors/src/main/scala/rockpaperscissors/Models.scala
@@ -23,14 +23,13 @@ position: absolute;
     }
   }
 
-  sealed trait Choice {
-    def diameter: String
-    def bgDark: String
-    def bgBright: String
-    def name: String
-    def iconPath: String
-  }
-  object Choice {
+  // yeah, maybe better to do in outer html div
+  final case class Styling(
+      var diameter: String,
+      var bgDark: String,
+      var bgBright: String
+  )
+  object Styling {
     val scissorsDark = "hsl(39, 89%, 49%)"
     val scissorsBright = "hsl(40, 84%, 53%)"
     val paperDark = "hsl(230, 89%, 62%)"
@@ -38,54 +37,86 @@ position: absolute;
     val rockDark = "hsl(349, 71%, 52%)"
     val rockBright = "hsl(349, 70%, 56%)"
 
-    case class Paper(
+    def paperStyling(
         diameter: String,
         bgDark: String = paperDark,
         bgBright: String = paperBright
-    ) extends Choice {
-      def name: String = "paper"
-      def iconPath: String = "public/images/icon-paper.svg"
-    }
-    case class Scissors(
+    ) = Styling(diameter, bgDark, bgBright)
+
+    def scissorsStyling(
         diameter: String,
         bgDark: String = scissorsDark,
         bgBright: String = scissorsBright
-    ) extends Choice {
-      def name: String = "scissors"
-      def iconPath: String = "public/images/icon-scissors.svg"
-    }
-    case class Rock(
+    ) = Styling(diameter, bgDark, bgBright)
+
+    def rockStyling(
         diameter: String,
         bgDark: String = rockDark,
         bgBright: String = rockBright
-    ) extends Choice {
+    ) = Styling(diameter, bgDark, bgBright)
+  }
+
+  sealed trait Choice {
+    def name: String
+    def iconPath: String
+  }
+  object Choice {
+    case object Paper extends Choice {
+      def name: String = "paper"
+      def iconPath: String = "public/images/icon-paper.svg"
+    }
+    case object Scissors extends Choice {
+      def name: String = "scissors"
+      def iconPath: String = "public/images/icon-scissors.svg"
+    }
+    case object Rock extends Choice {
       def name: String = "rock"
       def iconPath: String = "public/images/icon-rock.svg"
     }
+
   }
 
   /** this will be Data Transfer Object, because Thymeleaf wants var and i want
     * vals and enums
     */
   final case class ChoiceBadge(
+      var s: Styling,
       var c: Choice,
       var p: Positioning
   )
 
   val choiceSelectionItems = {
     List(
-      ChoiceBadge(Choice.Paper("8rem"), Positioning.Absolute("6rem", "6rem")),
       ChoiceBadge(
-        Choice.Scissors("8rem"),
+        Styling.paperStyling("8rem"),
+        Choice.Paper,
+        Positioning.Absolute("6rem", "6rem")
+      ),
+      ChoiceBadge(
+        Styling.scissorsStyling("8rem"),
+        Choice.Scissors,
         Positioning.Absolute("6rem", "17rem")
       ),
-      ChoiceBadge(Choice.Rock("8rem"), Positioning.Absolute("15rem", "11.5rem"))
+      ChoiceBadge(
+        Styling.rockStyling("8rem"),
+        Choice.Rock,
+        Positioning.Absolute("15rem", "11.5rem")
+      )
     )
   }
 
   final case class ShowdownState(
-    var playersChoice: ChoiceBadge,
-    var houseChoice: Option[ChoiceBadge],
-    var gameEnded: Boolean
-  )
+      var playersChoice: ChoiceBadge,
+      var houseChoice: Option[ChoiceBadge],
+      var gameEnded: Boolean
+  ) {
+    def isPlayerVictorious(): Option[Boolean] = {
+      houseChoice.map(houseSelectedChoice => {
+        val player = playersChoice.c
+        val house = houseSelectedChoice.c
+        false
+      })
+
+    }
+  }
 }