如何为ngClick添加多个受notInProgress控制的执行操作？

你遇到的$parse:lval错误，本质是AngularJS的表达式解析器不允许在逻辑运算符（比如||）的右侧直接做赋值操作（像activeLeft = 4这种）——它会误以为你试图给一个非左值的对象赋值，所以抛出了这个错误。下面给你几种可行的解决方案，按推荐程度排序：

1. 推荐：把逻辑封装到控制器函数中 #

这是最符合AngularJS最佳实践的方式，把业务逻辑从模板移到控制器里，不仅代码更清晰，后期维护和测试也更方便。

首先在你的控制器里新增一个处理点击的函数：

// 适配你的控制器写法（ES6类或传统Angular控制器均可）
$ctrl.handleGrantTabClick = function() {
  // 只有当notInProgress为false时，才执行后续操作（和你原来的||逻辑等价）
  if (!$ctrl.notInProgress) {
    $ctrl.setTab('grantandpublishing');
    // 建议把activeLeft也放到控制器实例里，避免用全局/作用域变量，更符合规范
    $ctrl.activeLeft = 4;
  }
};

然后模板里直接调用这个函数：

<a data-ng-click="$ctrl.handleGrantTabClick();"></a>

2. 模板内直接处理（应急可用，不推荐） #

如果实在不想修改控制器，可以调整表达式写法，用逗号运算符连接多个操作，同时用括号确保赋值操作被正确解析，并且把逻辑换成&&（原来的notInProgress || 操作等价于!notInProgress && 操作，利用短路逻辑控制执行）：

<a data-ng-click="!notInProgress && ($ctrl.setTab('grantandpublishing'), activeLeft = 4);"></a>

这里的逗号运算符会依次执行前后的操作，括号让Angular正确识别赋值语句。不过这种写法会让模板逻辑变复杂，后期排查问题麻烦，尽量少用。

为什么原来的写法会报错？ #

你原来的代码notInProgress || activeLeft = 4中，activeLeft = 4是赋值操作，Angular的表达式解析器在处理逻辑运算符右侧时，会把它当成“表达式”而非“执行语句”，所以不允许直接赋值——它认为你在试图给一个不能被赋值的对象赋值，于是抛出了$parse:lval错误。

内容的提问来源于stack exchange，提问作者Patrick McDermott