Review 1

[tshemsedinov]

• Use this in arrow function, don't need obj https://github.com/vladimirvikulin/Car-Racing/blob/c8ffffcda293a6529e45a8ba493dbb6cd82193fc/src/classes.js#L33
• Magic number https://github.com/vladimirvikulin/Car-Racing/blob/c8ffffcda293a6529e45a8ba493dbb6cd82193fc/src/classes.js#L42
• Rewrite https://github.com/vladimirvikulin/Car-Racing/blob/c8ffffcda293a6529e45a8ba493dbb6cd82193fc/src/classes.js#L47-L56
• Try not to use ternary operator for branching (it's ok just for expressions)
• Rewrite https://github.com/vladimirvikulin/Car-Racing/blob/c8ffffcda293a6529e45a8ba493dbb6cd82193fc/src/game.js#L35-L40

const CAR_SHIFT = 100;

const CAR_DATA = [['yellow', 15], ['red', 14], ['black', 12], ['purple', 11]];

const PLAYER1_CARS = CAR_DATA.map(([color, speed]) => {
  const x = canvas.width / 2 - CAR_SHIFT;
  const y = canvas.height / 2;
  const img = `./images/${color}Car.png`;
  return new Car(img, x, y, true, false, speed);
});

• Decompose: https://github.com/vladimirvikulin/Car-Racing/blob/c8ffffcda293a6529e45a8ba493dbb6cd82193fc/src/game.js#L107-L136