New methods

make constructor private for consistency with PaginatedMenu in favor of Menu#create.
add #clone()

Author: coderFlameyosFlow

src/main/java/me/flame/menus/menu/Menu.java

@@ -2,37 +2,69 @@
 
 import me.flame.menus.items.MenuItem;
 import me.flame.menus.modifiers.Modifier;
+import org.jetbrains.annotations.Contract;
 import org.jetbrains.annotations.NotNull;
 
 import java.util.EnumSet;
-import java.util.List;
 
+/**
+ * Most commonly used normal Menu extending BaseMenu.
+ */
 @SuppressWarnings("unused")
-public final class Menu extends BaseMenu<Menu> {
-    public Menu(int rows, String title, EnumSet<Modifier> modifiers, boolean colorize) {
-        super(rows, title, modifiers, colorize);
-    }
-
-    public Menu(int rows, String title, EnumSet<Modifier> modifiers) {
+public final class Menu extends BaseMenu implements Cloneable {
+    private Menu(int rows, String title, EnumSet<Modifier> modifiers) {
         super(rows, title, modifiers);
     }
 
-    public Menu(MenuType type, String title, EnumSet<Modifier> modifiers) {
+    private Menu(MenuType type, String title, EnumSet<Modifier> modifiers) {
         super(type, title, modifiers);
     }
 
-    public Menu(int rows, String title) {
+    private Menu(int rows, String title) {
         super(rows, title, EnumSet.noneOf(Modifier.class));
     }
 
-    public Menu(MenuType type, String title) {
+    private Menu(MenuType type, String title) {
         super(type, title, EnumSet.noneOf(Modifier.class));
     }
 
-    public Menu(MenuType type, String title, EnumSet<Modifier> modifiers, boolean colorize) {
+    private Menu(MenuType type, String title, EnumSet<Modifier> modifiers, boolean colorize) {
         super(type, title, modifiers, colorize);
     }
 
+    @NotNull
+    @Contract("_, _ -> new")
+    public static Menu create(String title, int rows) {
+        return new Menu(rows, title);
+    }
+
+    @NotNull
+    @Contract("_, _, _ -> new")
+    public static Menu create(String title, int rows, EnumSet<Modifier> modifiers) {
+        return new Menu(rows, title, modifiers);
+    }
+
+    @NotNull
+    @Contract("_, _ -> new")
+    public static Menu create(String title, MenuType type) {
+        return new Menu(type, title);
+    }
+
+    @NotNull
+    @Contract("_, _, _ -> new")
+    public static Menu create(String title, MenuType type, EnumSet<Modifier> modifiers) {
+        return new Menu(type, title, modifiers);
+    }
+
+    public static Menu create(MenuData menuData) {
+        MenuType type = menuData.getType();
+        Menu menu = type != MenuType.CHEST
+                ? new Menu(type, menuData.getTitle(), menuData.getModifiers())
+                : new Menu(menuData.getRows(), menuData.getTitle(), menuData.getModifiers());
+        menu.setContents(menuData.getItems().toArray(new MenuItem[0]));
+        return menu;
+    }
+
     /**
      * Generate a paginated menu; convert to a {@link PaginatedMenu} from a {@link Menu}
      *
@@ -42,9 +74,7 @@ public Menu(MenuType type, String title, EnumSet<Modifier> modifiers, boolean co
     @NotNull
     public PaginatedMenu pagination() {
         PaginatedMenu menu = PaginatedMenu.create(title, rows, 3, modifiers);
-        for (int i = 0; i < size; i++) {
-            menu.setItem(i, itemMap.get(i));
-        }
+        menu.setContents(itemMap);
         return menu;
     }
 
@@ -56,9 +86,16 @@ public PaginatedMenu pagination() {
     @NotNull
     public PaginatedMenu pagination(int pages) {
         PaginatedMenu menu = PaginatedMenu.create(title, rows, pages, modifiers);
-        for (int i = 0; i < size; i++) {
-            menu.setItem(i, itemMap.get(i));
-        }
+        menu.setContents(itemMap);
         return menu;
     }
+
+    @Override
+    public Menu clone() {
+        try {
+            return (Menu) super.clone();
+        } catch (CloneNotSupportedException e) {
+            throw new RuntimeException(e);
+        }
+    }
 }