Salve gente devo fare una classe che pesca dati da un certo database usando jdbc, volevo sapere se il codice seguente è corretto:
codice:
package sangregorio.suiteone.migra;


import it.suiteone.base.Prodotto;

import java.io.FileInputStream;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.Locale;
import java.util.ResourceBundle;

import org.apache.poi.hssf.usermodel.HSSFWorkbook;


public class DataImportArticoli {

	public static void run() throws Exception {	
		
		Connection con=null;
		Statement s=null;
		ResultSet rs=null;

		//Credenziali accesso DB cliente
		String url= "jdbc:jtds:sqlserver://192.168.1.50:1369";
		String id= "user";
		String pass = "password";
		
		try{
			Class.forName("net.sourceforge.jtds.jdbc.Driver");
			con = java.sql.DriverManager.getConnection(url, id, pass);

		}catch(ClassNotFoundException cnfex){
			cnfex.printStackTrace();

		}
		String sql = "select * from ARTICOLI";
		
		try{
			s = con.createStatement();
			rs = s.executeQuery(sql);
			
			while( rs.next() ){
				rs.getString("cust_id");
				rs.getString("rdate");
				rs.getString("email");
			}

		}	catch(Exception e){
			e.printStackTrace();
			}
		
		finally{
			if(rs!=null) rs.close();
			if(s!=null) s.close();
			if(con!=null) con.close();
		}
	}
}