Skip to content

Утечка ресурса FileInputStream и использование printStackTrace в Utils.java #157

@coderabbitai

Description

@coderabbitai

Описание

В файле src/main/java/com/clouds42/Utils.java обнаружены следующие проблемы:

  1. Утечка ресурса: FileInputStream внутри forEach лямбды (около строк 268-280) не закрывается автоматически
  2. Использование printStackTrace(): Вместо логгера используется printStackTrace() для вывода исключений

Предлагаемое решение

  1. Обернуть FileInputStream в try-with-resources для автоматического закрытия ресурса:
Document formXmlDocument;
try (FileInputStream formFileIS = new FileInputStream(formXmlFileName)) {
  formXmlDocument = builder.parse(formFileIS);
}
  1. Заменить printStackTrace() на использование логгера:
logger.error("Can't read forms directory: {}", e.getLocalizedMessage());

Ссылки

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions