Нормальна ли такая реализация функции?

Question

[Сергей]

Данную функцию

function closeInfoWindows(infoWindow, infoWindow2) {
  if (infoWindow) {
    infoWindow.close();
  }
  if (infoWindow2) {
    infoWindow2.close();
  }
}

переписал так:

function closeInfoWindows() {
  for (var i = 0; i < arguments.length; i++) {
    var window = arguments[i];
    if (window) {
      window.close();
    }
  }
}

Нормальна ли такая реализация функции с вашей точки зрения или же можно сделать как-то иначе?

Comments

[Ярослав Иванов]

норм

Answer 1

[0xD34F]

Дичь конечно.

Не надо определять переменные с именами как у каких-то глобальных объектов, так что window - долой. Ну и в целом - как-то многословно, это могла бы быть и одна строка:

const closeInfoWindows = (...windows) => windows.forEach(w => w && w.close());

Comments

[AngReload]

Думаю, что стоит предупредить: в старых браузерах такой код просто так не заработает, его нужно будет компилировать в код по старым стандартам с помощью Babel.

[Сергей]

Для большей совместимости я пишу код со старым синтаксисом, так что одной строкой не выйдет. Насчёт правильного именования переменных приму к сведению. А почему "дичь"? Код ведь работает, какой смысл в ещё одном цикле?

[0xD34F]

Сергей,

А почему "дичь"?

Да собственно только из-за window в качестве имени переменной. На самом деле всё остальное окей.

[Сергей]

AngReload, ясно. 0xD34F, спасибо!

[Сергей Соколов]

Не нужно ещё проверять, раз уж в цикле, есть ли у объекта метод close() ?
Оборачивать в try .. catch?

[0xD34F]

Сергей Соколов, нужно конечно. Но по-хорошему, наверное надо проверять не наличие метода, а что объект действительно является окном, а не чем-то ещё... Typescript? Мне кажется, автору рановато смотреть в эту сторону. Набить немного шишек будет нелишним.

[Владимир]

A если w.close у одного аргумента не определено свалится с ошибкой?

[0xD34F]

Владимир, предыдущий комментарий.

[Владимир]

0xD34F, Вопрос витал в ввоздухе.

[Сергей]

Мне кажется, автору рановато смотреть в эту сторону.

Да, я только начал изучать Javascript, определённо рано.

Сергей Соколов, Владимир, у всех передаваемых объектов будет данный метод. Перехватывать ошибки пока не научился, да и код вряд ли будет использоваться другими разработчиками, поэтому передача "случайных" аргументов исключена. Тем не менее, возьму это на заметку.

Answer 2

[AngReload]

Ничего плохого. Непонятно, зачем нужна проверка if window, какого-то окна может не быть?

Comments

[Сергей]

AngReload, да.