Зачем писать такой большой код?

Question

[WannaCreative]

Всем привет , дорогие друзья!
Вот потихоньку учу Java, и, тут возник один вопрос.
Зачем писать прочие методы, если из за этого получается код длинее?
В чем различия, и, нужно ли все таки писать с методами?

public class Main
{
    public static void main(String[] args) throws IOException{
        int[] array = new int[5];
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < array.length; i++){
            String s = reader.readLine();
            int value = Integer.parseInt(s);
            array[i] = value;
        }
        InitializeArray(array);
    }
    public static int[] InitializeArray(int[] array){
        int max = array[0];
        int min = array[0];
        for (int i = 0; i < array.length; i++){
            if (max < array[i])
                max = array[i];
            if (min > array[i])
                min = array[i];
        }
        WriteLine(min, max);
        return array;
    }
    private static void WriteLine(int minimum, int maximum){
        System.out.println("Minumum is: "+minimum);
        System.out.println("Maximum is: "+maximum);
    }
}

Это пример с методами, а вот без них

public class Solution
{
    public static void main(String[] args) throws IOException
    {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int[] array = new int[5];
        for (int i = 0; i < array.length; i++){
            String s = reader.readLine();
            int value = Integer.parseInt(s);
            array[i] = value;
        }

        int minimum = array[0];
        int maximum = array[0];

        for (int i = 0; i < array.length; i++){
            if (minimum > array[i])
                minimum = array[i];
            if (maximum < array[i])
                maximum = array[i];
        }
        System.out.println("Maximum is: "+maximum);
        System.out.println("Minimum is: "+minimum);
    }

}

P.S | Во втором случае код короче, и легче понять, лично мне

Answer 1

[Alexander]

Все зависит от ситуации - метод решает какую-то отдельную задачу, разделение на методы нужно для того чтобы не повторять один и тот же код, а просто вызвать готовый метод. Позволяет структурировать код. Избегать ошибок - вы решили немного поменять код и не нужно лезть и искать по всему проекту, где вы его использовали, просто исправляете метод. В вашем случаи, если вы не собираетесь вызывать те методы из других функций, то можно все совместить в одной.

Comments

[WannaCreative]

Спасибо за ответ! А Вы делаете приложение для IOS на чем? SWIFT?

[Владимир Грабко]

WannaCreative: на c++ КАНЕШ

[Alexander]

WannaCreative: на Objective C, свифт пока еще не смотрел

Answer 2

[Дмитрий Вапельник]

добавлю к вышесказанному: такой код проще тестировать

Answer 3

[Вячеслав]

Разница в том, что за лапшу из 200-300 строк без методов, вас побьют. Ибо читать это невозможно. И, возможно, нарушит кучу принципов проектирования. В вашем случае без методов допустимо сделать.

Answer 4

[Роман Румянцев]

Лучше выделять в отдельные методы блоки кода, решающие определённую задачу или используемые многократно, чтобы потом самому меньше путаться и сильно упростить жизнь людям, пытающимся разобраться в вашем коде.
Но в данном случае за такое разбиение программиста нужно бить большой дубиной: метод InitializeArray не инициализирует массив, а ищет минимум и максимум, печатает их в консоль и зачем-то возвращает свой параметр, хотя он его не изменял. Метод WriteLine печатает не одну строку, а две. Главный метод никак не использует возвращаемое значение метода InitializeArray. В общем, лучше не разбивать на методы совсем, чем разбивать так.

Answer 5

[asd111]

Насколько я понял метод InitializeArray находит минимум и максимум в массиве и затем печатает их.
Поэтому желательно назвать метод InitializeArray так чтобы из названия было понятно что он делает. Например вместо InitializeArray можно назвать метод find_min_and_max_and_print_them - т.е. найти минимум и максимум и вывести их. От метода WriteLine можно избавиться потому что он ничего по сути не делает.
Ещё можно обернуть в функцию место где создается массив и вводятся данные.
И ещё нужно сделать чтобы размер массива где то указывался.
В java конечно CamelCase, но на мой взгляд underscore легче читается.
Куски кода нужно оборачивать в функции для инкапсуляции, т.е. чтобы скрыть подробности работы за названием функции, поэтому функциям нужно давать имена из которых понятно что данная функция делает. Например InitializeArray - не очень понятное название для функции которая ищет и выводит максимум и минимум. А например если такую функцию назвать Найти_минимум_и_максимум_в_массиве_и_вывести_значения - то сразу всё понятно.

Код не проверял возможно есть ошибки, но думаю примерная суть понятна.
Т.е. в целом можно сделать так:

public class Main
{
    const int ARRAY_SIZE = 5;
    public static void main(String[] args) throws IOException{
        int[] working_array = create_array_and_fill_it_from_stdin();
        find_max_and_min_in_array_and_print_them(working_array);
    }
    public static int[] create_array_and_fill_it_from_stdin(){
        int[] array = new int[ARRAY_SIZE];
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < array.length; i++){
            String s = reader.readLine();
            int value = Integer.parseInt(s);
            array[i] = value;
        }
        return array;
    }
    public static void find_max_and_min_in_array_and_print_them(int[] array){
        int max = array[0];
        int min = array[0];
        for (int i = 0; i < array.length; i++){
            if (max < array[i])
                max = array[i];
            if (min > array[i])
                min = array[i];
        }
        System.out.println("Minumum is: "+min);
        System.out.println("Maximum is: "+max);        
    }
}

Comments

[Денис Загаевский]

И как только у вас в названии метода появляется and, надо ударить себя по рукам. Нарушение буковки S(SRP) из SOLID. За подчёркивания вообще убейтесь.

[asd111]

Денис Загаевский: lol, зачем убиваться если я пишу на python, я же написал что мне так удобнее, и для чтения на мой взгляд underscore проще чем CamelCase.
Вы могли бы выразить своё мнение не так грубо и вас было бы приятнее слушать.

Answer 6

[abcd0x00]

Изучи метод разработки "сверху вниз". Если его не освоишь, ничего длиннее ста строк писать не будешь.
wiki. структурное программирование

Answer 7

[mikhail_404]

И, конечно, не забываем про SOLID!