Moment: 특정 dow / doy 콤보에 대한 잘못된`weekOfYear`

이견있는 사람? 이것은 요일이 일요일이나 월요일이 아닌 많은 로케일에 영향을 미칩니다. 버그가 확인되면 더 나은 weekOfYear 계산을 위해 pull 요청을 제출해야합니까? 아니면 현재 알고리즘을 수정할 계획이 이미 있습니까?

나는 doy 설정이 나를 매우 혼란스럽게한다는 것을 인정해야합니다. { dow: 6, doy: 12 } 는 주가 토요일에 시작하고 1 주가 1 월 1 일이있는 주임을 의미합니다. 내가 올바르게 이해한다면 {dow:6, doy: 1} 는 첫 주에 12 월 26 일이 포함된다는 것을 의미합니다.

pull 요청에 { dow: 6, doy: 12 } 테스트를 추가했습니다.

{ dow: 6, doy: 1 } 를) 의미합니까? 7보다 큰 doy 값은 실제 의미가없는 것 같습니다 .. 오타 였나요?

아니요, {dow: 6, doy: 12} . doy 는 1 월 첫째 주에 해당하는 날이 _ 아닙니다 _. {dow: 6, doy: 12} 는 아랍어 국가의 표준입니다.주는 토요일에 시작하고 첫 번째주는 1 월 1 일을 포함하는 주입니다.

버그는 실제로 날짜로부터의주의 계산에있는 것이 아니라 단지 다른 방향이었습니다.

우와, 그렇다면 지금까지 doy 오해하고 계셨군요 ..

실제로 문서 에 따르면 localeData.firstDayOfYear() 는 0에서 15까지의 정수를 반환 할 수 있지만 나에게는 너무 이상해 보입니다.

이해를 돕기 위해 1이 아닌 12 인 이유를 설명해 주시겠습니까? 그리고 어떤 것 doy 의 1 또는 4 의미,이 경우에는? 여기서 수학은 정확히 어떻게 작동합니까? 잘 이해하신 것 같네요 ..

doy 가 무엇인지 더 분명하게 만드는 좋은 설명을 찾았지만, 6보다 클 수있는 방법은 아직 알 수 없습니다.

@icambron , 차임 할 수 있습니까? 현재 문서보다 더 자세히 설명하는 링크가 있습니까?

@usmonster 오랜만에 이걸 봤는데 머릿속에서 ... 그래, 나도 모르겠어. 더 도움을 드릴 수 없어서 죄송합니다. 내가 링크 설명을 썼다 때마다 나는 아마 다음을 이해할 수 있도록, 난 그냥 처리 주 동안 순간의 구현의 무리를 작성했지만, 그 설명은 함께 흔들하지 않는 것 doy 하므로, 값> 6 .. .

그 이유, doy 정의 방법은 설명 할 수 없지만 7 + dow - janX 로 설정할 수 있습니다. 여기서 janX 는 주에 속해야하는 x 번째 1 월입니다. 1 번.

따라서 1 월 1 일이 1 번 주에 속해야하고 토요일이주의 첫 번째 날이면 doy = 7 + 6 - 1 = 12 입니다. ISO의 경우 미국의 경우 doy = 7 + 1 - 4 = 4 이고 doy = 7 + 0 - 1 = 6 입니다.

아, 이런 식으로 _ 구현 _하면 구현 문제라고 생각합니다. 내가 본 다른 정의 (즉, ISO, 유니 코드 TR35)는 이와 같은 주 계산을 정의하지 않습니다. API의 주석은 이것이 무슨 일이 일어나고 있는지 암시조차하지 않지만, 지금은 바로이 논리를 가정하는 테스트가 작성되었음을 알 수 있습니다. 예를 들어 다음과 같은 줄이 표시됩니다.

dow : 1, // Monday is the first day of the week.
doy : 7  // The week that contains Jan 1st is the first week of the year.

그리고 다른 파일에서 :

dow : 6, // Saturday is the first day of the week.
doy : 12  // The week that contains Jan 1st is the first week of the year.

따라서 구현에 대한 평가가 정확 해 보입니다. 그래도 테스트가 깨진 구현과 일치하도록 작성된 것처럼 느껴집니다.

직관적 인 작업 (첫 주에 있어야하는 " janX "를 가져옴)을 수행하도록 구현을 수정하는 것이 더 깔끔한 작업이었을 것이기 때문에 특히 실망 스럽습니다. 그러나 지금 수정하면됩니다. ' t 이전 버전과 호환됩니다. 으.

대안은 doy 를 지원 중단하고 주 구성에 새 옵션 (예 : minDays 또는 dom )을 도입하여 최소 "1 월 일"(1- 7) 그 해의 첫 번째 주에 있어야합니다. 실수로 지정된 경우 doy 보다 우선합니다.

이견있는 사람?

내가 말할 수있는 한, iso 주에 대해 코드를 설정했음을 알 수있는 한, 우리는 사실을 남용하고 숫자가 제대로 작동하는지 확인했습니다.

의미를 사용자에게 더 의미있는 것으로 변경하면 더 좋을 수 있습니다. 그러나 이전 버전과 호환되어야합니다 (따라서 새로운 메서드는 새로운 숫자를 노출합니다).

예를 들어 doy에 7을 몇 번 더하면 dow에 상대적이기 때문에 작동하도록 만들 수 있습니다.

# 2336에 찬성하여 이것을 닫습니다.

안녕하세요 @ichernev. :) 1 년 후 다시 돌아와서, 구문을 변경하지 않고는 이전 버전과 호환되는 작업을 수행 할 수 없다고 확신하지만 더 의미있는 의미를 구현하는 것을 선호합니다.

doy에 7을 몇 번 더하면 dow에 상대적이기 때문에 작동하도록 만들 수 있습니다.

나는 이것이 항상 사실이라고 확신하지 않는다. 예를 들어, { dow: 0, doy: 6 } 는 제안 된 의미 변경 전후에 같은 것을 의미하지 않는다. (이것이 내가 다른 이름을 위해 doy 를 더 이상 사용하지 않도록 제안한 이유입니다.)

이에 대한 추가 피드백은 어디에서 검색해야합니까? 이 특정 문제는 종료되었지만 대화를 moment / momentjs.com # 279로 이동하거나 더 적절한 경우 새 문제를 열 ​​수 있습니다.

또한 doy 작동 방식을 파악하기 위해 추가로 하루를 보냈고이를 수행하기 위해 순간 소스를 다운로드해야했기 때문에이 문제를 해결하고 싶습니다. @usmonster가 제안한 것처럼 의미 상 관련성이 더 높은 새 설정 변수를 도입하는