Jawa / Говнокод #20081 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
@Override
public boolean isIsImpersonated() {
    return isIsImpersonated();
}

Геттер для isImpersonated.

Запостил: Mordavolt Mordavolt, (Updated )

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

    • ava inkanus-gray # Ссылка на оригинал 0 ⇈

      То ли дело "PHP":
      public __call ($name, $arguments ) {
    if ($name === 'Impersonated') {
        return $this->_impersonated;        // достигли последнего уровня, возвращаем поле 
    } else if (stripos($name, 'is') === 0) {
        $name = substr($name, 2);            // убираем префикс "is" 
        return $this->$name();                    // совершаем "рекурсивный" вызов
    } else {
        throw new Exception('Попытка вызвать несуществующий метод.');
    }
}
      Ответить

      • ava Mordavolt # Ссылка на оригинал 0 ⇈

        Ну не считая дикого названия, код внутри должен был быть тупо 
        return isImpersonated;

        Просто кто-то умудрился назвать метод по дебильному, заавтокомплитить его неверно и оставить эту бесконечную, бессмысленную и беспощадную рекурсию вместо простого ретурна.
        Ответить

  • ava Steve_Brown # Ссылка на оригинал 0 

    // Логические переменные должны начинаться на is!
// bool wasCollision;
bool isWasCollision;
    Ответить

    • ava gost # Ссылка на оригинал 0 ⇈

      > Логические переменные должны начинаться на is!
      А мне всегда казалось, что на is должны начинаться их геттеры...
      object.connected -> object.isConnected()
      Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