2012-03-16 14 views
5

Ho una tabella con 2 colonne per la memorizzazione delle impostazioni dell'applicazione nel database Oracle. Questo è un esempio davvero semplice:SQL come aggiornare le righe della tabella con una istruzione SQL

enter image description here

Voglio creare un metodo Java che aggiorna i valori con dichiarazione preparata.

codice Esempio:

AGGIORNAMENTO CODICE

public void updateDBSettings() throws SQLException { 


      String SQL_Statement = null; 

      if (ds == null) throw new SQLException();  
     Connection conn = ds.getConnection(); 
      if (conn == null) throw new SQLException();  

     try { 
      conn.setAutoCommit(false); 
      boolean committed = false; 
       try { 
         SQL_Statement = "UPDATE GLOBALSETTINGS SET (SettingName = ?, SettingValue = ?)"; 

         PreparedStatement updateQuery = conn.prepareStatement(SQL_Statement); 
         updateQuery.setString(1, "20"); 
         updateQuery.setString(2, "40"); 

         updateQuery.executeQuery(); 

         conn.commit(); 
         committed = true; 
       } finally { 
         if (!committed) conn.rollback(); 
         } 
      } 
       finally {    
       conn.close(); 

       } 

     }  

so che questa istruzione SQL è sbagliato. Qual è il modo corretto di scrivere la dichiarazione SQL?

migliori auguri Peter

P.S Dopo aver aggiornato il codice ottengo questo errore di stack:

javax.faces.el.EvaluationException: java.sql.SQLSyntaxErrorException: ORA-00907: manca parentesi

at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:102) 
at com.sun.faces.application.ActionListenerImpl.processAction(ActionListenerImpl.java:102) 
at javax.faces.component.UICommand.broadcast(UICommand.java:315) 
at javax.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:794) 
at javax.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1259) 
at com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:81) 
at com.sun.faces.lifecycle.Phase.doPhase(Phase.java:101) 
at com.sun.faces.lifecycle.LifecycleImpl.execute(LifecycleImpl.java:118) 
at javax.faces.webapp.FacesServlet.service(FacesServlet.java:593) 
at org.apache.catalina.core.StandardWrapper.service(StandardWrapper.java:1542) 
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:281) 
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175) 
at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:655) 
at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:595) 
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:161) 
at org.apache.catalina.connector.CoyoteAdapter.doService(CoyoteAdapter.java:331) 
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:231) 
at com.sun.enterprise.v3.services.impl.ContainerMapper$AdapterCallable.call(ContainerMapper.java:317) 
at com.sun.enterprise.v3.services.impl.ContainerMapper.service(ContainerMapper.java:195) 
at com.sun.grizzly.http.ProcessorTask.invokeAdapter(ProcessorTask.java:849) 
at com.sun.grizzly.http.ProcessorTask.doProcess(ProcessorTask.java:746) 
at com.sun.grizzly.http.ProcessorTask.process(ProcessorTask.java:1045) 
at com.sun.grizzly.http.DefaultProtocolFilter.execute(DefaultProtocolFilter.java:228) 
at com.sun.grizzly.DefaultProtocolChain.executeProtocolFilter(DefaultProtocolChain.java:137) 
at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:104) 
at com.sun.grizzly.DefaultProtocolChain.execute(DefaultProtocolChain.java:90) 
at com.sun.grizzly.http.HttpProtocolChain.execute(HttpProtocolChain.java:79) 
at com.sun.grizzly.ProtocolChainContextTask.doCall(ProtocolChainContextTask.java:54) 
at com.sun.grizzly.SelectionKeyContextTask.call(SelectionKeyContextTask.java:59) 
at com.sun.grizzly.ContextTask.run(ContextTask.java:71) 
at com.sun.grizzly.util.AbstractThreadPool$Worker.doWork(AbstractThreadPool.java:532) 
at com.sun.grizzly.util.AbstractThreadPool$Worker.run(AbstractThreadPool.java:513) 
at java.lang.Thread.run(Thread.java:722) 

causato da: java.sql.SQLSyntaxErrorException: ORA-00907: manca diritto parentesi

