Jshint: 일반 매개변수는 기본 매개변수 다음에 오면 안 됩니다.

이 메시지는 원래 JSHint "오류"로 구현되었습니다(즉,
무시할 수 없음): gh-1779. 초기에 SyntaxError였을 수 있지만
초안은 그런 식으로 마무리되지 않았으므로 gh-2543에서 우리는 "다운그레이드"했습니다.
경고 메시지. 이것은 다음을 통해 JSHint 2.9.1에서 무시할 수 있음을 의미합니다.
-W138 .

그러나 이 경고가 적절한지 여부는 여전히 불분명합니다. NS
개인적으로 이렇게 설계된 기능은 사용하기 어렵다고 생각하지만,
잠재적인 코드 안전 문제를 식별할 수 없습니다.

@rwaldron @caitp 둘 다 이것에 대해 생각이 있습니까?

그런 종류의 서명은 redux 코드 샘플의 일부라는 것을 언급하고 싶습니다. 다음 라인을 찾으십시오.

function counter(state = 0, action) {

그래서 널리 사용되는 IMHO입니다.

이러한 종류의 패턴은 _절대적으로_ "보풀"의 정의입니다.

그래서 널리 사용되는 IMHO입니다.

나는 redux에 나타나는 패턴이 "널리 사용되는" 무언가를 나타낸다는 암시에 강력하게 동의하지 않습니다. 나는 redux를 살펴보고 counter(undefined, action) 예를 찾았고, 이 중 하나가 실제로 action 인수를 _필요로 _하거나 런타임에 직면한다는 점을 고려할 때 이것이 무엇일 수 있는지 궁금합니다. 오류. action 가 _항상_ 필수이고 state 가 선택 사항인 경우 undefined 명시적으로 전달해야 하는 require 호출은 기본 매개변수 값의 목적을 무효화합니다 .

...버그를 신고하고 싶습니다.

그러나 이 경고가 적절한지 여부는 여전히 불분명합니다.

나는 그것이 있다고 믿으며 경고를 원하지 않는 사람은 그것을 끌 수 있습니다.

이 @jugglinmike 를 닫으

@rwaldron ok, 실제로 우리는 redux에 대해 논의하지 않습니다. 그러한 서명과 함께 나타날 수 있는 오류 샘플을 제공할 수 있습니까?
나에게 그것은 단지 언어의 속성입니다. 그렇다면 "잘못"으로 표시하는 이유는 무엇입니까?

그러한 서명과 함께 나타날 수 있는 오류 샘플을 제공할 수 있습니까?

발생하게 될 유일한 런타임 오류는 counter() 및 counter(undefined) 와 같이 호출하는 것입니다. 하지만 그것은 제 요점이 아닙니다. 내 요점은 그것이 끔찍한 디자인이고 프로그래머와 그들의 도구에 과도한 부담을 준다는 것입니다. 예를 들어, 축소기는 다음을 합리적으로 분석할 수 있습니다.

function counter(action, state = 0) {
  return [action, state];
}
counter({});
counter({}, 0);
counter({}, undefined);

그리고 생산:

function c(a,s=0){return[a,s]}
c({});
c({});
c({});

반면에 기본값을 먼저 두십시오.

function counter(state = 0, action) {
  return [state, action];
}
counter(0, {});
counter(undefined, {});

다음을 생성할 것입니다:

function c(s=0,a){return[s,a]}
c(0, {});
c(undefined, {});

이것은 상당히 인위적인 예이지만 여전히 기본 매개변수를 사용하는 것이 완전히 무의미하다는 제 요점을 보여줍니다.

나에게 그것은 단지 언어의 속성입니다.

할 수 있다고 해서 반드시 해야 하는 것은 아닙니다.

그래서 현재로서는 축소기의 버그에 대해 걱정하는 것 외에는 왜 나쁜 디자인인지 설명할 수 없습니다.

잘못 설계된 호출 서명을 피하기 위해 모든 호출 사이트에서 명시적인 undefined 를 전달하도록 요구하는 것은 잘못된 관행이자 기본 매개변수의 잘못된 사용으로 간주됩니다.

이 동작은 React/Redux에서 권장합니다. React/Redux를 제거하는 것보다 jshin을 제거할 가능성이 더 큽니다./

http://redux.js.org/docs/basics/Reducers.html

한 가지 깔끔한 트릭은 ES6 기본 인수 구문을 사용하여 더 간결한 방식으로 작성하는 것입니다.

기능 todoApp(상태 = 초기 상태, 작업) {
// 지금은 어떤 작업도 처리하지 않습니다.
// 그리고 우리에게 주어진 상태를 반환합니다.
반환 상태
}

@jugglinmike wdyt ^^ ?

@txm 한번도 사용하지 않은 action 매개변수는 어떻게 되나요?

나는 여전히 가시적인 위험이 없는 한(심지어
그렇지 않으면 바람직하지 않은 패턴의 존재), JSHint는 침묵을 유지해야 합니다.
즉, @txm 의 예제가 어떻게 구별되는지 이해하는 데 문제가 있습니다.

어떻게 비활성화 할 수 있습니까?

@thalesfsp : /* jshint -W138 */ 추가 파일의 시작 부분을 추가합니다. (jshint v2.9.1 이상을 사용하고 있는지 확인하십시오)

Redux와 동일한 문제가 발생했습니다. .jshintrc 구성 파일에서 비활성화할 수 있습니까?

@derwaldgeist 물론입니다! 다음은 경고를 무시하는 .jshintrc 파일의 예입니다.

{
  "esversion": 6,
  "-W138": true
}

(그런데 JSHint의 문서 에는 특정 경고 비활성화에 대한 자세한 정보가 있습니다.)

@jugglinmike 감사합니다. .jshintrc 파일에서도 "-Wxxx" 구문을 사용할 수 있는지 몰랐습니다. 나는 항상 이것을 파일의 맨 위에 사용했습니다. 알아두면 좋아요!

'콜백'이 함수의 마지막 매개변수로 널리 사용되기 때문에 이 보푸라기 경고는 나에게 다소 어리석은 것처럼 보입니다.

openDialog(URL, 이름, 인수 = {}, 위치) {
return new Promise(function(resolve, reject) {
chrome.windows.create({
URL: URL,
유형: "팝업",
너비: pos && pos.width || 찾으시는 주소가 없습니다,
높이: pos && pos.height || 찾으시는 주소가 없습니다,
왼쪽: pos && pos.left || 찾으시는 주소가 없습니다,
상단: pos && pos.top || 찾으시는 주소가 없습니다
}, 함수(w) {

분명히 다른 매개변수를 undefined 것이 유효하며 린트 프로세스를 통과합니다. 나는 그것이 더 나은 패턴이라고 말하지는 않지만 보풀 과정을 통과합니다.