Discussion :

[Erick_Claude]

Bonjour,
J'ai un problème avec la syntaxe dans ce code

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
package data;
 
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
import java.util.List;
 
import javax.swing.text.DefaultEditorKit.InsertBreakAction;
 
import modele.Contribuable;
 
public class ContribuableDAOImpl implements iDAOContribuable {
 
	private Connection con ;
 
 
 
	public ContribuableDAOImpl(Connection con) {
		super();
		this.con = con;
 
	}
 
 
	@Override
	public boolean ajouter(Contribuable c) {
		Statement pst =null ;
		ResultSet rst = null;
 
 
		try {
			if ((con == null)|| (con.isClosed()))   return false ;
			else {
 
		pst =con.createStatement();
		pst.executeUpdate("INSERT INTO  contribuable VALUES ('"  + c.getNif() + "','" +  c.getNom() + "')");
 
 
		//  INSERT INTO  contribuable VALUES (?,?);
 
		// pst.setString(1, c.getNif())
		// pst.setString(2, c.getNom())
		// pst.executeQuery()
 
 
			}
 
		} catch (SQLException e) {
			// TODO: handle exception
			System.out.println(e.getMessage());
			e.printStackTrace();
			return false  ;
		} 
 
 
			return true;
 
	}
 
 
 
	@Override
	public boolean supprimer(Contribuable c) {
		Statement pst =null ;
		ResultSet rst = null;
 
 
		try {
			if ((con == null)|| (con.isClosed()))   return false ;
			else {
 
		pst =con.createStatement();
		pst.executeUpdate("DELETE FROM contribuable WHERE  nif='"  + c.getNif() + "'") ;
 
 
			}
 
		} catch (SQLException e) {
			// TODO: handle exception
			System.out.println(e.getMessage());
			return false  ;
		} 
 
 
			return true;
 
	}
 
 
 
	@Override
 
	public List<Contribuable> rechercherParNom(String critere) {
 
		List<Contribuable> res = new ArrayList<Contribuable>();	
		Statement pst =null ;
		ResultSet rst = null;
 
 
		try {
			if ((con == null)|| (con.isClosed()))   return res ;
			else {
 
 
 
 
 
		pst =con.createStatement();
		rst = pst.executeQuery("SELECT * FROM contribuable WHERE nom LIKE 'h%'") ;
 
		//il manque quelque chose ici mais je sais pas comment ecrire le code
 
		return res;
 
			}
 
		} catch (SQLException e) {
			// TODO: handle exception
			System.out.println(e.getMessage());
			return res;
 
		} 
 
	  }
 
}

[wax78]

J'ai un problème avec la syntaxe dans ce code

Certes, mais quel problème au juste ?

[PhilippeGibault]

Bonjour,

Effectivement, ce code a énormément de problème.
En voici une liste (de ce que j'ai vu):

-Nommage: C'est IDAOContribuable et non iDAOContribuable. Le nom d'une classe (et d'une interface) commence par une majuscule. Je recommande d'ailleurs IContribuableDAO.

- La connexion JDBC: Je recommande un template Methode (https://fr.wikipedia.org/wiki/Patron_de_m%C3%A9thode) avec la Méthode getConnexion() ou à minima une méthode getConnexion().
Attention, la Connexion JDBC est auto-closable. C'est un flux, il doit être fermé (remarque suivante).

- Tout ce qui est auto-closable doit être fermé. Depuis Java 7, il faut utiliser le try-ressource soit:

Code :

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

1
2
3
 
try(Connection con = getConnexion();PreparedStatement ps = con.prepareStatement(sql)){
...

-Quand il y a des paramètres dans la requête, toujours utiliser le PreparedStatement, principalement pour des raisons de sécurité (comme l'injection de SQL).

Tu trouvera de l'aide ici:
https://java.developpez.com/faq/jdbc...paredStatement