Issue 148 #158
Issue 148 #158
Conversation
|
|
||
| public static void main (String[]args) throws NonexistentEntityException, Exception{ | ||
| PopulateDB.fullSetupDB("prosub", "root", ""); | ||
| //PopulateDB.recreateDB("prosub", "root", ""); |
There was a problem hiding this comment.
Remover este comentário. O próprio git faz controle de versão.
| public static void main(String[] args) { | ||
| try { | ||
| PopulateDB.fullSetupDB("prosub", "root", ""); | ||
| //PopulateDB.recreateDB("prosub", "root", ""); |
| Class.forName("com.mysql.jdbc.Driver"); | ||
|
|
||
| //STEP 3: Open a connection | ||
| //System.out.println("Connecting to a selected database..."); |
There was a problem hiding this comment.
Remover: System.out.println, uma vez que foi substituído por Log
| //System.out.println("Database deleted successfully..."); | ||
| } catch (SQLException | ClassNotFoundException se) { | ||
| //Handle errors for JDBC | ||
| //se.printStackTrace(); |
There was a problem hiding this comment.
Remover: Não faz sentido imprimir o stackTrace.
| } catch (SQLException | ClassNotFoundException se) { | ||
| //Handle errors for JDBC | ||
| //se.printStackTrace(); | ||
| LOG.info(("" + se)); |
There was a problem hiding this comment.
Não vejo log de exceptions como sendo info(), mas algo como severe().
Além disso, usar método toString() ao invés de concatenação ""+.
| LOG.info("Connecting to a selected database..."); | ||
| conn = DriverManager.getConnection(dbURL, username, password); | ||
| LOG.info("Connected database successfully..."); | ||
| //System.out.println("Connected database successfully..."); |
There was a problem hiding this comment.
Remover todos os comentários de System.out.println
| } | ||
| } catch (SQLException se) { | ||
| //se.printStackTrace(); | ||
| LOG.log(Level.INFO, "{0}", se); |
There was a problem hiding this comment.
Manter padrão do formato de log - veja o comentário anterior. Ou seja, ou utiliza-se curingas {0} ou utiliza-se toString() - escolha um como padrão!
Além disso, log de exception é algo severe().
Faça todos os ajustes necessários nesta classe.
|
Alterações feitas |
No description provided.