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

    +96

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public static Entity read(final URI path) {
    	if(!(path.items!= null && (0 < path.items.size()))){
    		assert (false);
    	}
                     //дальше сам метод

    Запостил: 3.14159265, 02 Июля 2010

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

    • двойное отрицание воспринимается конечно тяжеловато, но вполне терпимо
      if (path.items == null || path.items.size() <= 0) {
         assert false; // сказать фи в дебаге
      }
      Ответить
      • Нужно было так:
        assert(path.items == null || path.items.size() <= 0);
        Ответить
        • А если немного подумать, то правильнее всё таки так:
          assert( path.items != null && 0 < path.items.size() );

          Ведь assert срабатывает в случае ложности выражения.
          Ответить

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