Я использую IRepository правильно?

Я бы сказал, вы близки к репозиторию, который я использую в производственном решении для планирования ресурсов в транспортных компаниях. (также используя NHibernate) - так что для начала, на мой взгляд, вы на правильном пути. Я согласен с dbones на использование IEnumerables / IList вместо массивов - вы в конечном итоге напишете .ToArray () много раз :-).

Несколько вещей, которые вы могли бы рассмотреть:

Предпочитайте композицию наследованию - вместо наследования из абстрактного репозитория - пусть она будет не абстрактной, и вставьте ее в 'ctor и делегируйте вызовы - это сделает ваш дизайн более надежным в определенных ситуациях (например, для репозитория только для запросов и т. д.). Таким образом, у вас также есть возможность разрешить создание экземпляров абстрактного репозитория (это слово?) и контролировать, должен ли он использоваться во всех репозиториях.

В продолжение этого пункта - вы можете захотеть изменить базовый репозиторий, чтобы иметь общие методы вместо наследования от универсального интерфейса:

public class Repository
{
    public void Add<T>(T entity)
    {
        using(var session = GetSession())
        using(var tx = session.BeginTransaction())
        {
             session.Save(entity)
             //Transaction handling etc.
        }
    }
    .... //repeat ad nasseum :-)
}

Возможно, вы захотите предоставить конкретным репозиториям доступ к ISession - это значительно улучшает то, как гибким, вы можете делать свои запросы и контролировать нетерпеливую / ленивую выборку, и вы получаете все преимущества NHibernate и т. д.

public class Repository
{
    public IList<T> WrapQueryInSession<T>(Func<ISession,IList<T> query)
    {
        using(var session = GetSession())
        using(var tx = session.BeginTransaction())
        {
             var items = query(session);
             //Handle exceptions transacitons etc.
             return items;
        }
     }
 }

Использование:

public class MovieRepository : IMovieRepository
{
    private Repository _repository;
    public MovieRepository(Repository repository)
    {
        _repository = repository;
    }
    public IList<Movie> GetByYear(int year)
    {
        Func<ISession, IList<Movie> query = session =>
        {
            var query = session.CreateQuery("from Movie"); //or
            var query = session.CreateCriteria("from Movie"); //or
            var query = session.Linq<Movie>();
            //set criteria etc.
            return query.List<Movie>(); //ToList<Movie>() if you're using Linq2NHibernate
        }:
        return _repository.WrapQueryInSession(query);
    }
}

Вы также можете установить для ваших методов возвращаемое значение типа bool, если что-то пойдет не так, и, возможно, выходной IEnumerable для любых ошибок, которые будут иметь смысл в вызывающем коде.

Но в целом - это всего лишь мои лакомые кусочки, которые я добавил со временем, чтобы лучше соответствовать моему использованию - и они совершенно необязательны, просто пища для размышлений :-). Думаю, вы на правильном пути - я не вижу серьезных проблем в вашем коде.

Надеюсь, это имеет смысл: -)