2

Есть такой небольшой кусочек кода:

if (Rect.__instances__) {
  Rect.__instances__.push(this);
} else {
  Rect.__instances__ = [];
  Rect.__instances__ = [].push(this);
}

Суть: если у класса есть свойство __instances__, то пихаем в него this. Если нет, то сначала инициализируем __instances__ пустым массивом, а после этого пихаем в него this.

Нужно записать это условие в одну строку вида:

Rect.__instances__ ? Rect.__instances__.push(this) : Rect.__instances__ = [].push(this);

Но этот код вместо того, чтобы вернуть массив со значением this внутри, просто возвращает единицу.

Тогда я попробовал сделать так:

Rect.__instances__ ? Rect.__instances__.push(this) : Rect.__instances__ = (() => {return []}).push(this);

На что мне выдало ошибку вида:

Uncaught TypeError: (intermediate value).push is not a function

Пробовал еще и так:

Rect.__instances__ ? Rect.__instances__.push(this) : Rect.__instances__ = (() => {new Array()}).push(this);

Ошибка та же.

Теперь прошу помощи здесь. Кстати, если кто-то понял, что делает этот код и для того, что я пишу есть решение получше, вроде методов стандартной библиотеки, то такой подход тоже сгодится :)

P.S. Я добавил к вопросу тег lambda, потому что здесь, вроде как именно про это речь и идет. Дело в том, что лямбды в JavaScript'е я гуглил, и в ответ мне выдавало только инфу, связанную со стрелочными ф-ми. Поэтому рискнул предположить, что, поскольку дело тут, скорее всего, именно в стрелочной функции, то и тег "лямбда" сюда подходит.

smellyshovel
  • 5,224
  • но зачем переписывать код в одну строку??? – Grundy Jan 27 '17 at 17:48
  • @Grundy некрасиво он выглядит. Люблю симметрию :D Раз уж на то пошло, то не мог я просто оставить одну строчку в if и 2 в else. А когда в одну - вообще идеально. И да, про читабельность я в курсе :) – smellyshovel Jan 27 '17 at 18:01

3 Answers3

2

Выбирай. Вроде последний вариант хороший.

if (Rect.__instances__) {
  Rect.__instances__.push(this);
} else {
  Rect.__instances__ = [this];
}

Rect.__instances__ ? Rect.__instances__.push(this) : (Rect.__instances__ = [this]);

if (!Rect.__instances__) {
  Rect.__instances__ = [];
}

Rect.__instances__.push(this);

(Rect.__instances__ = Rect.__instances__ || []).push(this);
Qwertiy
  • 123,725
  • Да, последний интересный. Хотелось бы узнать, для чего во втором примере в else-statement скобки? – smellyshovel Jan 27 '17 at 16:48
  • @smellyshovel, для того же, для чего и в if? http://ru.stackoverflow.com/a/424351/178988 – Qwertiy Jan 27 '17 at 16:49
  • Но ведь можно было записать так? Rect.__instances__ ? Rect.__instances__.push(this) : Rect.__instances__ = [this]; – smellyshovel Jan 27 '17 at 16:50
  • @smellyshovel, ой. Я про первый подумал. Во втором - думал, что из-за из-за приоритетов операций, но ты прав, они там не нужны в js. Хм.. И не только в js. Просто очень редко присваивание попадает в это место. А на грабли с запятой я уже наступал)) – Qwertiy Jan 27 '17 at 16:51
  • Тогда ясно. В общем, первый и третий варианты не подходят однозначно, так как условие стояло в том, чтобы записать в одну строку. Но за четвертый вариант само собой отмечу решением. Очень уж понравилось :) Спасибо! – smellyshovel Jan 27 '17 at 16:53
1
(Rect.__instances__ = []).push(this);

или

Rect.__instances__ = [this];

а push возвращает новую длину массива, поэтому Вы и наблюдали единицу в первоначальном варианте кода.

0

var Rect={};

if (Rect.instances) {
  Rect.instances.push("1111");
} else {
  Rect.instances = new Array("1111");
}
console.log(Rect)
C.Raf.T
  • 4,034
  • Здесь тоже все как надо, благодарю, решение, по идее, аналогичное тому, что выше. Но я слышал, что там, где массив можно объявить литералом, там его при помощи конструктора лучше не объявлять. – smellyshovel Jan 27 '17 at 16:42
  • 1
    Только new Array и не хватало. – Qwertiy Jan 27 '17 at 16:43