javascript - AngularJS 'agnostic' 和 'component' 指令

Question

最近我和同事就 AngularJS 中最灵活的指令使用方式发生了激烈的争论。经过漫长而疲惫的讨论，我们同意请社区来裁决我们的争吵。请帮助我们:)

如果简单地说，我坚信要使某些东西真正灵活，应该遵循主要的 MVC 原则 - 严格将逻辑与表示分离，并且我们拥有什么样的“范围”并不重要。 这只是使用任何 MVC 框架的更好方法。我的同事不同意我的观点，并且坚信 AngularJS 中的指令对于 AngularJS 来说是非常“特殊”的，并且只有当它在“板”中包含所有内容(如模板+“自己的” Controller )时才有效。他还声称，在指令范围内直接操作 DOM 是非常好的，并且所有“指令”都应该以这种方式工作......

我们有一个想法来实现“就地”实体编辑功能。它应该按以下方式工作 - 用户只需单击一些“文本”，就可以更改它的值。

我建议采用以下方法:

模板(text-editor.html):

<label class="title">{{title}}</label>
<span class="value" ng-show="!editor.edited[name]" ng-click="editor.activateEditing(name)">{{editor.entity[name]}}</span>
<input ng-show="editor.edited[name]" ng-model="editor.entity[name]" ng-change="editor.validate(name)"></input> 

Controller :

function editorController ($scope, $controller) {
    function copy (source) {
      var result = {};
      for (var prop in source) {
        if (source.hasOwnProperty(prop) && prop.charAt(0) !== '_') {
          result[prop] = source[prop];
        }
      }

      return result;
    }

    $scope.edited = {};

    $scope.activateEditing = function (name) {
      var entity = $scope.entity;

      $scope.edited[name] = true;

      if (!entity._backup) {
        entity._backup = copy(entity);
      }
    };

    $scope.cancelEditing = function () {
      var entity = $scope.entity, backup = entity._backup;
      for (var prop in backup) {
        if (backup.hasOwnProperty(prop)) {
          entity[prop] = backup[prop];
        }
      }

      delete entity._backup;
      $scope.edited = {};
    };

    $scope.saveEditing = function () {
      var entity = $scope.entity, backup = entity._backup;
      $scope.$emit('saveEntity', entity, backup);

      delete entity._backup; 
      $scope.edited = {};
    };

    $scope.deleteEntity = function () {
      var entity = $scope.entity;
      $scope.$emit('deleteEntity', entity);

      delete entity._backup; 
      $scope.edited = {};
    };

    $scope.$on('trackEntity', function (event, entity) {
      $scope.entity = entity;
      $scope.edited = {};
    });
  }

  editorController.$inject = ['$scope', '$controller'];
  app.controller('EditorCtrl', editorController);

指令:

function textEditor() {
    return {
      scope: {
        title: '=',
        name: '=',
        editor: '='
      },
      templateUrl: 'partials/text-editor.html',
    };
  }

  app.directive('textEditor', textEditor);

用法:

<div ng-controller="EditorCtrl">
  <div class="stack-item" ng-if="show">
    <div class="stack-item-header" ng-include="'partials/section-header.html'"></div>

    <div class="details-menu">
      <div class="details-actions">
        <div class="details-edit-actions">
          <a ng-click="saveEditing()"><span class="icon-save"></span>Save</a>
          <a ng-click="cancelEditing()"><span class="icon-cancel"></span>Cancel</a>
        </div>
      </div>
    </div>

    <div class="details">
      <fieldset>
        <legend>Medication</legend>
        <div text-editor title="'Drug Name'" name="'drugName'" editor="this"></div>
        <div text-editor title="'Dosage'" name="'dosage'" editor="this"></div>
      </fieldset>
    </div>
  </div>
</div>

验证函数超出了我们的讨论范围，它以某种方式在范围中定义......

因此，正如您所看到的，我的指令完全不可知 - 它仅用于模板，为模板提供一些附加参数。

我的同事建议了其他解决方案:

使用示例:

<div content-link>
  <a link ng-bind="case.info.caseName"></a>
  <input content ng-model="case.info.caseName" />
</div>

指令:

(function () {
    'use strict';

    angular
        .module('xx')
        .directive('contentLink', contentLink)

    contentLink.$inject = ['$compile'];
    function contentLink($compile) {
        var directive = {
            restrict: 'A',
            transclude: true,
            scope: {
                contentActive: '=?'
            },
            link: link,
            controller: ContentLinkController,
            controllerAs: 'contentLink',
            template: '<div class="cl-link" ng-hide="contentLink.contentActive" ng-click="contentLink.toggleContent()"></div>' +
                      '<div class="cl-content" ng-show="contentLink.contentActive" outside-click="contentLink.toggleContent()" outside-if="contentLink.contentActive" ' +
                            'ignore-element-selectors="{{ contentLink.otherContentElements }}">' +
                      '</div>'
        };

        return directive;

        function link(scope, element, attrs, controller, transclude) {
            angular.extend(scope.contentLink, {
                toggleContent: toggleContent
            });

            element.removeAttr('content-link').addClass('content-link');

            transclude(scope.$parent, function (clone, innerScope) {
                clone.filter('[link]').removeAttr('link').appendTo(element.find('.cl-link'));
                clone.filter('[content]').removeAttr('content').appendTo(element.find('.cl-content'));
            });

            var deregisterScopeDestroyListener = scope.$on('$destroy', cleanup);

            function toggleContent() {
                scope.contentActive = scope.contentLink.contentActive = !scope.contentLink.contentActive;

                element.toggleClass('content-active', scope.contentLink.contentActive);
            }

            function cleanup() {
                scope.contentLink = null;
                deregisterScopeDestroyListener();
            }
        }
    }

    ContentLinkController.$inject = ['$scope', '$element', '$attrs'];
    function ContentLinkController($scope, $element, $attrs) {
        var vm = this;

        angular.extend(vm, {
            contentActive: $scope.contentActive || false,
            otherContentElements: $attrs.otherContentElements
        });
    }
})();

所以看起来它并没有解决取消编辑的问题，但我的同事只是指出这是没有必要的 - 可以通过某种方式在“不同”层上完成。

因此，我们希望在大型应用程序范围内创建一些“可重用”的东西 - 从您的 Angular 来看，哪种解决方案似乎更合理？

Answer 1

我会选择第一种方法。

我对第二种方法的问题是它违背了 Angular 方法论的原则。第二种方法更接近“组件”样式，这是捆绑整个组件的其他框架(例如 ExtJs)中有效且推荐的样式。我不认为应该将样式导入到 Angular 中。

在更简单的世界观中 - 我一眼就能理解第一种方法，而我干脆放弃了第二种方法。务实的开发还意味着我们在团队中工作，可维护性和易于更改胜过其他深刻的哲学问题。

另外 - 请不要进行暴力讨论:)