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

    +83

    1. 1
    2. 2
    3. 3
    private double calculateValue( int number, int rate ) {
    return Math.pow( number, rate );
}

    Запостил: galak, 31 Января 2011

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

    • ava Lure Of Chaos 31.01.2011 21:36 # −1

      поди туда ты знаешь куда = )
      Ответить

    • ava tir 01.02.2011 09:58 # +1

      отсутствует контекст, сложно сказать, действительно это ГК или нет
      Ответить

      • ava galak 01.02.2011 14:58 # 0

        Контекст здесь совершенно не важен. Главный перл этого отрывка -- наименование метода, говорящее буквально обо всем.
        Ответить

        • ava Lure Of Chaos 01.02.2011 15:02 # −1

          угу, из понятного наименования сделали абстрактное "посчитать"
          Ответить

        • ava tir 01.02.2011 15:51 # 0

          someObject.setCalculatedValue(calculateV alue(10, 20)) - по мне нормальное именование метода при таком использовании
          Ответить

          • ava galak 01.02.2011 20:39 # 0

            Врядли этот пример можно считать удачным. Вместо calculateValue вы бы скорее написали, что то о том какие именно вычисления производятся над значениями.
            Ответить

            • ava tir 01.02.2011 20:59 # +1

              а если формула оказалась бы чуть сложнее, например:
              Math.pow( number, rate ) - 1

              честно, я бы вряд ли придумал более подходящее название =) а вдруг формула будет меняться?

              я все это к тому, что без контекста достаточно сложно судить о качестве кода. если же из контекста очевидно, что ничего меняться не будет - это, конечно, далеко не самый лучший код.
              Ответить

              • ava galak 01.02.2011 21:27 # 0

                да алгоритм вычислений мог бы изменится, но их функциональное назначение -- нет
                Ответить

    • ava bugmenot 01.02.2011 10:04 # +1

      delegate || boilerplate
      Ответить

    • ava Lure Of Chaos 01.02.2011 15:05 # −2

      какого хуя так названо? (почти цитата)
      Ответить

    • ava Belorus 17.02.2011 14:52 # −1

      Не ГК.
      Ответить

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