Как проверить существование элемента в массиве?

Question

[KOPC1886]

Всем привет!

Не получается сделать через indexOf.

Вот код

$scope.selectedEditor = function(user)
            {
                user.originalObject.id = user.originalObject.id * 1;
                if($scope.selectedEditors.indexOf(user.originalObject.id) === -1)
                {
                    $scope.selectedEditors.push({
                        id: user.originalObject.id,
                        label: user.originalObject.label
                    });
                }
                console.log($scope.selectedEditors);
            }

<div id="selectedEditors">
                                <div class="selected_user" ng-repeat="editor in selectedCategory.editors">
                                    <span class="user">{{editor.label}}</span>
                                    <span class="delete_chosen glyphicon glyphicon-remove" ng-click="deleteEditor($index)" title="Удалить"></span>
                                </div>
                            </div>

При добавлении одного и того же пользователя, в массив вновь добавляется этот юзер. А мне нужно сделать проверку, чтобы юзеры не повторялись. Если есть такой в массиве уже, то не добавлять. А если нет, то добавить.

Answer 1

[Vladimir San]

user у вас - объект, который не является примитивом, и с ним indexOf не сработает.
Варианты:
1) Держать параллельно массив с id юзеров и проверять наличие id в нем
2) Проходить по массиву и каждый элемент сравнивать с новым по id.
или выдумать еще что-нибудь

Comments

[KOPC1886]

KwI мне нужно определить, есть ли в массиве уже объект с id таким или нет. А какой из этих вариантов будет лучше?

[Илья]

KOPC1886: 1 вариант даёт плюс к скорости, но минус к памяти. 2 вариант наоборот.

[KOPC1886]

FireGM: а еще есть какие то другие способы?

[Vladimir San]

KOPC1886: На StackOverflow есть красивый ответ:
pos = myArray.map(function(e) { return e.id; }).indexOf(your_id);
Но принципиально других вариантов наверно нет - все будут вращаться вокруг этих трех.
А у Вас НАСТОЛЬКО много пользователей, что в браузере надо экономить память/скорость именно на этом моменте?

[KOPC1886]

FireGM: да и при добавлении/удалении юзера, мне надо будет в двух scope'ах это проделывать получается

[KOPC1886]

KwI: мне не экономия нужна) А правильное решение)

[KOPC1886]

Я не знаю просто как лучше это сделать.

[KOPC1886]

FireGM: так получается что ли?

$scope.selectedEditors = [];
angular.forEach($scope.selectedCategory.editors, function(value, index){
$scope.selectedEditors.push(value.id);
});
$scope.selectedEditor = function(user)
{
console.log(user);
user.originalObject.id = user.originalObject.id * 1
if($scope.selectedEditors.indexOf(user.originalObject.id) === -1)
{
$scope.selectedEditors.push(user.originalObject.id);
$scope.selectedCategory.editors.push({
'id': user.originalObject.id,
'label': user.originalObject.label
});
}

[Vladimir San]

KOPC1886: Если несколько scope'ов, то наверно лучше в сервис вынести эти данные? Это будет правильно с точки зрения Angular.js, и тогда все манипуляции с данными происходить будут в одном месте.
А вообще правильность - сомнительная категория :) Можете использовать любой удобный Вам вариант. Хотя чувствую, что с map будет самым оптимальным.

[KOPC1886]

KwI
А с map я что-то не очень понял.

у меня есть $scope.selectedCategory.editors - тут юзеры.
а в map какой scope передавать?

[Vladimir San]

KOPC1886: В map передается callback, а вызывать map у того массива, где у вас лежат объекты.