Use functional options pattern

client.go

@@ -9,29 +9,39 @@ import (
 	"os"
 )
 
-type ClientOptions struct {
-	BaseUrl string
-	ApiKey  string
-}
-
 type Client struct {
 	BaseUrl string
 	ApiUrl  string
 	ApiKey  string
 }
 
-func NewClient(options *ClientOptions) *Client {
+func NewClient(options ...func(*Client)) *Client {
-	if options.BaseUrl == "" {
+	c := &Client{}
-		options.BaseUrl = "https://stacker.news"
+	for _, o := range options {
-	}
+		o(c)
-	if options.ApiKey == "" {
-		options.ApiKey = os.Getenv("SN_API_KEY")
 	}
 
-	return &Client{
+	// set defaults
-		BaseUrl: options.BaseUrl,
+	if c.BaseUrl == "" {
-		ApiUrl:  fmt.Sprintf("%s/api/graphql", options.BaseUrl),
+		c.BaseUrl = "https://stacker.news"
-		ApiKey:  options.ApiKey,
+	}
+	if c.ApiKey == "" {
+		c.ApiKey = os.Getenv("SN_API_KEY")
+	}
+	c.ApiUrl = fmt.Sprintf("%s/api/graphql", c.BaseUrl)
+
+	return c
+}
+
+func WithApiKey(apiKey string) func(*Client) {
+	return func(c *Client) {
+		c.ApiKey = apiKey
+	}
+}
+
+func WithBaseUrl(baseUrl string) func(*Client) {
+	return func(c *Client) {
+		c.BaseUrl = baseUrl
 	}
 }