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 기간을 0으로 사용하여 "성공"합니다.

moment() 가 잘못된 입력에 대해 단순히 예외를 throw했다면 이러한 문제는 즉시 식별되었을 것입니다. 조용히 "성공"했기 때문에 클래스 Garbage-In, Garbage-Out 버그가 지속되었습니다.

오타가 있는 기간의 동작은 현재 정의되지 않았으므로 잘못된 문자열에 대한 예외를 throw하기 시작하면 이전 버전과 호환됩니다.

오타가 있을 수 있는 신뢰할 수 없는 문자열을 사용하는 경우 문자열이 유효한 기간을 설명하는 데 사용할 수 있는지 테스트할 때 try/catch 를 명시적으로 사용할 수 있습니다.

+1 @markstos.

던지는 strict 옵션(날짜에 대한 순간 자체와 같은)을 지원하는 것은 어떻습니까? 구문 분석이 실패할 때 기간을 0으로 반환하는 것은 매우 위험합니다.

strict 옵션은 변경하지 않는 것보다 낫지 만 유효하지 않은 날짜를 0 기간으로 구문 분석하는 것이 처음에는 좋은 기본 동작이라고 생각하지 않습니다.

동의. 가능한 변경은 [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=isValid

코드 언급도 관련이 있지만 코드가 아닌 문서에 링크하는 것은 의도적이었습니다.

+1
파싱의 성공 여부를 알 수 있는 능력이 매우 필요합니다.
이전 버전과의 호환성이 우려되는 경우 입력이 올바르지 않고 strict 인수가 지정된 경우 throw되는 Duration.parse(input: string, strict?: boolean = true): Duration 메서드를 도입할 수 있습니다.
임의의 데이터에 대해 모두 0인 Duration을 얻을 때의 현재 동작은 매우 이상합니다.

+1
기간에 isValid() 메서드가 있으면(파싱이 성공했는지 확인하기 위해) 편리합니다.

안녕하세요, Moment 2.18.0 에는 이제 지속 시간에 대한 .isValid 메서드가 있습니다. 그러나 꽤 관대합니다. @markstos @theazureshadow 및 기타 사용자, 사용해 보고 제안 사항이 있으면 알려주십시오.

@마르와하하 ,

초기 테스트로 위의 예제 중 하나를 복사/붙여넣기했습니다. 예상대로 true 또는 false를 반환하는 대신 예외가 발생합니다(모멘트 2.18.0 포함).

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

(예외를 던지지 않고 유효하지 않은 기간을 생성했지만 여전히 "성공"하지만 유효한지 확인하면 예외가 발생합니다! )

아직 새 방법에 대한 문서가 없는 것으로 보이지만 이것이 의도된 용도가 아닐 수도 있습니다.

@markstos 2.18.0이 있는 Momentjs.com 의 콘솔에서 문제 없이 해당 코드를 방금 실행했습니다. 자세한 내용을 알아볼까요?

분의 철자가 틀리기 때문에 해당 코드가 유효하지 않을 수 있습니다.

2017년 3월 21일 오후 2시 35분에 "Maggie Pint" [email protected]이 다음과 같이 썼습니다.

@markstos https://github.com/markstos 방금 해당 코드를 실행했습니다.
2.18.0이 있는 Momentjs.com의 콘솔에는 문제가 없습니다. 우리가 얻을 수 있습니까?
그것에 대한 자세한 내용은?

—
당신이 언급되었기 때문에 이것을 받는 것입니다.
이 이메일에 직접 답장하고 GitHub에서 확인하세요.
https://github.com/moment/moment/issues/1805#issuecomment-288176838 또는 음소거
스레드
https://github.com/notifications/unsubscribe-auth/ACbGmWIf24KlRs8oiS2wTn206iJNMod7ks5roBhfgaJpZM4CRuVM
.

기간은 무효로해야하는 이유를 예 @marwahaha @maggiepint, "분"의 맞춤법입니다.

@markstos Momentjs.com 의 콘솔에서도 해당 예외를 재현할 수 없습니다. (지금은 2.18.1입니다.) isValid를 호출할 때 여전히 예외를 재현할 수 있다면 더 자세한 정보를 얻을 수 있습니까?

(분명히 Mark는 이 상황에서 isValid 가 false isValid 를 반환하기를 원했지만 이는 예외와는 별개의 문제입니다. 객체 입력으로 구문 분석 유효성을 처리하는 방법이 100% 명확하지 않습니다.)

2.18.1에서도 예외를 재현할 수 없습니다. 아마도 그것은 잘못된 경보였을 것입니다. "isValid()"에 대해 "true"를 반환하는 이러한 종류의 유효하지 않은 기간을 갖는 @butterflyhug 의 결과를 재현 할 수 있습니다 .

이 경우 결과로 "0분"을 잘못 유효한 출력으로 반환하는 대신 구문 분석이 실패하지 않아야 합니까?

해야 하는 것은 다른 질문이지만 IIRC가 하는 일은 다음과 같이 해석하는 것입니다.
단위 해시 테이블에서 단위를 찾을 수 없기 때문에 값을 밀리초로 표시합니다.

무효라고 해야 할까요? 아마 IMO.

2017년 3월 22일 오전 8시 42분에 "Mark Stosberg" [email protected]이 작성했습니다.

2.18.1에서도 예외를 재현할 수 없습니다. 아마도 거짓이었을 것이다.
경보. @butterflyhug https://github.com/butterflyhug 's를 재현 할 수 있습니다.
이러한 종류의 유효하지 않은 기간이 "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 mintues'를 '0 minutes'로 변환하고 결과를 "valid"로 선언하는 것은 버그입니다. 이를 수정하기 위한 브레이킹 체인지일 수 있지만 여전히 버그 수정입니다.

이 유효성 검사 동작이 이상적이지 않다는 데 동의하지만 이것이 반드시 버그 라는 데에는 동의하지 않습니다.

moment({'minutes': 3, '$cacheKey': 92619502}).isValid() 와 같은 것을 구문 분석하는 경우를 고려하십시오. 나는 그 구문 분석을 유효한 순간으로 평가하는 사용자가 있을 것으로 강력히 기대하며, 이 예제와 귀하가 작성한 버그 사이에 원칙적인 차이가 없다고 봅니다.

우리가 논의하고 있는 API는 moment 생성자를 위한 "객체 파서"이며 여기에 문서화되어 있습니다.

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

문서는 현재 알 수 없거나 추가 인수를 전달하는 처리에 대해 침묵하고 있습니다.

지금 여기에서 Moment의 동작은 "Garbage In, Garbage out"으로 설명할 수 있습니다. 가비지 입력은 자동으로 수락되고 결과는 "가비지 아웃"입니다. 철자가 잘못된 날짜는 다른 날짜로 변환되고 유효한 것으로 간주됩니다!

이러한 업데이트를 통해 개발자는 가능한 한 빨리 자신이 실수를 했음을 알 수 있습니다.

• 알 수 없는 인수에서 예외가 발생한다는 문서는 개체 생성자에 전달됩니다.
  
  
  
  • 알 수 없는 인수가 개체 생성자에 전달되면 예외를 throw합니다.
  
  

Moment가 계속해서 "Garbage In"을 유효한 것으로 받아들이는 한 개발자에게는 계속해서 손해가 될 것이며 사용자는 깨진 날짜가 "유효"하다고 Moment가 잘못 보고하게 됩니다.

이 개선 사항은 "중단" 변경할만한 가치가 있는 것 같습니다.

나는 이것에 물렸습니다. moment().subtract('1 day') ... 이것은 지속 시간을 표현하는 유효한 방법이 아닙니다. 그래도 .subtract(0) 와 같을 것으로 기대하지는 않습니다. IMO, 이 경우 가장 놀라운 일은 던지는 것입니다.

@johnvh 전적으로 동의합니다.

그것이 "기간을 표현하는 유효한 방법"이 아니라는 것을 알고 있지만 어쨌든 그 코드를 작성합니까? 당신은 무엇을 기대합니까? 나는 그것이 momentjs 작성자가 제공하는 훌륭한 문서에서 지속 시간을 표현하는 유효한 방법이 아니라는 것을 알아냈다고 가정합니다.

2017년 4월 25일 08:29에 Mark Stosberg [email protected] 은 다음과 같이 썼습니다.

@johnvh 전적으로 동의합니다.

—
이 스레드에 가입했기 때문에 이 메시지를 받고 있습니다.
이 이메일에 직접 답장하거나 GitHub에서 보거나 스레드를 음소거하세요.

@simonfox 개발자는 인간입니다. 우리는 실수를 합니다. 이것이 우리 코드 기반의 절반이 테스트인 이유입니다. Moment 문서는 /is/ 우수하지만 API를 올바르게 기억하고 문서를 참조할 필요가 없다고 생각하면 도움이 되지 않습니다. 그렇기 때문에 잘못된 입력을 깨진 "유효한" 입력으로 자동 변환하는 대신 훌륭한 문서뿐만 아니라 잘못된 입력에 실패하는 코드도 있으면 도움이 됩니다. @johnvh 가 코드가

@markstos 정확히. 100% 동의합니다.

이것은 추적해야 하는 버그였습니다. 물론 무효임을 알면서도 그렇게 쓰지는 않았습니다. 순간 조작 방법이 지원되는 서명에서 매우 자유주의적이기 때문에 옳은 것처럼 보입니다. 우리는 또한 자연어를 지속 시간으로 변환하기 위해 인간 간격 을 사용하는 동일한 프로젝트에서 의제 를 사용합니다. 그래서 우리는 다음과 같은 것을 가질 것입니다:

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

후자는 의도한 대로 작동하지만 전자는 작동하지 않습니다. 전자가 예외를 일으켰다면 우리의 실수를 경고했을 것입니다.

또한 Moment /has/ isValid() 메소드가 값을 유효하거나 무효로 검증한다는 것을 의미합니다. 유효하지 않은 값이 유효한 것으로 분류되는 경우 isValid() 메소드가 문서화된 대로 작동하지 않는 것으로 보입니다.

안녕하세요, 우리는 이 분야의 모든 PR을 검토하고 싶습니다!

이 버그 또는 이와 유사한 문제가 발생했습니다. Moment 2.18.1을 사용하여 moment.duration('3', 'minutes').asMinutes() 를 실행하면 결과가 0이 됩니다. 3을 반환하거나 예외를 throw해야 한다고 생각합니다. 최소한 moment.duration('3', 'minutes').isValid() 는 false 반환해야 합니다.

Moment가 지금 행동하는 방식은 여기 여러 사람들이 언급했듯이 감지 및 진단하기 어려울 수 있는 버그를 유발하여 많은 문제를 일으킬 수 있습니다.

이 문제가 해결되었습니까?

@TomJSmith 빠른 테스트는 버그가 여전히 존재함을 보여줍니다. 이 코드:

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

"성공"하고 예상대로 하루 대신 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 

좀 짜증나네요. 완전히의 목적 패배 isValid() 에 기능 Duration 경우라도,이 오브젝트 (시 isValid() 반환 false 순간 심지어 잘못된 해석 않기 때문에 입력?)

@robbiecloset 업데이트는 4월의 마지막 게시물 이후로 "date-fns" 프로젝트가 시간대 지원에서 더 좋아졌습니다. 이제 이를 돕기 위한 "date-fns-timezone" 및 "date-fns-tz" 프로젝트가 있습니다.

전환이 가능하다면 사용을 고려하겠습니다.

나는 moment.duration(value).toISOString() === value , 그 값은 구성에서 읽습니다.

나는 moment.duration(value).toISOString() === value를 사용합니다. 그 값은 구성에서 읽습니다.

@bors-ltd 내 생각에 문제의 일부는 동일한 인코딩 값을 생성하는 두 값을 디코딩할 수 있다는 것입니다.

예: 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(또는 적어도 유효하지 않을 가능성이 있음)임을 보장하는 메소드로 대체하는 것입니다.