[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