Angularjs, не работает ngClick, как исправить?

Question

Dmitrii @v_manprod

Angular

Angularjs, не работает ngClick, как исправить?

Учу ангулар, хочу сделать витрину товаров, при нажатии на кнопку с ценой итогавая сумма отображалась бы в total, что не так?

app.js
var drinksApp = angular.module('drinksApp',[]);
drinksApp.controller('drinkListController',
    function drinkListController($scope) {
    $scope.drinks =  [];

    $scope.toggleActive = function(d){
        d.active = !d.active;
    };

    $scope.total = function(){

        var total = 0;



        angular.forEach($scope.drinks, function(d){
            if (d.active){
                total = total + d.price;
            }
        });

        return total;
    };

});

drink-list.component.js
angular.
module('drinksApp').
    component('drinkList', {
    templateUrl:'app/list.template.html',

    controller: function drinkListController() {
        this.drinks = [
            {
                name: 'Кока Колла',
                snippet: 'Качественный освежающий напиток.',
                price: 50
            }, {
                name: 'Спрайт',
                snippet: 'Прозрачный цвет и вкус лимона.',
                price: 90
            }, {
                name: 'Тархун',
                snippet: 'Лучшее, что придумал свет.',
                price: 200
            }, {
                name: 'Колокольчик',
                snippet: 'Зазвенит в ушах 100проц.',
                price: 25
            }
        ];




    }
});
angular.
module('drinksApp').
component('sidebar',    {
        templateUrl:'app/sidebar.template.html'


});

<!doctype html>
<html lang="en" ng-app="drinksApp">
<head>
    <meta charset="UTF-8">
    <meta name="viewport"
          content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" integrity="sha384-BVYiiSIFeK1dGmJRAkycuHAHRg32OmUcww7on3RYdg4Va+PmSTsz/K68vbdEjh4u" crossorigin="anonymous">

    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>



</head>
<body  >
<h1>Автомат напитков</h1>

<p>1+2 = {{1+2}}</p>

<div class="row centered" ng-controller="drinkListController">
    <div class="col-lg-8">
        <div class="col-lg-3" ng-repeat="drink in $ctrl.drinks ">
        <h2>
            <span>{{drink.name}}</span>
            </h2>
        <p>{{drink.snippet}}</p>

            <button ng-click="toggleActive()" ng-class="{active:drink.active}">{{drink.price}}</button>



        </div>


    </div>
    <div class="col-lg-4">


                <sidebar></sidebar>
        <h3>Total: <span>{{total()}}</span></h3>

    </div>
</div>







<script src="angular.min.js"></script>
<script src="app.js"></script>
<script src="drink-list.component.js"> </script>
</body>
</html>

Вопрос задан более трёх лет назад
146 просмотров

9 комментариев

Подписаться 1 Простой 9 комментариев

Ptolemy_master @Ptolemy_master

Во-первых, очень нехорошо, что у вас функция и переменная внутри нее имеют одинаковые имена (total). Во-вторых, никто не вызывает эту функцию, вот она и не выполняется. При клике на кнопку у вас просто меняется d.active, но ничего больше не происходит. Попробуйте повесить watch на эту переменную и в случае ее измененения вызывайте функцию пересчета total (лучше переименовать), которая также будет менять переменную scope (а не возвращать), а в итоговой сумме поставьте эту переменную, а не функцию - она будет обновляться автоматически.

Написано более трёх лет назад

Dmitrii @v_manprod Автор вопроса

Ptolemy_master,

var drinksApp = angular.module('drinksApp',[]);
drinksApp.controller('drinkListController',
    function drinkListController($scope) {
        $scope.drinks =  [];

        $scope.toggleActive = function(d){
            d.active = !d.active;
        };



        $scope.money = 0;
        $scope.$watch('money', function () {

            angular.forEach($scope.drinks, function(d){
                if (d.active){
                    money = money + d.price;
                }
            });

        });




    });

как-то так?

Написано более трёх лет назад

Ptolemy_master @Ptolemy_master

Dmitrii, не совсем.

var drinksApp = angular.module('drinksApp',[]);
drinksApp.controller('drinkListController',
    function drinkListController($scope) {
        $scope.drinks =  [];

        $scope.toggleActive = function(d){
            d.active = !d.active;
        };

        $scope.money = 0;
        $scope.$watch('d.active', function (newValue, oldValue) {
        if (newValue != oldValue && newValue){
            money = 0; //Обнулим предыдущее значение
            angular.forEach($scope.drinks, function(d){       
                    money = money + d.price;
                }
            });

        });
    });

<h3>Total: <span>{{money}}</span></h3>

Написано более трёх лет назад

