Discussion :

[boubounagh]

bonsoir
l'erreur suivante s'affiche en exécutant mon prg tandis que j'ai pas fermé mon resultSet mais je n'arrive pas à comprendre pourquoi cela. voici un bout de code qui cause l'erreur:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 String queryString=("SELECT idPage, count(idPageAmi) AS nbreAmis FROM connectivite GROUP BY idPage");
   resultat = statement.executeQuery(queryString);
   while(resultat.next()){
 
	 System.out.println(resultat.getInt("idPage"));
	 System.out.println(resultat.getInt("nbreAmis"));
	   String query=("UPDATE connectivite SET nbreAmis ="+  resultat.getInt("nbreAmis")+" WHERE idPage="+ resultat.getInt("idPage"));
	     System.out.println(query);
	   statement.executeUpdate(query);
 
   }
 
        }

[boubounagh]

apparemment le pb est du à la requête update qui est à l’intérieur de while car si j'élimine cette instruction et je fais un simple affichage l'erreur n'est plus affichée. mais le pb que j'ai besoin de la requete update comment résoudre ça s'il vous plait?

[in]

tu as l'erreur car en effet, tu utilises ton statement dans la boucle alors qu'il est déjà en cours d'utilisation. Le statement.executeUpdate(query); doit passer le premier coup puis le next() sort en erreur car le statement a été fermé.

Pour résoudre le problème, il te suffit d'utiliser 2 objets statement différents.
Par contre, ça n'est pas très joli ...
Et surtout n'oublie pas de fermer tes resultset et statements après utilisation dans un bloc finally (cf la FAQ)

[HowardHyde]

Je crois qu'il serait mieux, au lieu d'avoir une instruction SQL SELECT suivi d'un boucle d'instructions UPDATE, metre q'une instruction SQL MERGE, comme celui-ci:

Code :

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

1
2
3
4
5
MERGE INTO connectivite x
USING (SELECT idPage, count(idPageAmi) AS nbreAmis FROM connectivite GROUP BY idPage) y
ON (y.idPage = x.idPage)
WHEN MATCHED THEN
UPDATE SET nbreAmis = y.nbreAmis

C'est un peu plus compliqué du point de vue de la sophistication en SQL, c'est vrai, mais cela simplifie le code Java et ameliore la performance de la base de données.
Voici un document sur l'instruction MERGE:
http://docs.oracle.com/cd/B19306_01/...ments_9016.htm

Le livre suivant aussi peut être interesant: 'Java Web Database Application Development' á: http://javawebdb.com

[tchize_]

Le mieux ca reste de faire tout ça en une seule opération. Il est dans 99.9% des cas inutile et contre performant de faire un select d'itérer dessus et, pour chaque row lu, de mettre à jour une base de données. Le language SQL permet de faire tout ça en une seule passe!

Ceci devrais faire l'affaire (je suis un peu rouillé en sql) et éviter le ping-pong réseau.

Code sql :

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

UPDATE connectivite c SET c.nbreAmis = (SELECT count(idPageAmi)  FROM connectivite c2 where c2.idPage = c.idPage)

edit: le MERGE est effectivement probablement plus performant, pas vu

[boubounagh]

bonjour

moi j'ai déjà résolu mon pb à l'aide du conseil présenté par in mais je ne comprends pas ce que HowardHyde a dit et concernant la solution de tchize_
j'ai compris mais ce que je comprends pas pourquoi vous me proposez d'autres solutions est ce que la première solution proposé par in n'est pas juste ?

[tchize_]

Imaginons que t'ais 100.000 lignes, correspondant à disons 5000 pages. Tu va construire, interpréter et exécuté dans la transaction 5001 requetes updates. alors qu'en faisant tout en une seule requete SQL, tu va n'avoir qu'une requete. Si le serveur met 200ms / requete, c'est la différence entre une opération qui se fera en 2 secondes et une opération qui se fera en 15 minutes. Sans compter que l'opération avec 5000 aller / retour va bien charger ton réseau et ton SGBD.

Bref, SQL est un language, utilisez le comme tel.

[boubounagh]

merci pour l'explication mais quand je mets la requete que tu m'as proposé l'erreur suivante est affichée:You can't specify target table 'c' for update in FROM clause

[in]

... est ce que la première solution proposé par in n'est pas juste ?

Ben c'était pas faux, , mais effectivement je n'ai pas fait attention à ce que tu voulais faire ...

C'est beaucoup mieux de faire ça en une seule requête. Le Merge est plus performant mais s'il te fait peur pour la maintenance, utilise l'update simple.

You can't specify target table 'c' for update in FROM clause

Tu as dû mal recopier la solution de tchize_. Il te manques sûrement un 2
Tu ne peux pas faire un select sur la table que tu update.

[boubounagh]

Tu as dû mal recopier la solution de tchize_. Il te manques sûrement un 2

en fait je suis sur que j'ai bien copié le code de tchize.

Tu ne peux pas faire un select sur la table que tu update.

selon la recherche que j'ai effectué concernant l'erreur affiché la cause de cet erreur est comme vous m'avez dit on ne peut pas faire un select dans un update donc la requete apparemment n'est pas totalement correcte.