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

    +75

    1. 1
    return new Integer(((Integer)var).intValue()+1);

    Autoboxing? Не, не слышал...

    Запостил: kostoprav, 08 Мая 2014

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

    • ava 0rt 10.05.2014 16:01 # +1

      Интересно при этом, каков изначальный тип переменной var. По ходу автор строки ищет приключений. А если там окажется Float?
      Ответить

      • ava Lure Of Chaos 10.05.2014 18:11 # +1

        если не int и не Integer, то не скомпилится, все ок
        Ответить

        • ava bormand 10.05.2014 19:32 # +2 

          Object var = Long.valueOf(0);
return new Integer(((Integer)var).intValue()+1);
          Все замечательно скомпилится, а потом замечательно кинет исключение.
          Ответить

          • ava 3.14159265 10.05.2014 22:03 # +3

            Это потому что кастим не к тому:
            >Object var = 0L;
            >return new Integer(((Number)var).intValue()+1);
            [fixed]
            Запись каста выглядит как говно и в рантайме говно. А если что-то выглядит как говно и работает как говно, то оно непременно им и является.
            Если в программе начинает плодится эта срань - первый признак что у кодера кривые руки.
            Ответить

    • ava LispGovno 11.05.2014 00:38 # 0

      А как это переписать? Всё негодование по поводу каста (Integer)var ?
      Ответить

      • ava bormand 11.05.2014 08:59 # 0

        Имхо как-то так:
        return var+1; // если var - Integer, Short или Long
return var.intValue()+1; // если var - Number
return ((Number)var).intValue() + 1; // если var - Object
        Ответить

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