1

这对许多人来说是一个琐事，但我以前从未使用过 Angular，我需要帮助来完成一项简单的任务，因为我被困住了。

我的 ng-controller processctrl声明了一个load()方法。我的视图有两次调用ng-controller="processoctrl"，导致load()方法运行两次。processctrl还有一个属性currentPhase，它以 null 开头并load()设置其值。我可以写($scope.currentPhase || load())来防止双重load()，但是与 currentPhase 的绑定发生在转发器的顶部（在 DOM 中）。

我可以将对象属性放入 $rootScope 并从 processoctrl.scope.load() 更新它，但它会变成疯狂。

我可以简单地将 $($(".headerCurrentPhase")[0]).html($scope.currentPhase) 放入我的 load() 方法中，但这也很疯狂。

我可以重写load()到其他地方getProcesses = function(){}并$rootScope.$emit('getProcesses',{})写入控制器$rootScope.$on('getProcesses',getProcesses)以防止这种双重load()，但我认为这是多余的，那么如何简单地调用控制器函数而不是 load()？或者如何实现这个简单的任务？使用指令？

看法：

<div ng-include src="'includes/overview-header.html'"></div>
<div ng-include src="'includes/process-info.html'"></div>

include/overview-header.html的摘录：

<div class="col-md-12">
....
    <h4><strong>currentPhase</strong></h4>
    <p ng-controller="processoctrl">
       <span class="label label-primary headerCurrentPhase" ng-bind="currentPhase"></span>
    </p>
</div>

include/process-info.html的摘录：

<tbody ng-controller="processctrl">
    <tr ng-repeat="Process in Processes|orderBy:'ID'">
        <td>{{Process.isCurrent}}</td>
        <td>{{Process.isCurrent}}</td>
        <td>{{Process.ID}}</td>
        <td>{{Process.Title}}</a></td>
    </tr>
</tbody>

进程控制

(function (app) {
   app.controller('processctrl', function ($scope, $rootScope, $routeParams, factorysvc) {
       $scope.Processes = [];
       $scope.currentPhase = null;
       load();

       //($scope.currentPhase || load())

       function load() {
            var promiseGet = factorysvc.getPsby($routeParams.itemId);

            promiseGet.then(function (data) {

                $scope.Processes  = data;

                $rootScope.root.empreendimento.currentPhase = $scope.currentPhase = data[0].currentPhase.Title;

            }, function (err) {
                $scope.Message = "Error " + err.status;
            });

    }

})}(angular.module('sgaapp')));
4

2 回答 2

1

伙计，我没听懂你想做什么，但我想给你一些建议：

  1. 你的逻辑真的很复杂，你应该重新考虑一下，因为现在它很糟糕；

  2. 也包括角度很烂。真的。你根本不应该使用它。如何避免？嗯，有两个选择。

    • 首先是使用客户端路由（用于页面）；与其使用内置路由，不如使用ui-router。您将能够使用嵌套状态和路由；
    • 其次是使用指令而不是包含。真的，想象一下 - 页眉或页脚，两者都可以是指令，您只需在需要时将其包含在页面中。

对我来说是什么 - 我同时使用第一个和第二个解决方案。

  1. 你应该避免把任何东西放进去$rootScope。也许它不能杀死你，有时它是唯一的解决方案，但是，你知道，这是一种不好的做法。

  2. 如果您需要存储数据并在控制器之间共享数据，更好的选择是我们的工厂（或其他服务，但工厂最普遍）。

例子：

app.factory('MyFactory', function () {
    return {
        myData: 'some default value',
        someCommonFuction: function (a, b) {
          return a + b;
        }
    };
});

除了数据存储，您还可以将工厂用作实用程序类，具有通用功能

于 2016-03-25T05:41:18.627 回答
1

有没有真正需要有2个包括？

如果overview-header 是跨视图共享的，那么我们可以在$rootScope 级别拥有currentPhase，或者拥有一个父级控制器并拥有它。

如果overview-header 仅适用于该视图，那么我们可以将两个html 合二为一，并在包装​​标签处使用processctrl。

于 2016-03-25T04:54:16.510 回答