I have a central package that provides several interfaces that other packages are dependent on (let us call one Client
). Those other packages, provide several implementations of those first interfaces (UDPClient
, TCPClient
). I instantiate a Client
by calling NewClient
in the central package, and it selects and invokes the appropriate client implementation from one of the dependent packages.
This falls apart when I want to tell the central package about those other packages, so it knows what clients it can create. Those dependent client implementations also import the central package, creating a cyclic dependency which Go does not allow.
What's the best way forward? I'd prefer not to mash all those implementations in a single package, and creating a separate registry package seems overkill. Currently I have each implementation register itself with the central package, but this requires that the user knows to import every implementation in every separate binary that makes use of client.
import (
_ udpclient
_ tcpclient
client
)
The standard library solves this problem in multiple ways:
1) Without a "Central" Registry
Example of this is the different hash algorithms. The
crypto
package just defines theHash
interface (the type and its methods). Concrete implementations are in different packages (actually subfolders but doesn't need to be) for examplecrypto/md5
andcrypto/sha256
.When you need a "hasher", you explicitly state which one you want and instantiate that one, e.g.
This is the simplest solution and it also gives you good separation: the
hash
package does not have to know or worry about implementations.This is the preferred solution if you know or you can decide which implementation you want prior.
2) With a "Central" Registry
This is basically your proposed solution. Implementations have to register themselves in some way (usually in a package
init()
function).An example of this is the
image
package. The package defines theImage
interface and several of its implementations. Different image formats are defined in different packages such asimage/gif
,image/jpeg
andimage/png
.The
image
package has aDecode()
function which decodes and returns anImage
from the specifiedio.Reader
. Often it is unknown what type of image comes from the reader and so you can't use the decoder algorithm of a specific image format.In this case if we want the image decoding mechanism to be extensible, a registration is unavoidable. The cleanest to do this is in package
init()
functions which is triggered by specifying the blank identifier for the package name when importing.Note that this solution also gives you the possibility to use a specific implementation to decode an image, the concrete implementations also provide the
Decode()
function, for examplepng.Decode()
.So the best way?
Depends on what your requirements are. If you know or you can decide which implementation you need, go with #1. If you can't decide or you don't know and you need extensibility, go with #2.
...Or go with #3 presented below.
3) Proposing a 3rd Solution: "Custom" Registry
You can still have the convenience of the "central" registry with interface and implementations separated with the expense of "auto-extensibility".
The idea is that you have the interface in package
pi
. You have implementations in packagepa
,pb
etc.And you create a package
pf
which will have the "factory" methods you want, e.g.pf.NewClient()
. Thepf
package can refer to packagespa
,pb
,pi
without creating a circular dependency.They should rely on another package defining interfaces that they need to rely on (and that are implemented by the first central package).
This is usually how an import cycle is broken (and/or using dependency inversion).
You have more options described in "Cyclic dependencies and interfaces in Golang".
go list -f
can also help visualizing those import cycles.