Откопал вот в одном проекте. Не знаю, как теперь назад закопать.

        public MyInternalDateClass extendDate(MyInternalDateClass date, int years) throws Exception {
            String existing = date.toString();
            int year = date.getYear() + years;
            String dateString = existing.replace(Integer.toString(date.getYear()), Integer.toString(year));

            return new MyInternalDateClass (dateString);
        }

Здравствуйте, elmal, Вы писали:

E>А что там закапывать. Поиск по всему проекту на этот класс, замена на использование стандартной даты


как показывает практика, велик шанс что после такой замены что-то перестанет работать, ибо в кастомном классе в каком-нить невинном методе его гениальный архитектор заложил какой-нить side effect

Здравствуйте, Kaifa, Вы писали:


AG>>29 февраля выстрелит

K>нет

Была дата 29.02.2012 + 1 год и получили 29.02.2013

Здравствуйте, andrey.t, Вы писали:

existing.replace(Integer.toString(date.getYear()), Integer.toString(year));

29 февраля выстрелит

Здравствуйте, BlackEric, Вы писали:

AG>>>29 февраля выстрелит

K>>нет

BE>Была дата 29.02.2012 + 1 год и получили 29.02.2013

Кстати, не факт. Если строка далее парсится чем-нибудь стандартным(SimpleDateFormat, например), то 29.02.2013 станет 01.03.2013(там можно и 43 февраля вставить — оно станет соответствующим марта) — этот класс по-умолчанию лоялен к таким датам.
Но код жесть, да.

Здравствуйте, andrey.t, Вы писали:

AT>Откопал вот в одном проекте. Не знаю, как теперь назад закопать.

напомнило: http://code-wtf.livejournal.com/216613.html

Здравствуйте, andrey.t, Вы писали:

AT>Откопал вот в одном проекте. Не знаю, как теперь назад закопать.
А что там закапывать. Поиск по всему проекту на этот класс, замена на использование стандартной даты плюс где нужно форматировать по хитрому — там вызов форматтера. Правится элементарно. Если бы весь говнокод так легко исправлялся, была бы жизнь прекрасна.

AG>29 февраля выстрелит

нет

Здравствуйте, BlackEric, Вы писали:

BE>Была дата 29.02.2012 + 1 год и получили 29.02.2013

Ну, это мягкий и легко диагностируемый глюк. Еггоги, эксепшены и изумления пользователей вылезут незамедлительно.
А вот замена 29.02.2012 на 28.02.2013 либо 01.03.2013 по неизвестному закону в непредсказуемом месте может дорого обойтись.
Либо ±1 день превратится в ±тырнадцать процентов и тугриков, либо в недолёт-перелёт расписания, и всё такое прочее.
Эти вещи должны быть прописаны в техзадании, но, судя по говнокоду, вряд ли индус об этом не то что задумывался, но даже и подозревал.

Здравствуйте, Кодт, Вы писали:

К>Здравствуйте, BlackEric, Вы писали:

BE>>Была дата 29.02.2012 + 1 год и получили 29.02.2013

К>Ну, это мягкий и легко диагностируемый глюк. Еггоги, эксепшены и изумления пользователей вылезут незамедлительно.

Можно нафантазировать, что внутреннее представление даты там — строка, и что конструктор не проверяет с учётом того, что 29 февраля не в каждом году.
Таким образом, можно завести будильник на дату, которая никогда не наступит.

Здравствуйте, Alexander G, Вы писали:

AG>Здравствуйте, andrey.t, Вы писали:


AG>

existing.replace(Integer.toString(date.getYear()), Integer.toString(year));

AG>29 февраля выстрелит
Выстрелить в любой день может: "10.10.10", и год меняем на 11-й. Тот ещё выстрел из бесшумного пистолета будет.