diff options
| author | Benjamin J. Culkin <bjculkin@mix.wvu.edu> | 2020-11-26 12:47:37 -0400 |
|---|---|---|
| committer | Benjamin J. Culkin <bjculkin@mix.wvu.edu> | 2020-11-26 12:47:37 -0400 |
| commit | 8933de7f646f0565edf700aa2f2fcab06d639855 (patch) | |
| tree | 7da4bf9f88477986a8af239a81138d7f499076eb /clformat/src/main/java/bjc/utils | |
| parent | 03de62016afa4e392f32069ec28ad58ee38699da (diff) | |
Fix some failing tests
This fixes two failing tests
* One was that when refactoring AestheticDirective, I accidentally
broke doing padding before the value.
* The other was that when allowing for using multiple ` on a directive,
I accidentally broke parsing for ` directives
Diffstat (limited to 'clformat/src/main/java/bjc/utils')
| -rw-r--r-- | clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java | 39 | ||||
| -rw-r--r-- | clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java | 7 |
2 files changed, 37 insertions, 9 deletions
diff --git a/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java b/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java index 856589c..3dfb45b 100644 --- a/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java +++ b/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java @@ -83,23 +83,50 @@ class AestheticEdict implements Edict { StringBuilder work = new StringBuilder(); - char padchar = padcharPar.asChar(itemTape, "padding character", "A", ' '); + char padchar = padcharPar.asChar(itemTape, + "padding character", "A", ' '); - int mincol = mincolPar.asInt(itemTape, "minimum column count", "A", 0); - int colinc = colincPar.asInt(itemTape, "padding increment", "A", 1); - int minpad = minpadPar.asInt(itemTape, "minumum amount of padding", "A", 0); + int mincol = mincolPar.asInt(itemTape, + "minimum column count", "A", 0); + int colinc = colincPar.asInt(itemTape, + "padding increment", "A", 1); + int minpad = minpadPar.asInt(itemTape, + "minumum amount of padding", "A", 0); - if (padBefore) doPadding(work, padchar, mincol, colinc, minpad); + // if (padBefore) doPadding(work, padchar, mincol, colinc, minpad); work.append(itemString); - if (!padBefore) doPadding(work, padchar, mincol, colinc, minpad); + // if (!padBefore) doPadding(work, padchar, mincol, colinc, minpad); + + String padding = createPadding(work, padchar, mincol, colinc, minpad); + if (padBefore) work.insert(0, padding); + else work.append(padding); formCTX.writer.write(work.toString()); itemTape.right(); } + private String createPadding( + StringBuilder work, + char padchar, + int mincol, + int colinc, + int minpad) + { + StringBuilder padding = new StringBuilder(); + for (int i = 0; i < minpad; i++) padding.append(padchar); + + for (int i = work.length(); i < mincol; i++) { + for (int k = 0; k < colinc; k++) { + padding.append(padchar); + } + } + + return padding.toString(); + } + private void doPadding(StringBuilder work, char padchar, int mincol, int colinc, int minpad) { for (int i = 0; i < minpad; i++) work.append(padchar); diff --git a/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java b/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java index 8c16917..0d1d0df 100644 --- a/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java +++ b/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java @@ -80,11 +80,12 @@ class DecimalEdict implements Edict { } } - StringBuffer work = new StringBuffer(); + //StringBuffer work = new StringBuffer(); - numForm.format(itemTape.item(), work, ZERO_FIELD); + //numForm.format(itemTape.item(), work, ZERO_FIELD); - formCTX.writer.write(work.toString()); + // formCTX.writer.write(work.toString()); + formCTX.writer.write(numForm.format(itemTape.item())); itemTape.right(); } } |
