関数Aの実行結果がnullかundefinedの時のみに関数内に変数を新たに作らず、関数Bを実行させるには？

Question

const hoge = (foo, bar) => (v) => foo(v) || bar(v);

foo(v)の結果がnullかundefinedの時にbar(v)を実行させたいので、
foo(v) || bar(v)のようにしているのですが、これだと、foo(v)の結果がfalsyな値の場合、
bar(v)が実行されてしまいます。

const zero = (v) => 0 * v;
const double = (v) => 2 * v;

console.log(
	hoge(zero, double)(10)
)

この場合だと、0はfalsyなので、20が`返って来てしまいます。

const hoge = (foo, bar) => (v) => {
	const result = foo(v)
	return result != null ? result : bar(v);
}

一応、このようにfoo(v)の結果をローカル変数に入れてから、foo(v)の結果がnullまたはundefinedではないかをチェックすれば、
期待する動きにはなるのですが、このようにローカル変数を作らないと期待する結果は得られないでしょうか？

const hoge = (foo, bar) => (v) => {
	return foo(v) != null ? foo(v) : bar(v);
}

foo(v)をこのように２回呼ぶことも出来るでしょうが、これは非効率なので避けたいところです。

const hoge = (foo, bar) => (v) => foo(v) || bar(v);

このように全て、引数に渡ってくる値のみで完結させたいのですが、何か方法があれば教えて頂ければ幸いです。

※ 不可能な場合はそれが答えだと思います。

Comments

[退会済みユーザー]

回答ではないですが、0!=nullはtrueです。
Number.isNaNか (result!==null && result!==undefine)な判定にする必要があります。

[murabito]

「0!=nullはtrue」で問題ありません。今回はfoo(v)の結果がundefinedかnullの時のみbar(v)を実行したいので大丈夫です。

[退会済みユーザー]

すみません。勘違いしてました。

Answer 1

こういうことでしょうか。

const f = (foo, bar) => v => (x => x != null ? x : bar(v))(foo(v))

f((x => 0), (x => 2 * x))(10)
// 0
f((x => null), (x => 2 * x))(10)
// 20
f((x => undefined), (x => 2 * x))(10)
// 20

Comments

[murabito]

ありがとうございます。即時実行関数使えば良かったのですね。

[think49]

この回答は x != null を別の関数に追い出す処理をしていますが、質問文のコードと比べてどのようなメリットが考えられるでしょうか。
個人的には、「関数コスト分、処理が増える」というデメリットが追加されているように読めてしまいます。

[karamarimo]

ローカル変数を使わず、foo を2回呼び出さずにやるにはどうするかという質問だったので、パズル的な問題だと思って回答しました。
確かに実用的にはメリットが無いので自分なら普通にローカル変数を使って書くと思います。

[think49]

なるほど、パズル的な問題としての回答でしたか。
高評価が6つ付いていたので、私の想定できない部分でメリットがあるのかと思って焦りました。パズル的思考なら納得です。

Answer 2

v == null;
v != null;

私はこの書き方が嫌いです。
変数には「入るべき型」を決めてから代入値を決めますが、nullが入りうるtruthyな型はObject型しかあり得ないからです。
つまり、truthy もしくは falsy で基本的に事足ります。

Re: murabito さん

Answer 3

ローカル変数が入るなぁ。

const hoge = ( foo, bar, i ) => v => (  ( i = foo( v ) ), ( i != null ? i : bar( v ) ) );
```**動くサンプル：**[https://jsfiddle.net/kuqtn37b/](https://jsfiddle.net/kuqtn37b/)