javascript - 如何修复我的过滤器中的这个 ESlint (no-param-reassign) 错误

Question

我正在尝试仅查找具有特定服务的用户。每个用户都有一系列服务。我需要找到的匹配如下:

userService.name === service.name && !user.disabled

下面的代码有效，但在处理参数重新分配时出现 ESlint 错误。

---

export const matchUserWithService = (user, userService, service) => {
  if (userService.name === service.name && !user.disabled) {
    user.isMatched = true;
    userService.rights = service.rights;
  }

  return userService;
};

export const renderServiceAdmins = (users, selectedService) => {
  const usersWithService = users.filter((user) => {
    user.services.map(usrSrv => matchUserWithService(user, usrSrv, selectedService));
    if (user.isMatched) return user;
    return false;
  });

  return usersWithService.map(user => user.services.map(service => service.rights.map((right) => {
    if (
      service.name === selectedService.name &&
      lowCase(right.name) === 'admin' &&
      !right.disabled
    ) {
      return (
        <li key={user.email + randId()}>
          { user.name } | <span className="info_blue">{ user.email }</span>
        </li>
      );
    }
    return null;
  })));
};

这可以用 .find 重构吗？

Answer 1

我建议重组代码，因为现在它依赖于 .map 调用中的突变，这可能变得很难推理。我无法检查我建议的代码是否真的有效，但我认为它表达了你想要更清楚地做的事情的意图。

编辑。解释:简而言之，我们有一个用户列表，其中只有一些是相关的。这告诉我们，我们可能希望代码以 users.filter 开头。一般的想法是过滤掉对所选服务具有管理员权限的用户。因此，我尝试将该逻辑提取到一个函数 (userHasAdminRightsForService) 中，我将其实现为一个返回我们将要用来过滤用户的函数的函数。由于这种设计，我们得到的代码读起来更像普通英语:users.filter(userHasAdminRightsForService(selectedService)) 而不是 users.filter(user => userHasAdminRightsForService(user, selectedService)) 。

Array.prototype.some用于检查数组是否至少有一个元素满足某些条件。所以像 userService.rights.some(right => lowCase(right.name) === 'admin') 这样的行意味着我们检查 userService 中是否至少有一个权限。 rights 满足其名称应为“admin”的标准。

// given a serviceToCheckFor, return a function that checks if 
// one specific user has any userService, with admin rights, that match the name of serviceToCheckFor
export const userHasAdminRightsForService = serviceToCheckFor = user => {
  return user.services.some(userService => 
    // name check
    userService.name === serviceToCheckFor.name &&
    // check if there exists a non-disabled admin right 
    userService.rights
      .filter(right => !right.disabled)
      .some(right => lowCase(right.name) === 'admin')
    );
}; 

export const renderServiceAdmins = (users, selectedService) => {
    const adminsForSelectedService = users
      .filter(user => !user.disabled)
      .filter(userHasAdminRightsForService(selectedService))

  return adminsForSelectedService.map( admin => 
     (<li key={admin.email + randId()}>
        { admin.name } | <span className="info_blue">{ admin.email }</span>
     </li>)
  );
};