Используйте контекстный менеджер, ловите только ожидаемые ошибки и сделайте результат однозначным. Пустой except скрывает сбои и затрудняет восстановление.
Как провести ревью функции обработки файла Python
Используйте контекстный менеджер, ловите только ожидаемые ошибки и сделайте результат однозначным. Пустой except скрывает сбои и затрудняет восстановление.
Короткий ответ
Что ответить на собеседовании
Подробный разбор
Ответ с пояснениями
Условие
Функция открывает файл без with, ловит все исключения пустым except и возвращает None как при успехе, так и при ошибке. Что нужно изменить?
Ответ
Используйте контекстный менеджер, ловите только ожидаемые ошибки и сделайте результат однозначным. Пустой except скрывает сбои и затрудняет восстановление.
Определите кодировку и предел размера входа. Разделите чтение, преобразование и сохранение, чтобы проверять чистую логику отдельно. Возвращайте результат либо выбрасывайте понятное исключение с причиной; не дублируйте чувствительное содержимое в логах. Контекстный менеджер гарантирует закрытие файла при выходе из блока.