Discussion :

[ifsera]

Salut,

CJe ne vois pas ce que je fais mal. J'ai tenté avec un finally, mais j'ai toujours cette exception.
Si quelqu'un peut m'expliquer. Le jdbc c'est loin

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
	public ArrayList<Temp> loadTemp() throws DAOException {
 
		ArrayList<Temp> imm = new ArrayList<Temp>();
		String sqlOrder = "";
 
		Connection myConnection = null;
		Statement stmt = null;
		ResultSet rs = null;
 
		try {
			sqlOrder = "SELECT * FROM Temp";
 
			myConnection = Connection.getInstance().getConnection();
 
 
			stmt = myConnection.createStatement();
 
			rs = DBConnector.executeSql(stmt, sqlOrder, myConnection);
			while (rs.next()) {
				imm.add(fillImm(rs));
			}
 
 
 
			 rs.close();
			stmt.close();
			myConnection.close();
 
			return immeubles;
		} catch (Exception e) {
			throw new DAOException(
					"Problem while loading data (sqlOrder="
							+ sqlOrder + ")", e);
		}
 
	}

Merci

[gandalf72000]

Dans ta boucle, tu dois récupéré le type de resultset(getObject(1) : type Object et colonne 1...)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
while (rs.next()) {
				imm.add(fillImm(rs.getObject(1)));
			}

[Nudger]

Bonjour,

Que fait la méthode fillImm ?
Peut-on avoir son code ?

[ifsera]

elle se sert du resultset pour remplir un objet que je mets après dans la liste.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	private static Temp fillImm (ResultSet rs) throws SQLException {
		Temp item = new Temp ();
		try {
			item.setCode(SQLHelper.readString(rs.getString("Code"), true));
			item.setAd(SQLHelper.readString(rs.getString("Ad"), true));
 
 
		} catch (SQLException e) {
			log.error(e);
			throw e;
		}
		return item;
	}

je fais autrement, du coup j'ai plus l'erreur.
Mais je sais pas vraiment pourquoi. La différence, c'est qu'avant
j'instancier la connexion et la fermer dans la méthode de load, tandis que je passe par un singleton maintenant.

Point de vue perf, cela vous semble correct ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
	public ArrayList<Temp> loadTemp() throws DAOException {
 
		ArrayList<Temp> imm = new ArrayList<Temp>();
		String sqlOrder = "";
 
		Statement stmt = null;
		ResultSet rs = null;
 
		try {
			sqlOrder = "SELECT * FROM Temp";
 
			stmt = Connection.getInstance().getConnection()
					.createStatement();
 
			rs = DBConnector.executeSql(stmt, sqlOrder, Connection
					.getInstance().getConnection());
			while (rs.next()) {
				imm.add(fillImm(rs));
			}
 
			return imm;
		} catch (Exception e) {
			throw new DAOException(
					"Problem while loading data (sqlOrder="
							+ sqlOrder + ")", e);
		} finally {
			try {
				if(rs != null) {rs.close();}
				if(stmt != null) {stmt.close();}
 
			} catch (SQLException e) {
				log.debug("", e);
			}
 
		}
 
	}

[tchize_]

Mais je sais pas vraiment pourquoi.

Etant donné que tu ne nous a ni donné la ligne à laquelle à lieu l'erreur ni sa stacktrace, ça risque d'être dur de te le dire. Mais ça se résume à "l'opération" (à la ligne en question) "n'est pas autorisée parce que tu as déjà fermé le ResultSet"

[ifsera]

Sorry, l'exception était levée au niveau de la méthode qui remplit l'objet (son appel, elle prend le resultset en paramètre).
Mais bon en reprenant mon dernier post, tel qu'écrit, je n'ai plus le souci. La connexion est dédiée à l'appli. Perso, j'aime pas trop, mais bon...