rename ifPresentOrElse to ifLeftOrElse

Author: h908714124

src/main/java/io/jbock/util/Either.java

@@ -187,7 +187,7 @@ public abstract <U> U fold(
      * @param leftAction action to run if this is a Left
      * @param rightAction action to run if this is a Right
      */
-    public abstract void ifPresentOrElse(
+    public abstract void ifLeftOrElse(
             Consumer<? super L> leftAction,
             Consumer<? super R> rightAction);
 
@@ -208,16 +208,16 @@ public final boolean isRight() {
     }
 
     /**
-     * If this is a Left, return a {@code LeftOptional} containing the LHS value.
-     * Otherwise return an empty {@code LeftOptional}.
+     * If this is a Left, returns a {@code LeftOptional} containing the LHS value.
+     * Otherwise returns an empty {@code LeftOptional}.
      *
      * @return the LHS value, or {@link java.util.Optional#empty()} if this is a Right
      */
     public abstract LeftOptional<L> getLeft();
 
     /**
-     * If this is a Right, return an {@code Optional} containing the RHS value.
-     * Otherwise return an empty {@code Optional}.
+     * If this is a Right, returns an {@code Optional} containing the RHS value.
+     * Otherwise returns an empty {@code Optional}.
      *
      * @return the RHS value if this is a Right, otherwise an empty {@code Optional}
      */

src/main/java/io/jbock/util/Left.java

@@ -79,7 +79,7 @@ public <U> U fold(
     }
 
     @Override
-    public void ifPresentOrElse(Consumer<? super L> leftAction, Consumer<? super R> rightAction) {
+    public void ifLeftOrElse(Consumer<? super L> leftAction, Consumer<? super R> rightAction) {
         leftAction.accept(value);
     }
 

src/main/java/io/jbock/util/Right.java

@@ -79,7 +79,7 @@ public <U> U fold(
     }
 
     @Override
-    public void ifPresentOrElse(Consumer<? super L> leftAction, Consumer<? super R> rightAction) {
+    public void ifLeftOrElse(Consumer<? super L> leftAction, Consumer<? super R> rightAction) {
         rightAction.accept(value);
     }
 

src/main/java/io/jbock/util/ValidatingCollector.java

@@ -26,7 +26,8 @@ void accumulate(Either<L, R> either) {
             if (left != null) {
                 return;
             }
-            either.ifPresentOrElse(l -> left = l,
+            either.ifLeftOrElse(
+                    left -> this.left = left,
                     right::add);
         }
 

src/main/java/io/jbock/util/ValidatingCollectorAll.java

@@ -24,10 +24,10 @@ static final class Acc<L, R> {
 
         void accumulate(Either<L, R> either) {
             if (!left.isEmpty()) {
-                either.ifPresentOrElse(left::add, r -> {
+                either.ifLeftOrElse(left::add, r -> {
                 });
             } else {
-                either.ifPresentOrElse(left::add, right::add);
+                either.ifLeftOrElse(left::add, right::add);
             }
         }
 

src/test/java/io/jbock/util/EitherTest.java

@@ -120,10 +120,10 @@ void testOrElseThrow() {
     void testAccept() {
         String[] output = {"1"};
         Either<Integer, Integer> left = Either.left(1);
-        left.ifPresentOrElse(l -> output[0] = "L", r -> output[0] = "R");
+        left.ifLeftOrElse(l -> output[0] = "L", r -> output[0] = "R");
         assertEquals("L", output[0]);
         Either<Integer, Integer> right = Either.right(1);
-        right.ifPresentOrElse(l -> output[0] = "L", r -> output[0] = "R");
+        right.ifLeftOrElse(l -> output[0] = "L", r -> output[0] = "R");
         assertEquals("R", output[0]);
     }