refactoring

jlangch · Oct 31, 2023 · d1c70d2 · d1c70d2
1 parent a052d74
commit d1c70d2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/github/jlangch/venice/impl/util/csv/CSVReader.java b/src/main/java/com/github/jlangch/venice/impl/util/csv/CSVReader.java
@@ -161,8 +161,8 @@ private String parseQuotedField(final CharacterReader rd) {
                 }
             }
             else {
-            	rd.consume();
                 sb.append((char)ch);
+            	rd.consume();
             }
         }
 

diff --git a/src/main/java/com/github/jlangch/venice/impl/util/csv/CSVWriter.java b/src/main/java/com/github/jlangch/venice/impl/util/csv/CSVWriter.java
@@ -147,10 +147,10 @@ private String escape(final String s) {
     }
 
     private String quote(final String s) {
-        return needsQuote(s) ? quote + escape(s) + quote : s;
+        return needsQuoting(s) ? quote + escape(s) + quote : s;
     }
 
-    private boolean needsQuote(final String s) {
+    private boolean needsQuoting(final String s) {
         return s != null && (s.contains(" ")
                                 || s.contains(quote)
                                 || s.contains(separator)
-Original file line number
+Diff line change
@@ Expand Up / @@ -161,8 +161,8 @@ private String parseQuotedField(final CharacterReader rd) { @@
                     }
                 }
                 else {
-                	rd.consume();
                     sb.append((char)ch);
+                	rd.consume();
                 }
             }
@@ Expand Down @@