0.123.516,410617 millions
at oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:445) 
at oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:396) 
at oracle.jdbc.driver.T4C8Oall.processError(T4C8Oall.java:879) 
at oracle.jdbc.driver.T4CTTIfun.receive(T4CTTIfun.java:450) 
at oracle.jdbc.driver.T4CTTIfun.doRPC(T4CTTIfun.java:192) 
at oracle.jdbc.driver.T4C8Oall.doOALL(T4C8Oall.java:531) 
at oracle.jdbc.driver.T4CPreparedStatement.doOall8(T4CPreparedStatement.java:207) 
at oracle.jdbc.driver.T4CPreparedStatement.executeForRows(T4CPreparedStatement.java:1044) 
at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1329) 
at oracle.jdbc.driver.OraclePreparedStatement.executeInternal(OraclePreparedStatement.java:3584) 
at oracle.jdbc.driver.OraclePreparedStatement.executeQuery(OraclePreparedStatement.java:3628) 
at oracle.jdbc.driver.OraclePreparedStatementWrapper.executeQuery(OraclePreparedStatementWrapper.java:1493) 
at com.sun.gjc.spi.jdbc40.PreparedStatementWrapper40.executeQuery(PreparedStatementWrapper40.java:642) 
at com.DX_57.SM_57.Application.updateDBSettings(Application.java:124) 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
at java.lang.reflect.Method.invoke(Method.java:601) 
at javax.el.BeanELResolver.invokeMethod(BeanELResolver.java:779) 
at javax.el.BeanELResolver.invoke(BeanELResolver.java:528) 
at javax.el.CompositeELResolver.invoke(CompositeELResolver.java:257) 
at com.sun.el.parser.AstValue.invoke(AstValue.java:248) 
at com.sun.el.MethodExpressionImpl.invoke(MethodExpressionImpl.java:302) 
at com.sun.faces.facelets.el.TagMethodExpression.invoke(TagMethodExpression.java:105) 
at javax.faces.component.MethodBindingMethodExpressionAdapter.invoke(MethodBindingMethodExpressionAdapter.java:88) 
... 32 more 

risposta

7

la sintassi corretta sarebbe qualcosa di simile a questo:

UPDATE GLOBALSETTINGS 
    SET settingValue = case 
         when settingName = 'SessionTTL' = then ? 
         when settingName = 'MaxUsersActive' = then ? 
         else settingValue 
         end 
WHERE settingName in ('SessionTTL', 'MaxUsersActive'); 

Comunque io non consiglio questo perché rende il codice difficile da leggi e mantieni.

si sono probabilmente meglio a correre due istruzioni di aggiornamento:

String sql = 
    "UPDATE GLOBALSETTINGS " + 
    " SET settingValue = ? " + 
    "WHERE settingName = ?"; 

PreparedStatement pstmt = conn.prepareStatement(sql); 
pstmt.setString(1, "40"); 
pstmt.setString(2, "SessionTTL"); 
pstmt.executeUpdate(); 

pstmt.setString(1, "20"); 
pstmt.setString(2, "MaxUsersActive"); 
pstmt.executeUpdate(); 

Se si vuole risparmiare qualche andata e ritorno al database, è possibile eseguire questo come una dichiarazione raggruppati:

pstmt.setString(1, "40"); 
pstmt.setString(2, "SessionTTL"); 
pstmt.addBatch(); 

pstmt.setString(1, "20"); 
pstmt.setString(2, "MaxUsersActive"); 
pstmt.addBatch(); 

pstmt.executeBatch(); 

Questo invia due istruzioni in "one go" sul server di database.

Nota: presumo che settingName sia univoco.

+0

Sei sicuro che il primo esempio sia corretto? Ricevo di nuovo java.sql.SQLSyntaxErrorException: ORA-00905: parola chiave mancante. Come posso eseguirlo in sqlplus per testarlo?Se lo eseguo in sqlplus in questo modo ricevo Error at Command Line: 3 Column: 56 Segnalazione errori: Errore SQL: Parametro IN o OUT mancante all'indice :: 1 –

+0

Il mio esempio è codice Java e SQL è destinato ad essere eseguire come una dichiarazione preparata (proprio come il tuo esempio). Non è possibile eseguire una tale istruzione SQL con '?' In SQL * Plus. Devi sostituirli con valori reali. –

+0

Eseguo l'esempio nello sviluppatore SQL: http://pastebin.com/aeb9FmQQ –

0
SQL_Statement = "UPDATE GLOBALSETTINGS SET (SettingName = ?, SettingValue = ?)"; 
+0

Ho aggiornato il codice come proposto ma ora ricevo questo messaggio di errore: javax.faces.el.EvaluationException: java.sql.SQLSyntaxErrorException: ORA-00907: parentesi mancante mancante –

3
String sqlStatement = 
    "update GLOBALSETTINGS " + 
    "set SettingName = ?, " + 
    "SettingValue = ?" + 
    "where id = ?"; // You have to put where condition here, otherwise all rows will get affected. I assume your serch-key-column as id. Change 'id' according to your table 
PreparedStatement updateQuery = con.prepareStatement(sqlStatement); 
updateQuery.setString(1, "20"); 
updateQuery.setString(2, "40"); 
updateQuery.setString(3, applicationSettingId); 

è possibile controllare tutorials from oracle

+2

Ciò probabilmente richiederà anche un 'WHERE' clausola –

+0

Ho pensato di aggiungere, ma ho guardato alla domanda Requisito da Pietro. –

+0

Sì, questo aggiornerà tutte le righe e sovrascriverà tutto. Quindi la persona che fa le domande mostra attenzione a questo. –

Problemi correlati