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

    +77

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    int min = 10;
    int max = 4095;
    for (SomeClass cc : someList) {
      if ( cc.getMinVlan( ) != null ) min = cc.getMinVlan( ) > min ? cc.getMinVlan( ) : min;
      if ( cc.getMaxVlan( ) != null ) max = cc.getMaxVlan( ) < max ? cc.getMaxVlan( ) : max;
    }

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

    Запостил: raorn, 20 Мая 2010

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

    • int min = 10;
      int max = 4095;

      какой замечательный выбор!
      Ответить
    • в чем говно?
      Ответить
      • Внимательно смотрим на строчки 4 и 5
        Ответить
        • типа min будет увеличиватся, а max уменьшатся?
          если так, то это гкод, а просто ошибка в знаке
          Ответить
      • Самое главное то, что при любом случае на итерации приравнивается значение, а не только в случае, когда это действительно необходимо.

        к тому же действительно, мин будет уменьшаться, а макс увеличиваться (что ты и сам заметил :)
        Ответить
        • Самый цимес в том, что если приходит два непересекающихся диапазона, то min получится больше max.
          Ответить
          • Резюмируя, можно только заметить большую концентрацию х%йни на строчку кода
            Ответить
            • Просто когда пишешь строчку кода - не ананаируй.
              Ответить
              • да на гк не должно подниматься ))
                Ответить
                • У извращенцев С++ ников это бывает, так как на девушек уже не стоит.
                  Ответить
    • getMinVlan и getMaxVlan вызываются по 2-3 раза.
      Ответить
      • Это не проблема, так в Ява таким образом организована система пропертей - через геттеры и сеттеры. Хотя, конечно, было бы более читабельней приравнять одной ссылке
        Ответить

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