Moment: अमान्य अवधियों का समर्थन करें

मैं सहमत हूं। शुरुआत के रूप में हमें एक वैध विधि की आवश्यकता है।

मुझे लगता है कि अमान्य अवधियों को अपवाद फेंकना चाहिए और अमान्य अवधियों की वर्तमान हैंडलिंग एक बग है जो वृद्धि नहीं है। इन मामलों पर विचार करें:

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

परिणाम क्या होगा? दस्तावेज़ परिभाषित नहीं करते कि खराब इनपुट पर क्या होता है।

इसका व्यापक प्रभाव पड़ता है, क्योंकि duration add() और subtract() बजाय duration का उपयोग किया जाता है, इसलिए इनका व्यवहार भी अपरिभाषित है:

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

मैन्युअल परीक्षण से मैं आपको बता सकता हूं कि क्या होता है: moment शून्य की अवधि का उपयोग करके "सफल" होता है।

यदि moment() ने खराब इनपुट पर केवल एक अपवाद फेंका होता, तो इन मुद्दों की तुरंत पहचान कर ली जाती। चूंकि यह चुपचाप "सफल" हुआ, इसलिए एक वर्ग गारबेज-इन, गारबेज-आउट बग बना रहा।

चूंकि टाइप की गई अवधि का व्यवहार वर्तमान में अपरिभाषित है, इसलिए खराब स्ट्रिंग्स पर अपवादों को फेंकना शुरू करना पश्चगामी संगत होगा।

यदि एक स्ट्रिंग का उपयोग किया जा रहा है जिस पर भरोसा नहीं किया जा सकता है जिसमें एक टाइपो हो सकता है, तो परीक्षण करते समय try/catch स्पष्ट रूप से उपयोग किया जा सकता है यदि स्ट्रिंग का उपयोग वैध अवधि का वर्णन करने के लिए किया जा सकता है।

+1 @ मार्कस्टोस।

एक strict विकल्प का समर्थन करने के बारे में क्या है (जैसे तारीखों के लिए क्षण ही) जो फेंकता है? पार्सिंग विफल होने पर शून्य की अवधि लौटाना काफी खतरनाक है।

एक strict विकल्प बिना किसी बदलाव के बेहतर होगा, लेकिन मुझे नहीं लगता कि अमान्य तिथियों को पार्स करना शून्य अवधि के रूप में पहली जगह में अच्छा डिफ़ॉल्ट व्यवहार है।

माना। क्या कोई संभावित परिवर्तन केवल क्षण@2.10 के साथ

शेष क्षण के समान पैटर्न का उपयोग क्यों न करें? एक _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=isValid

दस्तावेज़ीकरण से लिंक करना कोड जानबूझकर नहीं था, हालांकि कोड उल्लेख भी प्रासंगिक हैं।

+1
पार्सिंग सफल हुई या नहीं, यह पता लगाने की क्षमता की बहुत आवश्यकता है।
यदि पिछड़ी संगतता एक चिंता का विषय है तो एक विधि Duration.parse(input: string, strict?: boolean = true): Duration पेश करना संभव हो सकता है जो इनपुट गलत होने पर फेंक देगा और strict तर्क निर्दिष्ट किया गया था।
वर्तमान व्यवहार जब हमें मनमानी डेटा के लिए सभी शून्य के साथ अवधि मिल रही है तो बहुत अजीब है।

+1
एक अवधि पर एक isValid () विधि होना (यह देखने के लिए कि क्या पार्सिंग सफल थी) आसान होगा

हाय सब, मोमेंट 2.18.0 में अब अवधि के लिए .isValid विधि है। हालाँकि, यह काफी उदार है। @markstos @theazureshadow और अन्य, इसे आज़माएं और अगर आपके पास सुझाव हैं तो हमें बताएं।

@marwahaha ,

प्रारंभिक परीक्षण के रूप में, मैं ऊपर दिए गए उदाहरणों में से एक को कॉपी/पेस्ट करता हूं। अपेक्षित रूप से सही या गलत लौटने के बजाय, यह एक अपवाद फेंकता है (क्षण 2.18.0 के साथ):

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

(बिना किसी अपवाद को फेंके अमान्य अवधि अभी भी "सफल हुई", लेकिन अब यह देखने के लिए जांच कर रही है कि यह वैध है या नहीं अपवाद फेंकता है!)

