Discussion :

[houdaùmlk]

Bonjours ,
Comment inserer un vector dans database avec la requette insert into
j'ai essayée ça mais ne fonctionne pas

Code :

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

1
2
3
4
5
6
7
 
  int  id =0;
				           for(int x = 0; x <vec.size() ; x++){
 
 
					      	           	requete3.executeUpdate(" insert into Predicat values("+id+",'"+vec.elementAt(x)+"')");
					      	           }id++;

et voici ma table

Code :

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

1
2
 
requete1.executeQuery("CREATE TABLE Predicat(idPr integer PRIMARY KEY NOT NULL,NomV Varchar2(40) NOT NULL)");

et merci

[adiGuba]

Salut,

j'ai essayée ça mais ne fonctionne pas

C'est à dire ?



Sinon quelques remarques :

• Attention à l'indentation qui rend le tout difficilement lisible malgré qu'il n'y ait que 4 lignes de code.
• Y'a une raison pour utiliser Vector ? C'est plutôt obsolète et on lui privilégiera List/ArrayList.
• De même qu'il est préférable d'utiliser le for-each pour itérer sur les valeurs plutôt qu'une boucle avec index. Plus simple et plus lisible.
• Ne pas oublier de libérer les ressources (Statement et Connection), de préférence dans un try-with-ressources.
• Spécifies toujours le nom des champs de la table dans ton INSERT. Cela permettra à ton code de fonctionner même si la table évolue...
• Enfin attention aux injections SQL, surtout si tu ne contrôles pas la source de donnée. Il est très fortement préférable de passer par un PreparedStatement...

Ce qui donnerait :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
try (PreparedStatement st = requete3.prepareStatement("insert into Predicat(idPr, NomV) values(?, ?)")) {
    int id = 0;
    for (String value : vec) {
        st.setInt(1, id);
        st.setString(2, value);
        st.executeUpdate();
        id++;
    }
}

a++

[houdaùmlk]

Salut,



C'est à dire ?



Sinon quelques remarques :

• Attention à l'indentation qui rend le tout difficilement lisible malgré qu'il n'y ait que 4 lignes de code.
• Y'a une raison pour utiliser Vector ? C'est plutôt obsolète et on lui privilégiera List/ArrayList.
• De même qu'il est préférable d'utiliser le for-each pour itérer sur les valeurs plutôt qu'une boucle avec index. Plus simple et plus lisible.
• Ne pas oublier de libérer les ressources (Statement et Connection), de préférence dans un try-with-ressources.
• Spécifies toujours le nom des champs de la table dans ton INSERT. Cela permettra à ton code de fonctionner même si la table évolue...
• Enfin attention aux injections SQL, surtout si tu ne contrôles pas la source de donnée. Il est très fortement préférable de passer par un PreparedStatement...

Ce qui donnerait :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
try (PreparedStatement st = requete3.prepareStatement("insert into Predicat(idPr, NomV) values(?, ?)")) {
    int id = 0;
    for (String value : vec) {
        st.setInt(1, id);
        st.setString(2, value);
        st.executeUpdate();
        id++;
    }
}

a++

merci ms j'ai ce probleme

Code :

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

1
2
 
java.sql.SQLIntegrityConstraintViolationException: ORA-00001: violation de contrainte unique

[adiGuba]

Bah le message est clair : tu ne respectes pas les contraintes d'unicité de ta table.
L'id doit être unique et apparemment ce n'est pas le cas...


a++