1. 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) RSS

    • ava SmackMyBitchUp 14.05.2011 13:51 # 0

      Тернистый уже раза 2 появлялся.
      Ответить

    • ava burdakovd 14.05.2011 14:45 # 0

      Math.signum() не перегружен для int
      Ответить

      • ava Lure Of Chaos 14.05.2011 15:21 # −2

        intы нормально расширяются до floatов же
        Ответить

        • ava burdakovd 14.05.2011 15:39 # +6

          Но возвращаемое значение обратно не сужается же!

          Таким образом когда нужно реализовывать сортировку с кастомным компаратором, то приходится делать 

          int compare(int i, int j) {
    return (int) Math.signum(x[i] - x[j]);
}

          vs
          int compare(int i, int j) {
    return x[i] < x[j] ? -1 : x[i] > x[j] ? 1 : 0;
}


          Оба варианта выглядят не ахти, к тому же в первом варианте произойдут чудеса при переполнении. А всё потому, что не сделали нормальный Math.compare() для примитивных типов и .compareTo() для boxed объектов.

          По теме: использовать 
          Math.<double>signum()
          , когда аргументы имеют тип int - это как-то из пушки по воробьям. Более того, привыкнув так делать вы начнёте делать то же самое с long (то бишь с int64), а там и до <s>беды</s> потери точности недалеко.
          Ответить

    • ava guest8 08.04.2019 21:06 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 08.04.2019 22:12 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 09.04.2019 10:45 # −999

      показать все, что скрытоvanished
      Ответить

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