tehty korjauksia

NumeronArvaus.cs

@@ -6,13 +6,23 @@ using Jypeli;
 namespace NumeronArvaus;
 
 /// @author alper
-/// @version 20.02.2025
+/// @version 03.04.2025
 /// <summary>
 /// Numeron arvaus peli, jossa pelaaja koittaa arvata numeron väliltä 1-100
 /// </summary>
 
 public class NumeronArvaus : Game
 {
+    /// <summary>
+    /// Pelin minimiarvo satunnaislukua varten.
+    /// </summary>
+    private const int MinArvo = 1;
+    
+    /// <summary>
+    /// Pelin maksimiarvo satunnaislukua varten.
+    /// </summary>
+    private const int MaxArvo = 100;
+
     /// <summary>
     /// Arvottu numero, joka on piilotettu käyttäjältä.
     /// </summary>
@@ -54,9 +64,13 @@ public class NumeronArvaus : Game
     public override void Begin()
     {
         // Arvotaan satunnainen numero väliltä 1-100
-        _arvottuNumero = RandomGen.NextInt(1, 101);
+        _arvottuNumero = RandomGen.NextInt(MinArvo, MaxArvo + 1);
+        
+        //laskuri joka laskee pelaajan yritykset
         _yritykset = new IntMeter(0);
-        _arvausHistoria.Clear();
+        
+        // tyhjentää arvasuhistorian pelin alkaessa
+       _arvausHistoria.Clear();
 
         // Taustakuva
         Level.Background.CreateGradient(Color.DarkBlue, Color.Black);
@@ -136,11 +150,11 @@ public class NumeronArvaus : Game
     /// <summary>
     /// Tarkistaa käyttäjän arvauksen ja näyttää vihjeitä.
     /// </summary>
-    void TarkistaArvaus()
+    private void TarkistaArvaus()
     {
         if (int.TryParse(_kayttajanSyote, out int pelaajanArvaus))
         {
-            if (pelaajanArvaus < 1 || pelaajanArvaus > 100)
+            if (pelaajanArvaus < MinArvo || pelaajanArvaus > MaxArvo)
             {
                 _vihjeteksti.Text = "Syötä luku väliltä 1-100!";
                 return;
@@ -181,7 +195,7 @@ public class NumeronArvaus : Game
     /// </summary>
     /// <param name="uusiArvaus">Käyttäjän antama uusi arvaus.</param>
     /// <returns>Arvaushistorian tekstimuodossa.</returns>
-    string PaivitaArvausHistoria(int uusiArvaus)
+    private string PaivitaArvausHistoria(int uusiArvaus)
     {
         _arvausHistoria.Add(uusiArvaus);
         string historiaTeksti = "Arvaukset: ";
@@ -196,7 +210,7 @@ public class NumeronArvaus : Game
     /// <summary>
     /// Aloittaa pelin alusta.
     /// </summary>
-    void ResetoiPeli()
+   private void ResetoiPeli()
     {
         ClearAll();
         Begin();
@@ -207,7 +221,7 @@ public class NumeronArvaus : Game
     /// Lisää painettu numero syötteeseen.
     /// </summary>
     /// <param name="nappi">Painettu näppäin.</param>
-    void LisaaMerkki(Key nappi)
+    private void LisaaMerkki(Key nappi)
     {
         if (_kayttajanSyote.Length < 3)
         {
@@ -221,7 +235,7 @@ public class NumeronArvaus : Game
     /// <summary>
     /// Poistaa viimeisen numeron syötteestä.
     /// </summary>
-    void PoistaMerkki()
+    private void PoistaMerkki()
     {
         if (_kayttajanSyote.Length > 0)
         {