Discussion :

[JeanNoel53]

Bonjour
Je code sous Eclipse une application qui doit former un plugin.
J'ai une exception que je vous donne ci-après :

Code :

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

1
2
3
4
5
6
7
8
java.sql.SQLException: [Microsoft][Gestionnaire de pilotes ODBC] État de curseur non valide
	at sun.jdbc.odbc.JdbcOdbc.createSQLException(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbc.standardError(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbc.SQLGetDataString(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbcResultSet.getDataString(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbcResultSet.getString(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbcResultSet.getObject(Unknown Source)
	at sun.jdbc.odbc.JdbcOdbcResultSet.getObject(Unknown Source)

J'ai envoyé les variables suivantes:

Code :

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

1
2
3
4
5
6
7
8
9
 
                                stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				myReader = (SQLData)rs.getObject( cmd);
				while( myReader != null);
				{
					iNbResultFound++;
					myReader = (SQLData)rs.getObject( cmd);
				}

L'exception se produit sue la ligne myReader = (SQLData)rs.getObject( cmd);
ou rs est valable et cmd = "vcDocument" qui est une variable qui definit une colonne de la table qui est visitée
wsRequest est une chaine correcte qui retourne les résultat sous MSSMS

[joel.drigo]

Salut,

On doit toujours appeler next() sur un ResultSet, avant de pouvoir lire les valeurs d'un enregistrement : ceci à pour effet, pour le premier appel, de pointer sur l'eventuel premier enregistrement : si le ResultSet est vide, la méthode retourne false, sinon elle retourne true. Pour les appels suivant, elle avance vers l'enregistrement suivant, tant qu'il y en a.

On fait donc :

Code :

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

1
2
3
4
5
6
7
while(rs.next()){ 
 
    SQLData myReader = (SQLData)rs.getObject( cmd );
 
    iNbResultFound++;
 
}

Si c'est juste pour compter, tu n'as même pas besoin de faire le getObject() (surtout récupérer l'objet en le type par SQLData n'a pas vraiment d'intérêt, puisqu'on est pas censé appeler ses méthodes directement). D'ailleurs, tu pourrais faire une simple requête SQL du type count(), et récupérer le résultat par un iNbResultFound=rs.getInt(1).

[JeanNoel53]

Je reformule ce que j'ai compris
il faut faire rs.next() pour que ce soit valable J'ai essayé le programma à attendu le Time Out et est partie en carafe après
après je comprend que la méthode count() pourais je voir un exemple?
En fait j'ai deux objectifs dans cette methode: récupérer le nombre de ligne dans la base de donnée et avoir myReader posé sur le premier enregistrement.
je vous donne le code modifié en conséquence:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
				stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				if(rs.next())
				{
					myReader = (SQLData)rs.getObject( cmd);
					iNbResultFound++;
				}
				while( rs.next());
				{
					iNbResultFound++;
				}

il reste que la getObject ne marche pas et que je ne sait pas adresser count

[joel.drigo]

après je comprend que la méthode count() pourais je voir un exemple?

Code SQL :

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

SELECT COUNT(*) FROM table

Donc en Java :

Code :

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

1
2
3
4
5
6
7
8
9
public int getCount(Connection conn, String tableName) throws SQLException {
    try( PreparedStatement s = conn.prepareStatement("SELECT COUNT(*) AS rowcount FROM " + tableName );
         ResultSet r = s.executeQuery();) {
        if ( r.next() ) {
	   return r.getInt("rowcount") ;
	}
	throw new IllegalStateException();
}
}

Si tu as besoin d'une condition

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
public int getCount(Connection conn, String tableName, String condition, Object...params) throws SQLException {
   try( PreparedStatement s = createStatement(conn, tableName, condition, params);  
        ResultSet r = s.executeQuery();) {
      if ( r.next() ) {
          return r.getInt("rowcount") ;
      }
      throw new IllegalStateException();
   }
}
 
private static PreparedStatement createStatement(Connection conn, String tableName, String condition, Object...params) throws SQLException {
   String sql = "SELECT COUNT(*) AS rowcount FROM " + tableName;
   if ( condition!=null ) {
      sql += " WHERE " + condition; 
   }
   PreparedStatement s = conn.prepareStatement(sql);
   for(int i=1; i<params.length+1; i++) {
      s.setObject(i, params);
   }
   return s;
}

A n'utiliser plutôt que pour compter uniquement : si tu tu dois faire un SELECt pour récupérer des données, mais aussi connaitre leur nombre avant de les lire par la boucle while( rs.hasNext()), tu peux faire comme ça :

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
 
   try( PreparedStatement s = conn.preparedStatement( wsRequest, ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
        ResultSet rs = s.executeQuery();) {
 
      rs.last();
      int count = rs.getRow(); // lecture du nombre d'enregistrements trouvés
      rs.beforeFirst();
 
      while( rs.next() ) {
          // traitement
      } 
   }
   finally {
      r.close() ;
   }

...

[JeanNoel53]

ne part plus en exception mais renvoi 1 alors qu'il y a trois données a récolter par la requète

[joel.drigo]

ne part plus en exception mais renvoi 1 alors qu'il y a trois données a récolter par la requète

Si getRow() retourne 1, c'est qu'il y a qu'un enregistrement dans le résultat. Comment tu sais que la requête doit renvoyer 3 enregistrements ? Si tu parcours les résultats et que tu les affiches par un simple System.out.println(), ça en affiche combien ?

[JeanNoel53]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
				stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				if(rs.next())
				{
					iNbResultFound++;
					//stream = rs.getNString( cmd);
				}
				while( rs.next())
				{
					iNbResultFound++;
				}

Ce code renvoit 3
Mais avec ce code je n'ai pas myReader d'initialisé; myReader est de type SQLData.
rs.GetNString( cmd) provoque une exception. avec cmd = nom d'une colonne

[joel.drigo]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
				stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				if(rs.next())
				{
					iNbResultFound++;
					//stream = rs.getNString( cmd);
				}
				while( rs.next())
				{
					iNbResultFound++;
				}

Ce code renvoit 3

Oui, mais ce code est absurde : pourquoi faire un if suivi d'un while : un while entre dans la boucle qui la condition est vraie, donc ça fait la même chose que if. Sauf que la si le resultset dès le début est vide, ça plantera.

Code :

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

1
2
3
4
5
6
7
stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				while(rs.next())
				{
					iNbResultFound++;
 
				}

fonctionne dans tous les cas.

maintenant ce code permet de connaitre le nombre d'élément uniquement à la fin de la boucle.

Mais avec ce code je n'ai pas myReader d'initialisé; myReader est de type SQLData.
rs.GetNString( cmd) provoque une exception. avec cmd = nom d'une colonne

Bah c'est un peu normal, vu que getNString retourne un String et pas un SQLData !

C'est quoi le type de la donnée que tu cherches à lire ? C'est vraiment du SQLData ?

[JeanNoel53]

J'ai besoin de récupérer le pointeur myReader qui pointe sur le premier enregistrement

[joel.drigo]

J'ai besoin de récupérer le pointeur myReader qui pointe sur le premier enregistrement

Je ne vois pas de quoi tu parles. Un truc (un Reader ?) qui pointe sur le premier enregistrement ? Tu as des blobs dans ta base, et tu cherches à ouvrir un InputStream sur un binaire stocké dedans ou quelque chose comme ça ? En tout cas SQLData, ça n'a rien à voir avec ça (je ne connais pas bien, mais à priori, c'est plutôt pour des structures métiers, genre bean).

Je repose la question : c'est quoi les types des données dans le SELECT ? Parce que là tu me dis que tu essaies de récupérer je ne sais quoi, alors que je ne sais aps ce que tu devrais réellement récupérer.

[JeanNoel53]

des champs int et des champs String, c'est du text.
Si on veut changer le type SQLData en autre chose,je veux bien en discuter;
Actuellement le read se fait sur la couche de plus haut niveau qui manipule myReader

[joel.drigo]

Il faut lire les champs int avec getInt(numéro) et les champs String avec getString(numéro), ou numéro est le numéro de colonne (de 1 à n).

Par exemple :

Code :

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

1
2
3
4
5
6
while( rs.next() ) {
 
    int column1 = rs.getInt(1); // si la colonne 1 est de type int
    String column1 = rs.getString(2); // si la colonne 2 est de type String
 
}

On peut utiliser à la place du numéro le nom de la colonne

Code :

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

1
2
3
4
5
while( rs.next() ) {
 
    int columnTruc = rs.getInt("truc"); // si la colonne "truc" est de type int
 
}

A chaque itération, on récupère les valeurs de l'enregistrement correspondant à la position actuel du curseur (qui avant par rs.next()).

Si tu veux pouvoir récupérer les valeurs, pour les traiter ailleurs, tu as intérêt à faire une classe métier pour y charger les valeurs.

[JeanNoel53]

J'ai changé le type des myReader qui a pris un type ResultSet: Le code est devenu:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
			{
				stmt.execute( wsRequest);
				rs = stmt.getResultSet();
				myReader = rs;
				while( rs.next())
				{
					iNbResultFound++;
				}
			}