使用循环计算数组中新推送的数据

Question

我无法正确计算循环内数组中的数值。

场景

• tbody 生成动态值列表。
• td 内部的button 有一个名为function 的preBookThis() 和params。
• 当用户点击button时，会触发function然后pushdata变成array variable。
• 在推送时，将有一个for loop 将迭代和计算值。 （$scope.nights 是两个日期 - 即 2018-05-05 和 2018-05-07，因此其值为 2）

问题

• 假设tbody 中有三行，并且具有以下值。

  $scope.nights = 2; row1 = 100; row2 = 200; row3 = 300;

• 当用户点击 第 1 行中的button 时，计算工作并按预期执行

  amount_row1 = 200;

但是……

• 当用户点击第2行中的button时，第一行的计算值将加倍。

  amount_row1 = 400; amount_row2 = 400;

还有……

• 当用户点击第3行中的button时，第一行和第二行的计算值将再次翻倍。

  amount_row1 = 800; amount_row2 = 800; amount_row3 = 600;

最终输出应该是：

 amount_row1 = 200;
 amount_row2 = 400;
 amount_row3 = 600;

HTML

<tbody ng-repeat="room in roomsDisplay">
    <tr>
        <td>
            <div class="col-md-6"><img ng-src="images/rooms/{{room.room_image}}"></div>         
        </td>
        <td>
            <div style="margin-top: 20px;"><input type="button" 
                ng-click="preBookthis({room})" class="btn btn-primary" 
                value="Book this Room"></div>
        </td>
    </tr>
</tbody>

JS

$scope.preBookthis = function(data){
    $scope.totalAmount = 0;
    $scope.bookData.push({ 
        'room_name': data.room.room_name, 
        'price': data.room.price,
        'amount': $scope.amount,
        'id' : data.room.rooms_type_id
    });

    _.forEach($scope.bookData, function(value, key){
        value.amount = value.amount * $scope.nights;
    });
}

Answer 1

在添加新对象时只进行一次计算，不要遍历已经计算过的所有其他对象

类似：

$scope.preBookthis = function(data){
    $scope.totalAmount = 0;
    $scope.bookData.push({ 
        'room_name': data.room.room_name, 
        'price': data.room.price,
        'amount': $scope.amount * $scope.nights,// calculate only for this item
        'id' : data.room.rooms_type_id
    });  
}

Answer 2

问题源于 $scope.bookData 在函数被调用时被完全更改。

调用函数时，$scope.bookData 可能包含其他行的对象。这些对象具有amount 值。在您的循环中，将该值乘以 $scope.nights（即 2）。

因此，您将 所有 行的值乘以 2。

您反复更改amount 的值。

试试这个：

_.forEach($scope.bookData, function(value, key){
    value.amount = (key+1) * $scope.nights*100;
});

编辑：

其实，另一个answer 提供了更好的解决方案。