Добавлен пропуск битых файлов и вывод компактного описания причины #497
Conversation
WalkthroughИзменения в файле Changes
Sequence Diagram(s)sequenceDiagram
participant C as Клиент
participant X as ExtendXStream
C->>X: Вызов метода fromXML()
alt Без ошибок
X->>C: Возвращает корректный результат
else Если возникает ConversionException
X->>X: Логирует "file is broken (skipped)"
else Если возникает StreamException
X->>X: Логирует ошибку с указанием причины
else Если возникает CannotResolveClassException
X->>X: Логирует "file is broken (skipped)"
else Если возникает любое другое Exception
X->>X: Логирует сообщение об ошибке (пропущено)
end
X->>C: Завершает выполнение метода
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (18)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/main/java/com/github/_1c_syntax/bsl/reader/common/xstream/ExtendXStream.java (2)
129-130: Добавлен универсальный обработчик исключений.Добавлен блок для перехвата всех остальных исключений, что повышает отказоустойчивость программы. Однако стоит быть осторожным, так как такой подход может скрывать серьёзные проблемы.
Возможно, стоит рассмотреть более детальную обработку специфических исключений вместо использования общего Exception. Или добавить дополнительную диагностическую информацию для неизвестных ошибок, чтобы упростить отладку.
123-131: Обеспечьте единообразие в сообщениях об ошибках.Форматирование сообщений об ошибках немного отличается в разных блоках catch:
- В блоках для
ConversionException,CannotResolveClassExceptionи общегоExceptionиспользуется\nв конце- В блоке для
StreamExceptionсимвол переноса строки отсутствуетРекомендуется унифицировать стиль сообщений для облегчения поиска и анализа в логах.
- LOGGER.error("Can't read file '{}' - it's broken (skipped) \n", file, e); + LOGGER.error("Can't read file '{}' - it's broken (skipped)", file, e); - LOGGER.debug("Can't read file '{}' - unknown class (skipped) \n", file, e); + LOGGER.debug("Can't read file '{}' - unknown class (skipped)", file, e); - LOGGER.error("Can't read file '{}' - unknown error (skipped) \n", file, e); + LOGGER.error("Can't read file '{}' - unknown error (skipped)", file, e);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/main/java/com/github/_1c_syntax/bsl/reader/common/xstream/ExtendXStream.java(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (10)
- GitHub Check: build (21, windows-latest)
- GitHub Check: build (17, windows-latest)
- GitHub Check: build (21, ubuntu-latest)
- GitHub Check: build (23, ubuntu-latest)
- GitHub Check: build (23, windows-latest)
- GitHub Check: QA
- GitHub Check: build (23, macOS-latest)
- GitHub Check: build (17, macOS-latest)
- GitHub Check: build (17, ubuntu-latest)
- GitHub Check: build (21, macOS-latest)
🔇 Additional comments (5)
src/main/java/com/github/_1c_syntax/bsl/reader/common/xstream/ExtendXStream.java (5)
71-71: Добавление нового импорта для обработки StreamException.Импорт
com.thoughtworks.xstream.io.StreamExceptionдобавлен для обработки ошибок потока при чтении XML.
123-124: Улучшен механизм обработки ConversionException.Изменение обработки исключения
ConversionExceptionпозволяет пропускать битые файлы вместо прерывания процесса. Теперь ошибка логируется, а программа продолжает работу с другими файлами.
126-126: Дополнено сообщение для CannotResolveClassException.Добавлена отметка "(skipped)" в сообщение лога для обозначения того, что файл с неизвестным классом был пропущен.
127-128: Добавлена обработка StreamException.Добавлена обработка
StreamException, которая может возникать при проблемах с XML-потоком. Теперь такие ошибки логируются с указанием причины, а файл пропускается.
123-131: Рассмотрите добавление тестов для нового функционала.В соответствии с PR objectives отмечено, что изменения ещё не покрыты тестами. Рекомендуется добавить модульные тесты для проверки корректной обработки различных типов ошибок и правильного пропуска битых файлов.
Убедитесь, что поведение метода при различных типах исключений соответствует ожиданиям и что сообщения об ошибках содержат нужную информацию для диагностики проблем.
|
Описание
Пропускает битые файлы (в том числе дочерние) с выводом пути к файлу и описания проблемы (если удалось понять).
Пример
Связанные задачи
Closes
Чеклист
Общие
gradlew precommit)Дополнительно