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

[kair84]

Функция WeekOfYear(31\12\2012) возвращает 2
наверное после дня апокалипсиса нумерация недель начинается заново.

[kair84]

красота:

print(weekofYear(30\12\2012)); // 53
print(weekofYear(31\12\2012)); // 2
print(weekofYear(01\01\2013)); // 1

[George Nordic]

Еще интереснее:

    info(strfmt("d3:%1",WeekOfYear(21\12\2012))); //52
    info(strfmt("d4:%1",WeekOfYear(30\12\2012))); //53

Хотя между 21 и 30 явно больше 7 дней

Вау! Да класс global рулит! static int weekOfYear(date _curDate)

Цитата:

// The dayofwk function used above returns
// the ISO-8601 day of week, where Monday is 1
// and recall that the 1 was subtracted from the result
// to get a value between (0 - 6).
//
// If the locale settings (firstDayOfWeek) is
// not Monay (i.e. > 0) then the caclulated
// day of week results need to be adjusted
// so they align with the locale setting.

Там комментариев больше чем кода Рефакторинг блин...
А толку-то...

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

[mifi]

Цитата:

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

Еще интереснее:

    info(strfmt("d3:%1",WeekOfYear(21\12\2012))); //52
    info(strfmt("d4:%1",WeekOfYear(30\12\2012))); //53

Хотя между 21 и 30 явно больше 7 дней

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

Хотя между 21 и 30 и больше 7 дней, но 21.12.2012 - пятница, 30.12.2012 - воскресенье. Что не так-то? Вроде бы нормально, что между одним днем одной недели и другим днем следующей недели не 7 дней..

[Bober]

Цитата:

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

красота:

print(weekofYear(30\12\2012)); // 53
print(weekofYear(31\12\2012)); // 2
print(weekofYear(01\01\2013)); // 1

Это майя дали человечеству еще один день, чтобы оттянуться напоследок на Новый год. Чтобы президент в своей последней речи попросил у всех прощения. И т.п.

[ice]

Цитата:

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

Функция WeekOfYear(31\12\2012) возвращает 2
наверное после дня апокалипсиса нумерация недель начинается заново.

в AX2009 возвращает 1

[Poleax]

Цитата:

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

Функция WeekOfYear(31\12\2012) возвращает 2
наверное после дня апокалипсиса нумерация недель начинается заново.

2 не наблюдается (Ax 2009 SP1 EE RU6 5.0.1500.3761)

static void Job17(Args _args)
{
    ;
    info(strfmt("25.12.2011 - %1",WeekOfYear(25\12\2011)));  //52
    info(strfmt("30.12.2011 - %1",WeekOfYear(30\12\2011)));  //1
    info(strfmt("31.12.2011 - %1",WeekOfYear(31\12\2011)));  //1
    info(strfmt("01.01.2012 - %1",WeekOfYear(01\01\2012)));  //1

    info(strfmt("29.12.2012 - %1",WeekOfYear(29\12\2012)));  //53
    info(strfmt("30.12.2012 - %1",WeekOfYear(30\12\2012)));  //53
    info(strfmt("31.12.2012 - %1",WeekOfYear(31\12\2012)));  //1
    info(strfmt("01.01.2013 - %1",WeekOfYear(01\01\2013)));  //1
}

Info	Сообщение (13:48:46)	25.12.2011 - 52
Info	Сообщение (13:48:46)	30.12.2011 - 1
Info	Сообщение (13:48:46)	31.12.2011 - 1
Info	Сообщение (13:48:46)	01.01.2012 - 1
Info	Сообщение (13:48:46)	29.12.2012 - 53
Info	Сообщение (13:48:46)	30.12.2012 - 53
Info	Сообщение (13:48:46)	31.12.2012 - 1
Info	Сообщение (13:48:46)	01.01.2013 - 1

но, есть разночтения с SQL:

PHP код:

SET DATEFORMAT dmy; 
SELECT DATEPART(ww, '25/12/2011') as '25/12/2011'
      ,DATEPART(ww, '30/12/2011') as '30/12/2011'
      ,DATEPART(ww, '31/12/2011') as '31/12/2011'
      ,DATEPART(ww, '01/01/2012') as '01/01/2012';
    
SELECT DATEPART(ww, '30/12/2012') as '30/12/2012'
      ,DATEPART(ww, '31/12/2012') as '31/12/2012'
      ,DATEPART(ww, '01/01/2013') as '01/01/2013';

select @@version 


25/12/2011  30/12/2011  31/12/2011  01/01/2012
----------- ----------- ----------- -----------
52          53          53          1

(1 row(s) affected)

30/12/2012  31/12/2012  01/01/2013
----------- ----------- -----------
53          54          1

(1 row(s) affected)


