mirror/go-patterns
1
0
Fork 0
mirror of https://github.com/tmrts/go-patterns.git synced 2024-11-25 06:26:06 +03:00
Code Issues Projects Releases Wiki Activity

idiom/functional-options: Unexport options

Browse Source 
Options is an implementation detail and adds no value by being exported.
By unexporting Options we decrease the API surface and maintain the
ability to make changes without breaking user expectations.
This commit is contained in:
Mathias Fredriksson 2018-01-06 02:35:57 +02:00
parent f978e42036
commit 4cdebe4220
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
idiom/functional-options.md
View File

@ -10,7 +10,7 @@ Options implemented as a function set the state of that option.
```go
package file
type Options struct {
type options struct {
UID int
GID int
Flags int
@ -18,28 +18,28 @@ type Options struct {
Permissions os.FileMode
}
type Option func(*Options)
type Option func(*options)
func UID(userID int) Option {
return func(args *Options) {
return func(args *options) {
args.UID = userID
}
}
func GID(groupID int) Option {
return func(args *Options) {
return func(args *options) {
args.GID = groupID
}
}
func Contents(c string) Option {
return func(args *Options) {
return func(args *options) {
args.Contents = c
}
}
func Permissions(perms os.FileMode) Option {
return func(args *Options) {
return func(args *options) {
args.Permissions = perms
}
}
@ -51,8 +51,8 @@ func Permissions(perms os.FileMode) Option {
package file
func New(filepath string, setters ...Option) error {
// Default Options
args := &Options{
// Default options
args := &options{
UID: os.Getuid(),
GID: os.Getgid(),
Contents: "",