Servlets - Utilisation correcte ?

Servlets - Utilisation correcte ? - Java - Programmation

Marsh Posté le 19-10-2009 à 16:31:49    

: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 :
  1. package lu.dinow.ajax;
  3. import java.io.IOException;
  4. import java.io.PrintWriter;
  5. import java.sql.*;
  7. import javax.servlet.ServletException;
  8. import javax.servlet.http.HttpServletRequest;
  9. import javax.servlet.http.HttpServletResponse;
  10. import javax.servlet.http.HttpSession;
  12. public class StudentInfo extends javax.servlet.http.HttpServlet  {
  14. private Connection orclCon;
  16. public void doGet(HttpServletRequest req, HttpServletResponse res)
  17.     throws IOException, ServletException {
  18.  System.out.println("doGet" );
  20.  res.setContentType("text/xml" );
  21.  PrintWriter out = res.getWriter();
  22.  String ret = process(req,req.getSession());
  23.  out.print(ret);
  24.  out.close();
  25. }
  28. public void init() throws ServletException {
  29.  super.init();
  30.  try {
  31.   orclCon = connect();
  32.  } catch(ClassNotFoundException e) {
  33.   e.printStackTrace();
  34.  } catch(SQLException e) {
  35.   e.printStackTrace();
  36.  }
  37. }
  39. private String process(HttpServletRequest res, HttpSession session){
  40.  String ret = "";
  41.  try{
  42.   ret = "<root>";
  43.   String typedLetters = res.getParameter("roll" );
  44.   Connection con = (orclCon == null) ? connect() : orclCon;
  45.   Statement stmt = con.createStatement();
  46.   String query = "Select First_name from employees where lower(first_name) like lower('" + typedLetters +"%') order by First_name";
  47.   ResultSet rs = stmt.executeQuery(query);
  48.   int i = 0;
  49.   while(rs.next()){
  50.    ret +="<employee id='"+ (i++) +"'>"+rs.getString("First_name" )+"</employee>";
  51.   }
  52.   rs.close();
  53.   stmt.close();
  54.   ret +="</root>";
  55.  }catch(Exception ex){
  56.   System.out.println(ex.getMessage());
  57.  }
  58.  return  ret;
  59. }
  62. public void destroy() {
  63.  super.destroy();
  64.  if (orclCon != null){
  65.   try {
  66.    System.out.println("Disconnected from Oracle" );
  67.    orclCon.close();
  68.   } catch(SQLException e) {
  69.    e.printStackTrace();
  70.   }
  71.  }
  72. }
  74. private Connection connect() throws ClassNotFoundException, SQLException {
  75.    System.out.println("Attempt to connect to Oracle..." );
  76.    String driverName = "oracle.jdbc.driver.OracleDriver";
  77.    Class.forName(driverName);
  78.    String url = "jdbc:oracle:thin:@EMEALU6CZ5J2J:1521:ORCL";
  79.    return DriverManager.getConnection(url, "HR", "HR" );
  80. }
  82. public void doPost(HttpServletRequest req, HttpServletResponse res)
  83.     throws IOException, ServletException {
  84.  System.out.println("doPost" );
  85.  res.setContentType("text/xml" );
  86.  PrintWriter out = res.getWriter();
  87.  String ret = process(req, req.getSession());
  88.  out.print(ret);
  89.  out.close();
  90. }
  91. }


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

---------------
.
Reply

Marsh Posté le 19-10-2009 à 16:31:49   

Reply

Marsh Posté le 19-10-2009 à 16:46:10    

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...

Reply

Marsh Posté le 19-10-2009 à 16:50:08    

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 :
  1. String selectStatement = "Select First_name, Last_name from employees where lower(first_name) like ? order by First_name";
  2.   PreparedStatement prepStmt = conn.prepareStatement(selectStatement);
  3.   prepStmt.setString(1, (typedLetters+"%" ).toLowerCase());
  4.   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

---------------
.
Reply

Marsh Posté le 22-10-2009 à 08:34:46    

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 ?


---------------
.
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed