Skip to content

Проектная работа № 1 ver 2 - #2

Open
Satturas wants to merge 26 commits into
devfrom
work
Open

Satturas wants to merge 26 commits into
devfrom
work

Conversation

@Satturas

@Satturas Satturas commented Feb 8, 2024

Copy link
Copy Markdown
Owner

Спасибо за замечания, почти все рекомендации учел, с числами 11-14 обидно, что сразу не додумал)

Рекомендация: здесь удобно использовать scanner.hasNextInt()/hasNextFloat() - всё-таки оставил реализацию через try-catch parseInt, не смог пока найти решение корректно обрабатывать возможный ввод пустой строки через while(scanner.hasNextInt())

Во время тестирования обнаружил, что hashMap не выводит повторные товары (с одинаковыми key), поэтому переписал через arrayLists.

В первом PR понажимал лишнего, он сделался merged и closed. Не знаю как правильней, но создал новый.

Anastasia added 26 commits February 3, 2024 21:16
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
# Conflicts:
#	src/main/java/Calculator.java
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.

1 participant