Skip to content

Conversation

@meloman-team
Copy link

@meloman-team meloman-team commented Nov 23, 2025

Кажется с момента написания кода, путь к файлу изменился и программа работает не правильно.

Copy link
Member

@rekby rekby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Спасибо за участие и замечание по поводу работы. Путь к файлу не менялся, но важно какая папка выбрана в качестве "текущей" при запуске проекта.

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

Нужно сделать универсальное решение (описал в комментарии внутри файла), чтобы решить саму причину её возникновения.

Можете вы поправить, буду признателен. Или наша команда поправит - задачу я записал.


private static final Logger LOGGER = LoggerFactory.getLogger(Application.class);
private static final String PATH = "/lesson-6.2/java/file.txt";
private static final String PATH = "dev-1/lesson-6.2/java/file.txt";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это исправление позволит запускать код из корня репозитория, вместо папки dev-1, но не решает основную проблему - зависимость от папки запуска.

Код проекта ссылается на файл, но вычисляет его расположение используя текущую рабочую директорию.

String currentDirectory = System.getProperty("user.dir");

Нужно сделать код этого урока независимым от папки запуска проекта, например вычислять путь к файлу относительно расположения .java-исходников.

Если это невозможно или неразумно сложно - тогда предположить, что все проекты будут запускаться из папки своих исходников (в данном случае - dev-1/lesson-6.2/java) и прописать инструкцию по запуску проекта в README в папку каждого урока.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants