From ecf0483fb19e7ef6ec09feed49d4c57a51ea71c6 Mon Sep 17 00:00:00 2001
From: Susanne <mahler-susanne@web.de>
Date: Sun, 16 Feb 2025 17:34:03 +0100
Subject: [PATCH] Implementierung nukeAll() + kleinere random Refactorings

---
 .../GameOfLifeAssignment/Life.class           | Bin 2097 -> 2239 bytes
 src/Life.java                                 |  18 +++++++++---------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/out/production/GameOfLifeAssignment/Life.class b/out/production/GameOfLifeAssignment/Life.class
index 8a8ecef311a5da93e0e0d6a5639bfb022c5090a1..f9fdfd8016cba1e77a1387ca5b6b0af57ba098a4 100644
GIT binary patch
delta 671
zcmY*XO=}ZT6g_X=d?aJElhokUByFR}q)C(5REi%>BwEFOr1f)A7osv%BZAUmaTf~t
z1M)ynkg5pXiGgB4q3-<!ZglTb+?A+3GjStt=Du_9J@4In53l2Vj@w`UeB1_b7FRlU
zO(<Vv-J>y#p(0>4SLzR%t4jh#ZhCs;x>^$%QBmvSt<!faNX#!a@7}xp@bU7Bs%yUz
zvyc$HE1BOyJn#69ucl<S-+_buh@-|-tVd8!<hRx{(j{XhIho3D!rgv@BeI$fV5&kV
z5ukJ>C9k$MvGMmuThM;(@PtH__8?5!LF~a0Tnr=2-J@T`;jq>)d3(v4!Z~PM1)L`e
z8Z2?#GcLge{tommG|`1Eb}+C5L#^p$!$TA~)n|;T@A{}QO0O7rlESD{Mox?=-xzNt
zSi;~P%by??w=g*OuYY!bQ`R!Q0q8XHne+z~CiHMGKZl#tjdetmy7>yRLXh+}knLq(
zAn|`Z8WP6alJma{wRl;0&#JN&>QJGozyu;Vj((h=5+{+vBy&|xT*Eb-#tqcrGk2ey
zM`08=#?}Mg3{}yQBri}(!p5CYDJ>W*P>OqmCufO7C|Mej->@yDL?~Mqo`@Wva=njI
z7g18L%>_|bKh4FVIbLfrHh>0^g?(&lp2*;8U_mu4Ph3<_twVfVo7Spzi3K%W-US&l
Bdc6Pu

delta 494
zcmYjOO-mb56g_vQZ)TjPk{Gp-CMH%TI-`ji?5c?vKky^g4=6MXU5Ja)g(!$yH+ENC
z_*S~Hl!DNefl@&s?)?v4^%uAjM6G8s8+pUy+;i@nckX*vv0Bu<e!Qpyn7~TSEs5Ti
zR%<ANVT|Z$>DFJRLvs2=e(I0LO>`C}gw1j995K&v^e3|`-i#<(5JQPVB$V|x^Y4Jq
zI%t*}Hv>iPR)feqYZV?`1`btT^`Hs8`at#TRW+dcS$RQa4kD*7R3BsfQ-w8?5nj-0
z_YzV)M`y(>CcWQC)PG>ZETsaN9AUBSrILT(A0g8KdFuqPLZrSU;s;{(6un{BG1@~-
zZ&``GU;Bof{y*~1c#PS|ox@Aw0vwDYj(5o5J^3Q0IY|{`SjRX%li#Gi(_jIQG25t4
zVu~&<zL1~dJurGWGu<{Z$LR$#WM9L1fNMkA<Q}eMoj2+Z!N-4s*T~z8;_E%Tnx3bR
mg+X+nLS&<vDHezn*8)E1TRSU@I_<pWm6e@cdx`D_mY)Ex_fxR|

diff --git a/src/Life.java b/src/Life.java
index 68a4d7f..dada11d 100644
--- a/src/Life.java
+++ b/src/Life.java
@@ -27,8 +27,11 @@ public class Life implements ILife {
 
   @Override
   public void nukeAll() {
-    // TODO Auto-generated method stub
-
+    for (int y = 0; y < grid.length; y++) {
+      for (int x = 0; x < grid[y].length; x++) {
+        grid[x][y] = false;
+      }
+    }
   }
 
   @Override
@@ -52,17 +55,14 @@ public class Life implements ILife {
 
     for (int y = 0; y < grid.length; y++) {
       for (int x = 0; x < grid[y].length; x++) {
+        int aliveNeighbours = countAliveNeighbours(x, y);  // Nur einmal berechnen
 
-        // Dieser Schritt wäre nun noch ein mögliches Refactoring, um die Fallunterscheidungen zwischen keepAlive und
-        // setAlive nochmal deutlicher zu machen, ändert aber eigentlich (soweit ich es sehe) nichts am Codeverhalten.
-        // mMn Ansichtssache, ob das nun redundanter Code ist oder tatsächlich das Verständnis bedeutend verbessert.
-
-        if (isAlive(x, y) && (countAliveNeighbours(x, y) == 3 || countAliveNeighbours(x, y) == 2)) {
+        if (isAlive(x, y) && (aliveNeighbours == 2 || aliveNeighbours == 3)) {
           next.setAlive(x, y);
-        } else if (!isAlive(x, y) && countAliveNeighbours(x, y) == 3) {
+        } else if (!isAlive(x, y) && aliveNeighbours == 3) {
           next.setAlive(x, y);
         } else {
-          next.setDead(x, y);  // das hier ist mMn im Grunde ja auch redundant, da eben die Zellen standardmäßig tot initialisiert werden
+          next.setDead(x, y);
         }
       }
     }