Merge pull request #4 from go-andiamo/remove-do-with

Remove DoWith

README.md

@@ -116,14 +116,6 @@ func main() {
         </td>
         <td><em>nothing</em></td>
     </tr>
-    <tr>
-        <td>
-            <code>DoWith(f func(v T))</code><br>
-            if the value is present, calls the supplied function with the value<br>
-            <em>returns the original optional</em>
-        </td>
-        <td><code>Optional[T]</code></td>
-    </tr>
     <tr>
         <td>
             <code>Filter(f func(v T) bool)</code><br>

optional.go

@@ -186,16 +186,6 @@ func (o Optional[T]) OrElsePanic(v any) {
 	}
 }
 
-// DoWith if the value is present, calls the supplied function with the value
-//
-// Returns the original optional
-func (o Optional[T]) DoWith(f func(v T)) Optional[T] {
-	if o.present {
-		f(o.value)
-	}
-	return o
-}
-
 // Filter if the value is present and calling the supplied filter function returns true, returns a new optional describing the value
 //
 // Otherwise returns an empty optional

optional_test.go

@@ -193,23 +193,6 @@ func TestOptional_OrElsePanic(t *testing.T) {
 	})
 }
 
-func TestOptional_DoWith(t *testing.T) {
-	o := Empty[string]()
-	called := false
-	f := func(v string) {
-		called = true
-	}
-	o2 := o.DoWith(f)
-	require.False(t, called)
-	require.Equal(t, o, o2)
-
-	called = false
-	o = Of("aaa")
-	o2 = o.DoWith(f)
-	require.True(t, called)
-	require.Equal(t, o, o2)
-}
-
 func TestOptional_Filter(t *testing.T) {
 	o := Empty[string]()
 	called := false