Как спарсить кривой excel в java?

Question

[vitya_brodov]

Здравствуете!
Мне приходит кривой excel(структура не удобная для парсинга), мне надо все поля записать в массив объектов
excel:

мой pojo:

@Data
@AllArgsConstructor
@NoArgsConstructor
public class RegisterModel {
    private String date;
    private String suip;
    private String state;
    private String transactionID;
}

service:

public List<RegisterModel> parse(MultipartFile file) throws IOException {
        Workbook workbook = new HSSFWorkbook(file.getInputStream());
        Sheet worksheet = workbook.getSheetAt(0);
        List<RegisterModel> registerModelList = new ArrayList<>();
        for (int i = 3; i < 10; i++) {

            Row row = worksheet.getRow(i);
            RegisterModel registerModel = new RegisterModel();
            String str = String.valueOf(row.getCell(1));
            if (!str.contains("null")) {
//                если дата
                if (str.contains(":")){
                    registerModel.setDate(str);
                }
//                если число
                if (NumberUtils.isDigits(str)){
                    registerModel.setTransactionID(str);
                }
                registerModelList.add(registerModel);
            }
        }
        System.out.println(registerModelList);
        return registerModelList;
    }

результат: данные не полностью записывается в объект, и повторяются.

[RegisterModel(date=2022-03-01T00:58:03, suip=null, state=null, transactionID=null), RegisterModel(date=null, suip=null, state=null, transactionID=502553852624)]

P.s помогите пожалуйста, я уже в отчаянии(

Comments

[сергей кузьмин]

во первых есть ошибки в логике прогаммы слишком часто переинициализируете pojo
во вторых кто мешает левую колонку прочитать и в зависимости от выполнять сетер

переделывайте

[vitya_brodov]

сергей кузьмин, особо ничего не изменилось, также все дублируется:

public List<RegisterModel> parse(MultipartFile file) throws IOException {
        Workbook workbook = new HSSFWorkbook(file.getInputStream());
        Sheet worksheet = workbook.getSheetAt(0);
        List<RegisterModel> registerModelList = new ArrayList<>();
        for (int i = 3; i < worksheet.getPhysicalNumberOfRows(); i++) {
                 RegisterModel registerModel = new RegisterModel();

            Row row = worksheet.getRow(i);
            String headers = String.valueOf(row.getCell(0));
            String values = String.valueOf(row.getCell(1));

            if (headers.equals("OPERDAY")){
                registerModel.setDate(values);
            }
            if (headers.equals("SUIP")){
                registerModel.setSuip(values);
            }
            if (headers.equals("STATE")){
                registerModel.setSuip(values);
            }
            if (headers.equals("NOM_OPER")){
                registerModel.setTransactionID(values);
            }
            registerModelList.add(registerModel);

        }
        System.out.println(registerModelList);
        return registerModelList;
    }

ввод:

[RegisterModel(date=2022-03-01T23:48:47, suip=Проведён, state=null, transactionID=1888314188), RegisterModel(date=2022-03-01T23:48:47, suip=Проведён, state=null, transactionID=1888314188), RegisterModel(date=2022-03-01T23:48:47, suip=Проведён, state=null, transactionID=1888314188)]

[rPman]

vitya_brodov, что тут насоветовали и что сделали
зачем new RegisterModel убрали наружу цикла, в этом случае в список добавляется один и тот же элемент registerModel с последними значениями из цикла

в самом вопросе не вижу повторений
а данные не полностью записаны потому что в коде так и написано если тип один - то прописываем одни поля, если тип другой - то другие
соответственно не совсем понимаю что тогда надо

[сергей кузьмин]

vitya_brodov, продолжаете с ощибками разбираться

инициализировать после того когда получены все
когда говорит state надо брать state

[vitya_brodov]

rPman,

в этом случае в список добавляется один и тот же элемент registerModel с последними значениями из цикла

- прошу Вас подсказать в этом вопросе, суть вопроса в этом заключалось, просто не смог объяснится(

[rPman]

vitya_brodov, куда уж понятнее то? что в моем комментарии не понятно?
если что то не ясно с кодом, занимайся его отладкой, по шагам проходи каждую строчку, внимательно смотри значение всех переменных (в т.ч. адрес объекта, это покажет разные ли объекты в списке или один и тот же)

[сергей кузьмин]

чего не ясно - каждый раз когда встречаете oper кончается один и начинается следующий RegisterModel

[vitya_brodov]

сергей кузьмин, PRman - спасибо! дошло)

Answer 1

[Dmitry Roo]

Нужно парсить обе колонки.
Парсим первую - понимаем, что нужно сделать со второй.
1. Если в первой что-то типа "oper_//d*" - создаем новый объект RegisterModel, старый (если есть такой) записываем в registerModelList.
2. Если что-то типа OPERDAY - парсим вторую колонку, пытаемся вытащить дату (можно сделать несколько вариантов формата даты, если нужно)
3. Если SUITE, STATE, NUM_ORDER - аналогично.

Я бы написал какой-то отдельный метод, который будет парсить данные из первой колонки и возвращать команды из энама.