Java

Servlets - Utilisation correcte ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Servlets - Utilisation correcte ?

didier1809

${citation_perso}

:hello:

J'ai fait un essai avec ajax (une aide à la recherche, à la google style), qui va lire des enregistrement dans Oracle.

Donc à chaque frappe du clavier --> Un select executé.

J'aimerai vos avis sur plusieurs points :
- Est-ce que je gère bien les connexions/déconnexions ?
- Est-ce que ce code est correct au point de vue performances ?
- Que changeriez vous ?

Je cherche surtout à faire du code propre et performant, et je vous rassure, ce n'est pas un exercice à terminer pour un cours quelconque ;-)

Code :

package lu.dinow.ajax;
import java.io.IOException;
import java.io.PrintWriter;
import java.sql.*;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;
public class StudentInfo extends javax.servlet.http.HttpServlet {
private Connection orclCon;
public void doGet(HttpServletRequest req, HttpServletResponse res)
throws IOException, ServletException {
System.out.println("doGet" );
res.setContentType("text/xml" );
PrintWriter out = res.getWriter();
String ret = process(req,req.getSession());
out.print(ret);
out.close();
}
public void init() throws ServletException {
super.init();
try {
orclCon = connect();
} catch(ClassNotFoundException e) {
e.printStackTrace();
} catch(SQLException e) {
e.printStackTrace();
}
}
private String process(HttpServletRequest res, HttpSession session){
String ret = "";
try{
ret = "<root>";
String typedLetters = res.getParameter("roll" );
Connection con = (orclCon == null) ? connect() : orclCon;
Statement stmt = con.createStatement();
String query = "Select First_name from employees where lower(first_name) like lower('" + typedLetters +"%') order by First_name";
ResultSet rs = stmt.executeQuery(query);
int i = 0;
while(rs.next()){
ret +="<employee id='"+ (i++) +"'>"+rs.getString("First_name" )+"</employee>";
}
rs.close();
stmt.close();
ret +="</root>";
}catch(Exception ex){
System.out.println(ex.getMessage());
}
return ret;
}
public void destroy() {
super.destroy();
if (orclCon != null){
try {
System.out.println("Disconnected from Oracle" );
orclCon.close();
} catch(SQLException e) {
e.printStackTrace();
}
}
}
private Connection connect() throws ClassNotFoundException, SQLException {
System.out.println("Attempt to connect to Oracle..." );
String driverName = "oracle.jdbc.driver.OracleDriver";
Class.forName(driverName);
String url = "jdbc:oracle:thin:@EMEALU6CZ5J2J:1521:ORCL";
return DriverManager.getConnection(url, "HR", "HR" );
}
public void doPost(HttpServletRequest req, HttpServletResponse res)
throws IOException, ServletException {
System.out.println("doPost" );
res.setContentType("text/xml" );
PrintWriter out = res.getWriter();
String ret = process(req, req.getSession());
out.print(ret);
out.close();
}
}

Message édité par didier1809 le 19-10-2009 à 16:35:22

---------------
.

Publicité

pataluc

tu ferais pas mieux de passer par un pool de connexion? surtout pour ce genre de cas (auto completion en ajax) si t'as 10 gus en train de taper dans le champ, ca fait tout de suite une foultitude de requêtes et donc d'ouverture de connexions en un rien de temps...

et sinon pense à blinder ton code contre les injections sql, car la on peux lui faire renvoyer un peu n'importe quoi...

didier1809

${citation_perso}

Merci

Pour le pool de connexions, je fais comment en pratique ?

Je "met" ca dans le contexte de tomcat ou quoi ?

Pour blinder le code je vais y regarder oui

Edit:
Protection contre l'injection de code :

Code :

String selectStatement = "Select First_name, Last_name from employees where lower(first_name) like ? order by First_name";
PreparedStatement prepStmt = conn.prepareStatement(selectStatement);
prepStmt.setString(1, (typedLetters+"%" ).toLowerCase());
ResultSet rs = prepStmt.executeQuery();

Connection pool:

J'ai piqué le code ici : http://www.kgo.de/Pool.html

Message édité par didier1809 le 19-10-2009 à 17:03:25

---------------
.

didier1809

${citation_perso}

CdE a écrit :

Mets un timer dans ton champs pour l'ajax aussi (genre au moins 500ms sans modification avant de faire la requête) ça t'évitera de manger 200 requêtes pendant que le mec tape.

Ok, oui bonne idée, tu as un exemple ?

---------------
.

FORUM HardWare.fr

Programmation

Java

Servlets - Utilisation correcte ?

Sujets relatifs
Formule correcte, mais ecrite de #name? dans la cellule	Utilisation de polices/fonts sur internet,
[Javascript] Utilisation de balises BB dans un textarea	Fichier en cours d'utilisation
Utilisation de la méthode Cells	Utilisation de backreference avec REGEXP sous MySQL
utilisation d'un librairie C avec Ada.	[Divers][Lecture]Utilisation de la bibliothèque portmidi : Pm_Read()
[LaTeX][resolu] package listings et utilisation de emphstyle	PL/SQL : utilisation de variable pour liste de valeur dans requête
Plus de sujets relatifs à : Servlets - Utilisation correcte ?

Page générée en 0.096 secondes