Discussion :

[Tamzoro]

Bonsoir,

J'ai une table nommée collabo avec 3 champs (CodeA,IdentM,IdentI) elles sont toutes des clefs étrangères ; j'aimerais créer un formulaire qui me permettra de modifier les différentes valeurs; voici un aperçu.





Dans ma méthode update, que dois-je mettre après le WHERE ? Car je n'ai pas pk...

Code :

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

1
2
3
4
5
6
7
8
9
10
public boolean updateCollabo(Collabo col)
    {
        String req = "Update collabo set IdentM = '" + col.getMusicienApp().getIdentM() +
        "', IdentI = '" + col.getInstrumentApp().getIdentI() + "' where CodeA = '"
        + col.getCodeA() + "'";
 
        boolean ok = ConnexionMySQL.getInstance().actionQuery(req);
 
        return ok;
    }

Par contre ma méthode Insert fonctionne parfaitement

Code :

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

1
2
3
4
5
6
7
8
public boolean insertCollabo (Collabo col)
    {
        boolean ok = ConnexionMySQL.getInstance().actionQuery("Insert into collabo (CodeA, " +
        "IdentM, IdentI) values ('" + col.getCodeA() + "','" + col.getMusicienApp().getIdentM() + "','" + col.getInstrumentApp().getIdentI() +  
         "'" + ")");
 
        return ok;
    }

Edit : je sais que ce type de requête est obsolète, mais je suis obligé de faire ainsi.

[tchize_]

Alors c'est assez compliqué

d'abord, j'éviterais ce choix malheureux de nom de classe / table qu'est collabo. En Belgique, avec notre passé, c'est un terme assez empreint de négativité

ensuite, tes collaborateurs, tu les références comment en temps normal si il n'y a pas de primary key? Y as-t-il une clé unique quelconque sur cette table?

Quel est le type de base de données? Certaines base te créent une clé cachée genre le ROWID en oracle qui peut servir.

Autre option, quand tu fais ton select en jdbc, tu obtiens un rowset. En te positionnant dedans (rowset.absolute(XXX)) et en faisant un rowset.setReadonly(false), tu peux commencer à modifier cette ligne en faisant des appels rowset.updateXXX(columne,value) suivi de rowset.update().


Tu peux aussi faire un update avec le where qui reprend les valeur des trois anciennes colones.

Mais avoir une primary key ca reste plus pratique.

[Tamzoro]

Ma database est disposé ainsi...


J'aurais préféré faire un update avec le WHERE si c'est possible, par contre je ne connais pas du tout la syntaxe pour faire 3 WHERE dans ma requête?

ps: Désolé pour l'entité "Collabo" , le nom de cette table m'a été imposé.

[tchize_]

Tu ne fais pas trois WHERE, tu fais un seul WHERE, exactement comme dans le select, avec plusieurs critère:

Code sql :

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

WHERE A=... AND B=.... AND C=...

[Tamzoro]

Merci pour ton aide, j'ai compris ce que je devais faire par contre j'ai une erreur syntaxique en rapport avec les AND

com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Erreur de syntaxe près de '2'and'IdentI = '3'' Ã* la ligne 1
Requete :Update collabo set CodeA = 'AAAAA9', IdentM = '2', IdentI = '3' where CodeA = 'AAAAA9'and'IdentM = '2'and'IdentI = '3'

Comment adapter les AND dans ma requête SQL liée au Java ?

Code :

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

1
2
3
4
5
String req = "Update collabo set CodeA  = '" + col.getCodeA() +
        "', IdentM = '" + col.getMusicienApp().getIdentM() + 
        "',  IdentI = '" + col.getInstrumentApp().getIdentI() + 
        "' where CodeA = '" + col.getCodeA() + "'" + "and" + "'IdentM = '" + col.getMusicienApp().getIdentM() + "'" +
        "and" + "'IdentI = '" + col.getInstrumentApp().getIdentI() + "'";

[tchize_]

il te manque des espaces autour du and

[joel.drigo]

Et une erreur ici :

Code :

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

+ "'IdentI = '" + col.getInstrumentApp().getIdentI() + "'";

La quote devant IdentI. Pareil d'ailleurs pour IdentM.

En plus les quotes pour des int, ça fait de la conversion implicite inutile (si c'est supporté par le SGBD).

Au lieu de compliquer la vie (pour ne pas dire "te faire ch***") avec les quotes, utilise des PreparedStatement.

[tchize_]

Effectivement, préfère les PreparedStatements. Sinon, le jour où tu devra encoder un "Viola d'amore" comme instrument, tu va avoir de sales surprises

[Tamzoro]

Bonsoir,

Merci pour votre aide je vais utiliser PreparedStatement ça va être plus simple.