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

    +75.4

    1. 1
    2. 2
    3. 3
    4. 4
    if (!Float.valueOf("0.0").equals(price.getActualPrice()))
{
	return true;
}

    Сравнение чисел (float) нездоровым способом.

    Запостил: asolntsev, 04 Марта 2010

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

    • ava guest 04.03.2010 19:25 # 0

      паттерн java-style :)
      Ответить

      • ava Amok 04.03.2010 21:33 # 0

        Если предположить, что он боится NullPointerException, то код проканает. Хотя за данный Float.valueOf("0.0") надо руки отрывать! На порядок лучше Float.valueOf(0).

        В противном случае, если price.getActualPrice() - примитив, еще проще
        if (price.getActualPrice() == 0f)
{
	return true;
}
        Ответить

        • ava guest 05.03.2010 00:48 # +1

          а если price.getActualPrice() == 1E-6 ? вобще для денег нельзя использовать float.
          Ответить

        • ava Balza 05.03.2010 05:53 # +1

          очень не рекомендуется сравнивать вещественные переменные с помощью оператора ==
          Ответить

          • ava asolntsev 05.03.2010 09:44 # 0

            А как же тогда?
            Ответить

          • ava Amok 05.03.2010 13:25 # 0

            Что такое по-твоему примитив в Java?

            На самом деле если важна точность вычислений, то в принципе нужен класс BigDecimal - рекомендации Sun.
            Ответить

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