build.gradle.kts

@@ -6,7 +6,7 @@ plugins {
 
 allprojects {
     group = "net.tomatentum.cutin"
-    version = "0.1.1" + (if (!project.hasProperty("release")) ("-" + getGitHash()) else "")
+    version = "0.1.0" + (if (!project.hasProperty("release")) ("-" + getGitHash()) else "")
     description = "A lightweight Reflection abstraction specifically but not exclusively made for tueem/Marinara."
     plugins.withType<JavaPlugin> {
         tasks.withType<Jar>().configureEach {

lib/src/main/java/net/tomatentum/cutin/container/LoneMethodContainer.java

@@ -6,7 +6,6 @@ import java.util.Collection;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Optional;
-import java.util.Set;
 
 import net.tomatentum.cutin.ReflectedMethod;
 import net.tomatentum.cutin.ReflectedMethodFactory;
@@ -22,22 +21,15 @@ public class LoneMethodContainer<I extends Object> implements MethodContainer<I>
     }
 
     @Override
-    public MethodContainer<I> addMethod(ReflectedMethod<I> method) {
-        this.methodStore.put(method.identifier(), method);
-        return this;
+    public void addMethods(ReflectedMethod<I>... methods) {
+        for (ReflectedMethod<I> reflectedMethod : methods)
+            this.methodStore.put(reflectedMethod.identifier(), reflectedMethod);
     }
 
     @Override
-    public MethodContainer<I> addMethods(Object containingObject, Method... methods) {
+    public void addMethods(Object containingObject, Method... methods) {
         for (Method method : methods)
-            this.factory.produce(method, containingObject)
-                .ifPresent(this::addMethod);
-        return this;
-    }
-
-    @Override
-    public Set<I> identifiers() {
-        return methodStore.keySet();
+            this.addMethods(this.factory.produce(method, containingObject));
     }
 
     @Override

lib/src/main/java/net/tomatentum/cutin/container/MethodContainer.java

@@ -3,26 +3,17 @@ package net.tomatentum.cutin.container;
 import java.lang.reflect.Method;
 import java.util.Collection;
 import java.util.Optional;
-import java.util.Set;
 
 import net.tomatentum.cutin.ReflectedMethod;
 
 public interface MethodContainer<I extends Object> {
 
-    MethodContainer<I> addMethod(ReflectedMethod<I> method);
-    default MethodContainer<I> addMethods(ReflectedMethod<I>[] methods) {
-        for (ReflectedMethod<I> reflectedMethod : methods) {
-            this.addMethod(reflectedMethod);
-        }
-        return this;
-    }
-    MethodContainer<I> addMethods(Object containingObject, Method... methods);
-    default MethodContainer<I> addAllMethods(Object containingObject) {
+    void addMethods(ReflectedMethod<I>... methods);
+    void addMethods(Object containingObject, Method... methods);
+    default void addAllMethods(Object containingObject) {
         this.addMethods(containingObject, containingObject.getClass().getDeclaredMethods());
-        return this;
-    }
+    };
 
-    Set<I> identifiers();
     Collection<ReflectedMethod<I>> methods();
     Collection<ReflectedMethod<I>> findFor(I identifier);
     Optional<ReflectedMethod<I>> findFirstFor(I identifier);

lib/src/main/java/net/tomatentum/cutin/container/MultiMethodContainer.java

@@ -5,7 +5,6 @@ import java.util.Collection;
 import java.util.HashSet;
 import java.util.Optional;
 import java.util.Set;
-import java.util.stream.Collectors;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -25,28 +24,20 @@ public class MultiMethodContainer<I extends Object> implements MethodContainer<I
     }
 
     @Override
-    public MethodContainer<I> addMethod(ReflectedMethod<I> method) {
+    public void addMethods(ReflectedMethod<I>... methods) {
+        for (ReflectedMethod<I> rMethod : methods) {
             Optional<Entry<I>> oentry = this.entries.stream()
-                .filter(e -> method.identifier().equals(e.identifier()))
+                .filter(e -> rMethod.identifier().equals(e.identifier()))
                 .findFirst();
-            Entry<I> entry = oentry.orElse(new Entry<>(method.identifier())).addMethod(method);
+            Entry<I> entry = oentry.orElse(new Entry<>(rMethod.identifier())).addMethod(rMethod);
             if (oentry.isEmpty()) this.entries.add(entry);
-            return this;
+        }   
     }
 
     @Override
-    public MethodContainer<I> addMethods(Object containingObject, Method... methods) {
+    public void addMethods(Object containingObject, Method... methods) {
         for (Method method : methods)
-            this.factory.produce(method, containingObject)
-                .ifPresent(this::addMethod);
-        return this;
-    }
-
-    @Override
-    public Set<I> identifiers() {
-        return entries().stream()
-            .map(Entry::identifier)
-            .collect(Collectors.toSet());
+            this.addMethods(this.factory.produce(method, containingObject));
     }
 
     @Override
@@ -72,10 +63,6 @@ public class MultiMethodContainer<I extends Object> implements MethodContainer<I
             .findFirst();
     }
 
-    protected Set<Entry<I>> entries() {
-        return this.entries;
-    }
-
     public static record Entry<I extends Object>(I identifier, Set<ReflectedMethod<I>> methods) {
 
         public Entry(I identifier) {