ЗАПИСЬ В БЛОГЕ: https://javascriptlearner815.github.io/blog/2021/02/21/simplify-your-js-code.html

Плохая практика

Изменение прототипов встроенных объектов

Возможно, вы уже создали функцию для прототипа объекта String, подобную этой:

String.prototype.reverse = (() => {
  // stuff like this.toString(), etc.
});

Однако это плохая практика. Вместо этого вы должны использовать это:

function reverse(string) {
  // stuff like string.toString(), etc.
}

Бесполезно иначе, если

Допустим, у вас есть такая функция:

function isZero(number) {
  if (number === 0) {
    return true;
  } else if (number !== 0) {
    return false;
  }
}

Здесь плохая практика. Вы можете заметить это?

Если бы вы сказали, что else-if можно заменить на else, вы были бы правы! Если проверка до этого не выполнялась, мы знаем, что число не равно 0, поэтому нам не нужно проверять это после того, как мы уже знаем, что число не равно 0.

Теперь наша функция будет такой:

function isZero(number) {
  if (number === 0) {
    return true;
  } else {
    return false;
  }
}