Final version. Cosmetic changes2#115
Final version. Cosmetic changes2#115manualcheg wants to merge 1 commit intoYandex-Practicum:masterfrom
Conversation
| public static void input(int numberOfFriends) { | ||
| Scanner scanner = new Scanner(System.in); | ||
| String roubles; | ||
| String[] product = new String[100]; |
There was a problem hiding this comment.
В данном случае ввод ограничен 100 товарами, после чего программа завершится. Скорее всего до 100 товаров не дойдет, но лучше подстраховаться и использовать ArrayList
There was a problem hiding this comment.
Хотелось выполнить работу соответственно уровню знаний, которые нам дали.
Формально ни массивы, ни arrayList нам неизвестны.
Но со строковой переменной не хотелось, хотя и разобрался, как это можно было.
Ну и, да, в реальности выполнения программы до 100 товаров никто не доберётся. Впрочем, как и в жизни :)
Спасибо за ревью!
| @@ -0,0 +1,70 @@ | |||
| import java.util.Scanner; | |||
|
|
|||
| public class calc { | |||
There was a problem hiding this comment.
Классы в Java принято называть с большой буквы
Про конвенции названий можно почитать здесь: https://www.oracle.com/java/technologies/javase/codeconventions-namingconventions.html
There was a problem hiding this comment.
да, понял, что ошибся только перед pull request'ом :(
|
|
||
| System.out.println("Программа \"Калькулятор счёта\""); | ||
| numberOfFriends = inputOfFriends(); | ||
| calc.input(numberOfFriends); |
There was a problem hiding this comment.
Можно запустить программу одной строчкой
calc.input(inputOfFriends());
Приложение работает.
Выполняются все требования задания.
Реализовано 2 класса: Main, calc.