用于排序属性的 ember/avoid-leaking-in-ember-objects 错误

Question

import Controller from "@ember/controller";
import { computed } from "@ember/object";

export default Controller.extend({
  filter: "",
  filterTodos: computed("arrangedContent", "filter", function() {
    var filter = this.get("filter");
    var rx = new RegExp(filter, "gi");
    var todos = this.model;
    return todos.filter(function(todo) {
      return todo.get("title").match(rx) || todo.get("body").match(rx);
    });
  }),
  sortedProperties: ["date:asc"],
  sortedTodos: computed.sort("model", "sortedProperties")
});

14:3 错误只允许字符串、数字、符号、布尔值、null、未定义和函数作为默认属性 ember/avoid-leaking-state-in-ember-objects

✖ 1 个问题（1 个错误，0 个警告）

Answer 1

查看错误，你违反了 eslint 规则ember/avoid-leaking-state-in-ember-objects。通常，您可以在谷歌上搜索规则名称并找到规则的描述以及如何修复它。这是该规则的文档：https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/avoid-leaking-state-in-ember-objects.md

正如您在该文档中所读到的，您不应该使用数组和对象作为默认属性。我会阅读那里链接的 Dockyard 博客文章，以了解为什么这是不好的做法。

在您的情况下，您使用数组作为sortedProperties 的值。根据上面链接的 eslint 规则文档，您通常可以通过在 init 上设置初始值来解决此问题。但是，排序属性是一种特殊情况 - 您可以通过查看computed.sort 的文档来查看建议的方法：https://api.emberjs.com/ember/3.4/classes/@ember%2Fobject%2Fcomputed/methods/sort?anchor=sort

如您所见，他们建议使用Object.freeze，这将防止属性被更改。所以这应该有效：

sortedProperties: Object.freeze(["date:asc"]),