From 95474a32c4a63524d137db050233c110183e2e91 Mon Sep 17 00:00:00 2001
From: DmitriyMX <dimon550@gmail.com>
Date: Sun, 13 Jun 2021 14:32:00 +0300
Subject: [PATCH] refactoring: Cli-Parser

---
 cli-parser/build.gradle                       |  2 +
 cli-parser/gradle.properties                  |  2 +
 .../main/java/mc/cliparser/CommandLine.java   | 17 +++++
 .../java/mc/cliparser/CommandLineParser.java  | 52 +++++++++++++
 .../src/main/java/mc/cliparser/Option.java    | 25 +++++++
 .../mc/cliparser/CommandLineParserTest.java   | 73 +++++++++++++++++++
 settings.gradle                               |  1 +
 7 files changed, 172 insertions(+)
 create mode 100644 cli-parser/build.gradle
 create mode 100644 cli-parser/gradle.properties
 create mode 100644 cli-parser/src/main/java/mc/cliparser/CommandLine.java
 create mode 100644 cli-parser/src/main/java/mc/cliparser/CommandLineParser.java
 create mode 100644 cli-parser/src/main/java/mc/cliparser/Option.java
 create mode 100644 cli-parser/src/test/java/mc/cliparser/CommandLineParserTest.java

