Underscore: 1.9.0 回帰。 空の配列で first または last を呼び出すと、undefined が返されます。

作成日 2018年04月19日  ·  9コメント  ·  ソース: jashkenas/underscore

1.9.0 では、 _.first([], 10)undefined代わりに[] undefined返します。

これは大きな問題だと思います :stuck_out_tongue:

bug fixed

最も参考になるコメント

よし! Underscore 1.9.1 が公開され、このバグ修正が含まれています。

全てのコメント9件

うわ…これは意図した変更だったように見える… https://github.com/jashkenas/underscore/pull/2513

個人的には、 _.first([1,2,3], 2)[1,2]ことから、 _.first([], 2)[]と期待すると思います。

明示的に指定されたアイテムの数で呼び出された場合、それは私には厄介な小さな回帰のように見えます。 他の意見は? @jridgewell?

そうは言っても、 _.first([], 1)はどうあるべきかという疑問が生じます。

おっと、私の側のレビューが悪い。 n引数を渡すと、常に配列が返されます。

心配いりません。1.9.1 は、発生した他のクリーンアップで実行できます — 加えて、他の未解決の PR のいくつかを解決するかもしれません...

グラシアス。 これは今日も私たちを噛んだ。

これは、私の場合、より大きなアプリのさまざまな場所で、エッジケースで非常にあいまいなバグにつながります。

優先すべきではないでしょうか。

これも私たちを噛んだ。 幸いなことに、実際に公開する前に、いくつかの単体テストでキャッチされました。 早急に直していただけるとありがたいです。

よし! Underscore 1.9.1 が公開され、このバグ修正が含まれています。

ありがとう@jashkenas

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

xiaoliwang picture xiaoliwang  ·  3コメント

marcalj picture marcalj  ·  5コメント

sky0014 picture sky0014  ·  8コメント

umarfarooq125 picture umarfarooq125  ·  8コメント

githublyp picture githublyp  ·  3コメント