Movie database handler: Interface and Test cases done

Files to be reviewed:

Interface:
https://github.com/TaviscaSolutions/blue-repo/blob/DatabaseHandler/ASP-WCF/MvcMovie/MovieService/IDatabaseHandler.cs

Movie database handler class:
https://github.com/TaviscaSolutions/blue-repo/blob/DatabaseHandler/ASP-WCF/MvcMovie/MovieService/MovieDatabaseHandler.cs

Test cases for movie database handler class:
https://github.com/TaviscaSolutions/blue-repo/blob/DatabaseHandler/ASP-WCF/MvcMovie/MovieFixtures/DatabaseHandlerFixture.cs
Actually the handler class has the CRUD operations. The actual database,
which I have currently done using dictionary, will be defined in another
class. Here I have used MovieDatabase class to create the database.


On 24 August 2014 12:14, Nikhil Prasad <notifications@github.com> wrote:

> In ASP-WCF/MvcMovie/MovieService/IDatabaseHandler.cs:
>
> > @@ -0,0 +1,29 @@
> > +﻿using System;
> > +using System.Collections.Generic;
> > +using System.Linq;
> > +using System.Text;
> > +using System.Threading.Tasks;
> > +
> > +namespace MovieService
> > +{
> > +    public interface IDatabaseHandler
> > +    {
> > +        Movie Add(Movie movie);
> > +
> > +        List<Movie> GetList();
>
> Really really bad name. I am not too clear on why the fixation with
> "handler". The method that you would expect in a handler class would
> largely be Handle(). What is wrong with a simpler and cleaner name like
> IMovieDb
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TaviscaSolutions/blue-repo/pull/38/files#r16634583>.
>