Repository navigation
Conversation
Charmandik
left a comment
Charmandik
left a comment
There was a problem hiding this comment.
В целом работа очень хорошая
Я бы рекомендовал исправить 2 момента - не используй статические поля и функции, научись работать без них
Передавай все необходимые тебе зависимости через функции или же через конструктор класса
| public class Calculator { | ||
| static double totalAmount; | ||
|
|
||
| void calculator() { |
There was a problem hiding this comment.
Имена функций должны содержать в себе действие. Функция делает что-либо. Хорошо помогает практика переводить имена функций дословно, если после перевода понятно чем занимается функция, то название хорошее
| System.out.println("Давайте добавим товар в счет."); | ||
| while (true) { | ||
| System.out.print("Введите название товара: "); | ||
| product.productName = Main.scanner.next(); |
There was a problem hiding this comment.
старайся не использовать статические поля и функции, если только это не константы
| System.out.println("Давайте добавим товар в счет."); | ||
| while (true) { | ||
| System.out.print("Введите название товара: "); | ||
| product.productName = Main.scanner.next(); |
There was a problem hiding this comment.
используй метод nextLine, так как иначе название товара может содержать только одно слово
2 participants
Калькулятор счета "состоит" из 4 классов. Не совсем понял, когда нужно ставить staticм перед методами и параметрами. Android Studio подсказала и всё заработало.
Еще столкнулся со следующей ситуацией: Форкнул проект->открыл в Android Studio, начал писать код, а Android Studio предложил обновиться до Giraffe -> обновился-> открыл проект, а его невозможно запустить. Просто неактивна кнопка "run". Что я только не пробовал. Не знаю как надо было решать, но мне помогло удалить проект и скачать форкнутый в новое расположение. После этого, открыв его в Android Studio, снова всё заработало. На самом деле так до конца и не разобрался почему такое случилось.