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

    +119

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    private boolean isInOutConnected(StopPosition stopPoint) {
    boolean ret = false;
    List<Port> l = stopPoint.getLoadingPorts();
    
    if (l != null && l.size() > 0) {
        ret = true;
    }
    
    return ret;
}

    Запостил: someone, 16 Июля 2014

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

    • ava absolut 18.07.2014 21:06 # +1 

      private boolean isInOutConnected(StopPosition stopPoint) {
    List<Port> l = stopPoint.getLoadingPorts();
    return (l != null && l.size() > 0);
}

      Нужен IsNullOrEmpty
      Ответить

      • ava someone 19.07.2014 07:34 # +2

        Не нужен. Передавать null как эквивалент пустого списка - дурной стиль.

        А вот List.isEmpty() вместо size() > 0 использовать очень даже можно.
        Ответить

        • ava bormand 19.07.2014 08:44 # 0

          > Передавать null как эквивалент пустого списка - дурной стиль.
          Хех. В пыхе еще хуже - там зачастую передают сам элемент вместо списка из одного элемента, и считают это нормальным и удобным...
          Ответить

          • ava kegdan 19.07.2014 09:20 # +1

            это пиздец. поэтому я не люблю динамические языки - ебанутое коммьюнити
            Ответить

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