From fad2890e10175db696eb614265643e50552d9d75 Mon Sep 17 00:00:00 2001
From: Ben Woo <30431861+benwoo1110@users.noreply.github.com>
Date: Fri, 14 Apr 2023 22:30:25 +0800
Subject: [PATCH] refactor: Rename reason method name with get

---
 .../MultiverseCore/utils/result/Result.java   | 24 +++++++++----------
 .../utils/result/ResultGroup.java             |  4 ++--
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/main/java/com/onarandombox/MultiverseCore/utils/result/Result.java b/src/main/java/com/onarandombox/MultiverseCore/utils/result/Result.java
index 386e7a95..57fcd9d9 100644
--- a/src/main/java/com/onarandombox/MultiverseCore/utils/result/Result.java
+++ b/src/main/java/com/onarandombox/MultiverseCore/utils/result/Result.java
@@ -16,39 +16,39 @@ public sealed interface Result<S extends SuccessReason, F extends FailureReason>
 
     boolean isFailure();
 
-    S successReason();
+    S getSuccessReason();
 
-    F failureReason();
+    F getFailureReason();
 
     default Result<S, F> onSuccess(Consumer<S> consumer) {
         if (this.isSuccess()) {
-            consumer.accept(this.successReason());
+            consumer.accept(this.getSuccessReason());
         }
         return this;
     }
 
     default Result<S, F> onFailure(Consumer<F> consumer) {
         if (this.isFailure()) {
-            consumer.accept(this.failureReason());
+            consumer.accept(this.getFailureReason());
         }
         return this;
     }
 
     default Result<S, F> onSuccessReason(S successReason, Consumer<S> consumer) {
-        if (this.isSuccess() && this.successReason() == successReason) {
-            consumer.accept(this.successReason());
+        if (this.isSuccess() && this.getSuccessReason() == successReason) {
+            consumer.accept(this.getSuccessReason());
         }
         return this;
     }
 
     default Result<S, F> onFailureReason(F failureReason, Consumer<F> consumer) {
-        if (this.isFailure() && this.failureReason() == failureReason) {
-            consumer.accept(this.failureReason());
+        if (this.isFailure() && this.getFailureReason() == failureReason) {
+            consumer.accept(this.getFailureReason());
         }
         return this;
     }
 
-    record Success<F extends FailureReason, S extends SuccessReason>(S successReason) implements Result<S, F> {
+    record Success<F extends FailureReason, S extends SuccessReason>(S getSuccessReason) implements Result<S, F> {
         @Override
         public boolean isSuccess() {
             return true;
@@ -60,12 +60,12 @@ public sealed interface Result<S extends SuccessReason, F extends FailureReason>
         }
 
         @Override
-        public F failureReason() {
+        public F getFailureReason() {
             throw new NoSuchElementException("No failure reason for success");
         }
     }
 
-    record Failure<S extends SuccessReason, F extends FailureReason>(F failureReason) implements Result<S, F> {
+    record Failure<S extends SuccessReason, F extends FailureReason>(F getFailureReason) implements Result<S, F> {
         @Override
         public boolean isSuccess() {
             return false;
@@ -77,7 +77,7 @@ public sealed interface Result<S extends SuccessReason, F extends FailureReason>
         }
 
         @Override
-        public S successReason() {
+        public S getSuccessReason() {
             throw new NoSuchElementException("No success reason for failure");
         }
     }
diff --git a/src/main/java/com/onarandombox/MultiverseCore/utils/result/ResultGroup.java b/src/main/java/com/onarandombox/MultiverseCore/utils/result/ResultGroup.java
index 120f9d8c..ccd7976b 100644
--- a/src/main/java/com/onarandombox/MultiverseCore/utils/result/ResultGroup.java
+++ b/src/main/java/com/onarandombox/MultiverseCore/utils/result/ResultGroup.java
@@ -69,7 +69,7 @@ public class ResultGroup {
             return Option.none();
         }
         return Option.ofOptional(results.stream()
-                .map(Result::successReason)
+                .map(Result::getSuccessReason)
                 .filter(successReasonClass::isInstance)
                 .map(successReasonClass::cast)
                 .findFirst());
@@ -80,7 +80,7 @@ public class ResultGroup {
             return Option.none();
         }
         return Option.ofOptional(results.stream()
-                .map(Result::failureReason)
+                .map(Result::getFailureReason)
                 .filter(failureReasonClass::isInstance)
                 .map(failureReasonClass::cast)
                 .findFirst());