Comments

Реквест по заданию в спринте 2 Практикума#1

Open

sred75 wants to merge 9 commits intomain from

master

Open

Реквест по заданию в спринте 2 Практикума #1
sred75 wants to merge 9 commits intomain from
master

Conversation

@sred75

Copy link

Owner

@sred75 sred75 commented Oct 31, 2022

Рабочий коммит - 050

s-buvaka and others added 9 commits

July 27, 2022 23:03


 Remove classes

e1f6893


 Change project name

c521234

@sred75


 коммит 010

b68aa71

@sred75


 коммит 010

82a8dd3

@sred75


 Merge remote-tracking branch 'origin/master'

a54fd55

# Conflicts:
#	src/main/java/goodsAddingEngine.java

@sred75


 коммит 020 - набивка массива товаров работает, но в сканере не обраба...

ec91d41

...тывается некорректный ввод, прога вылетает

@sred75


 коммит 030 - убрал проверку скаенера >0 внутрь проверки ввода на doub...

e5858d6

...le, переписал логику в новом классе goodsEntering, вылетов нет

@sred75


 коммит 040 - отладил ввод списка товаров и цен через goodsEntering, в...

d50cf48

...ылетов нет

@sred75


 коммит 050 - отладил и удалил всё лишнее

b5d629f

MagicUnderHood

MagicUnderHood reviewed

Nov 1, 2022

View reviewed changes

src/main/java/goodsEntering.java

import java.util.Scanner;

public class goodsEntering {

public static int goodsArrayCreating(){

int qtyOfGoods= 0;

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Можешь применить в студии автоформатирование (в выбранном файле, сверху вкладка Code - Reformat Code, либо Ctrl+Alt+L), тогда автоматически код выправится.

src/main/java/Main.java

System.out.println(calculator.cart);

System.out.println("Каждому человеку к оплате: " + formatter.roundResult(result) + " " + formatter.formatValue(result));

double payment = (double) sum / qtyOfGuys;

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Приведение типа к double (код (double) перед sum) необязательно, потому что sum и так типа double

src/main/java/goodsEntering.java

public static int goodsArrayCreating(){

int qtyOfGoods= 0;

String enteredName = "";

Double enteredPrice = 0d;

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Double с большой буквы это класс-обертка, а здесь достаточно использовать примитив double. Подробнее про классы-обертки будет в спринте 3, тема 1, урок 2 Списки. ArrayList
Инициализация 0d здесь необязательно, поскольку до использования переменной её значение будет перезаписано enteredPrice = price.nextDouble();

Copy link

Owner Author

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

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

Очень долго промучился с проверками и типами данных(( В результате душевной травмы пробовал принудительно присваивать типы некоторым значениям переменных))
Стало большим сюрпризом, что нужно очищать буфер сканера, иначе случался вылет в ошибку...

src/main/java/goodsEntering.java

Double enteredPrice = 0d;

Scanner name = new Scanner(System.in);

Scanner price = new Scanner(System.in);

while (!enteredName.equalsIgnoreCase("Завершить")) {

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Можно сделать цикл бесконечным while(true), а при проверках ввода ставить break; для выхода из него

Copy link

Owner Author

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

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

Сэкономил строчки кода на проверках и break)) Спасибо, что обратили внимание 👍

src/main/java/Main.java

qtyOfGoods = goodsEntering.goodsArrayCreating();

System.out.println("\nДОБАВЛЕННЫЕ ТОВАРЫ: ");

while (i < qtyOfGoods){

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Можно вместо qtyOfGoods использовать goodsArray.size(). Это ещё и предотвратит от попытки получить элемент по индексу, который больше размера массива

src/main/java/howManyGuys.java Show resolved Hide resolved

src/main/java/goodsEntering.java

@@ -0,0 +1,34 @@

import java.util.Scanner;

public class goodsEntering {

public static int goodsArrayCreating(){

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Можно метод делать не статическим, тогда для его вызова надо создавать экземпляр класса goodsEntering, и вызывать метод от экземпляра

src/main/java/howManyGuys.java

import java.util.Scanner;

public class howManyGuys {

static int qtyOfGuys;

public static void main(String[] args) {

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Этот метод здесь лучше убрать, потому что точка входа в приложение в методе main класса Main

src/main/java/howManyGuys.java

public static int howManyGuys() {

boolean flag = false;

System.out.print("На какое кол-во человек разделить счёт? ");

while (!flag) {

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Достаточно здесь написать в скобках true, и необходимости в переменной flag не будет

src/main/java/rubWord.java

String payOutput;

int payInputNorm = (int) payInput;

String payString = Integer.toString(payInputNorm);

char endOfPayment = payString.charAt(payString.length() - 1);

Copy link

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

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

Можно более простые варианты: для определения последней цифры - payInputNorm % 10. Плюс учесть, что для чисел 11-19 окончание слова - "рублей", поможет payInputNorm % 100

Copy link

Owner Author

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

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

Классный подход, спасибо 👍)

Labels

None yet

2 participants

@sred75 @MagicUnderHood

Comments

Conversation

@sred75 sred75 commented Oct 31, 2022

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

@sred75 sred75 Nov 1, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants