olegmak02 / coursework Goto Github PK

• Use constants, for example here: outputMessage(str, x = 410, y = 800)
• This is a good place for ternary:

    let ind;
    if (this.direction) {
      ind = this.coordsX.indexOf(x);
    } else {
      ind = this.coordsY.indexOf(y);
    }

• Better use collection of functions instead of switch/case
• Use constants: if (clickX > 390 && clickX < 830 && clickY > 520 && clickY < 660)
• Decompose long expressions (see above)
• Let's rewrite this:

function mousemove(e) {
  if (e.clientY < 465 && e.clientX < 465 &&
  e.clientY >= 115 && e.clientX >= 115) {
    onField = true;
  } else {
    onField = false;
  }
}

Here is better implementation:

const MIN = 115;
const MAX = 465;

const between = (val, min, max) => val >= min && val < max;
const inArea = val => between(val, MIN, MAX);

const mousemove = ({ clientY, clientX }) => inArea(clientY) && inArea(clientX);

• Добавить eslint и конфигурацию, лучше взять эту: .eslintrc.json
• В файле README.md нужно 1-2 страницы описания проекта и кода
• Добавить лицензию, например MIT
• Добавить package.json с зависимостями, тем же eslint. Пример: package.json