내 필터에이 ESlint (no-param-reassign) 오류를 수정하는 방법

Question

특정 서비스가있는 사용자 만 찾으려고합니다. 각 사용자는 서비스 배열을 가지고 있습니다. 작품 아래

이

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

코드,하지만 PARAM - 재 할당을 처리 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

나는 코드를 구조 조정 건의 할 것입니다. 내 제안 된 코드가 실제로 작동하는지 검사 할 수는 없지만 더 명확하게하려는 의도를 표현한다고 생각합니다.

을 편집하십시오. 설명 : 간단히 말해, 일부 사용자 만 관련성이있는 사용자 목록이 있습니다. 그러면 코드가 users.filter으로 시작되기를 원할 것입니다. 일반적인 개념은 선택된 서비스에 대한 관리자 권한을 가진 사용자를 필터링하는 것입니다. 그래서 그 논리를 하나의 함수()로 추출하려고 시도했습니다.이 함수는 사용자를 필터링하는 데 사용할 함수를 반환합니다. 이 디자인 때문에 우리는 보통 영어와 비슷한 코드를 얻습니다 : users.filter(user => userHasAdminRightsForService(user, selectedService)) 대신 users.filter(userHasAdminRightsForService(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>) 
); 
}; 

Answer 2

코드가 유효하지만 as the rule points out, 함수 매개 변수를 수정하거나 다시 지정하면 의도하지 않은 동작이 발생할 수 있습니다.

일반적으로 .eslintrc 파일에서 규칙을 완전히 비활성화하고 싶지 않지만 특정 오류 발생을 억제하려는 경우 rule-disabling comments 중 하나를 사용하십시오. 또한 사용자가 수행중인 작업을 알 수 있음을 나타냅니다.

Answer 3

어쩌면 규칙을 사용 중지하지 않고 다시 작성할 수 있습니까? 지금은 대한 이유에 꽤 어려워 질 수있는 .map 전화의 내부 돌연변이에 의존하기 때문에

export const renderServiceAdmins = (users, selectedService) => { 
    var admins = users.reduce((serviceAdmins, user) => { 
        user.services 
         .forEach((service) =>{ 
          if(service.name === selectedService.name) { 
           service.rights 
            .forEach((right)=> { 
             if(lowCase(right.name) === 'admin' && !right.disabled) { 
              serviceAdmins.concat[{ 
               name: user.name, 
               name: user.email 
              }] 
             } 
            }) 
          } 
         }); 
        return serviceAdmins; 
       }, []); 
    return admins.map(admin=> { 
     return (
      <li key={admin.email + randId()}> 
       { admin.name } | <span className="info_blue">{ admin.email }</span> 
      </li> 
     ); 
    }); 
}; 

Answer 4

우리는 이런 식으로 해결 결국, 훨씬 깨끗하고 방법을 적은 코드가 너무 (2 개 기능을 제거)! 누락 된 부분은 다음

export const renderServiceAdmins = (users, theService) => 
    users 
    .filter(user => !user.disabled) 
    .filter(user => user.services.find(
     usrSrv => (
     usrSrv.name === theService.name && 
     usrSrv.rights.find(r => r.name.toLowerCase() === 'admin' && !r.disabled) 
    ) 
    )) 
    .map(user => (
     <li key={user.email + randId()}> 
     { user.name } | <span className="info_blue">{ user.email }</span> 
     </li> 
    ));