Java. Как «скормить» регулярку компилятору?

Question

[Nanahuatzin]

Здравствуйте, товарищи! Я - новичок как в деле программирования, так и в деле освоения регулярных выражений.

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

Так вот, первым делом решил разобраться с валидацией формы заказа. Он должен быть вида:
"Имя, адрес, заказ, сумма". И тут же напоролся на регулярку. Сделал сначала по-своему - не работает. Перепробовал многое, но ничего не выходит. Кое-кто предложил мне вариант попроще, и в его примере всё происходит так, как описано кодом, но у меня - нет.

Вот мой код (и да не вытекут у вас глаза):

spoiler

import java.io.IOException;
import java.util.Scanner;
import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class Pizza {

    public static String orderDescription() {

        System.out.println("Input a data about new order in format: \"name, address, order, sum\". Input data should go with commas and spaces.");
        Scanner scan = new Scanner(System.in);
        String record = scan.next();
        validateOrder(record);
        return record;
    }
    public static String validateOrder(String record) {

        Pattern patternRegexOrder = Pattern.compile("(.*), (.*), (.*), (\\d+(?:\\.\\d{0,2})?)");
        Matcher matcherRegexOrder = patternRegexOrder.matcher(record);
        while (matcherRegexOrder.find() == false) {
            System.out.println("Check input data:\n1) All data goes throw comma and space;\n2) Name - only literals;\n3) Address - no limits;\n4) Order - no limits;\n5) Sum - total sum in format \"000.00\"");
            orderDescription();
            }
        if (matcherRegexOrder.find()) {
            System.out.println("The order has been added to queue.");
            orderDescription();
        }
        return record;
    }
    public static void main(String[] args) throws IOException {
        orderDescription();
    }
}

Comments

[mayton2019]

Что должен делать этот метод?

public static String validateOrder(String record)

[dodo512]

scan.next() прочитает только до ближайшего пробела.
Вероятно нужен scan.nextLine().

[Nanahuatzin]

mayton2019, он проверяет, что введенные данные по заказу соответствуют шаблону, а если нет - выдаёт инструкцию по правильному шаблону.

[Nanahuatzin]

dodo512, это мысль!) Не знал, что scan.Next() принимает только до пробела.
Однако, поменял, и теперь вообще странности пошли.

Изначально, я использовал простую регулярку типа ".+", чтобы просто проверить ход программы, и она работала, то есть выдавала, что заказ добавлен, или что выдавала инструкцию по правильному заполнению.

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

Поменял (и только это поменял) scan.Next() на scan.nextLineСейчас, почему-то вообще появляется сообщение, что введите заказ, а после просто завершается программа. При этом, я так понимаю, что он скипает вывод сообщения про добавление заказ в очередь.
Чую, это вовсе не из-за этой перемены.

[mayton2019]

Nanahuatzin, ну чел. Так не делается. Надо писать логику так чтоб она была максимально тестабельная.

Например,

class OrderEntity {
  String name;
  String text;
  String addr;
  BigDecimal sum;
}

Optional<OrderEntity> validateAndParseOrder(String orderText) {
   ...
}

Вот. Если заказ успешно разобран на составные части - то можно вернуть объект в котором уже
сделана декомпозиция. И дальше с ним работать. Если не разобран - то будет пустой объект опшионал.

Или еще более развитой вариант - пара объектов. Взаимоисключающая. Либо левый шаблон - когда ошибка. Либо правый когда ордер успешно разобран. В левом варианте строка будет содержать причину ошибки.

Either<String, OrderEntity> validateAndParseOrder(String orderText) {
   ..
}

Вот. И изучай модульные тесты. Как твою логику тестировать. Потому что этот ужас с выводом на экран
тестировать крайне неудобно.

[Nanahuatzin]

mayton2019, спасибо за ответ, но начинать от сложного к простому - как-то непродуктивно) Ещё только-только начал.
Ну, разумеется, понимаю, что нужно все разбивать на классы, чтобы было в дальнейшем тестировать легко, но эта задачка не для того, просто взаимодействие некоторое посмотреть хочу, не более.

