Feature/review#1
Conversation
| import java.util.Arrays; | ||
| import java.util.Scanner; | ||
|
|
||
| public class Main { |
There was a problem hiding this comment.
- Формат компилируемых файлов должен быть *.java
- Класс с точкой входа принято называть Main
- Наименование файла должно совпадать с наименованием класса
- package подразумевает, что файл находится в директории, а не в корне проекта
- Желательно добавлять комментарий к классу в формате
/* <Что делает класс>
@autor Имя_Фамилия
*/
| import java.util.Scanner; | ||
|
|
||
| public class Main { | ||
| public static void checkYear(int year) { |
There was a problem hiding this comment.
наименование checkIsLeap(int year) лучше отражает функциональность
| if (deliveryDistance > 20) { | ||
| deliveryTime++; | ||
| } | ||
| if (deliveryDistance > 60) { |
There was a problem hiding this comment.
условия >20 и >60 перекрывают друг-друга, соответственно
if (deliveryDistance > 60) можно убрать
|
|
||
| public static void checkOS(int deviceYear, int clientOS) { | ||
| int currentYear = LocalDate.now().getYear(); | ||
| if (clientOS == 1) { |
There was a problem hiding this comment.
вынести в константы значения, которые отвечают за определение OS
private static final int ANDROID_OS = 0;
private static final int IOS_OS = 1;
| } | ||
| } | ||
|
|
||
| public static void checkOS(int deviceYear, int clientOS) { |
There was a problem hiding this comment.
предлагаю выделить из данного метода два метода:
- метод получения OS - возвращать iOS или Android
- метод, который определяет, какое обновление нужно lite-version или full-version
при этом нужно не забыть про модификаторы доступа
а в основной метод передавать результат выполнения предыдущих двух и печатать информацию через
System.out.println(String.format("Install the %s for %s at the link", typeOfUpdate, osName));
из плюсов- уходим от вложенности if-else.
There was a problem hiding this comment.
также нужно переименовать метод т.к. данное название не совсем отображает то, что делает метод
| return averageValue; | ||
| } | ||
|
|
||
| public static double getSum(int[] arr) { |
There was a problem hiding this comment.
сумма целых чисел не может быть вещественным числом. возвращаемым значением должен быть int
| } | ||
|
|
||
|
|
||
| public static void main(String[] args) { |
There was a problem hiding this comment.
перенести данный метод в начало класса, так будет проще отследить что делает данный класс
There was a problem hiding this comment.
блоки(таски) предлагаю вынести в отдельные методы
| return deliveryTime; | ||
| } | ||
|
|
||
| public static void checkDoubles(String str) { |
There was a problem hiding this comment.
предлагаю переименовать метод в checkDuplicateCharacter
1 participant
No description provided.