Насколько сильно стоит завязываться на абстракции на примере репозитория?

Question

[Егор Полянский]

В рамках написания проекта дошел до описания пула БД. В момент изучения темы встал вопрос, есть ли смысл в полной абстракции от БД. Что я имею в виду:
Первый пример.
Интерфейс пула запрашивает необходимые методы, которые завязаны на конкретной реализации pgx.

type DBPool interface {
    QueryRow(ctx context.Context, sql string, args ...any) pgx.Row
    Query(ctx context.Context, sql string, args ...any) (pgx.Rows, error)
    Exec(ctx context.Context, sql string, args ...any) (pgconn.CommandTag, error)
}

А второй пример заключается в том, что мы будем в методах интерфейса возвращать стандартные типы, не завязанные на какой-либо конкретной реализации.

В связи с чем я задаюсь вопросом. Приходилось ли Вам опираться на полную абстракцию? Есть ли какой-то весомый выигрыш от этого в разработке?

Заранее извиняюсь в возможных неточностях моего объяснения, так как сам еще "плаваю" в этой теме

Answer 1

[Пума Тайланд]

Интерфейс с pgx-типами — это не абстракция от БД, это просто обёртка. Реально нужно абстрагироваться выше: пусть репозиторий возвращает доменные структуры (GetUser(ctx, id) (User, error)), тогда pgx остаётся скрытым внутри. Делать DB-агностичный пул «на всякий случай» это YAGNI, смысл только если есть реальная вторая реализация.

Comments

[Егор Полянский]

пусть репозиторий возвращает доменные структуры (GetUser(ctx, id) (User, error)), тогда pgx остаётся скрытым внутри

. А для интерфейса пула, как я приводил выше, возврат pgx-типов не является ошибкой в данном случае?

[Пума Тайланд]

Нет, для пула pgx-типы — нормально, это внутренняя инфраструктура. Главное не тащить их выше репозитория.

[Егор Полянский]

Пума Тайланд, спасибо большое за объяснение!

[Пума Тайланд]

пожалуйста, удачи с проектом!

Answer 2

[Dhwtj]

На абстракции надо завязываться настолько насколько вам/команде потом будет удобно с этим работать, мыслить, насколько часто придется менять.
Если в планах есть переход на другую БД или есть такой риск и заказчик оплатит ресурсы делать абстракцию, то делайте. Иначе, не делайте.