Some more minor refactoring

Author: pesse

src/main/java/org/utplsql/cli/ReportersCommand.java

@@ -20,11 +20,12 @@ public class ReportersCommand implements ICommand {
             description = ConnectionInfo.COMMANDLINE_PARAM_DESCRIPTION)
     private List<ConnectionInfo> connectionInfoList = new ArrayList<>();
 
-    public ConnectionInfo getConnectionInfo() {
-        if ( connectionInfoList != null && connectionInfoList.size() > 0 )
-            return connectionInfoList.get(0);
-        else
-            return null;
+    private ConnectionInfo getConnectionInfo() {
+        assert connectionInfoList != null;
+        assert connectionInfoList.size() > 0;
+        assert connectionInfoList.get(0) != null;
+
+        return connectionInfoList.get(0);
     }
 
     @Override

src/main/java/org/utplsql/cli/VersionInfoCommand.java

@@ -10,6 +10,7 @@
 
 import javax.sql.DataSource;
 import java.sql.Connection;
+import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -34,7 +35,18 @@ public int run() {
         System.out.println(CliVersionInfo.getInfo());
         System.out.println(JavaApiVersionInfo.getInfo());
 
-        ConnectionInfo ci = getConnectionInfo();
+        try {
+            writeUtPlsqlVersion(getConnectionInfo());
+        }
+        catch (SQLException e) {
+            e.printStackTrace();
+            return 1;
+        }
+
+        return 0;
+    }
+
+    private void writeUtPlsqlVersion( ConnectionInfo ci ) throws SQLException {
         if ( ci != null ) {
 
             DataSource dataSource = DataSourceProvider.getDataSource(ci, 1);
@@ -46,13 +58,7 @@ public int run() {
             catch ( UtPLSQLNotInstalledException e ) {
                 System.out.println("utPLSQL framework is not installed in database.");
             }
-            catch ( Exception e ) {
-                e.printStackTrace();
-                return 1;
-            }
         }
-
-        return 0;
     }
 
     @Override