--------------------------------------------------------------------------------------------------------------------------------
Microsoft SQL Server 2008 R2 (RTM) - 10.50.1600.1 (Intel X86) 
	Apr  2 2010 15:53:02 
	Copyright (c) Microsoft Corporation
	Enterprise Edition on Windows NT 5.2 <X86> (Build 3790: Service Pack 2) (Hypervisor)

[George Nordic]

Да, в 2009 возвращает 1. Я свои примеры тоже с 2009 брал.

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

[George Nordic]

Цитата:

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

Хотя между 21 и 30 и больше 7 дней, но 21.12.2012 - пятница, 30.12.2012 - воскресенье. Что не так-то? Вроде бы нормально, что между одним днем одной недели и другим днем следующей недели не 7 дней..

Да, именно поэтому в самой последней версии DAX 31.12.2012 - это 1я рабочая неделя.

[nix0root]

Не верно отрабатывает функция wkofyr(_date), которая вызывается из weekofyear.

[oip]

Цитата:

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

наверное после дня апокалипсиса нумерация недель начинается заново.

Майя тут ни при чем. Например:

info(strfmt("%1",WeekOfYear(31\12\1928))); // Ax 3.0, 4.0: 2. Ax 2009: 1
info(strfmt("%1",WeekOfYear(31\12\1956))); // Ax 3.0, 4.0: 2. Ax 2009: 1
info(strfmt("%1",WeekOfYear(31\12\1984))); // Ax 3.0, 4.0: 2. Ax 2009: 1
info(strfmt("%1",WeekOfYear(31\12\2040))); // Ax 3.0, 4.0: 2. Ax 2009: 1

info(strfmt("%1",WeekOfYear(31\12\2001))); //1
info(strfmt("%1",WeekOfYear(30\12\2003))); //1
info(strfmt("%1",WeekOfYear(29\12\2008))); //1

[George Nordic]

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

Да, wkofyr(_date) - работает некорректно. Но для этого и был написан пафосный класс-обертка weekOfYear(), который учитывает локальные настройки, в частности, с какого дня начинается неделя, но при этом при формировании тестовой даты не учитывает, что год может быть високосным

Ошибка возникает здесь, как я говорил:

            if (week > 52)
            {
                // Look at the last day of the week and compare it
                // to the target date.  If the year is different
                // then the target date is part of week 1 for next year.
                testDate = _curDate + ((#DaysPerWeek - 1) - dowCurDate); //Ошибка

Непонятна необходимость данной проверки. В России 31.12.2012 является 54 неделей, а не 1й.
Ладно, хорошо. Может, в мире и не так. Тогда надо учитывать, что каждый 4й год происходит такая беда как високосный год, и testDate формируется некорректно.

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

[S.Kuskov]

Цитата:

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

надо учитывать, что каждый 4й год происходит такая беда как високосный год, и testDate формируется некорректно.

А при чём тут високосный год? testDate формируется с одной лишь целью - узнать на какой год попадает конец недели. В случае если неделя начинается в одном году а коньчается в другом, то ей присваивают первый номер! Зачем это сделано? Наверное для того чтобы дни одной и той же недели с понедельника по воскресенье всегда оказывались в одном номере недели.

[George Nordic]

Цитата:

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

А при чём тут високосный год? testDate формируется с одной лишь целью - узнать на какой год попадает конец недели. В случае если неделя начинается в одном году а коньчается в другом, то ей присваивают первый номер!

Честно - я не помню уже. Давно разбирал, году в 2004 столкнулся с этой проблемой. Но давайте разберем вместе. Так вот, поставьте брекпоинт на эту строчку и проверьте, корректно ли формируется testDate в обычном и високосном году. В обычном году была ошибка на 2 дня, кажется, в високосный - тоже ошибка была.

Цитата:

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

Зачем это сделано?

Да вот сам в шоке. Наверное, что бы понять, если последняя неделя года приходится с переходом на другой год, то дать ей номер 1?

Цитата:

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

Наверное для того чтобы дни одной и той же недели с понедельника по воскресенье всегда оказывались в одном номере недели.

Угу. А вот с переходом через год этот фокус не срабатывает.

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

[S.Kuskov]

Цитата:

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

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

Давайте ещё раз посмотрим на строчку которую вы привели

testDate = _curDate + ((#DaysPerWeek - 1) - dowCurDate);

Здесь testDate получается смещением даты в перёд на количество дней оставшееся до конца недели. Т.е. так чтобы попасть на воскресенье. Единственное место где здесь может быть засада - это функция dayofwk (используется для расчёта dowCurDate), возвращающая номер дня недели. Но она работает корректоно и в високосный год и в не високосный.

Цитата:

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

В обычном году была ошибка на 2 дня, кажется, в високосный - тоже ошибка была.

Т.е. дата testDate не попадала на воскресенье? Мне это воспроизвести не удалось.

[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.