Changeset 34900 for gs2-extensions/tipple-bridge

Timestamp:

2021-02-18T17:16:49+13:00 (3 years ago)

Author:

su41

Message:

Added another error message handler for incorrect database name.

File:

• gs2-extensions/tipple-bridge/trunk/src/src/main/java/sqlCli.java (modified) (8 diffs)

gs2-extensions/tipple-bridge/trunk/src/src/main/java/sqlCli.java

@@ -13 +13 @@
 /*
 TODO:
-for the CLI:
-- Error handling
-- Subcommands?
 
  */
 
-//-u su41Tipple -p --port 3306 --dbname tippleTest1
+//-u su41Tipple -p --port 3306 --dbname koicarp_test
 //@/home/su41/configFileTest
 
@@ -27 +24 @@
 @Command (name="sqlCli", description = "Arguments for database connection", mixinStandardHelpOptions = true)
 public class sqlCli implements Callable<Integer> {
+
     //our cli arguments
-
     @Option(names = {"-u", "--user"}, description = "The user name",
             required = true)
@@ -35 +32 @@
     //interactive allows the user to omit the argument value and instead enter it as a prompt
     /*
-    NOTE: if you are running an interactive option on an IDE, most likely will be using something
-    like the javaw.exe library which tries to mimic the terminal.
+    NOTE: if you are running an interactive option on an IDE, most likely the IDE will be using
+    the javaw.exe library which tries to mimic the terminal.
     The point is that interactive option would usually make whatever you enter hidden (it doesn't
     echo back) but on the IDE it will echo back.
@@ -62 +59 @@
         this parameter can be read in at any point on the options line.
         but that's only because it's the only parameter at the moment.
-        If we add more parameters, will either have to put them next to each other
-        OR add in indexes
+        If we add more parameters, will have to start indexing
         https://picocli.info/quick-guide.html#_options_and_parameters
          */
@@ -93 +89 @@
         Connection conn;
         Statement stmt;
+        //return status for code to send back to commandLine.execute
         int returnStatus = 0;
        // String dbUrl = "jdbc:mysql://localhost:" + port +"/"+ dbName;
@@ -103 +100 @@
     System.out.println("Saving output to:");
     System.out.println("  " + fileName);
-    
+
         try (FileWriter file = new FileWriter(fileName)){
             //register the jdbc driver
@@ -200 +197 @@
         }
         catch(SQLException sqlError){
+            // looking at the sql error code is good for debugging.
+            // can ref: https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-reference-error-sqlstates.html
+            // to see what they're for
+
            // System.out.println("MySQL error code: " + sqlError.getErrorCode());
+
+
             //prints out the error message from sql, then check to see which error code
             // came back and what the code matches.
@@ -210 +213 @@
                System.out.println("    -'Username'");
                System.out.println("    -'Password' ");
-               System.out.println("    -'Database name' ");
+            }
+
+            if(sqlError.getErrorCode() == MysqlErrorNumbers.ER_DBACCESS_DENIED_ERROR){
+                System.out.println("Error with database credentials. Double check your: ");
+                System.out.println("    -'Database name' ");
             }
            returnStatus = 1;