Dmitrii @v_manprod Автор вопроса
Ptolemy_master, а кнопка такая же остается?
<button ng-click="toggleActive()" ng-class="{active:drink.active}">{{drink.price}}</button>
Написано более трёх лет назад
Ptolemy_master @Ptolemy_master
Dmitrii, да, кнопкой вы переключаете состояние. В принципе, можно было обойтись и без watch, а общую сумму считать в этой функции переключения. Если у вас переключение идет для каждого отдельного продукта, имеет смысл сделать по-другому:

var drinksApp = angular.module('drinksApp',[]); drinksApp.controller('drinkListController', function drinkListController($scope) { $scope.drinks = []; $scope.toggleActive = function(d){ d.active = !d.active; calculateTotal(); }; $scope.money = 0; $scope.calculateTotal = function() { money = 0; //Обнулим предыдущее значение angular.forEach($scope.drinks, function(d){ money = money + d.price; } }; }); });

То есть мы просто на любой клик пересчитывам общую сумму.
Написано более трёх лет назад
Dmitrii @v_manprod Автор вопроса

Ptolemy_master, но вот всеравно, показывает, что total: 0 и не обновляет((

Написано более трёх лет назад
Ptolemy_master @Ptolemy_master

Dmitrii, скопируйте сюда тот код, который не работает.

Написано более трёх лет назад

Dmitrii @v_manprod Автор вопроса

Ptolemy_master,

<!doctype html>
<html lang="en" ng-app="drinksApp">
<head>
    <meta charset="UTF-8">
    <meta name="viewport"
          content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" integrity="sha384-BVYiiSIFeK1dGmJRAkycuHAHRg32OmUcww7on3RYdg4Va+PmSTsz/K68vbdEjh4u" crossorigin="anonymous">

    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>



</head>
<body  >
<h1>Автомат напитков</h1>

<p>1+2 = {{1+2}}</p>


<div class="row centered" ng-controller="drinkListController">
    <div class="col-lg-8">
        <drink-List></drink-List>
        <h3>Total: <span>{{money}}</span></h3>


    </div>
    <div class="col-lg-4">


        <sidebar></sidebar>


    </div>
</div>







<script src="angular.min.js"></script>
<script src="app.js"></script>
<script src="drink-list.component.js"> </script>
</body>
</html>

app.js
var drinksApp = angular.module('drinksApp',[]);
drinksApp.controller('drinkListController',
    function drinkListController($scope) {
        $scope.drinks =  [];

        $scope.toggleActive = function(d){
            d.active = !d.active;
            calculateTotal();
        };

        $scope.money = 0;
        $scope.calculateTotal = function() {

            money = 0; //Обнулим предыдущее значение
            angular.forEach($scope.drinks, function(d){
                money = money + d.price;
            })
        };

    });

component.js
angular.
module('drinksApp').
component('drinkList', {
    templateUrl:'app/list.template.html',

    controller: function drinkListController() {
        this.drinks = [
            {
                name: 'Кока Колла',
                snippet: 'Качественный освежающий напиток.',
                price: 50
            }, {
                name: 'Спрайт',
                snippet: 'Прозрачный цвет и вкус лимона.',
                price: 90
            }, {
                name: 'Тархун',
                snippet: 'Лучшее, что придумал свет.',
                price: 200
            }, {
                name: 'Колокольчик',
                snippet: 'Зазвенит в ушах 100проц.',
                price: 25
            }
        ];





    }
});
angular.
module('drinksApp').
component('sidebar',    {
    templateUrl:'app/sidebar.template.html'


});

и сам шаблончик

        <div class="col-lg-3" ng-repeat="drink in $ctrl.drinks ">
        <h2>
            <span>{{drink.name}}</span>
            </h2>
        <p>{{drink.snippet}}</p>


            <button ng-click="toggleActive(d)" ng-class="{active:d.active}" >{{drink.price | currency }}</button>




        </div>

Написано более трёх лет назад

Ptolemy_master @Ptolemy_master

Dmitrii, чето у вас странно контроллер прописан в двух местах. И в одном список пустой. Перенесите весь код из компонента в ту вторую функцию контроллера.
Также в дебаггере (вы каким пользуетесь? Я - батарангом) посмотрите на список, когда пересчитывается функция, заполнен ли он и нет ли ошибок при расчете.

Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы

Nginx

+3 ещё

Средний
Как настроить Nginx на отдачу файла main.js Angular SPA на микрофронтенде?
- 2 подписчика
- 26 окт.
- 112 просмотров
0

ответов
Angular

Простой
Как приостановить все запросы в Angular при получении ошибки 401, чтобы новые запросы не отправлялись?
- 3 подписчика
- 21 окт.
- 458 просмотров
0

ответов
JavaScript

+2 ещё

Средний
Как поправить ошибку Cannot read properties of undefined (reading 'form')?
- 1 подписчик
- 21 окт.
- 126 просмотров
2

ответа
Angular

Простой
Как вывести сложный объект на график в DevExtreme Angular 17?
- 1 подписчик
- 16 сент.
- 91 просмотр
1

ответ
Angular

Простой
Как исправить ошибку?
- 1 подписчик
- 06 сент.
- 37 просмотров
1

ответ
Angular

Простой
Как именно работает markForCheck в Angular при detection changes?
- 1 подписчик
- 19 июл.
- 76 просмотров
1

ответ
Angular

Средний
Приходилось ли Вам кастомизировать компоненты из библиотеки Angular Material и какой подход Вы использовали?
- 1 подписчик
- 19 июл.
- 54 просмотра
0

ответов
JavaScript

+1 ещё

Сложный
Как оптимизировать рендеринг таблицы?
- 2 подписчика
- 17 июл.
- 190 просмотров
1

ответ
Angular

+1 ещё

Простой
Как в Ionic Framework слушать сообщения?
- 1 подписчик
- 15 июл.
- 19 просмотров
0

ответов
Angular

+1 ещё

Простой
В чём проблема с запросами?
- 2 подписчика
- 14 июл.
- 119 просмотров
0

ответов
Показать ещё Загружается…

Системный Аналитик 1С

Wanted. • Москва

До 260 000 ₽

Ведущий программист 1С

Автомакон

от 250 000 до 400 000 ₽

Системный аналитик

HR Prime

от 200 000 до 300 000 ₽

Поправить сайт (мобильная версия) на WordPress

24 нояб. 2024, в 04:16

5000 руб./за проект

Необходимо поменять пароль в WINDOWS

24 нояб. 2024, в 03:11

500 руб./за проект

Требуется консультация по UX-дизайну казино

24 нояб. 2024, в 01:35

5000 руб./за проект

Во-первых, очень нехорошо, что у вас функция и переменная внутри нее имеют одинаковые имена (total). Во-вторых, никто не вызывает эту функцию, вот она и не выполняется. При клике на кнопку у вас просто меняется d.active, но ничего больше не происходит. Попробуйте повесить watch на эту переменную и в случае ее измененения вызывайте функцию пересчета total (лучше переименовать), которая также будет менять переменную scope (а не возвращать), а в итоговой сумме поставьте эту переменную, а не функцию - она будет обновляться автоматически.
Ptolemy_master,
var drinksApp = angular.module('drinksApp',[]); drinksApp.controller('drinkListController', function drinkListController($scope) { $scope.drinks = []; $scope.toggleActive = function(d){ d.active = !d.active; }; $scope.money = 0; $scope.$watch('money', function () { angular.forEach($scope.drinks, function(d){ if (d.active){ money = money + d.price; } }); }); });

как-то так?
Dmitrii, не совсем.

var drinksApp = angular.module('drinksApp',[]); drinksApp.controller('drinkListController', function drinkListController($scope) { $scope.drinks = []; $scope.toggleActive = function(d){ d.active = !d.active; }; $scope.money = 0; $scope.$watch('d.active', function (newValue, oldValue) { if (newValue != oldValue && newValue){ money = 0; //Обнулим предыдущее значение angular.forEach($scope.drinks, function(d){ money = money + d.price; } }); }); });

<h3>Total: <span>{{money}}</span></h3>
Ptolemy_master, а кнопка такая же остается?
<button ng-click="toggleActive()" ng-class="{active:drink.active}">{{drink.price}}</button>
Dmitrii, да, кнопкой вы переключаете состояние. В принципе, можно было обойтись и без watch, а общую сумму считать в этой функции переключения. Если у вас переключение идет для каждого отдельного продукта, имеет смысл сделать по-другому:

var drinksApp = angular.module('drinksApp',[]); drinksApp.controller('drinkListController', function drinkListController($scope) { $scope.drinks = []; $scope.toggleActive = function(d){ d.active = !d.active; calculateTotal(); }; $scope.money = 0; $scope.calculateTotal = function() { money = 0; //Обнулим предыдущее значение angular.forEach($scope.drinks, function(d){ money = money + d.price; } }; }); });

То есть мы просто на любой клик пересчитывам общую сумму.
Ptolemy_master, но вот всеравно, показывает, что total: 0 и не обновляет((
Dmitrii, скопируйте сюда тот код, который не работает.
Dmitrii, чето у вас странно контроллер прописан в двух местах. И в одном список пустой. Перенесите весь код из компонента в ту вторую функцию контроллера.
Также в дебаггере (вы каким пользуетесь? Я - батарангом) посмотрите на список, когда пересчитывается функция, заполнен ли он и нет ли ошибок при расчете.

Angularjs, не работает ngClick, как исправить?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт