Repository navigation
Conversation
…прашивает количество персон для разделения счета, но пока еще не может отработать ввод числа буквами. Также создал класс Product, с помощью которого буду создавать объекты данного класса, для создания списка заказанных позиций из общего счета.
…тформатированный вывод стоимости позиций из счета и суммы распила, а также добавить обозначение валюты.
… счету, данное значение выводся в консоль с двумя знаками после запятой. + чуть переписал README.md и добавил новый счетчик в класс Main.
…, но он пока не до конца правильно работает, буду исправлять.
…а Рубль работает коректно.
| @@ -0,0 +1,9 @@ | |||
| public class Product { | |||
| String nameProduct = "name"; | |||
There was a problem hiding this comment.
значение name для этой переменной никогда не будет достигнуто, т к при создание объекта (вызове конструктора) оно перезатрется. об этом подсказывает idea ;)
| int a = roundingUp(value); | ||
| int val = String.valueOf(a).length(); | ||
|
|
||
| while(true) { |
There was a problem hiding this comment.
кажется, что этот цикл здесь совсем не обязателен - его можно удалить
|
|
||
| String formatter(Double value){ | ||
| String str = ""; | ||
| int a = roundingUp(value); |
There was a problem hiding this comment.
переменным нужно стараться присваивать говорящие, а не абстрактные имена :)
| while(true) { | ||
| switch (val) { | ||
| case 0: { | ||
| str = "рублей"; |
There was a problem hiding this comment.
слово рублей повторяется трех местах данного класса - его лучше вынести в константы (как и другие слова)
private static final String RUB = "рублей"
|
|
||
| void addingProductsToConsole(){ | ||
| for (int i = 0; i < listProduct.size(); i++ ){ | ||
| String text = String.format("%d-я позиция по счету: %s - стоимость которой = %.2f %s", i |
There was a problem hiding this comment.
довольно громоздкая конструкция для восприятия. для удобочитаемости можно вынести подставляемые значения в отдельные переменные, либо воспользоваться StringBuilderом
| } | ||
| } | ||
| static int checkingUserInputString (){ | ||
| Scanner scanner = new Scanner(System.in); |
There was a problem hiding this comment.
не нужно создавать много сканнеров - достаточно одного экземпляра этого класса
public class Main {
private static final Scanner scanner = new Scanner(System.in)
public static void main(String[] args) {...
2 participants
Выполнил практическое задание 2-ого спринта, реализовал вроде все, что просили в ТЗ к заданию.