हालांकि ऐसा प्रतीत होता है कि अभी तक नई विधि के लिए दस्तावेज़ नहीं हैं, इसलिए शायद यह इच्छित उपयोग नहीं है।

@markstos मैंने उस कोड को Momentjs.com के कंसोल में चलाया - जिसमें 2.18.0 है - बिना किसी समस्या के। क्या हम इसके बारे में अधिक जानकारी प्राप्त कर सकते हैं?

शायद, वह कोड अमान्य होना चाहिए क्योंकि मिनटों की वर्तनी गलत है।

21 मार्च 2017 को दोपहर 2:35 बजे, "मैगी पिंट" नोटिफिकेशन @github.com ने लिखा:

@markstos https://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
.

@maggiepint @marwahaha हां, "मिनट" की गलत वर्तनी यह है कि अवधि अमान्य क्यों होनी चाहिए।

@markstos मैं

(जाहिर है कि मार्क इस स्थिति में isValid वापस करने के लिए false था, लेकिन यह अपवाद से अलग मुद्दा है। यह मेरे लिए 100% स्पष्ट नहीं है कि हमें ऑब्जेक्ट इनपुट के साथ पार्सिंग वैधता को कैसे संभालना चाहिए।)

मैं 2.18.1 के साथ अपवाद को पुन: पेश नहीं कर सकता। शायद यह एक झूठा अलार्म था। मैं "isValid ()" के लिए "सत्य" लौटने वाली इस तरह की अमान्य अवधि होने के @butterflyhug के परिणाम को पुन: उत्पन्न

परिणाम के रूप में गलत-वैध आउटपुट के रूप में "0 मिनट" वापस करने के बजाय इस मामले में पार्सिंग विफल नहीं होनी चाहिए?

चाहिए एक अलग सवाल है, लेकिन IIRC जो करता है उसकी व्याख्या करता है
मान मिलीसेकंड के रूप में क्योंकि इकाई हैशटेबल में इकाई नहीं मिली है।

क्या यह अमान्य होना चाहिए? आईएमओ शायद।

22 मार्च, 2017 को सुबह 8:42 बजे, "मार्क स्टोसबर्ग" नोटिफिकेशन @github.com ने लिखा:

मैं 2.18.1 के साथ अपवाद को पुन: पेश नहीं कर सकता। शायद यह झूठ था
अलार्म। मैं @butterflyhug https://github.com/butterflyhug 's . को पुन: पेश
इस प्रकार की अमान्य अवधि के लिए "सत्य" लौटने का परिणाम
"यह सही है()"।

इस मामले में "0 मिनट" को a . के रूप में वापस करने के बजाय पार्सिंग विफल नहीं होनी चाहिए
परिणाम के रूप में गलत-वैध आउटपुट?

-
आप इसे प्राप्त कर रहे हैं क्योंकि आपका उल्लेख किया गया था।
इस ईमेल का सीधे उत्तर दें, इसे 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() जैसे कुछ पार्स करते हैं। मैं दृढ़ता से उम्मीद करता हूं कि हमारे पास ऐसे उपयोगकर्ता हैं जो उस पार्स को एक वैध क्षण के रूप में पसंद करते हैं, और मुझे इस उदाहरण और आपके द्वारा लिखे गए बग के बीच एक सैद्धांतिक अंतर नहीं दिखता है।

हम जिस एपीआई पर चर्चा कर रहे हैं, वह moment कंस्ट्रक्टर के लिए "ऑब्जेक्ट पार्सर" है, जिसे यहां प्रलेखित किया गया है:

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

दस्तावेज़ वर्तमान में अज्ञात या अतिरिक्त तर्क पारित करने के उपचार पर चुप है।

अभी, यहां मोमेंट के व्यवहार को "कचरा अंदर, कचरा बाहर" के रूप में वर्णित किया जा सकता है। कचरा इनपुट चुपचाप स्वीकार कर लिया जाता है और परिणाम "कचरा बाहर" होता है - गलत वर्तनी वाली तिथियों को अन्य तिथियों में परिवर्तित किया जा रहा है और वैध माना जा रहा है!

ये अपडेट डेवलपर को जल्द से जल्द यह पता लगाने की अनुमति देंगे कि उन्होंने कोई गलती की है:

• दस्तावेज़ कि एक अपवाद को अज्ञात तर्कों में फेंक दिया जाएगा, ऑब्जेक्ट कंस्ट्रक्टर को पास कर दिया जाता है।
  
  
  
  • यदि ऑब्जेक्ट कंस्ट्रक्टर को अज्ञात तर्क दिए जाते हैं, तो एक अपवाद फेंक दें
  
  

जब तक मोमेंट "गारबेज इन" को वैध के रूप में स्वीकार करना जारी रखता है, यह डेवलपर्स के लिए एक अहितकारी बना रहेगा और उपयोगकर्ताओं को मोमेंट के साथ गलत तरीके से रिपोर्ट करने के लिए छोड़ दिया जाता है कि टूटी हुई तिथियां "वैध" हैं।

यह सुधार एक "ब्रेकिंग" बदलाव के लायक लगता है।

मुझे बस इसके द्वारा काट लिया गया था: moment().subtract('1 day') ... यह अवधि व्यक्त करने का एक वैध तरीका नहीं है। मैं उम्मीद नहीं करता कि यह .subtract(0) बराबर होगा। आईएमओ, इस मामले में करने के लिए कम से कम आश्चर्यजनक बात फेंकना है।

@johnvh मैं पूरी तरह से सहमत हूं।

आप जानते हैं कि यह "अवधि व्यक्त करने का वैध तरीका" नहीं है, फिर भी आप उस कोड को वैसे भी लिखते हैं? आप क्या उम्मीद करते हैं? मुझे लगता है कि आपने काम किया है, लेखकों द्वारा प्रदान किए गए उत्कृष्ट दस्तावेज़ीकरण क्षण से अवधि व्यक्त करने का यह एक वैध तरीका नहीं था?

25/04/2017 को, 08:29 बजे, मार्क स्टोसबर्ग नोटिफिकेशन @github.com ने लिखा:

@johnvh मैं पूरी तरह से सहमत हूं।

-
आप इसे प्राप्त कर रहे हैं क्योंकि आपने इस थ्रेड की सदस्यता ली है।
इस ईमेल का सीधे उत्तर दें, इसे GitHub पर देखें, या थ्रेड को म्यूट करें।

@simonfox डेवलपर्स इंसान हैं। हम गलती करते हैं। इसलिए हमारा आधा कोड बेस टेस्ट है। मोमेंट डॉक्यूमेंटेशन /is/उत्कृष्ट है, लेकिन यह मदद नहीं करता है अगर आपको लगता है कि आपने एपीआई को सही ढंग से याद किया है और दस्तावेज़ीकरण को संदर्भित करने की आवश्यकता नहीं है। यही कारण है कि न केवल महान दस्तावेज़ीकरण बल्कि कोड भी सहायक होता है जो अमान्य इनपुट पर चुपचाप अमान्य इनपुट को टूटे हुए "वैध" इनपुट में परिवर्तित करने के बजाय विफल रहता है। यदि @johnvh को उसके कोड के चलने पर तुरंत एक सत्यापन विफलता मिली थी, तो वह यह देखने के लिए उत्कृष्ट दस्तावेज़ीकरण की जाँच कर सकता था कि समस्या क्या हो सकती है। दुर्भाग्य से, मोमेंट ने अमान्य इनपुट को मान्य माना और बिना किसी अलार्म के उठाया।

@markstos बिल्कुल। मैं 100% सहमत हूँ।

यह एक बग था जिसे ट्रैक किया जाना था। हमने इसे इस तरह नहीं लिखा था, यह जानते हुए कि यह निश्चित रूप से अमान्य था। यह सही लगता है, क्योंकि इस समय उनके समर्थित हस्ताक्षरों में हेरफेर के तरीके बहुत उदार हैं। हम उसी परियोजना में एजेंडा का भी उपयोग करते मानव-अंतराल का उपयोग करता है। तो हमारे पास कुछ ऐसा होगा जो केवल पंक्तियों को अलग करता है:

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

बाद वाला इरादा के अनुसार काम करता है, पूर्व नहीं करता है। यदि पूर्व ने अपवाद उठाया होता, तो यह हमें हमारी गलती के प्रति सचेत कर देता।

साथ ही, यह तथ्य कि Moment /has/ a isValid() विधि का तात्पर्य है कि यह मानों को मान्य या अमान्य के रूप में मान्य करता है। ऐसे मामलों में जहां अमान्य मानों को मान्य के रूप में वर्गीकृत किया जाता है, isValid() विधि दस्तावेज के रूप में काम नहीं करती प्रतीत होती है।