diff --git a/cli-parser/build.gradle b/cli-parser/build.gradle
new file mode 100644
index 0000000..4edad6d
--- /dev/null
+++ b/cli-parser/build.gradle
@@ -0,0 +1,2 @@
+//file:noinspection GrUnresolvedAccess
+apply from: rootDir.toPath().resolve('logic.gradle').toFile()
diff --git a/cli-parser/gradle.properties b/cli-parser/gradle.properties
new file mode 100644
index 0000000..afd47c0
--- /dev/null
+++ b/cli-parser/gradle.properties
@@ -0,0 +1,2 @@
+# suppress inspection "UnusedProperty" for whole file
+module.name=cli-parser
\ No newline at end of file
diff --git a/cli-parser/src/main/java/mc/cliparser/CommandLine.java b/cli-parser/src/main/java/mc/cliparser/CommandLine.java
new file mode 100644
index 0000000..4f115d9
--- /dev/null
+++ b/cli-parser/src/main/java/mc/cliparser/CommandLine.java
@@ -0,0 +1,17 @@
+package mc.cliparser;
+
+import lombok.RequiredArgsConstructor;
+import lombok.ToString;
+
+import java.util.Set;
+
+@RequiredArgsConstructor
+@ToString
+public class CommandLine {
+
+	private final Set<Option> options;
+
+	public boolean has(Option option) {
+		return options.contains(option);
+	}
+}
diff --git a/cli-parser/src/main/java/mc/cliparser/CommandLineParser.java b/cli-parser/src/main/java/mc/cliparser/CommandLineParser.java
new file mode 100644
index 0000000..cc0df69
--- /dev/null
+++ b/cli-parser/src/main/java/mc/cliparser/CommandLineParser.java
@@ -0,0 +1,52 @@
+package mc.cliparser;
+
+import java.util.HashSet;
+import java.util.Set;
+import java.util.concurrent.atomic.AtomicReference;
+
+public class CommandLineParser {
+
+	private final Set<Option> options = new HashSet<>();
+
+	public void addOption(Option option) {
+		options.add(option);
+	}
+
+	public CommandLine parse(String[] args) {
+
+		Set<Option> foundOptions = new HashSet<>();
+		AtomicReference<Option> refCurrentOption = new AtomicReference<>(null);
+
+		for (String arg : args) {
+			if (refCurrentOption.get() != null) {
+				refCurrentOption.get().value(arg);
+				foundOptions.add(refCurrentOption.get());
+				refCurrentOption.set(null);
+			} else {
+				parseOptArgs(arg, foundOptions, refCurrentOption);
+			}
+		}
+
+		return new CommandLine(foundOptions);
+	}
+
+	@SuppressWarnings("java:S125")
+	private void parseOptArgs(String arg, Set<Option> foundOptions, AtomicReference<Option> refCurrentOption) {
+		String optName;
+		if (arg.startsWith("--")) {
+			optName = arg.substring(2);
+		} else /*if (args[i].startsWith("-"))*/ {
+			optName = arg.substring(1);
+		}
+
+		for (Option option : options) {
+			if (optName.equals(option.shortName()) || optName.equals(option.longName())) {
+				if (option.hasArgs()) {
+					refCurrentOption.set(option);
+				} else {
+					foundOptions.add(option);
+				}
+			}
+		}
+	}
+}
diff --git a/cli-parser/src/main/java/mc/cliparser/Option.java b/cli-parser/src/main/java/mc/cliparser/Option.java
new file mode 100644
index 0000000..74b917a
--- /dev/null
+++ b/cli-parser/src/main/java/mc/cliparser/Option.java
@@ -0,0 +1,25 @@
+package mc.cliparser;
+
+import lombok.*;
+import lombok.experimental.Accessors;
+
+@Accessors(fluent = true)
+@Builder
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
+@EqualsAndHashCode
+@ToString
+public class Option {
+
+	@Getter
+	private final String shortName;
+
+	@Getter
+	private final String longName;
+
+	@Getter
+	private final boolean hasArgs;
+
+	@Getter
+	@Setter
+	private String value;
+}
diff --git a/cli-parser/src/test/java/mc/cliparser/CommandLineParserTest.java b/cli-parser/src/test/java/mc/cliparser/CommandLineParserTest.java
new file mode 100644
index 0000000..75fb5e8
--- /dev/null
+++ b/cli-parser/src/test/java/mc/cliparser/CommandLineParserTest.java
@@ -0,0 +1,73 @@
+package mc.cliparser;
+
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+class CommandLineParserTest {
+
+	@Test
+	void optionTest() {
+		Option option = Option.builder().shortName("v").build();
+		assertNull(option.longName());
+		assertFalse(option.hasArgs());
+
+		option = Option.builder().longName("version").build();
+		assertNull(option.shortName());
+		assertFalse(option.hasArgs());
+
+		option = Option.builder().shortName("v").longName("value1").hasArgs(true).build();
+		assertNotNull(option.shortName());
+		assertNotNull(option.longName());
+		assertTrue(option.hasArgs());
+	}
+
+	@Test
+	void shortOptionFlag() {
+		Option option = Option.builder().shortName("v").build();
+
+		var parser = new CommandLineParser();
+		parser.addOption(option);
+
+		CommandLine commandLine = parser.parse(new String[]{ "-v" });
+		assertTrue(commandLine.has(option));
+	}
+
+	@Test
+	void longOptionFlag() {
+		Option option = Option.builder().longName("version").build();
+
+		var parser = new CommandLineParser();
+		parser.addOption(option);
+
+		CommandLine commandLine = parser.parse(new String[]{ "--version" });
+		assertTrue(commandLine.has(option));
+	}
+
+	@Test
+	void argsOption() {
+		Option option1 = Option.builder().shortName("v").longName("value1").hasArgs(true).build();
+		Option option2 = Option.builder().shortName("a").longName("value2").hasArgs(true).build();
+
+		var parser = new CommandLineParser();
+		parser.addOption(option1);
+		parser.addOption(option2);
+
+		CommandLine commandLine = parser.parse(new String[]{ "--value1", "arg1", "-a", "arg2"  });
+		assertTrue(commandLine.has(option1));
+		assertEquals("arg1", option1.value());
+		assertTrue(commandLine.has(option2));
+		assertEquals("arg2", option2.value());
+	}
+
+	@Test
+	void noPassOption() {
+		Option option = Option.builder().longName("version").build();
+
+		var parser = new CommandLineParser();
+		parser.addOption(option);
+
+		CommandLine commandLine = parser.parse(new String[]{ "--any-opt" });
+		assertFalse(commandLine.has(option));
+	}
+}
diff --git a/settings.gradle b/settings.gradle
index d109887..0257633 100644
--- a/settings.gradle
+++ b/settings.gradle
@@ -13,5 +13,6 @@ rootProject.name = map.get('project.name')
 include('utils')
 include('protocol-new')
 include('server-new')
+include('cli-parser')
 //include('protocol')
 //include('server')
\ No newline at end of file