Как заставить 'reduce' изменять 'this' а не возвращать новый объект?

Question

[Vlad Harbarchuk]

Здравствуйте! Разбираюсь немного с объектами и this, и никак не могу понять как решить следующую проблему:

1. У нас есть некий массив с объектами под названием 'beers'

var beers = [ {}, {}, ... другие объекты массива ];

2. Добавляем методы к beers:

beers.reduceBeers = function () {
			 this.reduce(function (all, item, index) {
				if (item.locale === 'domestic') {
					all.push(item);
				}
				return all;
			}, []);
			return this;
		};
    
    beers.logBeers = function () {
        console.log(this);
        return this;
        };

    beers.otherBeers = function () {
        console.log('some other method that doing something with beers');
        return this;
	};

3. В итоге нам нужно, чтобы могли делать цепные вызовы этих методов
beers.reduceBeers().logBeers().otherBeers();

Для того, чтобы делать цепные вызовы мы должны возвращать 'this' после каждого выполнения метода. Но в методе beers.reduceBeers мы не изменяем сам массив beers (так как reduce возвращает новый ) и this просто возвращает beers без изменений. И следующий метод будет применятся к изначальному массиву beers, а не к отсортированному, с применением reduce. То есть нужно что-то вроде "this = this.reduce(function ... " но так делать нельзя, выдает ошибку. А если возвращать что-то другое, а не this, то следующий метод в цепочке (logBeers в нашем случае) просто не будет работать.

Буду очень благодарен если кто-то подскажет как это можно решить, единственное что пришло в голову это добавлять методы в Array.prototype но это как я читал очень нежелательно делать, особенно в таком случае как у меня...

Comments

[Lynn «Кофеман»]

У меня ощущение, что вы не понимаете что происходит, что делает reduce и что видимо вам нужен метод splice.

[Vlad Harbarchuk]

Алексей Тен: я только учусь, поэтому все возможно. splice не подходит, так как мне нужно именно отфильтровать массив по определенным критериям, или например вернуть новый массив который будет только из определенных свойств определенных объектов, которые являются элементами этого массива. но вот как заставить чтоб this изменилось в итоге...

Answer 1

[zemka99]

Если у вас есть возможность поменять структуру переменной beers и сделать вот так

var beers ={
		array:[{
			...
		},{
			...
		},{
			..
		}]
	};

то дальше можно без проблем работать с this.array

beers.reduceBeers = function(){
		this.array = [];
		this.array.reduce(function (all, item, index) {
			if (item.locale === 'domestic') {
				this.array.push(item);
			}
			return this.array;
		}, []);
		return this;
	};

вроде должно сработать

Comments

[Vlad Harbarchuk]

да, возможность поменять есть. Сделал как вы написали, но тогда просто возвращается пустой массив всегда, подозреваю из-за того, что мы изменили его тут this.array = [], и reduce стал применятся уже на пустом массиве

[zemka99]

beers.reduceBeers = function(){

this.array = this.array.reduce(function (all, item, index) {

if (item.locale === 'domestic') {
all.push(item);
}
return all;
}, []);

return this;
};
давайте вот так

[Vlad Harbarchuk]

zemka99: да, спасибо Вам большое, теперь все работает и все понятно!

[zemka99]

ну кстати как написали ниже filter и правда больше сюда подходит
beers.reduceBeers = function(){

this.array = this.array.filter(function (item) {

return (item.locale === 'domestic')? true : false;
});

return this;
};

Answer 2

[Dmitriy Mozgovoy]

var beers=[1,{locale:'domestic'},5,6,{locale:'domestic'},0,{locale:'domestic'}];

       beers.reduceBeers = function () {

           var stack=[];

           this.forEach(function(item,index){
               if (item.locale === 'domestic') {
                   stack.push(item);
               }
           });

           this.length = 0;//empty an Array;

           this.push.apply(this,stack);

           return this;
       };

       beers.reduceBeers();

Насчет целесообразности reduce терзают смутные сомнения ... Хотя можете запихать в stack что хотите, все что в stack будет добавлено в "this". Тоже самое:

beers.reduceBeers = function () {

           var stack=this.filter(function(item,index){
                return item.locale === 'domestic';
           })

           this.length = 0;//empty an Array;
           this.push.apply(this,stack);
           return this;
       };

Comments

[Vlad Harbarchuk]

Да, спасибо Ваш способ тоже работает, с reduce погорячился :)

Answer 3

[Сергей Сергей]

Вместо редюс Array.prototype.filter - https://developer.mozilla.org/en-US/docs/Web/JavaS...

В методе reduceBeers результирующий массив после фильтра удержать в переменную (filterResult, например), потом опустошить массив this.splice( 0, this.length );
Затем сделать this.concat( filterResult );

Comments

[Dmitriy Mozgovoy]

P.S Метод concat тоже не изменяет массив. Не смог удержатся =)

[Сергей Сергей]

Digital Brain: ах ты ж оказия какая!
ну тогда splice.apply( this, [0,0].concat(fR) );

Answer 4

[Nikolay Talanov]

Почему бы не написать что-то такое:
beers.reduceBeers = function () {
return this.filter(function(el) {return el === 'domestic'});
};

Конкретно у вас ошибка в том, что вы на сам this никак не влияете. То что вы там юзаете reduce, это никак не затрагивает this за рамками самого reduce. Вот если бы вы юзали splice например, то тогда данные бы менялись.
В моем примере делается все тоже что и у вас, только правильным методом (тут явно нужен фильтр, а не редьюс, который юзается для всяких сумм и прочих вещей) и изменяется сам this, который тут же возвращается.

Comments

[Vlad Harbarchuk]

спасибо за ответ, Ваш способ сразу возвращать this.reduce (или this.filter) пробовал, но тогда будет возвращен просто новый объект, который не есть beers и у него нету его методов. И при таком вызове beers.reduceBeers().logBeers() будет ошибка, ведь что reduce что filter возвращают новый массив а не изменяют тот, на котором они были вызваны.

да, в этом случае filter будет более правильным, буду использовать его, спасибо) хотя все равно я дальше буду юзать reduce для подсчета среднего арифметического и так чтобы это работало с ним мне тоже нужно)

p.s. проблему решил способом zemka99 к комментарии, всем большое спасибо за помощь!