?
Solved

Why am I getting this error with my SQL statement

Posted on 2010-11-16
5
Medium Priority
?
393 Views
Last Modified: 2012-08-14
Hi,
I am getting an error with my JDBC program, for some reason or another I get the below error can someone tell me what I need to do to correct this. Thanks. The code is attached and the error is below

Welcome to JDBC program!
05/01/10
Error: Error retrieving data!
java.sql.SQLException: Operation not allowed after ResultSet closed
      at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:1075)
      at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:989)
      at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:984)
      at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:929)
      at com.mysql.jdbc.ResultSetImpl.checkClosed(ResultSetImpl.java:794)
      at com.mysql.jdbc.ResultSetImpl.next(ResultSetImpl.java:7145)
      at JDBCApp.ConvertData(JDBCApp.java:62)
      at JDBCApp.main(JDBCApp.java:120)
import java.sql.*;


public class JDBCApp 
{
    Connection link;
    Statement statement =null;
    ResultSet results;
    String dbURL = "blahblah";
	String username = "blah";
	String password = "blah";
	int ID;
	String Date;
    
    /** Creates a new instance of JDBCApp */
    public JDBCApp() 
    {
    }
    public void Connect(){
    	   System.out.println("Welcome to JDBC program!");
           try 
           {
               Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
               link = DriverManager.getConnection(dbURL,username,password);
   	}
   	catch(ClassNotFoundException e)
   	{
   		System.out.println("Error: Unable to load driver!");
   		System.exit(1);
   	}
   	catch(SQLException e)
   	{
   		System.out.println("Error: Cannot connect to database!");
   		System.exit(1);
   	}
    }
    
    public void ConvertData(){
    	try
    	{
    		statement = link.createStatement();
    		results = statement.executeQuery("SELECT id,date FROM wind_data WHERE date LIKE '%2010'");
    	}
    	catch(SQLException e)
    	{
    		System.out.println("Error: Cannot execute query!");
    		e.printStackTrace();
    		System.exit(1);
    	}

    	try
    	{
    		while (results.next())
    		{
    			ID = (results.getInt("ID"));
    			Date = (results.getString("DATE"));
    			
    			String newdate = FormatDate(Date);
    			System.out.println(newdate);
    			
    			try
    			{
    				//statement = link.prepareStatement("UPDATE wind_data    SET date = '"+newdate+"'   WHERE ID = '"+ID+"'");
    				statement.executeUpdate("UPDATE wind_data    SET date = '"+newdate+"'   WHERE ID = '"+ID+"'"); 
    				//System.out.println("Getting Data");
    			}
    			catch(SQLException e)
    			{
    				System.out.println("Error: Cannot execute query!");
    				e.printStackTrace();
    				System.exit(1);
    			
    		}
    		}
    	}
    	catch(SQLException e)
    	{
    		System.out.println("Error: Error retrieving data!");
    		e.printStackTrace();
    		System.exit(1);
    	}
    }
    
    public void Disconnect(){
    	try
    	{
    		link.close();
    	}
    	catch(SQLException e)
    	{
    		System.out.println("Error: Unable to disconnect!");
    		e.printStackTrace();
    	}
    }
    
    public String FormatDate(String Date){
		String day = Date.substring(0, 2);
		String month = Date.substring(3,5);
		String year = Date.substring(8,10);
		String newdate = day+"/"+month+"/"+year;
		return newdate;
		
	}
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) 
    {
    	JDBCApp a = new JDBCApp();
     a.Connect();
     a.ConvertData();
     a.Disconnect();

	

	
    }
    
   
}

Open in new window

0
Comment
Question by:bhession
5 Comments
 
LVL 3

Accepted Solution

by:
indigo23 earned 1000 total points
ID: 34147137
try creating a different statement object for the update query
0
 
LVL 86

Expert Comment

by:CEHJ
ID: 34147328
Storing a date type as a string is guaranteed to give you problems, as well as being inefficient
0
 
LVL 2

Expert Comment

by:jzkidding
ID: 34154629
in place on newdate in the update statement  try todate('+newdate+','dd/mm/yy')
0
 
LVL 10

Expert Comment

by:gordon_vt02
ID: 34155454
As indigo said, you need a new Statement object for the UPDATE query.  If you reuse the existing 'statement' object to run executeUpdate(), it will close the ResultSet from the SELECT query, causing the exception you are seeing.

You should probably consider moving your Statement and ResultSet variable declarations into the ConvertData() method as there is really no need for them to be class members.
0
 

Author Closing Comment

by:bhession
ID: 34281547
Yeah that sorted the problem alright. Thanks sorry for the late reply
0

Featured Post

Never miss a deadline with monday.com

The revolutionary project management tool is here!   Plan visually with a single glance and make sure your projects get done.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Instead of error trapping or hard-coding for non-updateable fields when using QODBC, let VBA automatically disable them when forms open. This way, users can view but not change the data. Part 1 explained how to use schema tables to do this. Part 2 h…
How much do you know about the future of data centers? If you're like 50% of organizations, then it's probably not enough. Read on to get up to speed on this emerging field.
Viewers will learn how to properly install Eclipse with the necessary JDK, and will take a look at an introductory Java program. Download Eclipse installation zip file: Extract files from zip file: Download and install JDK 8: Open Eclipse and …
How to fix incompatible JVM issue while installing Eclipse While installing Eclipse in windows, got one error like above and unable to proceed with the installation. This video describes how to successfully install Eclipse. How to solve incompa…

599 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question