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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
try {
    order.setAddress(address.clone());
} catch (CloneNotSupportedException e) {
    // We will not get here, because priorAddressDtoToAdd.address is of Address type.
    // Moreover - none of the subclasses throws this exception
    LOG.error("Address clone is not supported? Nonsense...", e);
}

Это было бы даже весело, если бы не повторялось вместе с коментами 3 раза в одном классе.

Запостил: roman-kashitsyn roman-kashitsyn, (Updated )

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

  • Что ж делать, если в Жабе механизм Cloneable реализован настолько криво, что его лучше вообще не использовать ни для чего (ну разве что массивы копировать).

    Хотя автор, переопределивший Cloneable и не убравший CloneNotSupportedException - конечно, ССЗБ.
    Ответить
  • Спасибо авторам Жавы. И после этого ещё говорят о длине верёвки в C++.
    Ответить
    • Joshua Bloch рекомендует для копирования использовать копирующий конструктор, и я с ним солидарна. Многие серьёзные API используют именно такой подход.
      Ответить

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

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

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


    8