1. Java / Говнокод #8593

    +80

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    Calendar startCal = Calendar.getInstance();
startCal.setTime(DateUtils.truncateDateAndTimeToDateOnly(startDate));
Date strtDt = startCal.getTime();

Calendar endCal = Calendar.getInstance();
endCal.setTime(DateUtils.truncateDateAndTimeToDateOnly(endDate));
Date endDt = endCal.getTime();

    Далее по тексту strtDt и endDt используются ровно один раз, startCal и endCal не используются больше вообще. Утилитный метод возвращает требуемый экземпляр класса Date.

    Запостил: roman-kashitsyn, 21 Ноября 2011

    Комментарии (27) RSS

    • ava Lure Of Chaos 21.11.2011 12:31 # 0

      походу 6 строчек можно ужать в 3 (с учетом "далее по тексту") и использовать один инстанс календаря
      Ответить

      • ava roman-kashitsyn 21.11.2011 13:01 # +1

        нет, можно вообще все 6 строчек выкинуть и заинлайнить strtDt -> DateUtils.truncateDateAndTimeToDateOnly( startDate).
        Ответить

        • ava Lure Of Chaos 21.11.2011 13:08 # 0

          тьфу ты, точно, календарь тут полезного ничего не делает. спасибо )
          Ответить

        • ava gegMOPO4 21.11.2011 22:14 # 0

          С учётом громоздкости вызова, не грех и оставить strtDt.
          Ответить

    • ava KirAmp 21.11.2011 13:31 # 0

      Языки меняются, даты остаются..
      Ответить

      • ava lucidfoxGovno 21.11.2011 14:06 # 0

        Ещё на С++ и Scala не было.
        Ответить

      • ava roman-kashitsyn 21.11.2011 14:25 # 0

        Тут скорее не в датах дело, а в долбоебизме автора кода. Там четыреста строк сплошных лулзов.
        Ответить

        • ava tir 21.11.2011 14:29 # +2

          Java Calendar API все-таки не для слабонервных =)
          Ответить

          • ava Lure Of Chaos 21.11.2011 18:52 # 0

            кстати, да. По мне, лучше .setYear(y), чем .set(.YEAR,y)
            Ответить

            • ava absolut 21.11.2011 21:20 # +1

              >.set(.YEAR,y)
              это как ?
              Ответить

              • ava Lure Of Chaos 21.11.2011 21:27 # 0

                Calendar сal = Calendar.getInstance();
                cal.set(Calendar.YEAR,1984);

                как-то так.
                Ответить

                • ava 3.14159265 21.11.2011 21:31 # +3

                  Какая мерзость этот нативный джавовский календарь.А год клацный.
                  Ответить

                  • ava Lure Of Chaos 21.11.2011 21:38 # 0

                    был вполне удобный java.util.Date, но к нему доебались, что он не поддерживает интернационализацию (только по григорианскому календарю), и обьявили его почти полностью deprecated.
                    Ответить

                    • ava 3.14159265 21.11.2011 21:42 # 0

                      Угу. Все конструкторы хорошие подепрекейтили.
                      Но все равно joda-time лучче.
                      Ответить

                      • ava Lure Of Chaos 21.11.2011 21:45 # 0

                        спасибо, не знал раньше о таком.
                        Ответить

                      • ava tir 21.11.2011 21:58 # 0

                        Это какие такие хорошие конструкторы? Где год - это "the year minus 1900", а месяц - "the month between 0-11"?
                        Ответить

                        • ava 3.14159265 22.11.2011 13:17 # +1

                          Нет. Date(String s), вернее метод parse, который он вызывает.
                          А те конструкторы больше чем на тесты не годятся.

                          И повторю свою мысль - большинство библиотечных функций и классов, что есть в жабе по дефолту - кривое неюзабельное говно.
                          Ответить

                          • ava Desktop 05.09.2023 18:43 # 0

                            а нахуя она тогда нужна?
                            Ответить

                            • ava guest6 05.09.2023 18:49 # 0

                              Чтобы показывать как не нужно делать фреймворки.

                              Замечательная книга "Effective Java" Блоха почти целиком состоит из примеров типа: "вот мы сделали так в джаве, и это была ошибка: никогда так не делайте". Работа с датами, writeObject, открытость классов по-умолчанию и пр.

                              А еще есть книга Ousterhoutа (который TCL) "A Philosophy of Software Design" и там тоже постоянные примеры того, как джавабляди обосрались.
                              Например, он обсирает необходимость вручную каждый стрим заворачивать в буферизирующий стрим и пр
                              Ответить

                  • ava gegMOPO4 21.11.2011 22:20 # +1

                    Если делать всё отдельными методами, пришлось бы реализовать ещё приблизительно 16*17=272 метода.
                    Ответить

        • ava absolut 21.11.2011 14:59 # +2

          может быть дело все-таки в дате ... рождения автора
          Ответить

    • ava dos_ 21.11.2011 18:41 # −2

      >Calendar
      >startCal
      >endCal
      >>Cal
      Ответить

    • ava guest8 09.04.2019 12:11 # −999

      показать все, что скрытоvanished
      Ответить

    • ava emfs 25.08.2021 21:00 # 0

      – Вот то-то и оно… – проговорила, Марго вытирая салфеткой губы, – любишь. А эта блядища этого не ценит!
      Ответить

    Добавить комментарий