Conversation
HW1
| public double calculateArea(){ | ||
| return Math.PI*Math.pow(this.radius,2); | ||
| } | ||
| public void vvodRadius(){ |
There was a problem hiding this comment.
вместо vvodRadius принятописать английскими словами, потому что код может попасть и к англоязычным разработчикам или тем, кто назвал бы иначе. Для универсальности лучше использовать английские слова в названиях.
Вместо vvodRadius - inputRadius
| public static void main(String[] args) { | ||
| Circle oneCircle = new Circle(); | ||
| oneCircle.vvodRadius(); | ||
| double ploschad = oneCircle.calculateArea(); |
There was a problem hiding this comment.
заменить русскоязычное на английское слово
ploschad -> area
| } | ||
|
|
||
| public void printInfo() { | ||
| System.out.println(); |
There was a problem hiding this comment.
попробовать здесь использовать класс StringBuffer или StringBuilder c методом .append()
Меньше конкатенаций (сложения строк через +) - быстрее и легче в оперативке программа
| System.out.println("Hi! My name is " + this.name + ", I'm " + this.age + " years old"); | ||
| } | ||
|
|
||
| public void vvodInfo() { |
There was a problem hiding this comment.
заменить русскоязычное на английское слово
vvodInfo -> inputInfo
|
|
||
| Scanner in = new Scanner(System.in); // вызов функции для ввода числа из консоли в программу// | ||
| System.out.println("введите первое слогаемое"); | ||
| int a = in.nextInt(); // ввод числа в программу// | ||
| System.out.println("введите второе слогаемое"); | ||
| int b = in.nextInt(); | ||
| System.out.println("сумма чисел равна " + (a + b)); | ||
| System.out.println(); | ||
|
|
||
| Scanner in1 = new Scanner(System.in); | ||
| System.out.println("введите уменьшаемое"); | ||
| a = in1.nextInt(); | ||
| System.out.println("введите вычетаемое"); | ||
| b = in1.nextInt(); | ||
| System.out.println("разность чисел равна " + (a - b)); | ||
| System.out.println(); | ||
|
|
||
| Scanner in2 = new Scanner(System.in); | ||
| System.out.println("введите первый множитель "); | ||
| a = in2.nextInt(); | ||
| System.out.println("введите второй множитель"); | ||
| b = in2.nextInt(); | ||
| System.out.println("произведение равно " + (a * b)); | ||
| System.out.println(); | ||
|
|
||
| Scanner in3 = new Scanner(System.in); | ||
| System.out.println("введите делимое "); | ||
| float c = in3.nextInt(); | ||
| System.out.println("введите делитель"); | ||
| float d = in3.nextInt(); | ||
| System.out.println("частное равно " + (c / d)); |
There was a problem hiding this comment.
каждый блок по одной операции можно вывести в private методы в этом же классе, чтобы были компактнее методы и легче сам main. Если что - идея из книги "Чистый код". Компактные методы легче читать и поддерживать
| int num1 = ran.nextInt(); | ||
| int num2 = ran.nextInt(); | ||
| int num3 = ran.nextInt(); | ||
| System.out.println("сгенерированы 3 случайных числа " + num1 +", " + num2 + " ," + num3 ); |
There was a problem hiding this comment.
попробуй StringBuilder вместо + для строк и параметров
shpaser2
left a comment
There was a problem hiding this comment.
посмотри комментарии к строкам у файлов. Дал дельные советы исходя из своего опыта
KaydunovDenis
left a comment
There was a problem hiding this comment.
Необходимо внести исправления.
No description provided.