-
Notifications
You must be signed in to change notification settings - Fork 220
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Переменная не используется, можно удалить
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Кажется, переменная не используется, только инициализируется
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Лучше сначала округлить, а потом взять остаток от деления по модулю 100
Стоит добавить if с проверкой, что addsInvert >= 11 и <=19, тогда возвращать рублей
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А здесь добавить деление по модулю 10, чтобы брать последнюю цифру
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Можно здесь поставить просто else, потому что с предыдущими проверками тут members >0 будет всегда true
No description provided.