Simplify SetActiveRenderer logic

adamgraham · adamgraham · commit 4a7b14668098 · 2022-05-07T01:10:19.000-05:00
diff --git a/Assets/Scripts/Explosion.cs b/Assets/Scripts/Explosion.cs
@@ -8,11 +8,9 @@ public class Explosion : MonoBehaviour
 
     public void SetActiveRenderer(AnimatedSpriteRenderer renderer)
     {
-        start.enabled = false;
-        middle.enabled = false;
-        end.enabled = false;
-
-        renderer.enabled = true;
+        start.enabled = renderer == start;
+        middle.enabled = renderer == middle;
+        end.enabled = renderer == end;
     }
 
     public void SetDirection(Vector2 direction)

Original file line number	Diff line number	Diff line change
`@@ -8,11 +8,9 @@ public class Explosion : MonoBehaviour`
`8`	`8`
`9`	`9`	`public void SetActiveRenderer(AnimatedSpriteRenderer renderer)`
`10`	`10`	`{`
`11`		`- start.enabled = false;`
`12`		`- middle.enabled = false;`
`13`		`- end.enabled = false;`
`14`		`-`
`15`		`- renderer.enabled = true;`
	`11`	`+ start.enabled = renderer == start;`
	`12`	`+ middle.enabled = renderer == middle;`
	`13`	`+ end.enabled = renderer == end;`
`16`	`14`	`}`
`17`	`15`
`18`	`16`	`public void SetDirection(Vector2 direction)`