refactor(evaluation): fix yet another evaluation bug by refactoring

src/game/mod.rs

@@ -133,11 +133,9 @@ impl<'wl, WL: WordList> Game<'wl, WL> {
 
         #[cfg(debug_assertions)]
         let buflen = solution.len();
-        #[cfg(debug_assertions)]
+        assert_eq!(buflen, buf.len());
-        {
+        assert_eq!(buflen, evaluation.len());
-            assert_eq!(buflen, buf.len());
+        debug!("buf: {buf:?}");
-            assert_eq!(buflen, evaluation.len());
-        }
 
         // first the correct solutions
         for ((idx, c_guess), c_sol) in guess.chars().enumerate().zip(solution.chars()) {
@@ -148,11 +146,8 @@ impl<'wl, WL: WordList> Game<'wl, WL> {
             }
         }
 
-        #[cfg(debug_assertions)]
+        assert_eq!(buflen, buf.len());
-        {
+        assert_eq!(buflen, evaluation.len());
-            assert_eq!(buflen, buf.len());
-            assert_eq!(buflen, evaluation.len());
-        }
 
         // then check if the char exists, but was not guessed to be at the correct position
         //
@@ -176,11 +171,8 @@ impl<'wl, WL: WordList> Game<'wl, WL> {
             evaluation[idx] = (c_guess, status);
         }
 
-        #[cfg(debug_assertions)]
+        assert_eq!(buflen, buf.len());
-        {
+        assert_eq!(buflen, evaluation.len());
-            assert_eq!(buflen, buf.len());
-            assert_eq!(buflen, evaluation.len());
-        }
         evaluation.into()
     }