Чтобы перенести туда, например, загруженные адреса потоков из объекта VideoItem
// потоки видео и аудио
// все доступные для текущего ролика потоки
@Ignore
private StreamHelper.StreamSources streamSources;
// кэш - адрес потока видео, загруженный для этого ролика в последний раз
// кэш - адрес потока аудио, загруженный для этого ролика в последний раз
// (может быть null, если поток видео содержал дорожку аудион)
@Ignore
private StreamHelper.StreamPair playbackStreams;
Плюсы:
не тащить в объект, связанный с базой данных, все подряд, не связанное с базой данных
т.е. просто красота кода. Было бы оправдано, еслибы классы базы данных были в отдельном проекте
Минусы:
если потребуется показать информацию о потоке где-то за пределами экрана плеера, будет удобнее иметь это поле внутри VideoItem.
Например, может произойти в интерфейсе загрузки видео. Или при проигрывании в режиме плеера.
Итого: пожалуй, пока минусы перевешивают плюсы. Никакой прямой практической пользы от рефактора нет. От текущиего положения дел потенциальная польза есть, плюс сейчас уже сделано и так. Но тикет пусть висит пока, раз уж начала обдумывать.
Чтобы перенести туда, например, загруженные адреса потоков из объекта VideoItem
Плюсы:
Минусы:
Итого: пожалуй, пока минусы перевешивают плюсы. Никакой прямой практической пользы от рефактора нет. От текущиего положения дел потенциальная польза есть, плюс сейчас уже сделано и так. Но тикет пусть висит пока, раз уж начала обдумывать.