Moment: 無効な期間をサポートする

同意します。 まず、isValidメソッドが必要です。

無効な期間は例外をスローする必要があると思います。無効な期間の現在の処理はバグであり、機能拡張ではありません。 次の場合を考慮してください。

`var wrong = moment.duration(3,'mintues');`

結果はどうなりますか？ ドキュメントは、悪い入力で何が起こるかを定義していません。

add()とsubtract()代わりにdurationが使用されるため、これにはカスケード効果があります。したがって、これらにも未定義の動作があります。

var hmm = moment().subtract(3,'mintues').toDate();
var uhoh = moment().add(3,'mintues').toDate();

手動テストから、何が起こるかを知ることができます。 momentは、ゼロの期間を使用することで「成功」します。

moment()が不正な入力に対して単に例外をスローした場合、これらの問題はすぐに特定されます。 サイレントに「成功」​​したため、Garbage-In、Garbage-Outクラスのバグが解決しませんでした。

タイプミスされた期間の動作は現在定義されていないため、不正な文字列で例外をスローし始めると、下位互換性があります。

タイプミスの可能性がある信頼されていない文字列が使用されている場合、その文字列を使用して有効な期間を記述することができるかどうかをテストするときに、 try/catchを明示的に使用できます。

+ 1 @ markstos。

スローするstrictオプション（日付のモーメント自体など）をサポートするのはどうですか？ 解析が失敗したときにゼロの期間を返すことは非常に危険です。

strictオプションは変更しないよりはましですが、最初から期間がゼロであることがデフォルトの動作として適切であるため、無効な日付を解析することはないと思います。

同意しました。 可能な変更は[email protected]でのみ着陸し

残りの瞬間と同じパターンを使用してみませんか？ _isValidフィールドとisValid（）メソッドを用意し、解析が失敗した場合は期間を無効に設定します。

参考までに、isValid（）が使用されている他の文書化された場所は次のとおりです。

https://github.com/moment/momentjs.com/search?utf8=%E2%9C%93&q=isValid

訂正^ https：//github.com/moment/moment/search？utf8 =％E2％9C％93＆ q =

コードではなくドキュメントへのリンクは意図的なものでしたが、コードの言及も関連しています。

+1
解析が成功したかどうかを確認する機能が非常に必要です。
下位互換性が懸念される場合は、入力が正しくなく、 strict引数が指定された場合にスローされるメソッドDuration.parse(input: string, strict?: boolean = true): Durationを導入できる可能性があります。
任意のデータに対してすべてゼロのDurationを取得しているときの現在の動作は、非常に奇妙です。

+1
（解析が成功したかどうかを確認するために）期間にisValid（）メソッドがあると便利です

みなさん、こんにちは.isValidメソッドがあります。 しかし、それはかなり寛大です。 @markstos @theazureshadowなど、ぜひお試しいただき、ご提案があればお知らせください。

@marwahaha 、

最初のテストとして、上記の例の1つをコピーして貼り付けました。 期待どおりにtrueまたはfalseを返す代わりに、例外をスローします（モーメント2.18.0）。

 moment.duration(3,'mintues').isValid();

（作成された無効な期間は例外をスローせずに「成功」​​しますが、有効かどうかを確認すると例外がスローされます！）

新しいメソッドのドキュメントはまだないようですが、おそらくこれは意図された使用法ではありません。

@markstos Momentjs.com（2.18.0）のコンソールでそのコードを問題なく実行しました。 詳細を教えてもらえますか？

おそらく、議事録のつづりが間違っているため、そのコードは無効であるはずです。

2017年3月21日14:35、「MaggiePint」 [email protected]は次のように書いています。

@markstoshttps ：//github.com/markstos私はちょうどそのコードをで実行しました
Momentjs.comのコンソール（2.18.0あり）は問題ありません。 もらえますか
それについての詳細は？

—
あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/moment/moment/issues/1805#issuecomment-288176838 、またはミュート
スレッド
https://github.com/notifications/unsubscribe-auth/ACbGmWIf24KlRs8oiS2wTn206iJNMod7ks5roBhfgaJpZM4CRuVM
。

期間は無効であるべき理由はい@marwahaha @maggiepint、「分」のスペルミスです。

@ markstosMomentjs.comのコンソールでもその例外を再現することはできません。 （ただし、現在は2.18.1です。）isValidを呼び出したときに例外を再現できる場合は、詳細を取得できますか？

（明らかに、マークはこの状況でisValidがfalse isValidを返すことを望んでいましたが、それは例外とは別の問題です。オブジェクト入力で有効性の解析をどのように処理するかは私には100％明白ではありません。）

2.18.1でも例外を再現できません。 おそらくそれは誤警報でした。 この種の無効な期間が「isValid（）」に対して「true」を返すという@butterflyhugの結果を再現できます。

この場合、結果として誤って有効な出力として「0分」を返す代わりに、解析が失敗するべきではありませんか？

別の質問である必要がありますが、IIRCが行うことはそれを解釈することです
ユニットがユニットハッシュテーブルに見つからないため、ミリ秒単位の値。

無効にする必要がありますか？ おそらくIMO。

2017年3月22日午前8時42分、「MarkStosberg」 [email protected]は次のように書いています。

2.18.1でも例外を再現できません。 おそらくそれは誤りでした
警報。 @butterflyhughttps ：//github.com/butterflyhugを再現できます
この種の無効な期間が「true」を返す結果
「isValid（）」。

この場合、「0分」をとして返す代わりに、解析が失敗するべきではありません。
結果として誤って有効な出力？

—
あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/moment/moment/issues/1805#issuecomment-288440827 、またはミュート
スレッド
https://github.com/notifications/unsubscribe-auth/AFxi0nZAz8gwAJM8fVx5rYPyIWjeEfHMks5roUF4gaJpZM4CRuVM
。

ここにバグがあるようです：

https://github.com/moment/moment/blob/497f918515ae6ab900f008f19523b1e4ae5e2450/src/lib/duration/create.js#L34

文字列が有効であり、「期間」マップに表示されていると想定されます。 修正は、文字列が設定する有効な値であることを確認するために、ここにチェックを追加することのようです。 そうでない場合は、 _isValid:false

この動作は、標準のisValidメソッドと一致しています。

moment({'mintues': 3}).isValid()
> true

このソースはnormalizeObjectUnits https://github.com/moment/moment/blob/497f918515ae6ab900f008f19523b1e4ae5e2450/src/lib/units/aliases.js#L14 -L29で、有効な属性のみを追加します（そして問題を削除します）もの）。

@maggiepint @ichernevこの動作を変更する必要がありますか？ 間もなく最新リリースを行います...

「3分」を「0分」に変換し、結果を「有効」と宣言するのはバグです。 それを修正することは重大な変更かもしれませんが、それでもバグ修正です。

この検証動作は理想的ではないことに同意しますが、これが必ずしもバグであることに同意しません。

moment({'minutes': 3, '$cacheKey': 92619502}).isValid()ようなものを解析する場合を考えてみましょう。 この解析を有効な瞬間として評価してくれるユーザーがいることを強く期待しています。この例とあなたが書いたバグとの間に原則的な違いは見られません。

ここで説明しているAPIは、 momentコンストラクターの「オブジェクトパーサー」です。これについては、以下で説明しています。

http://momentjs.com/docs/#/parsing/object/

ドキュメントは現在、未知のまたは追加の引数を渡すことの扱いについて沈黙しています。

現在、ここでのMomentの動作は、「Garbage In、Garbageout」として説明できます。 ガベージ入力はサイレントに受け入れられ、結果は「ガベージアウト」になります。スペルミスのある日付は他の日付に変換され、有効と見なされます。

これらの更新により、開発者は間違いを犯したことをできるだけ早く見つけることができます。

• 不明な引数で例外がスローされることを文書化して、オブジェクトコンストラクターに渡します。
  
  
  
  • 不明な引数がオブジェクトコンストラクターに渡された場合は、例外をスローします
  
  

Momentが「GarbageIn」を有効として受け入れ続ける限り、それは開発者にとって不利益であり続け、ユーザーは、壊れた日付が「有効」であると誤って報告するMomentを残されます。

この改善は、「破壊的な」変更の価値があるようです。

私はこれに噛まれました： moment().subtract('1 day') ...これは期間を表す有効な方法ではありません。 ただし、 .subtract(0)と同等になるとは思いません。 IMO、この場合に行う最も驚くべきことは投げることです。

@johnvh私は完全に同意します。

それが「期間を表現する有効な方法」ではないことを知っていますが、とにかくそのコードを記述しますか？ 何を期待しますか？ 私はあなたがそれがmomentjsの作者が提供する優れたドキュメントから期間を表現するための有効な方法ではなかったことを理解したと思いますか？

25/04/2017では、午前8時29分で、マークStosbergの[email protected]は書きました：

@johnvh私は完全に同意します。

—
このスレッドにサブスクライブしているため、これを受け取っています。
このメールに直接返信するか、GitHubで表示するか、スレッドをミュートしてください。

@simonfoxの開発者は人間です。 私たちは間違いを犯します。 そのため、コードベースの半分はテストです。 Momentのドキュメントは/ is /優れていますが、APIを正しく覚えていて、ドキュメントを参照する必要がないと思われる場合は役に立ちません。 そのため、優れたドキュメントだけでなく、無効な入力を壊れた「有効な」入力にサイレントに変換するのではなく、無効な入力で失敗するコードを用意しておくと便利です。 @johnvhがコードの実行時に検証の失敗をすぐに受け取った場合、彼は優れたドキュメントをチェックして、問題が何であるかを確認できたはずです。 残念ながら、Momentは無効な入力を有効なものとして扱い、アラームなしに発生しました。

@markstos正確に。 私は100％同意します。

これは追跡しなければならないバグでした。 もちろん無効であることを知って、そのように書いたわけではありません。 モーメント操作メソッドは、サポートされているシグニチャで非常に自由であるため、正しく見えます。 また、同じプロジェクトでアジェンダを使用します。このプロジェクトでは、自然言語を期間に変換するために人間の間隔を使用します。 したがって、このようなものは1行だけ離れています。

moment().subtract('1 day');
agenda.processEvery('1 day');

後者は意図したとおりに機能しますが、前者は機能しません。 前者が例外を提起した場合、それは私たちの過ちを警告していたでしょう。

また、Moment / has / isValid（）メソッドは、値を有効または無効として検証することを意味します。 無効な値が有効として分類される場合、 isValid()メソッドは文書化されているように機能しないように見えます。

みなさん、こんにちは。この分野のPRを確認したいと思います。

私はこのバグ、またはそれに類似したものに遭遇しました。 Moment2.18.1を使用してmoment.duration('3', 'minutes').asMinutes()を実行すると、結果は0になります。3を返すか、例外をスローする必要があると思います。 少なくとも、 moment.duration('3', 'minutes').isValid()はfalseを返す必要があります。

ここで何人かの人々が言及しているように、Momentの現在の動作は、検出と診断が難しいバグを引き起こすことによって多くの問題を引き起こす可能性があります。

これは対処されましたか？

@TomJSmith簡単なテストでは、バグがまだ存在していることが示されています。 このコード：

 moment().subtract('1 day');

「成功」し、予想どおり1日ではなく0日を減算します。 結果の日付に.isValid()を呼び出して、結果が有効でない場合でも結果が有効であることを示すtrue応答を返すことができます。

現在の動作は、一部のユースケースで悪い結果につながる可能性があるため、警告とともに明確に文書化する必要があります。

使用事例
保持期間に基づいてリソースを削除したい：

const duration = moment.duration('invalid');
const currentDate = moment();
const removeBeforeDate  = moment().subtract(duration);

console.log(`currentDate      : ${currentDate}`);
console.log(`removeBeforeDate : ${removeBeforeDate}`);

結果：

currentDate      : Fri Apr 13 2018 13:15:04 GMT+0200
removeBeforeDate : Fri Apr 13 2018 13:15:04 GMT+0200

結果：
現在の日付までのすべてのリソースが削除されます...

これまでのところ、解析後に値をテストする場合にこれを回避するために私が見つけた唯一の方法は次のとおりです。

const duration = moment.duration('invalid');
if (duration.toISOString() === 'P0D') {
    // throw an Error
}

はい、このバグはひどく、2014年から開いています。これは「date-fns」プロジェクトが正しく行うものです。 実際には、無効な入力が無効かどうかがわかります。 https://date-fns.org/残念ながら、date-fnsはタイムゾーン変換を処理しませんが、外部ライブラリを使用してそれらを処理する方法があるかもしれません。

Momentプロジェクトでこれを修正するには、ガベージ入力の受け入れを終了し、それを有効な日付として扱う必要があります。 これはより良い動作のように見えますが、技術的には「後方破壊動作」の変更であるため、変更を行うことには抵抗があるようです。 しかし、このプロジェクトが過去4年間行動を起こしていないことを考えると、すぐに修正されるとは思いません。

これに関する更新はありますか？

moment 2.23.0を使用すると、無効なISO8601文字列から期間を作成しようとすると、 isValid()関数もtrue返します。

例：

const mom = moment.duration('asdf')
console.log(mom.isValid()) // This returns true, expected would be false 

そのちょっと迷惑なイモ。 そして、 DurationオブジェクトのisValid()関数の目的を完全に無効にします（ isValid()がfalse返す場合もあります。これは、モーメントが無効なものでさえ解釈するためです。入力？）

@robbiecloset更新は、4月の前回の投稿以降、「date-fns」プロジェクトのタイムゾーンサポートが改善されました。 現在、これを支援する「date-fns-timezone」および「date-fns-tz」プロジェクトがあります。

切り替えが可能な場合は、使用を検討します。

moment.duration(value).toISOString() === valueを使用します。その値は、構成から読み取られます。

moment.duration（value）.toISOString（）=== valueを使用します。その値は、構成から読み取られます。

@ bors-ltd問題の一部は、同じエンコードされた値を生成する2つの値をモーメントがデコードできる可能性があることだと思います。

例： moment.duration('PT0M0S').toISOString() === moment.duration('PT0S').toISOString() 。

テストを見ると、無効な値は実際には仕様の一部であるように見えます。

https://github.com/moment/moment/blob/2.24.0/src/test/moment/duration.js#L419 -L420

ISO期間標準に一致しないパターンは、「0」秒としてテストされています。

こんにちは、 isValid()メソッドでも同じ問題が発生し、手動で実行することになりました。
const isValidDuration = duration => { return !!duration.match( /^(-?)P(?=\d|T\d)(?:(\d+)Y)?(?:(\d+)M)?(?:(\d+)([DW]))?(?:T(?:(\d+)H)?(?:(\d+)M)?(?:(\d+(?:\.\d+)?)S)?)?$/ ); };
それが誰にでも役立つことを願っています。

https://momentjs.com/docs/#/ -project-status /を参照して

ここでのすべての議論に感謝します。

ハッキーな修正は、isValidメソッドを、データ部分の合計が0より大きい（または少なくとも無効である可能性が高い）ことを保証するメソッドに置き換えることです。