Ответь честно — тестовое задание?

Т.е. тебе прямо сказали «ничего готового не юзать, плить с нуля»?

нет, так не говорили

откуда столько злости то?

в моём случае под "выше" имеется в виду СТО

всем привет. пишу либу для хранения файлов, есть 3 класса: Manager, Storage и File.
Manager является, по сути, основным классом, внутри которого реализованы методы store, get, delete. Он же работает с базой.
Storage - класс чисто для работы с файлами и директориями. в нём есть upload, отдающий объект File после загрузки файла. этот метод дёргается из Manager::store перед тем, как записать файл в базу.
File, если по-простому, то модель файла.

вопрос заключается в распределении функционала. сейчас File умеет сам себя удалять из хранилища - просто вызывает анлинк на абсолютный путь. правильно ли это? или этим должен заниматься Storage, который сейчас ничего не знает о File, кроме как то, что его нужно отдавать после аплоада?

плюс хочу организовать очистку пустых директорий.
название файла при загрузке генерится из md5+timestamp. после этого из этого имени нарезаются названия подпапок (не помню названия этого способа, но до определённого уровня вложенности с начала названия файла отщипываются куски и с их названиями создаются папки). например, файл b4bfcf6443db07298095fd21bf58479b_1584557369.jpg будет лежать в uploads/b4/bf/. после удаления файла папки b4/bf останутся, а хотелось бы их подчищать, убедившись, что они пустые. есть ли смысл закладывать такой функционал в тот же File::delete()? или лучше вынести это в Storage и там уже сканировать uploads и удалять всё, что пустое?
спасибо

если в вашей предметной области файл может сам себя удалить - ОК
иначе переделывать