[mayton2019]

Nanahuatzin, как будет угодно. Но ты начал изучать регулярки в крайне неудобных условиях тестирования.

Дело твое вобщемто.

[Nanahuatzin]

mayton2019, да не изучал я регулярки)) Просто хотел валидировать инпут, изучая вообще работу с файлами, и изначально показалось, что дело пяти минут, но не тут-то было. Но гештальт был не закрыт, а это плохо, вот поэтому я тут и оказался).

[mayton2019]

Nanahuatzin, кстати твое тех-задание неудачное. Задача решается без регулярок проще.
По крайней мере в тов виде как ты описал - проще сделать String.split по comma separator.

Ищи всегда простые решения. Как АК-47.

Он - надежен потому что простой.

Answer 1

[Nanahuatzin]

В общем, как правильно подсказал dodo512, метод для сканера был явно неверный.
Однако, в дебаге метод matcherRegexOrder.find() принимал значение True и тут же сбрасывался в False, поэтому тело if-a не читалось. Почему, я так и не понял.
Поменял на метод matcherRegexOrder.matches(), и дело пошло на лад.
Теперь работает так, как задумывалось. Кому интересно:

spoiler

import java.io.IOException;
import java.util.Scanner;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
import org.fusesource.jansi.AnsiConsole;

public class Pizza {

    public static final String ANSI_RESET = "\u001B[0m";
    public static final String ANSI_BLACK = "\u001B[30m";
    public static final String ANSI_RED = "\u001B[31m";
    public static final String ANSI_GREEN = "\u001B[32m";
    public static final String ANSI_YELLOW = "\u001B[33m";
    public static final String ANSI_BLUE = "\u001B[34m";
    public static final String ANSI_PURPLE = "\u001B[35m";
    public static final String ANSI_CYAN = "\u001B[36m";
    public static final String ANSI_WHITE = "\u001B[37m";

    public static String orderDescription() {

        System.out.println(ANSI_GREEN + "\nВведите данные о новом заказе в формате: \"имя, адрес, заказ, сумма\". Данные нужно вводить через запятую с пробелом.\nЕсли вы хотите завершить программу - просто отправьте \"123\"" + ANSI_RESET);
        Scanner scan = new Scanner(System.in);
        String record = scan.nextLine();
        if (record.equals("123")) {
            System.out.println(ANSI_GREEN + "\nПрограмма завершена." + ANSI_RESET);
            scan.close();
            System.exit(0);
        };

        validateOrder(record);
        return record;
    };

    public static String validateOrder(String record) {

        Pattern patternRegexOrder = Pattern.compile("(\\D*), (.*), (.*), (\\d{1,}\\.\\d{2})");
        Matcher matcherRegexOrder = patternRegexOrder.matcher(record);
        while (matcherRegexOrder.matches() == false) {
            System.out.println(ANSI_RED + "\nПроверьте правильность ввода:\n1) Все данные вводятся через запятую с пробелом;\n2) Имя - только буквы;\n3) Адрес - цифры, буквы и любые символы;\n4) Заказ - цифры, буквы и любые символы;\n5) Сумма - итоговая сумма в формате \"000.00\"" + ANSI_RESET);
            orderDescription();
        };

        if (matcherRegexOrder.matches()) {
            System.out.println(ANSI_BLUE + "\nЗаказ добавлен в очередь ожидания." + ANSI_RESET);
            orderDescription();
        };
        
        return record;
    };
    public static void main(String[] args) throws IOException {
        AnsiConsole.systemInstall();
        orderDescription();
        };
};

Answer 2

[SVM]

вы уверены, что правильно ставите вопрос?
почему бы не скормить регулярку ещё кому/чему то ))

Answer 3

[Dmitry Roo]

Тут проблема не в регулярке, а в логике программы.
Если в validateOrder попадет неправильная строка - шанса исправиться программа не дает.

Comments

[Nanahuatzin]

Проблема в том была как раз, что если туда попадала правильная строка, то программа всё равно считала её за неверную)