Outro problema após a atualização do AngularFire 0.5.0 – o cálculo retorna NaN
-
22-12-2019 - |
Pergunta
Estou cada vez mais decepcionado com o angularFire 0.5.0, porque nada mais funciona corretamente.Depois que consegui corrigir a remoção de um único item com a ajuda de vocês, me deparei com outro problema.
Cada item consiste em uma data, descrição e preço.Antes de atualizar, consegui calcular o total de todos os preços e devolvê-lo na página.Agora diz apenas NaN ou Null.Eu já estava tentando descobrir o porquê, mas ambos os valores (earning.price, $scope.totalEarning) no cálculo são números.Eu não entendo.O novo fogo angular faz alguma coisa?Estou tentando consertar isso há algum tempo e simplesmente não consigo.Eu estaria realmente, se alguém pudesse descobrir.Provavelmente não estou vendo isso e é uma questão muito idiota.
Veja no plunkr: http://embed.plnkr.co/jb1iWOcjcm0alFchmzOH/preview
Aqui está o código:
$scope.addEarning = function() {
$scope.earnings.$add({date:$scope.FormEarningDate, description:$scope.FormEarningDescription, price:$scope.FormEarningPrice});
$scope.FormEarningDate = '';
$scope.FormEarningDescription = '';
$scope.FormEarningPrice = '';
$scope.updateEarning();
}
$scope.updateEarning = function() {
$scope.totalEarning = 0;
angular.forEach($scope.earnings, function (earning) {
price = parseFloat(earning.price)
$scope.totalEarning += price;
$log.log(typeof $scope.totalEarning);
})
$scope.totalBalance = $scope.totalEarning - $scope.totalCost;
}
E o HTML:
<form for="Profit" class="form-inline" style="margin-bottom: 20px" ng-submit="addEarning()">
<div class="form-group">
<div class="col-sm-3">
<input type="date" name="idate" ng-model="FormEarningDate" class="form-control" id="idate" placeholder="Date">
</div>
<div class="col-sm-5">
<input type="text" name="idesc" ng-model="FormEarningDescription" required class="form-control" id="idesc" placeholder="Description">
</div>
<div class="col-sm-2">
<input type="text" name="iprice" ng-model="FormEarningPrice" required class="form-control" id="iprice" placeholder="Amount">
</div>
</div>
<tr ng-repeat="earning in earnings | orderByPriority | orderBy : 'date'">
<td>{{earning.date}}</td>
<td>{{earning.description}}</td>
<td>{{earning.price}} €</td>
<td><button class="btn btn-danger" ng-click="earnings.$remove(earning.$id)">Löschen</button></td>
</tr>
<tr>
<td colspan="2"></td>
<td><strong>Total:</strong> {{totalEarning}} €</td>
<td></td>
</tr>
Solução
Resposta atualizada
Existem várias coisas que você está fazendo incorretamente.Conforme declarado na minha resposta original, você precisa iterar as chaves do Firebase, não o próprio objeto do Firebase.Além disso, você precisa garantir que todas as atualizações ocorram após a atualização do Firebase (por meio de um $on('change', ...)
manipulador de eventos) e dentro do ciclo de vida AngularJS (realizado abaixo usando $timeout
).
var app = angular.module('balance', ['firebase']);
app.controller('BalanceCtrl', function($scope, $log, $http, $timeout, $firebase) {
$scope.earnings = $firebase(new Firebase('https://dgdemo.firebaseio.com/Earnings'));
$scope.totalBalance = 'Nothing yet.';
$scope.totalEarning = 0;
$scope.totalCost = 0;
$scope.updateEarning = function() {
$scope.totalEarning = 0;
angular.forEach($scope.earnings.$getIndex(), function (id) {
var earning = $scope.earnings[id];
$scope.totalEarning += parseFloat(earning.price);
});
$scope.totalBalance = $scope.totalEarning - $scope.totalCost;
};
// Ensure any call to updateCallback will execute within the AngularJS lifecycle.
var updateCallback = angular.bind(null, $timeout, $scope.updateEarning);
$scope.addEarning = function() {
var earning = $scope.earnings.$add({
date: $scope.FormEarningDate,
description: $scope.FormEarningDescription,
price: $scope.FormEarningPrice
});
$scope.FormEarningDate = '';
$scope.FormEarningDescription = '';
$scope.FormEarningPrice = '';
};
$scope.earnings.$on('change', updateCallback);
});
Resposta original
Isto é porque angular.forEach
está iterando sobre todas as propriedades do $scope.earnings
objeto (que é um objeto Firebase).O que você deseja fazer é iterar os itens individuais.
$scope.updateEarning = function() {
$scope.totalEarning = 0;
angular.forEach($scope.earnings.$getIndex(), function (id) {
var earning = $scope.earnings[id];
$scope.totalEarning += parseFloat(earning.price);
});
$scope.totalBalance = $scope.totalEarning - $scope.totalCost;
}