Minor bugs and sample config file

Author: dlopez

src/main/java/es/jdlopez/sqlcmd/MainRunner.java

@@ -62,19 +62,20 @@ private static void writeResultSet(ResultSet resultSet, PrintWriter out, RunnerC
         if (conf.getPrintHeader()) {
             for (int i = 1; i <= colCount; i++) {
                 out.print(rsmd.getColumnName(i));
-                if (i <= colCount)
+                if (i < colCount)
                     out.print(conf.getPrintFieldSeparator());
             }
             out.println();
         } // header
         while (resultSet.next()) {
             for (int i = 1; i <= colCount; i++) {
                 out.print(resultSet.getString(i));
-                if (i <= colCount)
+                if (i < colCount)
                     out.print(conf.getPrintFieldSeparator());
             } // for col
             out.println();
         } // while row
+        out.flush();
     }
 
     private static String readAll(InputStream in) throws IOException {
@@ -103,6 +104,11 @@ private static RunnerConfig readArgs(String[] args) throws ConfigException {
             }
         else
             throw new ConfigException(null, "No arguments");
+        // default values (cant use constructor, explicit or implicit)
+        if (conf.getPrintFieldSeparator() == null)
+            conf.setPrintFieldSeparator("\t");
+        if (conf.getPrintHeader() == null)
+            conf.setPrintHeader(true);
         return conf;
     }
 }

src/main/java/es/jdlopez/sqlcmd/RunnerConfig.java

@@ -1,15 +1,15 @@
 package es.jdlopez.sqlcmd;
 
 public class RunnerConfig {
-    private String inputSQL;
     private String jdbcDriverPath;
     private String jdbcDriverClass;
     private String jdbcUrl;
     private String jdbcUser;
     private String jdbcPass;
+    private String inputSQL;
     private String outputResult;
-    private boolean printHeader;
-    private String printFieldSeparator;
+    private Boolean printHeader = true;
+    private String printFieldSeparator = "\t";
 
     public String getInputSQL() {
         return inputSQL;
@@ -67,11 +67,11 @@ public void setOutputResult(String outputResult) {
         this.outputResult = outputResult;
     }
 
-    public boolean getPrintHeader() {
+    public Boolean getPrintHeader() {
         return printHeader;
     }
 
-    public void setPrintHeader(boolean printHeader) {
+    public void setPrintHeader(Boolean printHeader) {
         this.printHeader = printHeader;
     }
 

target/test-classes/sample.properties

@@ -0,0 +1,6 @@
+# database connection properties
+jdbcDriverPath=
+jdbcDriverClass=
+jdbcUrl=
+jdbcUser=
+jdbcPass=