Repository navigation
Проектная работа №1 - #1
VirusDobra wants to merge 1 commit into
Conversation
| public class Result { | ||
| Calculator menuMany = new Calculator(); | ||
| Friends kolVoPeople = new Friends(); | ||
| double Finish(){ |
There was a problem hiding this comment.
⏫ Лучше стараться называть все функции с маленькой буквы, такой принцип нейминга общепринятый в java и kotlin
| Calculator menuMany = new Calculator(); | ||
| Friends kolVoPeople = new Friends(); | ||
| double Finish(){ | ||
| double zrachKa,peOple,reShala,celoe; |
There was a problem hiding this comment.
⏫ Также при нейминг переменных лучше придерживаться camelCase способа - каждое новое слово в названии пишется с большой буквы
| } | ||
|
|
||
|
|
||
| /* double i = 4; |
There was a problem hiding this comment.
⏫ Лучше в итоговой версии удалять закомментированный код и лишние отступы
|
|
||
| public class Friends { | ||
|
|
||
| boolean restart = true; |
There was a problem hiding this comment.
⏫ Переменная restart используется только в методе friends, поэтому нам не обязательно создавать как поле класса. Мы можем объявить ее лишь в поле friends
| if (numberFriends.hasNextInt()) { | ||
| int number = numberFriends.nextInt(); | ||
| if (number <=1 ){ | ||
| System.out.print("Количество человек, равно 1. В этом случае нет смысла ничего считать и делить.\nВведите сколько вас гостей: "); |
There was a problem hiding this comment.
⏫ Комментарий по выводимому тексту пользователю: мы не уверены, что он ввел именно 1. Он также мог ввести 0 или отрицательное число
| double Calculator() { | ||
|
|
||
| boolean restart = true; | ||
| String chek = " ", menu = " ", stop = "Завершить"; |
There was a problem hiding this comment.
⏫ Обычно в нейминге переменных используются английские слова:
chek->receiptobshag->commonmany->money
2 participants
Спасибо за замечания, становлюсь лучше