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

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    public boolean intersects(final Range range) {
      return (((range.indexFrom >= this.indexFrom) ? range.indexFrom : this.indexFrom) <=
        ((range.indexTo <= this.indexTo) ? range.indexTo : this.indexTo));
    }

    проверка пересечения двух диапазонов

    Lure Of Chaos, 17 Мая 2011

    Комментарии (15)

  2. Java / Говнокод #6685

    +83

    1. 1
    2. 2
    Float.parseFloat(obj.getPrice() + ""));
// int obj.getPrice()

    int to float

    dshulgin, 17 Мая 2011

    Комментарии (5)

  3. Java / Говнокод #6683

    +79

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    Object sync;

try {
    synchronized (sync){
        sync.wait();
    }
} catch (NullPointerException npe){
    sync = new Object();
    synchronized (sync){
        sync.wait();
    }
}

    Вот так чел пытался избежать "ненужного" создания объекта-монитора. Память экономил...

    Skipy, 17 Мая 2011

    Комментарии (14)

  4. Java / Говнокод #6681

    +118

    1. 1
    return EVIL_PAGE;

    извините

    3.14159265, 16 Мая 2011

    Комментарии (15)

  5. Java / Говнокод #6677

    +82

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    //request reading
        String allData = "";
        try {
            int data = input.read();
            while (true) {
                allData += (char) data;
                if (input.available() < 1) {
                    break;
                }
                data = input.read();
            }
        } catch (IOException ex) {
            Logger.getLogger(HttpProtocolProcessor.class.getName()).log(Level.SEVERE,
                    "Problems occured while reading the stream.", ex);
        }

    Прямо свежего наклада. Вот таким вот нетривиальным способом член команды читает из InputStream'a всё в одну строку. Он не только читает по байтно, но еще и для каждого байта создаёт новую строку. О мой боже. Т_Т.

    sogekoder, 15 Мая 2011

    Комментарии (5)

  6. Java / Говнокод #6670

    +74

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    public boolean setBit(byte _position, boolean _state) {
		if ( !valid ) {
			LOG.error("value is INVALID");
			
			return false;
		} else if ( _position<0 ) {
			LOG.error("NEGATIVE _position");
			
			return false;
		} else if ( _position > capacity ) {
			LOG.warn("_position("+_position+") > cacity("+capacity+") "+
					"for value "+this);
			
			return false;
		}
		
		value|=( (_state ? 1 : 0) << (_position+1) );
		
		return true;
}

    ога, разбежался

    ilardm, 14 Мая 2011

    Комментарии (16)

  7. Java / Говнокод #6663

    +145

    1. 1
    2. 2
    3. 3
    @NotNull List<@NotNull @NotBlank String>
getFromCache(@NotNull @NotBlank final String key,
@NotNull @NotEmpty final Map<@NotNull @NotBlank String, @NotNull List<@NotNull @NotBlank String>> cache) @ReadOnly;

    пустили козла в аннотации

    Lure Of Chaos, 14 Мая 2011

    Комментарии (10)

  8. Java / Говнокод #6661

    +147

    1. 1
    2. 2
    case KeyEvent.VK_ESCAPE:
      case 65385:

    я так и не узнал, какая клавиша еще сгодится....

    Lure Of Chaos, 14 Мая 2011

    Комментарии (11)

  9. Java / Говнокод #6658

    +147

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (nn.isleaf) {
      this.traverse_ssector((BSPLeaf) nn);
      return;
    }
    final BSPBranch n = (BSPBranch) nn;

    правильное наследование:
    в родительском классе BSPNode определено поле isleaf
    в конструкторе класса BSPLeaf оно выставляется true и никогда не меняется
    в конструкторе класса BSPBranch соответственно false

    а ведь так просто сделать проверку instanceof

    Lure Of Chaos, 14 Мая 2011

    Комментарии (7)

  10. Java / Говнокод #6657

    +147

    1. 1
    2. 2
    3. 3
    static int getSign(final int num) {
    return (num < 0) ? -1 : (num > 0) ? 1 : 0;
  }

    лисапед, бо есть Math.signum()

    Lure Of Chaos, 14 Мая 2011

    Комментарии (8)