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

[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 не попадала на воскресенье? Мне это воспроизвести не удалось.