module 1, distance lab

[GoogleCodeExporter]

1) imin1 - имя переменной не в camelcase. vect и другие: 
лучше сделать чуть длиннее но осмысленней 
(см. Clean Code)
2) mass() - 
  а) название невыразительное 
  б) имена производные не от английских слов - плохая практика ;
     можно   назвать getArray(), retrievArray()
calc() и остальные по аналогии
3) пузырьковая сортировка - самая медленная 
из всех возможных ,  не нужно писать свою, 
т.к. есть сортировка из коробки
4) форматирование. отступы, пробелы около 
бинарных операторов. нужно делать по convention

Original issue reported on code.google.com by kolesa.a...@gmail.com on 17 Jan 2015 at 9:00

[GoogleCodeExporter]

1) Done
2) Done
3) Done
4) Done

Original comment by aleks.po...@gmail.com on 27 Jan 2015 at 9:20

[GoogleCodeExporter]

1) два раза создавать сканер - лишнее
2) по заданию на вход поступает ряд чисел 
через пробел
3) calcArray() - по названию не ясно, что делает 
метод. А сам метод с побочным эффектом

Original comment by kolesa.a...@gmail.com on 28 Jan 2015 at 9:57

[GoogleCodeExporter]

1) Deleted excessive Scanner;
2) String from user is now parsed into INT array;
3) Methods renamed;

Original comment by aleks.po...@gmail.com on 29 Jan 2015 at 9:32

[GoogleCodeExporter]

1) падает при запуске с:
Input array elements with spaces:
0 
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 1
    at ua.com.goit.gojava.alekspole.Distance.Distance.findMinElements(Distance.java:51)
    at ua.com.goit.gojava.alekspole.Distance.Distance.main(Distance.java:17)
2) пустой обработчик:
catch (NumberFormatException nfe) {};
в идеалу нужно проверять на валидность 
любой ввод юзера и выдавать адекватное 
ссобщение

Original comment by kolesa.a...@gmail.com on 30 Jan 2015 at 2:03

[GoogleCodeExporter]

Fixed this problem

Original comment by aleks.po...@gmail.com on 30 Jan 2015 at 2:44

[GoogleCodeExporter]

Exception нужно выбрасывать

Original comment by kolesa.a...@gmail.com on 30 Jan 2015 at 4:43