search

alekszalata / NumberWithDimensionSecondTry

0 stars 0 forks source link

Замечания #1

Open maximpetrov opened 7 years ago

maximpetrov commented 7 years ago
  1. Конструктор DimensionException не должен выводить сообщение в консоль.
  2. Для NumberWithDimension определите equals, hashCode и переименуйте compare в compareTo и реализуйте интерфейс Comparable.
  3. Не нужно перехватывать исключение прямо в том месте, где вы его выбрасываете.
  4. Храните размерность не строкой а объектом Dimension.
  5. Перенесите большую часть Normalise в конструктор Dimension из строки.
  6. Избавьтесь от дублирования кода в этом методе.
  7. Для Dimension определите equals, hashCode и toString.
alekszalata commented 7 years ago

@maximpetrov Все исправил. Не могли бы вы проверить?