Merge branch 'master' of https://github.com/bculkin2442/rgens

1 files changed, 58 insertions, 12 deletions

diff --git a/src/main/java/bjc/rgens/parser/elements/vars/VariableElement.java b/src/main/java/bjc/rgens/parser/elements/vars/VariableElement.java
index eb36af8..c73e249 100644
--- a/src/main/java/bjc/rgens/parser/elements/vars/VariableElement.java
+++ b/src/main/java/bjc/rgens/parser/elements/vars/VariableElement.java
@@ -7,30 +7,73 @@ import bjc.rgens.parser.GenerationState;
 import bjc.rgens.parser.GrammarException;
 import bjc.utils.ioutils.LevelSplitter;
 
+/**
+ * Case element which references a variable.
+ * 
+ * @author Ben Culkin
+ *
+ */
 public abstract class VariableElement {
+	/**
+	 * Whether or not to forbid spaces in this element.
+	 */
 	public boolean forbidSpaces;
 
+	/**
+	 * Create a new variable element.
+	 * 
+	 * @param forbidSpacing
+	 *                      Whether spacing should be forbidden in this element.
+	 */
 	protected VariableElement(boolean forbidSpacing) {
 		forbidSpaces = forbidSpacing;
 	}
 
+	/**
+	 * Generate this element.
+	 * 
+	 * @param state
+	 *              The state of generation.
+	 */
 	public abstract void generate(GenerationState state);
 
+	/**
+	 * Parse a variable element from a string.
+	 * 
+	 * @param varElm
+	 *               The string to parse.
+	 * 
+	 * @return The variable elements which make up the string.
+	 */
 	public static List<VariableElement> parseElementString(String varElm) {
 		boolean forbidSpaces = LevelSplitter.def.levelContains(varElm, "-", "+");
 
 		String[] parts;
 
-		if(forbidSpaces) {
-			parts = LevelSplitter.def.levelSplit(varElm, true, "-", "+").toArray(new String[0]);
+		if (forbidSpaces) {
+			parts = LevelSplitter.def.levelSplit(varElm, true, "-", "+")
+					.toArray(new String[0]);
 		} else {
-			parts = new String[] { varElm };
+			parts = new String[] {
+					varElm
+			};
 		}
 
 		return parseElementString(forbidSpaces, parts);
 	}
 
-	public static List<VariableElement> parseElementString(boolean forbidSpaces, String... parts) {
+	/**
+	 * Parse a string of variable elements.
+	 * 
+	 * @param forbidSpaces
+	 *                     Whether or not to forbid spacing in this variable.
+	 * @param parts
+	 *                     The parts to parse into variable elements.
+	 * 
+	 * @return The variable elements from the string.
+	 */
+	public static List<VariableElement> parseElementString(boolean forbidSpaces,
+			String... parts) {
 		List<VariableElement> elms = new ArrayList<>(parts.length);
 
 		VariableElement prevElement = null;
@@ -44,23 +87,26 @@ public abstract class VariableElement {
 
 			VariableElement elm = null;
 
-			if(part.startsWith("$")) {
+			if (part.startsWith("$")) {
 				elm = new VRefVariableElement(forbidSpaces, part.substring(1));
 			} else if (part.startsWith("@")) {
-				if(forbidSpaces)
-					throw new GrammarException("Arrays references aren't allowed in rule names");
+				if (forbidSpaces)
+					throw new GrammarException(
+							"Arrays references aren't allowed in rule names");
 
 				elm = new ARefVariableElement(part.substring(1));
 			} else if (part.startsWith("%")) {
-				elm = new RRefVariableElement(forbidSpaces, String.format("[%s]", part.substring(1)));
+				elm = new RRefVariableElement(forbidSpaces,
+						String.format("[%s]", part.substring(1)));
 			} else if (part.startsWith("/")) {
 				throw new GrammarException("Template variables aren't implemented yet");
 			} else {
-				if(prevElement != null && prevElement instanceof LiteralVariableElement) {
+				if (prevElement != null
+						&& prevElement instanceof LiteralVariableElement) {
 					/* Aggregate chain literals together */
-					((LiteralVariableElement)prevElement).val += part;
+					((LiteralVariableElement) prevElement).val += part;
 				} else {
-					if(part.contains(" ")) {
+					if (part.contains(" ")) {
 						elm = new LiteralVariableElement(false, part);
 					} else {
 						elm = new LiteralVariableElement(true, part);
@@ -68,7 +114,7 @@ public abstract class VariableElement {
 				}
 			}
 
-			if(elm != null) {
+			if (elm != null) {
 				elms.add(elm);
 
 				prevElement = elm;