From 46c1a3e1e7cf21f3c63dcb028fd1807e870dfbef Mon Sep 17 00:00:00 2001 From: Benjamin Culkin Date: Mon, 5 Feb 2018 14:04:27 -0800 Subject: Warning cleanup --- .../main/java/bjc/utils/ioutils/CLFormatter.java | 420 +++++++++++---------- 1 file changed, 215 insertions(+), 205 deletions(-) (limited to 'base/src/main/java/bjc/utils/ioutils/CLFormatter.java') diff --git a/base/src/main/java/bjc/utils/ioutils/CLFormatter.java b/base/src/main/java/bjc/utils/ioutils/CLFormatter.java index eefd532..1f65607 100644 --- a/base/src/main/java/bjc/utils/ioutils/CLFormatter.java +++ b/base/src/main/java/bjc/utils/ioutils/CLFormatter.java @@ -9,12 +9,11 @@ import java.util.UnknownFormatConversionException; import java.util.regex.Matcher; import java.util.regex.Pattern; -import bjc.utils.PropertyDB; import bjc.utils.esodata.Tape; import bjc.utils.esodata.SingleTape; import static bjc.utils.PropertyDB.applyFormat; -import static bjc.utils.PropertyDB.getCompiledRegex; + import static bjc.utils.PropertyDB.getRegex; public class CLFormatter { @@ -23,15 +22,15 @@ public class CLFormatter { public final boolean colonMod; public CLModifiers(boolean at, boolean colon) { - atMod = at; + atMod = at; colonMod = colon; } public static CLModifiers fromString(String modString) { - boolean atMod = false; + boolean atMod = false; boolean colonMod = false; - if(modString != null) { - atMod = modString.contains("@"); + if (modString != null) { + atMod = modString.contains("@"); colonMod = modString.contains(":"); } @@ -40,6 +39,8 @@ public class CLFormatter { } public static class EscapeException extends RuntimeException { + private static final long serialVersionUID = -4552821131068559005L; + public final boolean endIteration; public EscapeException() { @@ -59,16 +60,17 @@ public class CLFormatter { public void format(); } - private static final String prefixParam = getRegex("clFormatPrefix"); + private static final String prefixParam = getRegex("clFormatPrefix"); private static final Pattern pPrefixParam = Pattern.compile(prefixParam); - private static final String formatMod = getRegex("clFormatModifier"); + private static final String formatMod = getRegex("clFormatModifier"); private static final String prefixList = applyFormat("delimSeparatedList", prefixParam, ","); private static final String directiveName = getRegex("clFormatName"); - private static final String formatDirective = applyFormat("clFormatDirective", prefixList, formatMod, directiveName); + private static final String formatDirective = applyFormat("clFormatDirective", prefixList, formatMod, + directiveName); private static final Pattern pFormatDirective = Pattern.compile(formatDirective); private Map extraDirectives; @@ -77,15 +79,16 @@ public class CLFormatter { extraDirectives = new HashMap<>(); } - private void checkItem(Object itm, char directive) { - if(itm == null) - throw new IllegalArgumentException(String.format("No argument provided for %c directive", directive)); + private static void checkItem(Object itm, char directive) { + if (itm == null) + throw new IllegalArgumentException( + String.format("No argument provided for %c directive", directive)); } public String formatString(String format, Object... params) { StringBuffer sb = new StringBuffer(); /* Put the parameters where we can easily handle them. */ - Tape tParams = new SingleTape(params); + Tape tParams = new SingleTape<>(params); doFormatString(format, sb, tParams); @@ -95,124 +98,123 @@ public class CLFormatter { private void doFormatString(String format, StringBuffer sb, Tape tParams) { Matcher dirMatcher = pFormatDirective.matcher(format); - while(dirMatcher.find()) { + while (dirMatcher.find()) { dirMatcher.appendReplacement(sb, ""); - String dirName = dirMatcher.group("name"); - String dirFunc = dirMatcher.group("funcname"); - String dirMods = dirMatcher.group("modifiers"); + String dirName = dirMatcher.group("name"); + String dirFunc = dirMatcher.group("funcname"); + String dirMods = dirMatcher.group("modifiers"); String dirParams = dirMatcher.group("params"); CLParameters arrParams = CLParameters.fromDirective(dirParams.split("(?": - /* @TODO - * Figure out how to implement - * tabulation/justification in a - * reasonable manner. - */ - throw new IllegalArgumentException("Layout-control directives aren't implemented yet."); - case "F": - case "E": - case "G": - case "$": - /* @TODO implement floating point directives. */ - throw new IllegalArgumentException("Floating-point directives aren't implemented yet."); - case "S": - case "W": - /* @TODO - * figure out if we want to implement - * someting for these directives instead - * of punting. - * */ - throw new IllegalArgumentException("S and W aren't implemented. Use A instead"); - default: - String msg = String.format("Unknown format directive '%s'", dirName); - throw new UnknownFormatConversionException(msg); + switch (dirName) { + case "A": + checkItem(item, 'A'); + handleAestheticDirective(sb, item, mods, arrParams); + tParams.right(); + break; + case "B": + checkItem(item, 'B'); + if (!(item instanceof Number)) { + throw new IllegalFormatConversionException('B', item.getClass()); + } + handleNumberDirective(sb, mods, arrParams, -1, ((Number) item).longValue(), 2); + tParams.right(); + break; + case "C": + checkItem(item, 'C'); + handleCDirective(sb, item, mods); + tParams.right(); + break; + case "D": + checkItem(item, 'D'); + if (!(item instanceof Number)) { + throw new IllegalFormatConversionException('D', item.getClass()); + } + handleNumberDirective(sb, mods, arrParams, -1, ((Number) item).longValue(), 10); + tParams.right(); + break; + case "O": + checkItem(item, 'O'); + if (!(item instanceof Number)) { + throw new IllegalFormatConversionException('O', item.getClass()); + } + handleNumberDirective(sb, mods, arrParams, -1, ((Number) item).longValue(), 8); + tParams.right(); + break; + case "R": + checkItem(item, 'R'); + handleRadixDirective(sb, mods, arrParams, item); + tParams.right(); + break; + case "X": + checkItem(item, 'X'); + if (!(item instanceof Number)) { + throw new IllegalFormatConversionException('X', item.getClass()); + } + handleNumberDirective(sb, mods, arrParams, -1, ((Number) item).longValue(), 16); + tParams.right(); + break; + case "&": + handleFreshlineDirective(sb, arrParams); + break; + case "%": + handleLiteralDirective(sb, arrParams, "\n", '%'); + break; + case "|": + handleLiteralDirective(sb, arrParams, "\f", '|'); + break; + case "~": + handleLiteralDirective(sb, arrParams, "~", '~'); + break; + case "*": + handleGotoDirective(mods, arrParams, tParams); + break; + case "^": + handleEscapeDirective(mods, arrParams, tParams); + break; + case "[": + handleConditionalDirective(sb, mods, arrParams, tParams, dirMatcher); + break; + case "]": + throw new IllegalArgumentException("Found conditional-end outside of conditional."); + case ";": + throw new IllegalArgumentException( + "Found conditional-seperator outside of conditional."); + case "T": + case "<": + case ">": + /* + * @TODO Figure out how to implement tabulation/justification in a reasonable + * manner. + */ + throw new IllegalArgumentException("Layout-control directives aren't implemented yet."); + case "F": + case "E": + case "G": + case "$": + /* @TODO implement floating point directives. */ + throw new IllegalArgumentException("Floating-point directives aren't implemented yet."); + case "S": + case "W": + /* + * @TODO figure out if we want to implement someting for these directives + * instead of punting. + */ + throw new IllegalArgumentException("S and W aren't implemented. Use A instead"); + default: + String msg = String.format("Unknown format directive '%s'", dirName); + throw new UnknownFormatConversionException(msg); } } @@ -220,14 +222,14 @@ public class CLFormatter { } private void handleCDirective(StringBuffer buff, Object parm, CLModifiers mods) { - if(!(parm instanceof Character)) { + if (!(parm instanceof Character)) { throw new IllegalFormatConversionException('C', parm.getClass()); } char ch = (Character) parm; int codepoint = (int) ch; - if(mods.colonMod) { + if (mods.colonMod) { /* * Colon mod means print Unicode character name. */ @@ -237,58 +239,59 @@ public class CLFormatter { } } - private void handleFreshlineDirective(StringBuffer buff, CLParameters params) { + private void handleFreshlineDirective(StringBuffer buff, CLParameters params) { int nTimes = 1; - if(params.length() > 1) { + if (params.length() > 1) { nTimes = params.getInt(0, "occurance count", '&'); } - if(buff.charAt(buff.length() - 1) == '\n') nTimes -= 1; + if (buff.charAt(buff.length() - 1) == '\n') + nTimes -= 1; - for(int i = 0; i < nTimes; i++) { + for (int i = 0; i < nTimes; i++) { buff.append("\n"); } } - private void handleLiteralDirective(StringBuffer buff, CLParameters params, String lit, char directive) { + private void handleLiteralDirective(StringBuffer buff, CLParameters params, String lit, char directive) { int nTimes = 1; - if(params.length() > 1) { + if (params.length() > 1) { nTimes = params.getInt(0, "occurance count", directive); } - for(int i = 0; i < nTimes; i++) { + for (int i = 0; i < nTimes; i++) { buff.append(lit); } } - private void handleNumberDirective(StringBuffer buff, CLModifiers mods, CLParameters params, int argidx, long val, int radix) { + private void handleNumberDirective(StringBuffer buff, CLModifiers mods, CLParameters params, int argidx, + long val, int radix) { /* - * Initialize the two padding related parameters, and - * then fill them in from the directive parameters if - * they are present. + * Initialize the two padding related parameters, and then fill them in from the + * directive parameters if they are present. */ - int mincol = 0; + int mincol = 0; char padchar = ' '; - if(params.length() > (argidx + 2)) { + if (params.length() > (argidx + 2)) { mincol = params.getIntDefault(argidx + 1, "minimum column count", 'R', 0); } - if(params.length() > (argidx + 3)) { + if (params.length() > (argidx + 3)) { padchar = params.getCharDefault(argidx + 2, "padding character", 'R', ' '); } - if(mods.colonMod) { + if (mods.colonMod) { /* - * We're doing commas, so check if the two - * comma-related parameters were supplied. + * We're doing commas, so check if the two comma-related parameters were + * supplied. */ - int commaInterval = 0; - char commaChar = ','; - if(params.length() > (argidx + 3)) { + int commaInterval = 0; + char commaChar = ','; + if (params.length() > (argidx + 3)) { commaChar = params.getCharDefault((argidx + 3), "comma character", 'R', ' '); } - if(params.length() > (argidx + 4)) { + if (params.length() > (argidx + 4)) { commaInterval = params.getIntDefault((argidx + 4), "comma interval", 'R', 0); } @@ -299,26 +302,27 @@ public class CLFormatter { } private void handleRadixDirective(StringBuffer buff, CLModifiers mods, CLParameters params, Object arg) { - if(!(arg instanceof Number)) { + if (!(arg instanceof Number)) { throw new IllegalFormatConversionException('R', arg.getClass()); } /* * @TODO see if this is the way we want to do this. */ - long val = ((Number)arg).longValue(); + long val = ((Number) arg).longValue(); - if(params.length() == 0) { - if(mods.atMod) { - buff.append(NumberUtils.toRoman((Long)val, mods.colonMod)); - } else if(mods.colonMod) { + if (params.length() == 0) { + if (mods.atMod) { + buff.append(NumberUtils.toRoman((Long) val, mods.colonMod)); + } else if (mods.colonMod) { buff.append(NumberUtils.toOrdinal(val)); } else { buff.append(NumberUtils.toCardinal(val)); } } else { - if(params.length() < 1) - throw new IllegalArgumentException("R directive requires at least one parameter, the radix"); + if (params.length() < 1) + throw new IllegalArgumentException( + "R directive requires at least one parameter, the radix"); int radix = params.getInt(0, "radix", 'R'); @@ -330,27 +334,28 @@ public class CLFormatter { int mincol = 0, colinc = 1, minpad = 0; char padchar = ' '; - if(params.length() > 1) { + if (params.length() > 1) { mincol = params.getIntDefault(0, "minimum column count", 'A', 0); } - if(params.length() < 4) { - throw new IllegalArgumentException("Must provide either zero, one or four arguments to A directive"); + if (params.length() < 4) { + throw new IllegalArgumentException( + "Must provide either zero, one or four arguments to A directive"); } - colinc = params.getIntDefault(1, "padding increment", 'A', 1); - minpad = params.getIntDefault(2, "minimum amount of padding", 'A', 0); + colinc = params.getIntDefault(1, "padding increment", 'A', 1); + minpad = params.getIntDefault(2, "minimum amount of padding", 'A', 0); padchar = params.getCharDefault(3, "padding character", 'A', ' '); StringBuilder work = new StringBuilder(); - if(mods.atMod) { - for(int i = 0; i < minpad; i++) { + if (mods.atMod) { + for (int i = 0; i < minpad; i++) { work.append(padchar); } - for(int i = work.length(); i < mincol; i++) { - for(int k = 0; k < colinc; k++) { + for (int i = work.length(); i < mincol; i++) { + for (int k = 0; k < colinc; k++) { work.append(padchar); } } @@ -358,13 +363,13 @@ public class CLFormatter { work.append(item.toString()); - if(!mods.atMod) { - for(int i = 0; i < minpad; i++) { + if (!mods.atMod) { + for (int i = 0; i < minpad; i++) { work.append(padchar); } - for(int i = work.length(); i < mincol; i++) { - for(int k = 0; k < colinc; k++) { + for (int i = work.length(); i < mincol; i++) { + for (int k = 0; k < colinc; k++) { work.append(padchar); } } @@ -372,16 +377,16 @@ public class CLFormatter { } private void handleGotoDirective(CLModifiers mods, CLParameters params, Tape formatParams) { - if(mods.colonMod) { + if (mods.colonMod) { int num = 1; - if(params.length() > 1) { + if (params.length() > 1) { num = params.getIntDefault(0, "number of arguments backward", '*', 1); } formatParams.left(num); - } else if(mods.atMod) { + } else if (mods.atMod) { int num = 0; - if(params.length() > 1) { + if (params.length() > 1) { num = params.getIntDefault(0, "argument index", '*', 0); } @@ -389,7 +394,7 @@ public class CLFormatter { formatParams.right(num); } else { int num = 1; - if(params.length() > 1) { + if (params.length() > 1) { num = params.getIntDefault(0, "number of arguments forward", '*', 1); } @@ -397,43 +402,44 @@ public class CLFormatter { } } - private void handleConditionalDirective(StringBuffer sb, CLModifiers mods, CLParameters arrParams, Tape formatParams, Matcher dirMatcher) { + private void handleConditionalDirective(StringBuffer sb, CLModifiers mods, CLParameters arrParams, + Tape formatParams, Matcher dirMatcher) { StringBuffer condBody = new StringBuffer(); - List clauses = new ArrayList<>(); - String defClause = null; - boolean isDefault = false; + List clauses = new ArrayList<>(); + String defClause = null; + boolean isDefault = false; - while(dirMatcher.find()) { + while (dirMatcher.find()) { /* Process a list of clauses. */ String dirName = dirMatcher.group("name"); String dirMods = dirMatcher.group("modifiers"); - if(dirName != null) { + if (dirName != null) { /* Append everything up to this directive. */ dirMatcher.appendReplacement(condBody, ""); - if(dirName.equals("]")) { + if (dirName.equals("]")) { /* End the conditional. */ String clause = condBody.toString(); - if(isDefault) { + if (isDefault) { defClause = clause; } else { clauses.add(clause); } break; - } else if(dirName.equals(";")) { + } else if (dirName.equals(";")) { /* End the clause. */ String clause = condBody.toString(); - if(isDefault) { + if (isDefault) { defClause = clause; } else { clauses.add(clause); } /* Mark the next clause as the default. */ - if(dirMods.contains(":")) { + if (dirMods.contains(":")) { isDefault = true; } } else { @@ -444,51 +450,54 @@ public class CLFormatter { } Object par = formatParams.item(); - if(mods.colonMod) { + if (mods.colonMod) { formatParams.right(); - if(par == null) { + if (par == null) { throw new IllegalArgumentException("No parameter provided for [ directive."); - } else if(!(par instanceof Boolean)) { + } else if (!(par instanceof Boolean)) { throw new IllegalFormatConversionException('[', par.getClass()); } - boolean res = (Boolean)par; + boolean res = (Boolean) par; String fmt; - if(res) fmt = clauses.get(1); - else fmt = clauses.get(0); + if (res) + fmt = clauses.get(1); + else + fmt = clauses.get(0); doFormatString(fmt, sb, formatParams); - } else if(mods.atMod) { - if(par == null) { + } else if (mods.atMod) { + if (par == null) { throw new IllegalArgumentException("No parameter provided for [ directive."); - } else if(!(par instanceof Boolean)) { + } else if (!(par instanceof Boolean)) { throw new IllegalFormatConversionException('[', par.getClass()); } - boolean res = (Boolean)par; + boolean res = (Boolean) par; - if(res) { + if (res) { doFormatString(clauses.get(0), sb, formatParams); } else { formatParams.right(); } } else { int res; - if(arrParams.length() > 1) { + if (arrParams.length() > 1) { res = arrParams.getInt(0, "conditional choice", '['); } else { - if(par == null) { + if (par == null) { throw new IllegalArgumentException("No parameter provided for [ directive."); - } else if(!(par instanceof Number)) { + } else if (!(par instanceof Number)) { throw new IllegalFormatConversionException('[', par.getClass()); } - res = ((Number)par).intValue(); + res = ((Number) par).intValue(); formatParams.right(); } - if(res < 0 || res > clauses.size()) { - if(defClause != null) doFormatString(defClause, sb, formatParams); + if (res < 0 || res > clauses.size()) { + if (defClause != null) + doFormatString(defClause, sb, formatParams); } else { doFormatString(clauses.get(res), sb, formatParams); } @@ -499,33 +508,34 @@ public class CLFormatter { private void handleEscapeDirective(CLModifiers mods, CLParameters params, Tape formatParams) { boolean shouldExit; - switch(params.length()) { + switch (params.length()) { case 0: shouldExit = formatParams.size() == 0; break; case 1: - int num = params.getInt(0, "condition count", '^'); + int num = params.getInt(0, "condition count", '^'); shouldExit = num == 0; break; case 2: - int left = params.getInt(0, "left-hand condition", '^'); - int right = params.getInt(1, "right-hand condition", '^'); + int left = params.getInt(0, "left-hand condition", '^'); + int right = params.getInt(1, "right-hand condition", '^'); shouldExit = left == right; break; case 3: default: - int low = params.getInt(0, "lower-bound condition", '^'); - int mid = params.getInt(1, "interval condition", '^'); - int high = params.getInt(2, "upper-bound condition", '^'); + int low = params.getInt(0, "lower-bound condition", '^'); + int mid = params.getInt(1, "interval condition", '^'); + int high = params.getInt(2, "upper-bound condition", '^'); shouldExit = (low <= mid) && (mid <= high); break; } /* At negates it. */ - if(mods.atMod) shouldExit = !shouldExit; + if (mods.atMod) + shouldExit = !shouldExit; - if(shouldExit) throw new EscapeException(mods.colonMod); + if (shouldExit) + throw new EscapeException(mods.colonMod); } - } -- cgit v1.2.3