Как можно упростить этот java код?

Question

[GromTron121]

Ребята, подскажите как можно было бы упростить этот код :

String path1 = "";  // Пути к файлам 
        String path2 = "";

        File file = new File(path1);
        Scanner sc = new Scanner(file);
        PrintWriter pw = new PrintWriter(path2);

        String match1 = sc.nextLine();
        String match2 = sc.nextLine();
        String match3 = sc.nextLine();
        String match4 = sc.nextLine();
        
        String scMatch1[] = match1.split(" ");
        String scMatch2[] = match2.split(" ");
        String scMatch3[] = match3.split(" ");
        String scMatch4[] = match4.split(" ");

        int a1 = Integer.parseInt(scMatch1[0]);
        int a2 = Integer.parseInt(scMatch2[0]);
        int a3 = Integer.parseInt(scMatch3[0]);
        int a4 = Integer.parseInt(scMatch4[0]);

        int b1 = Integer.parseInt(scMatch1[1]);
        int b2 = Integer.parseInt(scMatch2[1]);
        int b3 = Integer.parseInt(scMatch3[1]);
        int b4 = Integer.parseInt(scMatch4[1]);

        if(a1+a2+a3+a4 > b1+b2+b3+b4){
            System.out.println("1");
            pw.print("1");
        } else if(a1+a2+a3+a4 < b1+b2+b3+b4){
            System.out.println("2");
            pw.print("2");
        } else {
            System.out.println("DRAW");
            pw.print("DRAW");
        }
        sc.close();
        pw.close();

Answer 1

[p_A_m_A_git_I]

Не использовать в продакшене. ̶Е̶с̶т̶ь̶ ̶ш̶а̶н̶с̶ ̶ч̶т̶о̶ ̶в̶а̶с̶ ̶п̶о̶б̶ь̶ю̶т̶.̶

public class Main {
    public static void main(String[] args) throws Exception {
        String inputFile = "", outputFile = "";
        String res = Files.lines(Paths.get(inputFile))
                .map(line -> new int[]{
                        Integer.parseInt(line.split(" ")[0]),
                        Integer.parseInt(line.split(" ")[1])}
                ).reduce((identity, accumulator) -> new int[]{
                        identity[0] + accumulator[0],
                        identity[1] + accumulator[1]
                }).map(resArr -> {
                    if (resArr[0] > resArr[1]) return "1";
                    if (resArr[0] < resArr[1]) return "2";
                    return "DRAW";
                }).get();

        PrintWriter printWriter = new PrintWriter(outputFile);
        printWriter.write(res);
        printWriter.close();
    }
}

Comments

[Dmitry Roo]

Чуть-чуть еще докрутить чтобы не побили:

public class Main {

    private final static Pattern pattern = Pattern.compile("(?<left>\\d+)\\s(?<right>\\d+)");

    public static void main(String[] args) throws Exception {
        var inputFile = "in.txt";
        var outputFile = "out.txt";
        Files.lines(Paths.get(inputFile))
                .limit(5) // Limit if needed
                .map(Main::parseToResult)
                .filter(Optional::isPresent)
                .map(Optional::get)
                .reduce((identity, accumulator) -> new Result(identity.left + accumulator.left,
                        identity.right + accumulator.right))
                .map(Result::getResultAsString)
                .ifPresent(s -> {
                    try (PrintWriter printWriter = new PrintWriter(outputFile)) {
                        printWriter.write(s);
                    } catch (Exception ignored) {/*NooP*/}
                });
    }

   
    private static Optional<Result> parseToResult(String line) {
        var matcher = pattern.matcher(line);
        if (matcher.find()) {
            var left = Integer.valueOf(matcher.group("left"));
            var right = Integer.valueOf(matcher.group("right"));
            return Optional.of(new Result(left, right));
        } else {
            return Optional.empty();
        }
    }

    private record Result(Integer left, Integer right) {
        String getResultAsString() {
            if (left > right) {
                return "1";
            } else if (left < right) {
                return "2";
            } else {
                return "draw";
            }
        }
    }
}

[p_A_m_A_git_I]

Dmitry Roo, еще немного ...

public class Main {

    private final static Pattern pattern = Pattern.compile("(?<left>\\d+)\\s(?<right>\\d+)");
    private final static String fileSuffix = ".txt";
    private final static String ifLeftIsBiggerMessage = "1";
    private final static String ifRightIsBiggerMessage = "2";
    private final static String ifEqualMessage = "DRAW";

    private final static int defaultLimit = 5;
    
    @SuppressWarnings(value = "all")
    public static void main(String... args) throws Exception {
        //To set the limit when launching the application.
        var limit = getLimitFromArr(args);
        var inputFile = String.format("in.%s", fileSuffix);
        var outputFile = String.format("out.%s", fileSuffix);
        Files.lines(Paths.get(inputFile))
                .limit(limit) // Limit if needed
                .map(Main::parseToResult)
                .filter(Optional::isPresent)
                .map(Optional::get)
                .reduce((identity, accumulator) ->
                        new Result(identity.left + accumulator.left,
                                identity.right + accumulator.right))
                .map(Result::getResultAsString)
                .ifPresent(s -> {
                    try (PrintWriter printWriter = new PrintWriter(outputFile)) {
                        printWriter.write(s);
                    } catch (Exception ignored) {/*NooP*/}
                });
    }

    private static int getLimitFromArr(String... args) {
        return args.length > 0 ? Integer.parseInt(args[0]) : defaultLimit;
    }

    private static Optional<Result> parseToResult(String line) {
        Matcher matcher = pattern.matcher(line);
        return matcher.find() ?
                Optional.of(new Result(
                        Integer.valueOf(matcher.group("left")),
                        Integer.valueOf(matcher.group("right"))
                )) : Optional.empty();
    }

    private record Result(Integer left, Integer right) {
        String getResultAsString() {
            return !Objects.equals(left, right) ?
                    String.valueOf(Integer.compare(left, right))
                            .replace("-1", ifRightIsBiggerMessage)
                            .replace("1", ifLeftIsBiggerMessage) : ifEqualMessage;
        }
    }
}

[Dmitry Roo]

p_A_m_A_git_I, ну тут с record Result вы уж перемудрили. И логика не прозрачная и константы вполне он сам может инкапсулировать.

Answer 2

[Орхан Гасанлы]

Добрый день. А можно ли узнать вашу конечную цель? Что вы пытаетесь реализовать?
Код действительно можно сократить, а то и лучше улучшить восприятие.
1) создайте 2 метода, первый из которых будет читать файл, а второй писать в файл. Можно воспользоваться возможностями класса Files из Java 8
2)

String match1 = sc.nextLine();
        String match2 = sc.nextLine();
        String match3 = sc.nextLine();
        String match4 = sc.nextLine();
        
        String scMatch1[] = match1.split(" ");
        String scMatch2[] = match2.split(" ");
        String scMatch3[] = match3.split(" ");
        String scMatch4[] = match4.split(" ");

        int a1 = Integer.parseInt(scMatch1[0]);
        int a2 = Integer.parseInt(scMatch2[0]);
        int a3 = Integer.parseInt(scMatch3[0]);
        int a4 = Integer.parseInt(scMatch4[0]);

        int b1 = Integer.parseInt(scMatch1[1]);
        int b2 = Integer.parseInt(scMatch2[1]);
        int b3 = Integer.parseInt(scMatch3[1]);
        int b4 = Integer.parseInt(scMatch4[1]);

тут происходит что-то странное. Скорее всего тут есть ошибка.
Вы сперва читаете строку, затем разделяете ее пробелом в массив строк, но потом почему-то присваиваете a1, а2. а3. а4 одно и то же число.

int a1 = Integer.parseInt(scMatch1[0]);
        int a2 = Integer.parseInt(scMatch2[0]);
        int a3 = Integer.parseInt(scMatch3[0]);
        int a4 = Integer.parseInt(scMatch4[0]);

скорее всего должно быть так:

int a1 = Integer.parseInt(scMatch1[0]);
        int a2 = Integer.parseInt(scMatch2[1]);
        int a3 = Integer.parseInt(scMatch3[2]);
        int a4 = Integer.parseInt(scMatch4[3]);

Такая же ситуация с перемеными b1, b2, b3, b4.
А что если человек введет не 4 элемента через пробел, а 5 и более? Или наоборот менее 4-х элементов?

Подобные выражения a1+a2+a3+a4 > b1+b2+b3+b4 с точки зрения clean code можно вынести в отдельную переменную типа boolean.
Например,

boolean isSumAGreaterThanSumB = a1+a2+a3+a4 > b1+b2+b3+b4;

Ну а далее if (isSumAGreaterThanSumB ) { ... }