[Android] [PATCH 2/2] Playlists in database
Edward Wang
edward.c.wang at compdigitec.com
Tue Mar 24 00:49:16 CET 2015
Hi Geoffrey,
On Mon, Mar 23, 2015 at 4:46 PM, Geoffrey Métais
<geoffrey.metais at gmail.com> wrote:
> Thing is, MediaDatabase.java aggregates all db accesses.
>
> I'd rather split different db helpers, so I'm not against this idea but for
> now all is done in this class.
>
> Thus, I already use present playlistDelete, playlistExists,
> playlistInsertItem and playlistAdd methods in this very class.
Indeed. Perhaps we can leave them like this for now but ideally it
would be nicer to do it like this in the future:
UI classes/fragments <====> PlaylistDatabase <====> MediaDatabase
<====> SQLiteDatabase
If so we can add a TODO in the code or a low-priority enhancement
ticket on the trac.
Regards,
Edward
More information about the Android
mailing list