Code cleanup

Author: adamgraham

Assets/Scenes/Flappy Bird.unity

@@ -270,6 +270,7 @@ MonoBehaviour:
   m_Script: {fileID: 11500000, guid: ae79f9d99372a6b438f800230f434b9d, type: 3}
   m_Name: 
   m_EditorClassIdentifier: 
+  player: {fileID: 527830928}
   scoreText: {fileID: 1474871343}
   playButton: {fileID: 892178073}
   gameOver: {fileID: 298509898}

Assets/Scripts/GameManager.cs

@@ -3,22 +3,29 @@
 
 public class GameManager : MonoBehaviour
 {
-    private Player player;
-    private Spawner spawner;
+    public static GameManager Instance { get; private set; }
 
-    public Text scoreText;
-    public GameObject playButton;
-    public GameObject gameOver;
-    public int score { get; private set; }
+    [SerializeField] private Player player;
+    [SerializeField] private Text scoreText;
+    [SerializeField] private GameObject playButton;
+    [SerializeField] private GameObject gameOver;
+
+    private int score;
+    public int Score => score;
 
     private void Awake()
     {
-        Application.targetFrameRate = 60;
-
-        player = FindObjectOfType<Player>();
-        spawner = FindObjectOfType<Spawner>();
-
-        Pause();
+        if (Instance != null)
+        {
+            DestroyImmediate(gameObject);
+        }
+        else
+        {
+            Instance = this;
+            Application.targetFrameRate = 60;
+            DontDestroyOnLoad(gameObject);
+            Pause();
+        }
     }
 
     public void Play()

Assets/Scripts/Parallax.cs

@@ -2,8 +2,8 @@
 
 public class Parallax : MonoBehaviour
 {
-    private MeshRenderer meshRenderer;
     public float animationSpeed = 1f;
+    private MeshRenderer meshRenderer;
 
     private void Awake()
     {

Assets/Scripts/Pipes.cs

@@ -4,8 +4,8 @@ public class Pipes : MonoBehaviour
 {
     public Transform top;
     public Transform bottom;
-
     public float speed = 5f;
+
     private float leftEdge;
 
     private void Start()
@@ -15,7 +15,7 @@ private void Start()
 
     private void Update()
     {
-        transform.position += Vector3.left * speed * Time.deltaTime;
+        transform.position += speed * Time.deltaTime * Vector3.left;
 
         if (transform.position.x < leftEdge) {
             Destroy(gameObject);

Assets/Scripts/Player.cs

@@ -2,15 +2,14 @@
 
 public class Player : MonoBehaviour
 {
-    private SpriteRenderer spriteRenderer;
     public Sprite[] sprites;
-    private int spriteIndex;
-
     public float strength = 5f;
     public float gravity = -9.81f;
     public float tilt = 5f;
 
+    private SpriteRenderer spriteRenderer;
     private Vector3 direction;
+    private int spriteIndex;
 
     private void Awake()
     {
@@ -62,9 +61,9 @@ private void AnimateSprite()
     private void OnTriggerEnter2D(Collider2D other)
     {
         if (other.gameObject.CompareTag("Obstacle")) {
-            FindObjectOfType<GameManager>().GameOver();
+            GameManager.Instance.GameOver();
         } else if (other.gameObject.CompareTag("Scoring")) {
-            FindObjectOfType<GameManager>().IncreaseScore();
+            GameManager.Instance.IncreaseScore();
         }
     }