From 92020705879adb1d5c4c1980d88af50eda43717f Mon Sep 17 00:00:00 2001 From: Byeonghoon Yoo Date: Thu, 8 Dec 2022 13:42:30 +0900 Subject: [PATCH] refactor!: rename `OrElse()` to `Or()` --- README.md | 2 +- examples_test.go | 6 +++--- option.go | 4 ++-- option_test.go | 6 +++--- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/README.md b/README.md index a3aef9b9..cbd2fd78 100644 --- a/README.md +++ b/README.md @@ -60,7 +60,7 @@ and more detailed examples are here: [./examples_test.go](./examples_test.go). - [Option[T]#Take() (T, error)](https://pkg.go.dev/github.com/moznion/go-optional#Option.Take) - [Option[T]#TakeOr(fallbackValue T) T](https://pkg.go.dev/github.com/moznion/go-optional#Option.TakeOr) - [Option[T]#TakeOrElse(fallbackFunc func() T) T](https://pkg.go.dev/github.com/moznion/go-optional#Option.TakeOrElse) -- [Option[T]#OrElse(fallbackOptionValue Option[T]) Option[T]](https://pkg.go.dev/github.com/moznion/go-optional#Option.OrElse) +- [Option[T]#Or(fallbackOptionValue Option[T]) Option[T]](https://pkg.go.dev/github.com/moznion/go-optional#Option.Or) - [Option[T]#Filter(predicate func(v T) bool) Option[T]](https://pkg.go.dev/github.com/moznion/go-optional#Option.Filter) - [Option[T]#IfSome(f func(v T))](https://pkg.go.dev/github.com/moznion/go-optional#Option.IfSome) - [Option[T]#IfSomeWithError(f func(v T) error) error](https://pkg.go.dev/github.com/moznion/go-optional#Option.IfSomeWithError) diff --git a/examples_test.go b/examples_test.go index 6dbf2f70..f90dfdbf 100644 --- a/examples_test.go +++ b/examples_test.go @@ -551,14 +551,14 @@ func ExampleFlatMapOrWithError() { // } -func ExampleOption_OrElse() { +func ExampleOption_Or() { fallback := Some[string]("fallback") some := Some[string]("actual") - fmt.Printf("%s\n", some.OrElse(fallback)) + fmt.Printf("%s\n", some.Or(fallback)) none := None[string]() - fmt.Printf("%s\n", none.OrElse(fallback)) + fmt.Printf("%s\n", none.Or(fallback)) // Output: // Some[actual] diff --git a/option.go b/option.go index bb2dd9a9..290b9a20 100644 --- a/option.go +++ b/option.go @@ -102,9 +102,9 @@ func (o Option[T]) TakeOrElse(fallbackFunc func() T) T { return o[value] } -// OrElse returns the Option value according to the actual value existence. +// Or returns the Option value according to the actual value existence. // If the receiver's Option value is Some, this function pass-through that to return. Otherwise, this value returns the `fallbackOptionValue`. -func (o Option[T]) OrElse(fallbackOptionValue Option[T]) Option[T] { +func (o Option[T]) Or(fallbackOptionValue Option[T]) Option[T] { if o.IsNone() { return fallbackOptionValue } diff --git a/option_test.go b/option_test.go index d338f196..ce5fc6c7 100644 --- a/option_test.go +++ b/option_test.go @@ -569,9 +569,9 @@ func TestOption_String(t *testing.T) { assert.Equal(t, "None[]", None[*MyStringer]().String()) } -func TestOption_OrElse(t *testing.T) { +func TestOption_Or(t *testing.T) { fallback := Some[string]("fallback") - assert.EqualValues(t, Some[string]("actual").OrElse(fallback).Unwrap(), "actual") - assert.EqualValues(t, None[string]().OrElse(fallback).Unwrap(), "fallback") + assert.EqualValues(t, Some[string]("actual").Or(fallback).Unwrap(), "actual") + assert.EqualValues(t, None[string]().Or(fallback).Unwrap(), "fallback") }