선택적 콜백을위한 JavaScript 스타일


93

때때로 (항상은 아님) 콜백을 받아 실행하는 일부 함수가 있습니다. 콜백이 정의되어 있는지 / 기능이 좋은지 확인하고 있습니까? 아니면 더 좋은 방법이 있습니까?

예:

function save (callback){
   .....do stuff......
   if(typeof callback !== 'undefined'){
     callback();
   };
};

최신 브라우저에서는 그냥 사용할 수 typeof callback !== undefined있으므로'
Snickbrack

1
그리고 당신이 단순히 전화하면 save()? 인수가 누락되어 오류 또는 Linting 경고가 표시되지 않습니까? 아니면 완벽하게 괜찮고 콜백은 간단 undefined합니까?
João Pimentel Ferreira

답변:


139

나는 개인적으로 선호한다

typeof callback === 'function' && callback();

typeof명령은 그러나 사기이며에만 사용되어야 "undefined"하고"function"

의 문제 typeof !== undefined는 사용자가 함수가 아닌 정의 된 값을 전달할 수 있다는 것입니다.


3
typeof어리석지 않습니다. 때때로 모호하지만 나는 그것을 바보라고 부르지 않을 것입니다. 그러나 콜백을 함수로 호출하려는 경우 실제로 숫자가 아닌 함수인지 확인하는 것이 가장 좋습니다. :-)
TJ Crowder

1
@TJCrowder dodgy는 잘못된 단어 일 수 있습니다. 잘 정의되어 있지만 복싱과 "object"95 %의 시간 반환으로 인해 쓸모가 없습니다 .
Raynos 2011

1
typeof권투를 일으키지 않습니다. typeof "foo"있다 "string", 없다 "object". 실제로 문자열 프리미티브 또는 String객체를 처리하는지 여부를 알 수있는 유일한 실제 방법 입니다. (아마도 당신이있는 거 생각 Object.prototype.toString하고, 매우 편리 원인 권투,하지만하지 않습니다.)
TJ 크라우

4
그건 &&그렇고 훌륭하게 관용적 사용 .
TJ Crowder

@TJCrowder 당신은 요점이 있습니다. 문자열 프리미티브와 박스형 문자열 객체를 비교하는 값이 무엇인지 모르겠습니다. 또한 동의 .toString을 찾기위한 사랑스러운 [[Class]].
Raynos 2011

50

다음을 수행 할 수도 있습니다.

var noop = function(){}; // do nothing.

function save (callback){
   callback = callback || noop;
   .....do stuff......
};

callback몇 군데에서 를 사용하는 경우 특히 유용합니다 .

또한을 사용하는 경우 jQuery이미 $ .noop 라는 함수가 있습니다.


4
제 생각에는 이것이 가장 우아한 해결책입니다.
Nicolas Le Thierry d' Ennequin

2
동의합니다. 또한 if 조건이 없기 때문에 테스트가 더 쉬워집니다.

5
그러나 이것은 유형 문제를 해결하지 못합니다. 배열을 전달하면 어떻게됩니까? 또는 문자열?
Zerho 2016

1
단순화callback = callback || function(){};
NorCalKnockOut

