Skip to content

Conversation

@Egorov02
Copy link
Owner

Калькулятор счета "состоит" из 4 классов. Не совсем понял, когда нужно ставить staticм перед методами и параметрами. Android Studio подсказала и всё заработало.
Еще столкнулся со следующей ситуацией: Форкнул проект->открыл в Android Studio, начал писать код, а Android Studio предложил обновиться до Giraffe -> обновился-> открыл проект, а его невозможно запустить. Просто неактивна кнопка "run". Что я только не пробовал. Не знаю как надо было решать, но мне помогло удалить проект и скачать форкнутый в новое расположение. После этого, открыв его в Android Studio, снова всё заработало. На самом деле так до конца и не разобрался почему такое случилось.

Copy link

@Charmandik Charmandik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом работа очень хорошая
Я бы рекомендовал исправить 2 момента - не используй статические поля и функции, научись работать без них
Передавай все необходимые тебе зависимости через функции или же через конструктор класса

public class Calculator {
static double totalAmount;

void calculator() {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Имена функций должны содержать в себе действие. Функция делает что-либо. Хорошо помогает практика переводить имена функций дословно, если после перевода понятно чем занимается функция, то название хорошее

System.out.println("Давайте добавим товар в счет.");
while (true) {
System.out.print("Введите название товара: ");
product.productName = Main.scanner.next();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

старайся не использовать статические поля и функции, если только это не константы

System.out.println("Давайте добавим товар в счет.");
while (true) {
System.out.print("Введите название товара: ");
product.productName = Main.scanner.next();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

используй метод nextLine, так как иначе название товара может содержать только одно слово

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants