trjhtr

As a part of studying C#, I was given a naive Windows Forms game code. The task is to try and make it more MVC-like and optimize where possible.

I've already read what MVC is, but having difficulties what to start with. Here's the code itself, TODOs are just my thoughts.

I'm not asking to do my job, but I would love to get some kind of a set of instructions, any ideas are welcomed and appreciated thou. Thank you

To have an idea what a game looks like, check this image

using System;
using System.Drawing;
using System.Windows.Forms;

namespace OldSchoolRacingGame

 public partial class CarGame : Form
 
 private int carSpeed = 5;
 private int roadSpeed = 5;
 private bool carLeft;
 private bool carRight;
 private int trafficSpeed = 5;
 private int score = 0;
 Random rnd = new Random();

 public CarGame()
 
 InitializeComponent();

 Reset();
 

 private void Reset()
 

 trophy.Visible = false;
 button1.Enabled = false;
 explosion.Visible = false;
 trafficSpeed = 5;
 roadSpeed = 5;
 score = 0;

 Player.Left = 161; // placing player
 Player.Top = 286;

 carLeft = false;
 carRight = false;

 Car1.Left = 66; // placing obstacles
 Car1.Top = -120;

 Car2.Left = 294;
 Car2.Top = -185;

 roadTrack2.Left = -3; // placing background images
 roadTrack2.Top = -222;
 roadTrack1.Left = -2;
 roadTrack1.Top = -638;

 timer1.Start(); // timer starts
 

 private void timer1_Tick(object sender, EventArgs e)
 

 private void CarGame_Load(object sender, EventArgs e)
 
 //TODO: ?
 

 private void MoveCar(object sender, KeyEventArgs e)
 
 if (e.KeyCode == Keys.Left && Player.Left > 0)
 carLeft = true;

 if (e.KeyCode == Keys.Right &&
 Player.Left + Player.Width < panel1.Width)
 carRight = true;
 

 private void KeepCarInside(object sender, KeyEventArgs e)
 
 if (e.KeyCode == Keys.Left)
 carLeft = false;

 if (e.KeyCode == Keys.Right)
 carRight = false;
 

 private void SwitchCar1Image() 
 
 //TODO: try to make a separate method based on RandomNumberGenerator
 //TODO: move to GameModel.cs
 int number = rnd.Next(1, 8);

 switch (number)
 
 case 1:
 Car1.Image = Properties.Resources.carGreen;
 break;
 case 2:
 Car1.Image = Properties.Resources.carOrange;
 break;
 case 3:
 Car1.Image = Properties.Resources.carGrey;
 break;
 case 4:
 Car1.Image = Properties.Resources.CarRed;
 break;
 case 5:
 Car1.Image = Properties.Resources.carPink;
 break;
 case 6:
 Car1.Image = Properties.Resources.TruckBlue;
 break;
 case 7:
 Car1.Image = Properties.Resources.TruckWhite;
 break;
 case 8:
 Car1.Image = Properties.Resources.ambulance;
 break;
 

 

 private void SwitchCar2Image()
 
 // TODO: same as SwitchCar1Image()
 int number = rnd.Next(1, 8);

 switch (number)
 
 case 1:
 Car2.Image = Properties.Resources.carGreen;
 break;
 case 2:
 Car2.Image = Properties.Resources.carOrange;
 break;
 case 3:
 Car2.Image = Properties.Resources.carGrey;
 break;
 case 4:
 Car2.Image = Properties.Resources.CarRed;
 break;
 case 5:
 Car2.Image = Properties.Resources.carPink;
 break;
 case 6:
 Car2.Image = Properties.Resources.TruckBlue;
 break;
 case 7:
 Car2.Image = Properties.Resources.TruckWhite;
 break;
 case 8:
 Car2.Image = Properties.Resources.ambulance;
 break;
 
 

 private void GameOver()
 
 // TODO: move to GameModel.cs
 trophy.Visible = true;

 timer1.Stop();

 button1.Enabled = true;

 explosion.Visible = true;
 Player.Controls.Add(explosion);
 explosion.Location = new Point(-8, 5);
 explosion.BackColor = Color.Transparent;
 explosion.BringToFront();

 if (score < 1000)
 trophy.Image = Properties.Resources.bronze;

 if (score > 2000)
 trophy.Image = Properties.Resources.silver;

 if (score > 3500)
 trophy.Image = Properties.Resources.gold;
 

 private void button1_Click(object sender, EventArgs e)
 
 Reset();
 

 private void roadTrack2_Click(object sender, EventArgs e)