Как решить проблему X does not implement Y при работе с интерфесами?

Question

[Sergey Petrov]

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

import "fmt"

//import repository

type RepositoryInterface interface {
 GetSubrepo() SubrepoInterface // << Не работает
 // GetSubrepo() Subrepo // << Работает
}

type SubrepoInterface interface {
 HelloSubrepo()
}

func main() {
 r := Repository{subrepo: Subrepo{}}
 useRepository(r)
}

// func touchRepository(c Repository) {
func useRepository(r RepositoryInterface) {
 sr := r.GetSubrepo()
 sr.HelloSubrepo()
}

// ******************** Repository package ****************
type Repository struct {
 subrepo Subrepo
}

func (r *Repository) GetSubrepo() Subrepo {
 return r.subrepo
}

type Subrepo struct {
}

func (sr *Subrepo) HelloSubrepo() {
 fmt.Printf("hello subrepo\n")
}

Ошибка:

./prog.go:18:16: cannot use r (variable of type Repository) as RepositoryInterface value in argument to useRepository: Repository does not implement RepositoryInterface (wrong type for method GetSubrepo)
have GetSubrepo() Subrepo
want GetSubrepo() SubrepoInterface

Подскажите, как правильно решить такую задачу, сохранив слабую связанность через интерфейсы.

Answer 1

[Sergey Petrov]

Как вариант, мне подсказали решение через дженерики

package main

import "fmt"

//import repository

type RepositoryInterface[T SubrepoInterface] interface {
	GetSubrepo() T
}

type SubrepoInterface interface {
	HelloSubrepo()
}

func main() {
	r := &Repository{subrepo: &Subrepo{}}
	useRepository[*Subrepo](r)
}

func useRepository[T SubrepoInterface](r RepositoryInterface[T]) {
	sr := r.GetSubrepo()
	sr.HelloSubrepo()
}

// ******************** Repository package ****************
type Repository struct {
	subrepo *Subrepo
}

func (r *Repository) GetSubrepo() *Subrepo {
	return r.subrepo
}

type Subrepo struct {
}

func (sr *Subrepo) HelloSubrepo() {
	fmt.Printf("hello subrepo\n")
}

Answer 2

[Александр Павлюк]

По-хорошему, у вас репозиторий не должен никогда возвращать интерфейс. В Го принято возвращать конкретный тип, а не интерфейс. https://github.com/golang/go/wiki/CodeReviewCommen...

Плюсом еще замечу, что обычно возвращаемый тип является внутренней моделью данных. Это обязанность пакета repository конвертировать из какого-то внушнего представления во внутреннюю модель.

Если сказать по-другому: в domain-пакетах у вас не должно быть импортов каких-то структур из пакета БД, только наоборот — из пакета БД импортить типы из domain.

Answer 3

[Максим Федоров]

package main

import "fmt"

type RepositoryInterface interface {
	GetSubrepo() SubrepoInterface
}
type SubrepoInterface interface {
	HelloSubrepo()
}

func main() {
	r := Repository{subrepo: Subrepo{}}
	useRepository(&r)
}

// func touchRepository(c Repository) {
func useRepository(r RepositoryInterface) {
	sr := r.GetSubrepo()
	sr.HelloSubrepo()
}

// ******************** Repository package ****************
type Repository struct {
	subrepo Subrepo
}

func (r *Repository) GetSubrepo() SubrepoInterface {
	return &r.subrepo
}

type Subrepo struct {
}

func (sr *Subrepo) HelloSubrepo() {
	fmt.Printf("hello subrepo\n")
}

Comments

[Sergey Petrov]

Но тогда описание SubrepoInterface должно находиться в каком-то общем месте.

Если проект имеет например такую структуру:

cmd/
  - main.go
internal/
  repository/
    mysql/
      - repository.go
  controller/
    - controller.go

в main.go создается экземпляр Repository и передается в новый экземпляр контроллера.
То где должно располагаться описание SubrepoInterface ?

[Sergey Petrov]

UPD: лишние поинтеры я из вопроса убрал. Но это все равно не работает

[Максим Федоров]

Sergey Petrov, я никогда не видел абстракции subRepository, потому не скажу где она должна находиться

ну и вообще по иному проектирую структуру приложения :(

[Максим Федоров]

Sergey Petrov,

> Но тогда описание SubrepoInterface должно находиться в каком-то общем месте.

я просто изменил указатели и пофиксил до раочего варианта, почему это повляло на вопрос о нахождении репозитория? :)

[Sergey Petrov]

Максим Федоров, да тут скорее всего неверная архитектура и стоит переделать. Но все же интересно как можно реализовать вот такое.
В любом случае, спасибо за ответ!