Правильно ли я использую IDisposible C# ASP.NET Core?

Question

[User99]

Правильно ли я использую IDisposible в репозиториях, чтобы после выполнения тело using он уничтожался?

Интерфейс репозитории:

public interface IExamsRepository<T> where T:BaseEntity
    {
        Task<IList<Exam>> ShowAll();
        Task New(Exam exam);
  }

Репозитории

public class ReposExams : IExamsRepository<Exam>,IDisposible
    {
        private string connectionString;

        public ReposExams(IConfiguration configuration)
        {
            connectionString = configuration.GetValue<string>("DBInfo:ConnectionString");
        }

        internal IDbConnection Connection
        {
            get
            {
                return new NpgsqlConnection(connectionString);
            }
        }

public void Dispose()
        {
            Dispose();
        }

}

то есть достаточно ли этой строки:

public void Dispose()
{
Dispose();
}

Comments

[edward_freedom]

https://docs.microsoft.com/en-us/dotnet/api/system...

[ayazer]

public void Dispose()
        {
            Dispose();
        }

вы сами себя диспоузите?

Answer 1

[Василий Банников]

Произошла рекурсия.
Наверное вы хотели задиспозить Connection

Comments

[User99]

При обращении к БД я использую так:

public async Task ThemeDelete(long id)
        {
           using(IDbConnection dbConnection = Connection)
            {
                await dbConnection.ExecuteAsync(@"DELETE FROM journal.journal_themes WHERE id=@id", new { id = id });
            }
        }

Как я понял мои запросы обернуты в Using, то есть после выполнения блока он должен самоуничтожиться, я правильно понимаю? Но в msdn я читал, чтобы самоуничтожиться блок using, должен быть реализован от IDispose. поэтому делаю так:

public class ReposExams : IExamsRepository<Exam>,IDisposible

но кажется это не правильно

[Василий Банников]

User99, using просто вызывает метод Dispose у того объекта, к которому он применён. В этом случае - у Connection.
Вам не нужно реализовывать IDisposable, тк ваш репозиторий всё равно не владеет никакими ресурсами

Answer 2

[Роман]

А зачем тут IDispose? У вас нет неуправляемых ресурсов. NpgsqlConnection это управляемая обертка над подключением, при сборке мусора все само соберется и освободится. Конечно, если у вас высоконагруженная система, то тогда имеет смысл освобождать подключение. https://habr.com/ru/post/89720/ вот почитайте, как правильно делать.

PS и создавать подключение в репозитории, не айс, зачем DI придумали люди.

Comments

[User99]

Я использую вот так,

public async Task ThemeDelete(long id)
        {
           using(IDbConnection dbConnection = Connection)
            {
                await dbConnection.ExecuteAsync(@"DELETE FROM journal.journal_themes WHERE id=@id", new { id = id });
            }
        }

Был случай когда пользователей стало больше 1000, то Npgsql стал ругаться что пулы превысили 100. потом стал закрывать соединение вручную, вот так:

public async Task ThemeDelete(long id)
        {
           using(IDbConnection dbConnection = Connection)
            {
                await dbConnection.ExecuteAsync(@"DELETE FROM journal.journal_themes WHERE id=@id", new { id = id });

dbConnection .Dispose();
            }
        }

PS и создавать подключение в репозитории, не айс, зачем DI придумали люди.

Можно пример пожалуйста?