Conversation
added Alcohol.java, Color.java, Point.java
| private Color color; | ||
| private double transparency; | ||
| private double fillPercentage; | ||
| private double age; |
There was a problem hiding this comment.
Мы обычно не выравниваем поля по форматированию. Достаточно поставить пробел между типом и именем поля.
There was a problem hiding this comment.
Вообще есть волшебное сочетание ctrl alt L в idea, которое тебе выровняет весь код в текущем файле. Можно также применять, выбрав пакет - как результат получим форматирование по всем файлам в текущем пакете и вложенных.
| this.volume = 0.; | ||
| this.color = null; | ||
| this.transparency = 0.; | ||
| this.fillPercentage = 0.; |
There was a problem hiding this comment.
Никогда не видел такой конструкции, сходу не нагуглил. Что она означает?
| return this.manufacturer; | ||
| } | ||
|
|
||
| public void setManufacturer(String mname) { |
There was a problem hiding this comment.
Обычно стараются в качестве имени аргумента в сеттере использовать такое же имя, как и у поля.
| } | ||
|
|
||
| public String getName() { | ||
| return this.name; |
There was a problem hiding this comment.
В геттерах можно this не использовать
|
|
||
| public void setAge(double age) { | ||
| this.age = age; | ||
| } |
There was a problem hiding this comment.
Обычно геттеры-сеттеры идут в начале класса, а остальные методы - после них
| return 0.; | ||
| this.fillPercentage -= percent; | ||
| if (this.fillPercentage < 0.) | ||
| { |
There was a problem hiding this comment.
Скобочка по-сишному проскочила. Мы ставим на той же строке.
| double pouredOut = percent; | ||
|
|
||
| if (percent < 0.) | ||
| return 0.; |
There was a problem hiding this comment.
Минутка занудства. Есть рекомендация любые операции в циклах, ифах, элсах заключать в фигурные скобки, чтобы при внесении изменений не написать код мимо структурного блока.
| private int rgb; | ||
|
|
||
| Color(int r, int g, int b) { | ||
| this.rgb = ((r & 0xff) << 16) | ((g & 0xff) << 8) | b & 0xff; |
There was a problem hiding this comment.
Обычно без нужды мы не занимаемся разрядными сдвигами, побитовыми операции и тп. Здорово, что ты это знаешь, без шуток, но такие конструкции тяжело читать, а в масштабах работы всего приложения (как правило, большого) выигрыш они дают пшиковый.
| this.age = age; | ||
| } | ||
|
|
||
| Alcohol(String name, String manuf, double degree, double volume) { |
There was a problem hiding this comment.
А ты конструкторы намеренно в паблик не выводишь?
2 participants
No description provided.