a instanceof Promise

해결됨
후리덤 프로필

fxjs 와 mql 를 애용하는 개발자입니다.

 

최근 사용한 어떤 모듈에서 parameter 수에 따라 비동기 처리에 대한 응답을

callback 패턴과 Promise 패턴 모두 지원하는 함수가 있었습니다.

 

go 함수를 통해 값을 다루어왔기 때문에 당연히 Promise 패턴을 생각하고 코딩을 했습니다.

그런데 예상과는 다르게 동작했고, 무엇보다 go 함수 결과에 catch 메소드 자체가 존재하지 않았습니다.

 

알고보니 해당 모듈의 함수에서 bluebird Promise 를 사용하고 있어서

fxjs 내부의 a instanceof Promise 부분에 걸려들지 않았던 것입니다.

그래서 메소드 전체를 new Promise 로 감싸고 callback 함수에서 resolve/reject 처리하여 진행하였습니다.

 

그런데 문득, Promise 값인지 아닌지를 판단하는 a instanceof Promise 부분까지도,

함수로 받게끔 할 수 있다면 어떨까 하는 생각이 들었습니다.

is_promise = a => a instanceof Promise

대신 is_promise = a => a instanceof Promise || a instanceof blue_bird_Promise

 

물론 함수를 사용할 때마다 해당 함수를 넘기는 것이 아니라,

fxjs 내부에 is_promise = a => instanceof Promise 부분은 이미 정의되어 있고,

fxjs 초기 설정시 is_promise 함수를 다른 함수로 대체하여 사용할 수 있도록 허용하는 것을 의미합니다.

유인동 프로필
유인동 5달 전

안녕하세요 :)

사실 이전에는 그래서 thenable 값을 모두 지원하기 위해서 a && typeof a.then == 'function' 같은 식으로 했었어요.

사실 함수로 만들어도 되는데요. 함수콜 하나라도 줄여주고자하는 것도 있구요.

후리덤님도 저와 같은 생각이시겠지만, 개인적으로 Promise 스펙을 지원하는 또 다른 Promise를 만드는 것을 반대하는 입장이라서 이번엔 thenable이 아니라 기본 Promise만 지원하는 방향으로 가긴 했어요. 

Promise라는 클래스에 새로운 메서드들을 정의하는 식으로 구현되고, 그 메서드들의 종류로 경쟁(?)하는 라이브러리들이 함수형 프로그래밍에 맞지 않다고 생각하거든요. 약속된 스펙만 가진 값일 때 여러 함수로 그 값을 잘 다룰 수 있으니까요. 

thenable 값을 모두 지원하지 않는 것이 "그런식으로 하는게 별로 안좋아." 라는 의견을 내고 있는 것이기도 한달까요.

근데 어쨌든 개발자가 사용하다가 '왜 안되지?' 하고 당황스러울 수 있는 부분이긴 할거 같아요. 

이 부분은 고민을 좀 해볼께요 :)

지식공유자 되기
많은 사람들에게 배움의 기회를 주고,
경제적 보상을 받아보세요.
지식공유참여
기업 교육을 위한 인프런
“인프런 비즈니스” 를 통해 모든 팀원이 인프런의 강의들을
자유롭게 학습하는 환경을 제공하세요.
인프런 비즈니스