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

    +71

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    String currentDate = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").format(new Date());
 accountNumber.append(String.valueOf(1900 + new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getYear()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMonth()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getDate()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getHours()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMinutes()));
 accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getSeconds()));

    ну а чо, зато внушительно выглядит

    Запостил: nafania217518, 01 Марта 2013

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

    • ava bormand 01.03.2013 17:03 # +2

      It's just a SimpleDateFormatFactory.
      Ответить

    • ava roman-kashitsyn 01.03.2013 17:10 # +9

      Всем известно, что лишние переменные замедляют выполнение кода
      Ответить

    • ava 3.14159265 01.03.2013 17:13 # 0

      Во всей красе. Полагаю accountNumber потом еще и парсится обратно в число.
      Человек не знавший про System.nanoTime ().
      Ответить

      • ava bormand 01.03.2013 17:22 # 0

        И про то, что append у стрингбилдера жрет любые типы без String.valueOf()/toString()...
        Ответить

        • ava 3.14159265 01.03.2013 17:29 # 0

          И про то что это всё, наверняка, можно было тупо сплюсовать.
          Ответить

          • ava bormand 01.03.2013 17:31 # 0

            И про то, что в яве есть не только String, но и много других типов.
            Ответить

    • ava roman-kashitsyn 01.03.2013 17:16 # 0

      Кстати, акцессоры вроде Date.getYear() задепрекейчены ещё при царе горохе
      Ответить

      • ava bormand 01.03.2013 17:30 # +5

        Тссс... Не надо ему говорить про Calendar...
        String currentDate = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").format(new Date());
Calendar calendar = Calendar.getInstance();
calendar.setTime(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate));
accountNumber.append(String.valueOf(calendar.get(Calendar.MONTH)));
calendar = Calendar.getInstance();
calendar.setTime(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate));
accountNumber.append(String.valueOf(calendar.get(Calendar.DAY_OF_MONTH)));
...
        Ответить

      • ava Lure Of Chaos 03.03.2013 14:01 # +1

        это одно из мест, которые приводят в бешенство. был простой, удобный объект Date... Но сановцы посчитали, что жизнь с ним слишком легка и скучна, и сделали Calendar, которым пользоваться, в смысле написания кода, очень неудобно.
        Приходится юзать всякие сторонние JodaTime.
        Ответить

        • ava 3.14159265 04.03.2013 16:38 # 0

          >был простой, удобный объект Date
          O RLY? Я б не сказал что он шибко удобный.
          Ответить

          • ava Lure Of Chaos 04.03.2013 16:48 # 0

            3 поля: год, месяц, день, с аксессорами - что может быть удобнее?
            Ответить

            • ava roman-kashitsyn 04.03.2013 18:07 # +2

              Для реального удобства нужны таймспаны с аксессорами и умные методы конструирования дат с addDays, addYears, etc. Мелкософтовцы в этом плане лучше сделали, DateTime у них в сишарпике в меру удобный.
              Ответить

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