Баг или "пророчество"

[George Nordic]

Так. Стоп.

1. Вы согласны, что приведенная функция работает некорректно?
Или Вы считаете, что 27.12.10 - , 26.12.11 - , 24-30.12.12 - это 1 рабочая неделя??
2. Если Вы согласны, что это не так, что это 53 недели в году, тогда идем дальше.
Вы согласны, что ошибка возникает именно в методе WeekOfYear?
3. Вы согласны, что они возникает именно при первой проверке if (week > 52)?

Итак, если Вы согласны, то осталось понять, а как исправить функцию, что бы она возвращала корректные значения для России.

Этого можно добиться, исключив данную проверку (для России). Рассматривать расчет testDate, как Вы уже указали, некорректно, так как цель данного расчет - это посмотреть, не переходит ли неделя через год и сбросить счетчик текущей недели.

Согласны? Или есть какие-либо подводные камни, которые я не учел?

С Уважением,
Георгий

[S.Kuskov]

Цитата:

Сообщение от George Nordic

1. Вы согласны, что приведенная функция работает некорректно?
Или Вы считаете, что 27.12.10 - , 26.12.11 - , 24-30.12.12 - это 1 рабочая неделя??

Не знаю. Никогда не приходилось сталкиваться с этой функцией, а тут нужно смотреть именно на контекст применения этой функции. Теоретически оба варианты имеют право на существование.

Цитата:

Сообщение от George Nordic

Итак, если Вы согласны, то осталось понять, а как исправить функцию, что бы она возвращала корректные значения для России.
Этого можно добиться, исключив данную проверку (для России).

Да, если убрать данную проверку, то результат будет другим, возможно для кого-то более ожидаемым.

P.S.: И ещё, чтобы не вводить никого в заблуждение, мы же говорим не о рабочих неделях, а о календарных. В том смысле, что праздничные дни никак на результат не влияют

[oip]

Цитата:

Сообщение от George Nordic

1. Вы согласны, что приведенная функция работает некорректно?

Не согласен. Некорректно в каком случае? В стандартной Аксапте она используется только для отображения номера недели в форме SysDateLookUp. И в этом контексте выглядит все логично.

А если кто-то будет использовать данную функцию в ином контексте, не разобравшись в принципе ее работы, посмотрев только на название, то он сам себе злобный буратино. Хотя да, могли бы в методе сверху написать об "особенности" определения номера недели в конце года, чтобы не было соблазна воспользоваться ей в неподобающем месте.

[George Nordic]

Цитата:

Сообщение от S.Kuskov

P.S.: И ещё, чтобы не вводить никого в заблуждение, мы же говорим не о рабочих неделях, а о календарных. В том смысле, что праздничные дни никак на результат не влияют

Рабочая неделя - это продолжительность работы в течение календарной недели в часах. Так что это немного не то... Я понял, о чем Вы говорите. Да, для этих целей необходимо использовать функциональность календарей. Нумерация рабочих недель совпадает с нумерацией календарных недель. 1 рабочая неделя начинается 1 января, а не 11 января.

Цитата:

Сообщение от oip

В стандартной Аксапте она используется только для отображения номера недели в форме SysDateLookUp. И в этом контексте выглядит все логично.

Ох же ты! Надо же, не ожидал, спасибо. Не ожидал, что в Sys используется функция куда высшего слоя..

Цитата:

Сообщение от oip

А если кто-то будет использовать данную функцию в ином контексте, не разобравшись в принципе ее работы, посмотрев только на название, то он сам себе злобный буратино. Хотя да, могли бы в методе сверху написать об "особенности" определения номера недели в конце года, чтобы не было соблазна воспользоваться ей в неподобающем месте.

Хм. а что еще ожидать от функции с таким названием? Или предлагаешь под каждый чих свою функцию писать? Неразумно. Стандарт - он на то и стандарт, что ждешь от него стандартного предсказуемого поведения, а не кренделей.

С Уважением,
Георгий

[Raven Melancholic]

Если учитывать, что в DAX2009 серьезно переписали подсистему работы с датами, то удивляться различию между DAX4 (и Ax3.0) и DAX2009 в результатах в общем-то не стоит (например, метод firstWeekOfYear класса Global полностью переписан на использование NET, поэтому в DAX2009 работает общая логика Win, а не особенности Ax).
Ну, и тем более, влияет серьезное изменение метода weekOfYear того же класса Global.