नमस्ते, हम इस क्षेत्र में किसी भी पीआर की समीक्षा करना पसंद करेंगे!

मैं इस बग, या इसके समान कुछ में भाग गया। अगर मैं मोमेंट 2.18.1 का उपयोग करके moment.duration('3', 'minutes').asMinutes() चलाता हूं, तो मुझे moment.duration('3', 'minutes').isValid() को false वापस करना चाहिए।

जिस तरह से मोमेंट अब व्यवहार करता है, वह बग पैदा करके बहुत परेशानी पैदा कर सकता है, जिसका पता लगाना और निदान करना मुश्किल हो सकता है, जैसा कि यहां कई लोगों ने उल्लेख किया है।

क्या इस पर ध्यान दिया गया है?

@TomJSmith एक त्वरित परीक्षण से पता चलता है कि बग अभी भी मौजूद है। यह कोड:

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

"सफल", अपेक्षानुसार एक दिन के बजाय शून्य दिन घटाना। आप परिणामी तिथि पर .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 से खुला है। यह कुछ ऐसा है जो "डेट-एफएनएस" प्रोजेक्ट सही हो जाता है। वे वास्तव में आपको बताते हैं कि क्या अमान्य इनपुट अमान्य है। https://date-fns.org/ दुर्भाग्य से, date-fns समय क्षेत्र रूपांतरणों को संभाल नहीं पाता है, लेकिन शायद बाहरी पुस्तकालय वाले लोगों को संभालने का एक तरीका है।

मोमेंट प्रोजेक्ट के लिए इसे ठीक करने के लिए, उन्हें कचरा इनपुट स्वीकार करना और इसे एक वैध तिथि के रूप में मानना ​​​​छोड़ना होगा। हालांकि यह एक बेहतर व्यवहार की तरह लगता है, यह तकनीकी रूप से एक "पिछड़ा तोड़ने वाला व्यवहार" परिवर्तन है, इसलिए ऐसा लगता है कि परिवर्तन करने के लिए प्रतिरोध है। लेकिन यह देखते हुए कि परियोजना ने पिछले लगभग चार वर्षों में कार्रवाई नहीं की है, मैं जल्द ही ठीक होने की उम्मीद नहीं करता।

क्या इस पर कोई अपडेट हैं?

moment 2.23.0 के साथ isValid() फ़ंक्शन भी true लौटाता है जब एक अमान्य ISO8601 स्ट्रिंग से अवधि बनाने का प्रयास किया जाता है।

उदाहरण:

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

इसकी तरह कष्टप्रद इमो। और Duration ऑब्जेक्ट्स पर isValid() फ़ंक्शन के उद्देश्य को पूरी तरह से हरा देता है (क्या ऐसा कोई मामला भी है, जब isValid() false लौटाता है क्योंकि पल एक अमान्य भी व्याख्या करता है इनपुट?)

@robbiecloset अपडेट अप्रैल में मेरी आखिरी पोस्ट के बाद से है, "डेट-एफएनएस" प्रोजेक्ट समय क्षेत्र समर्थन पर बेहतर हो गया है। इसमें मदद के लिए अब "date-fns-timezone" और "date-fns-tz" प्रोजेक्ट हैं।

यदि स्विच करना संभव है तो मैं इसका उपयोग करने पर विचार करूंगा।

मैं moment.duration(value).toISOString() === value उपयोग करता हूं, वह मान कॉन्फ़िगरेशन से पढ़ा जाता है।

मैं क्षण का उपयोग करता हूं। अवधि (मान)। toISOString() === मान, वह मान कॉन्फ़िगरेशन से पढ़ा जाता है।

@ बोर्स-लि मुझे लगता है कि समस्या का एक हिस्सा यह है कि पल दो मानों को डीकोड करने में सक्षम हो सकता है जो समान एन्कोडेड मान उत्पन्न करते हैं।

उदाहरण के लिए: moment.duration('PT0M0S').toISOString() === moment.duration('PT0S').toISOString() ।

यदि आप परीक्षणों को देखते हैं, तो ऐसा लगता है कि अमान्य मान वास्तव में विनिर्देश से अलग हैं:

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

पैटर्न जो आईएसओ अवधि मानक से मेल नहीं खाते हैं उनका परीक्षण "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 है (या कम से कम अमान्य होने की संभावना है)