如何摆脱jQuery堆积物并以角度的方式改善我当前的指令？

Question

I would like to have button or link with icon, default glyphicon-play or glyphicon-pause if interval is enabled. How can I refactor this directive especially $element.hasClass("glyphicon-pause") or $element.removeClass("glyphicon-pause").addClass("glyphicon-play"); in more "angular way"?

<button play class="btn glyphicon glyphicon-play"></button>

Current directive:

app.directive('play', ['$interval', function ($interval) {
    return {
        restrict: 'A',
        link: function ($scope, $element, attrs) {
            var i = 0,
                interval;


            var play = function () {
                $interval.cancel(interval);
                interval = $interval(function () {
                    $scope.states[i].active = false;
                    $scope.states[i++].active = true;
                    i = i % 3;
                }, 1000);
            };


            var stop = function () {
                $interval.cancel(interval);
            };
            console.log($element, attrs);


            $element.on('click', function ($event) {
                if ($element.hasClass("glyphicon-pause")) {
                    $element.removeClass("glyphicon-pause").addClass("glyphicon-play");
                    stop();
                } else {
                    $element.removeClass("glyphicon-play").addClass("glyphicon-pause");
                    play();
                }
            });
        }
    };
}]);

Answer 1

Using ng-class and ng-click would be the two most angular-like improvements here.

<button play class="btn glyphicon" ng-class="{glyphicon-play: isPlaying, glyphicon-pause: !isPlaying}" ng-click="togglePlay()"></button>

---

app.directive('play', ['$interval', function ($interval) {
    return {
        restrict: 'A',
        link: function ($scope, $element, attrs) {
            $scope.isPlaying = false;
            var i = 0,
                interval;

            var play = function () {
                $scope.isPlaying = true;
                $interval.cancel(interval);
                interval = $interval(function () {
                    $scope.states[i].active = false;
                    $scope.states[i++].active = true;
                    i = i % 3;
                }, 1000);
            };


            var stop = function () {
                $scope.isPlaying = false;
                $interval.cancel(interval);
            };
            console.log($element, attrs);

            $scope.togglePlay = function() {
              if($scope.isPlaying){
                stop();
              }else{
                play();
              }
            };
        }
    };
}]);