Как добавить к массиву объектов новый ключ-значение?

Question

[Илья]

Есть такой код:

$scope.messages = data;
			for (var i = 0; i < $scope.messages.response.length; ++i) {
				if ($scope.messages.response[i].type == 'location') {
					var strings = $scope.messages.response[i].content.split(' ');
					var lat1 = strings[0],
						lat2 = strings[1];
					$http.get('http://maps.googleapis.com/maps/api/geocode/json?latlng='+lat1+','+lat2+'&sensor=true_or_false')
					.success(function(data)
					{
						$ionicLoading.hide();
						$scope.mapAdd = data;
						$scope.messages.response[i].formatted_address = $scope.mapAdd.results[0].formatted_address; вот здесь нужно добавить новое значение с ключом formatted_address, но если делать так, то  TypeError: Cannot set property 'formatted_address' of undefined
						console.log($scope.messages.response[i].formatted_address);
						console.log($scope.mapAdd.results[0].formatted_address);
					});
				};
			};

Подскажите как реализовать. Спасибо.

Comments

[sdo]

Не очень понял что требуется реализовывать

[Илья]

sdo: У нас есть массив объектов(это видно на скрине). Так вот. К объекту нужно добавить новый ключ formatted_address со значением $scope.mapAdd.results[0].formatted_address

Answer 1

[vsvladimir]

Тут наверное проблема с переменной i. Результат $http.get приходит уже после завершения цикла. i=16 и $scope.messages.response[16] не существует. Нужно исходные значения i где-то запомнить. Советую почитать про замыкания в JavaScript, как одно из решений.

Comments

[Илья]

Вы правы. Вот рабочий код:

$http.get('http://api.getchill.co/api/v1/messages/index/id_user/'+$rootScope.userData.id_user+'/id_contact/'+$stateParams.id)
      .success(function(data) {
          $scope.messages = data;
          angular.forEach($scope.messages.response, function(response, i){
              if (response.type == 'location') {
                  var strings = response.content.split(' ');
                  var lat1 = strings[0], lat2 = strings[1];
                  $http.get('http://maps.googleapis.com/maps/api/geocode/json?latlng='+lat1+','+lat2+'&sensor=false')
                  .success(function(data) {
                      $ionicLoading.hide();
                      response.address = data.results[0].formatted_address;
                  });
              };
          });
          $ionicSlideBoxDelegate.update();
      });

Answer 2

[Вячеслав]

Если вам просто нужно добавить элемент в начало массива, используйте unshift.
Если нет - то вопрос непонятен.

Comments

[Илья]

У нас есть массив объектов(это видно на скрине). Так вот. К объекту нужно добавить новый ключ formatted_address со значением $scope.mapAdd.results[0].formatted_address

[Вячеслав]

Илья: к какому объекту ? :)
ваш код перегружен лишними скобочками, точками, вложенностью элементов.
добавить к объекту новый ключ элементарно, я думаю, вы и сами знаете ответ на этот вопрос:
myObj.formatted_address = value;
Значит, вам нужно нечто другое, но что конкретно - все еще непонятно.

[Илья]

Вячеслав: Да-да, мне нужно именно добавить новый ключ. Я так и делаю, но по логам ошибка TypeError: Cannot set property 'formatted_address' of undefined.

[Вячеслав]

Илья: окей. ошибка очевидна - объект, к которому вы обращаетесь - не существует (undefined).
Меня смущает, например, что на скрине у вас - response, а обращаетесь в коде вы к results

Answer 3

[sdo]

ну самое простое (не вижу у вас в коде $ а других библиотек не знаю, может у вас там есть map) это (также не вкурил, к какому именно объекту.. и в чем сложность вопроса?)

for(var i=0;i<MyObject.response.length;i++){
    MyObject.response[i]['formatted_address ']= $scope.mapAdd.results[0].formatted_address;
}

У меня четкое ощущение что я не понимаю, а задача сложнее.

Comments

[Илья]

Из библиотек там только angularjs.
Делаю также как и Вы, но в логах TypeError: Cannot set property 'formatted_address' of undefined.

Если говорить прямо, то у нас есть массив response. Там есть объекты. И в каждый объект нужно добавить новый ключ(которого еще нет), и ему присвоить некое значение. Как еще объяснить я не знаю, к сожалению.(

[sdo]

Илья: покажите код с моей вставкой

[Илья]

sdo:

$http.get('http://api.getchill.co/api/v1/messages/index/id_user/'+$rootScope.userData.id_user+'/id_contact/'+$stateParams.id)
		.success(function(data)	{
			$scope.messages = data;
			for (var i = 0; i < $scope.messages.response.length; ++i) {
				if ($scope.messages.response[i].type == 'location') {
					var strings = $scope.messages.response[i].content.split(' ');
					var lat1 = strings[0], lat2 = strings[1];
					$http.get('http://maps.googleapis.com/maps/api/geocode/json?latlng='+lat1+','+lat2+'&sensor=false')
					.success(function(data)	{
						$ionicLoading.hide();
						$scope.messages.response[i]['formatted_address'] = data.results[0].formatted_address;
					});
				};
			};
			$ionicSlideBoxDelegate.update();
		});

[Илья]

sdo: Как я понял, вся проблема в том, что массив объектов просто не виден в .success(function(data) {}
Вот я тему создал отдельную - Почему не виден(undefined) массив объектов в методе?