Помогите разделить код на методы

Question

/* 
Online Java - IDE, Code Editor, Compiler 
 
Online Java is a quick and easy tool that helps you to build, compile, test your programs online. 
*/ 
import java.util.Scanner; 
 
public class Main 
{ 
    public static void main(String[] args) throws Exception { 
        Scanner scn = new Scanner( System.in ); 
        String exp = scn.nextLine(); 
        char action; 
        String[] data; 
        if (exp.contains(&#34; + &#34;)) { 
            data = exp.split(&#34; \+ &#34;); 
            action = '+'; 
        } else if (exp.contains(&#34; - &#34;)) { 
            data = exp.split(&#34; - &#34;); 
            action = '-'; 
        } else if (exp.contains(&#34; * &#34;)) { 
            data = exp.split(&#34; \* &#34;); 
            action = '*'; 
        } else if (exp.contains(&#34; / &#34;)) { 
            data = exp.split(&#34; / &#34;); 
            action = '/'; 
        }else{ 
            throw new Exception(&#34;Некорректный знак действия&#34;); 
        } 
        if (action == '*' || action == '/') { 
            if (data[1].contains(&#34;\&#34;&#34;)) throw new Exception(&#34;Строчку можно делить или умножать только на число&#34;); 
        } 
        for (int i = 0; i < data.length; i++) { 
            data[i] = data[i].replace(&#34;\&#34;&#34;, &#34;&#34;); 
        } 
 
        if (action == '+') { 
            printInQuotes(data[0] + data[1]); 
        } else if (action == '*') { 
            int multiplier = Integer.parseInt(data[1]); 
            String result = &#34;&#34;; 
            for (int i = 0; i < multiplier; i++) { 
                result+=data[0]; 
            } 
            printInQuotes(result); 
        } else if (action == '-') { 
            int index = data[0].indexOf(data[1]); 
            if(index == -1){ 
                printInQuotes(data[0]); 
            }else{ 
                String result = data[0].substring(0, index); 
                result+=data[0].substring(index+data[1].length()); 
                printInQuotes(result); 
            } 
        }else{ 
            int newLen = data[0].length()/Integer.parseInt(data[1]); 
            String result = data[0].substring(0,newLen); 
            printInQuotes(result); 
        }

} 
    static void printInQuotes(String text){ 
        if (text.length()>40){ 
            System.out.println(&#34;&#34;&#34;+text.substring(0,40)+&#34;...&#34;+&#34;&#34;&#34;); 
        } 
        else{ 
        System.out.println(&#34;\&#34;&#34;+text+&#34;\&#34;&#34;);} 
         
    } 
}

Татьяна · Accepted Answer

import java.util.Scanner; 
 
public class Main { 
    public static void main(String[] args) throws Exception { 
        Scanner scn = new Scanner(System.in); 
        String exp = scn.nextLine(); 
        char action = getAction(exp); 
        String[] data = getData(exp, action); 
 
        if (action == '*' || action == '/') { 
            validateMultiplicationOrDivision(data[1]); 
        } 
 
        for (int i = 0; i < data.length; i++) { 
            data[i] = data[i].replace(""", ""); 
        } 
 
        String result = calculateResult(action, data); 
        printInQuotes(result); 
    } 
 
    static char getAction(String exp) throws Exception { 
        if (exp.contains(" + ")) { 
            return '+'; 
        } else if (exp.contains(" - ")) { 
            return '-'; 
        } else if (exp.contains(" * ")) { 
            return '*'; 
        } else if (exp.contains(" / ")) { 
            return '/'; 
        } else { 
            throw new Exception("Некорректный знак действия"); 
        } 
    } 
 
    static String[] getData(String exp, char action) { 
        String[] data; 
        switch (action) { 
            case '+': 
                data = exp.split(" \+ "); 
                break; 
            case '-': 
                data = exp.split(" - "); 
                break; 
            case '*': 
                data = exp.split(" \* "); 
                break; 
            case '/': 
                data = exp.split(" / "); 
                break; 
            default: 
                data = new String[0]; // technically unreachable 
        } 
        return data; 
    } 
 
    static void validateMultiplicationOrDivision(String data) throws Exception { 
        if (data.contains(""")) { 
            throw new Exception("Строчку можно делить или умножать только на число"); 
        } 
    } 
 
    static String calculateResult(char action, String[] data) throws Exception { 
        String result = ""; 
        switch (action) { 
            case '+': 
                result = data[0] + data[1]; 
                break; 
            case '*': 
                int multiplier = Integer.parseInt(data[1]); 
                for (int i = 0; i < multiplier; i++) { 
                    result += data[0]; 
                } 
                break; 
            case '-': 
                int index = data[0].indexOf(data[1]); 
                if (index == -1) { 
                    result = data[0]; 
                } else { 
                    result = data[0].substring(0, index) + data[0].substring(index + data[1].length()); 
                } 
                break; 
            case '/': 
                int newLen = data[0].length() / Integer.parseInt(data[1]); 
                result = data[0].substring(0, newLen); 
                break; 
        } 
        return result; 
    } 
 
    static void printInQuotes(String text) { 
        if (text.length() > 40) { 
            System.out.println(""" + text.substring(0, 40) + "...""); 
        } else { 
            System.out.println(""" + text + """); 
        } 
    } 
}  Теперь код разбит на методы, что облегчает его чтение и сопровождение: 
 
getAction определяет действие на основе введенного выражения. 
getData разбивает строку на данные в зависимости от действия. 
validateMultiplicationOrDivision проверяет корректность деления или умножения строки. 
calculateResult вычисляет результат в зависимости от действия. 
printInQuotes выводит результат в кавычках. 
Этот подход помогает сделать код более структурированным и модульным.

Максим · Answer

Такой код лучше писать по типу: readInt, readOperator без всяких contains. Тем более что Scanner это хорошо умеет.