Fixed receiving error

Замечания по лаб№1

1. summaryByMod
1.1 Почему ошибка если задан массив нулевой длинны? В это случае
результат = 0, но не ошибка. Проверка лишняя
1.2 System.exit()  - останавливает текущую JVM. Это зачем? И представь,
если твой класс используется в серверной программе – остановится весь
сервер. Вообще ситуаций, когда реально нужен System.exit() в обычной
жизни не возникает
1.3 Полный if – имелся ввиду, не тернарная операция sum = mas[i]%mod==0
? mas[i] : 0. Т.е. else там не нужен
2. comparePositive
2.1 value == 0 && value == -0 – а это как? Зачем «-0»?
3. comparePositiveWithError
3.1 Здесь нужно было не дублировать логику метода comparePositive, а им
воспользоваться. Т.е. метод comparePositive должен вызываться из метода
comparePositiveWithError
3.2 Кстати, здесь уже value == 0 без «-0». В чем разница?
4. scalarProduct
4.1 «Arrays have the same length.» Видимо, надо «don’t have» или «have
to have» или «should have»
5. scalarProduct
5.1 А зачем вот это?
if (mass1.length < i)
break;
6. divisionPositivesByNegatives
6.1. else if (mass[i] > 0) – здесь if лишний.
6.2 нет проверки «деления на «0»

Замечания по лаб№2

1. absolutOfComparison.
1.1 здесь можно было воспользоваться Math.max() вместо sinCos > cosSin ?
sinCos : cosSin.
2. deleteElementsWithLengthByMod3
2.1 Входной параметр должен быть именно List, а не LinkedList.
2.2 Нужно было использовать Iterator
3. drawStringWithoutStringsByMod3
3.1 То же самое: входной параметр должен быть List
3.2 Вместо null лучше возвращать пустую строку («») – меньше вероятности
NullPointerException
4. convertToList
4.1 То же самое: вместо Null лучше вернуть путую коллекцию (например,
Collections.emptyList())
4.2 Integer.parseInt(string) – возвращает int. За счет автобоксинга у
тебя int преобразуется в Integer. Но можно было бы сразу получить
Integer, используя метод Integer.valueOf
5. convertToSortedArray
5.1 Тип возвращаемого арзумента должен быть именно int[], а не Integer[]