1
선언에서 기본값을 지정하여 인수를 선택적으로 만들 수도 있습니다.function save (callback=noop) { ...
Keith

39

간단하게

if (callback) callback();

어떤 유형이든 상관없이 제공된 경우 콜백을 호출하는 것을 선호합니다. 자동으로 실패하지 않도록 구현자가 잘못된 인수를 전달했음을 알고 수정할 수 있습니다.


11

ECMAScript 6

// @param callback Default value is a noop fn.
function save(callback = ()=>{}) {
   // do stuff...
   callback();
}

3

콜백을 선택 사항으로 만드는 대신 기본값을 할당하고 아무리

const identity = x =>
  x

const save (..., callback = identity) {
  // ...
  return callback (...)
}

사용시

save (...)              // callback has no effect
save (..., console.log) // console.log is used as callback

이러한 스타일을 연속 전달 스타일 이라고 합니다. 다음 combinations은 배열 입력의 가능한 모든 조합을 생성 하는 실제 예 입니다.

const identity = x =>
  x

const None =
  Symbol ()

const combinations = ([ x = None, ...rest ], callback = identity) =>
  x === None
    ? callback ([[]])
    : combinations
        ( rest
        , combs =>
            callback (combs .concat (combs .map (c => [ x, ...c ])))
        )

console.log (combinations (['A', 'B', 'C']))
// [ []
// , [ 'C' ]
// , [ 'B' ]
// , [ 'B', 'C' ]
// , [ 'A' ]
// , [ 'A', 'C' ]
// , [ 'A', 'B' ]
// , [ 'A', 'B', 'C' ]
// ]

combinations연속 전달 스타일로 정의 되기 때문에 위의 호출은 사실상 동일합니다.

combinations (['A', 'B', 'C'], console.log)
// [ []
// , [ 'C' ]
// , [ 'B' ]
// , [ 'B', 'C' ]
// , [ 'A' ]
// , [ 'A', 'C' ]
// , [ 'A', 'B' ]
// , [ 'A', 'B', 'C' ]
// ]

결과로 다른 작업을 수행하는 사용자 지정 연속을 전달할 수도 있습니다.

console.log (combinations (['A', 'B', 'C'], combs => combs.length))
// 8
// (8 total combinations)

연속 통과 스타일은 놀랍도록 우아한 결과와 함께 사용할 수 있습니다.

const first = (x, y) =>
  x

const fibonacci = (n, callback = first) =>
  n === 0
    ? callback (0, 1)
    : fibonacci
        ( n - 1
        , (a, b) => callback (b, a + b)
        )
        
console.log (fibonacci (10)) // 55
// 55 is the 10th fibonacci number
// (0, 1, 1, 2, 3, 5, 8, 13, 21, 34, 55, ...)


1

나는 같은 스 니펫을 반복해서 보는 것에 너무 지쳤습니다.

  var cb = function(g) {
    if (g) {
      var args = Array.prototype.slice.call(arguments); 
      args.shift(); 
      g.apply(null, args); 
    }
  };

나는 같은 일을하는 수백 개의 기능을 가지고 있습니다.

  cb(callback, { error : null }, [0, 3, 5], true);

또는 무엇이든...

나는 전체 "기능을 확인하라"전략에 회의적이다. 유일한 합법적 인 값은 함수 또는 허위입니다. 누군가가 0이 아닌 숫자 나 비어 있지 않은 문자열을 전달하면 어떻게 하시겠습니까? 문제를 무시하면 어떻게 해결됩니까?


함수는 일부 값을 첫 번째 인수로, 함수를 첫 번째 인수로 또는 두 매개 변수로 허용하도록 오버로드 될 수 있습니다. 그것이 함수인지 확인하는 유스 케이스가 있습니다. 그것은 비 기능을 실행 오류가 발생하는 다음 잘못된 매개 변수를 무시하는 것이 낫다
Raynos

@Raynos-매우 구체적인 사용 사례입니다. 약한 유형의 JavaScript는 유형을 구별하는 데 좋지 않으며 일반적으로 유형을 구별하고 호출자가 원하는 것을 추측하는 것보다 명명 된 매개 변수를 전달하는 것이 좋습니다.save( { callback : confirmProfileSaved, priority: "low" })
Malvolio

동의하지 않습니다. 충분한 유형 검사 능력이 있습니다. 저는 메서드 오버로딩을 선호하지만 그것은 개인적인 선호입니다. 이것은 jQuery가 많이하는 일입니다.
Raynos

@Raynos- 잘못된 매개 변수를 무시한 다음 비 기능 실행에 대한 오류를 던지는 것이 더 나쁩니다 . 일반적인 경우를 고려하십시오. 라이브러리 함수가 일부 비동기 활동을 수행하고 호출 함수에 알림이 필요합니다. 따라서 정교하지 않은 사용자, 무시하고 실패하는 것은 똑같이 보입니다. 활동이 완료되지 않는 것처럼 보입니다. 그러나 정교한 사용자 (예 : 원래 프로그래머)에게 실패하면 오류 메시지와 문제를 직접 가리키는 스택 추적이 제공됩니다. 매개 변수 무시는 "아무것도"발생하지 않는 원인을 결정하기위한 지루한 분석을 의미합니다.
Malvolio 2011

그래도 절충안이 있습니다. 오류가 발생하면 런타임이 중단되지만 무시하면 주변의 다른 코드가 정상적으로 실행됩니다.
Raynos 2011

1

유효한 함수는 Function 프로토 타입을 기반으로합니다. 다음을 사용하십시오.

if (callback instanceof Function)

콜백이 함수인지 확인하려면


0

콜백을 실행하는 기준이 정의 여부와 관계없이이면 괜찮습니다. 또한 실제로 기능이 있는지 확인하는 것이 좋습니다.


0

나는 이후 커피 스크립트로 옮겼고 기본 인수가이 문제를 해결하는 좋은 방법이라는 것을 알았습니다.

doSomething = (arg1, arg2, callback = ()->)->
    callback()

0

ArgueJS쉽게 할 수 있습니다 .

function save (){
  arguments = __({callback: [Function]})
.....do stuff......
  if(arguments.callback){
    callback();
  };
};
당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.