[SOLUCIONADO] Enviar emails desde java

[rgf1987]

Hola gente!!!

estoy enviando emails desde java utilizando javamails, el problema es que me tarda demasiado en enviar un email, una media de 2-4 segundos, lo cual, desde mi punto de vista es muchisimo

el codigo es el siguiente:

Código JAVA:

public class Correo {
    Properties props = new Properties();
    
    public void enviarCorreo(String emailDestino, String subject, String body, String otherDates)
    {
        Correo obj = new Correo();
        String server = "smtp.gmail.com";
        String userName = "[email protected]";
        String password = "rootroot";
        String fromAddres = "todomotorrevista";
        String toAddres = emailDestino;
        String cc = "";
        String bcc = "";
        boolean htmlFormat = false;        
        
        obj.sendMail(server, userName, password, fromAddres, toAddres, cc, bcc,
                     htmlFormat, subject, body+otherDates);
        
    }
 
    private void sendMail(String server, String userName, String password, String fromAddress, String toAddress, String cc, String bcc, boolean htmlFormat, String subject, String body)
    {
    
        Properties properties = System.getProperties();
        properties.put("mail.smtps.host", server);
        properties.put("mail.smtps.auth", "true");
        Session ses  = Session.getInstance(properties);
 
        ses.setDebug(true);
 
        try{
        
            MimeMessage msg = new MimeMessage(ses);
    
            msg.setFrom(new InternetAddress(fromAddress));
    
            if (toAddress != null){
               msg.addRecipients(Message.RecipientType.TO, toAddress);
            }
    
            if (cc != null){
                msg.setRecipients(Message.RecipientType.CC,InternetAddress.parse(cc, false));
            }
    
            if (bcc != null){
                msg.setRecipients(Message.RecipientType.BCC,InternetAddress.parse(bcc, false));
            }
    
            if (htmlFormat){
                msg.setContent(body, "text/html");
            }
            else{
                msg.setContent(body, "text/plain");
            }
    
            msg.setSubject(subject);
            msg.saveChanges();
    
            Transport tr = ses.getTransport("smtps");
            tr.connect(server,userName, password);
            tr.sendMessage(msg, msg.getAllRecipients());
            tr.close();
        }
        
        catch(MessagingException e){
            e.printStackTrace();
        }  
        }
}
 
class MyPasswordAuthenticator extends Authenticator{
   String user;
   String pw;
 
   public MyPasswordAuthenticator (String username, String password)   {
      super();
      this.user = username;
      this.pw = password;
   }
   public PasswordAuthentication getPasswordAuthentication()   {
      return new PasswordAuthentication(user, pw);
   }    
}
 

Igual tarda tanto porque se tiene que conectar con la cuenta de gmail antes de poder enviar el mensaje, pero no estoy seguro... alguna ayuda / alternativa para que los mensajes se envien más rapido?

Un saludo!!!

[chuidiang]

Tarda mucho .... ¿desde dónde hasta dónde? ¿Desde que arrancas el programa hasta que ves el correo en tu buzón? ¿Desde que llamas a enviaCorreo() hasta que sale de ese método?...

Por medidas de tiempos para tratar de determinar qué es exactamente lo que tarda, si en establecer conexión, si en enviar el correo, etc.

Código java:

long inicio = System.currentTimeMillis();
// Aquí código a medir
System.out.println ("Tarda " + (System.currentTimeMills()-inicio) + " milisegundos");
 

En cualquier caso, 2 segundos puede o no ser mucho según qué estés midiendo. Si es desde que envías el correo hasta que lo ves en tu buzón de entrada, es más bien poco y el tiempo perdido puede no tener nada que ver con tu programa.

Se bueno.

[rgf1987]

Al darle a un botón "Guardar" almaceno datos en la base de datos y posteriormente si el guardado es correcto, envio un email al usuario.
Al darle a guardar, mientras envio el email la web se me bloquea y no puedo hacer nada más hasta que terminan de enviarse el email/s

Código Java:

@RequestMapping("confirmarArticulo")
    public String confirmarArticulo(@RequestParam("idArticulo") int idArticulo, ModelMap modelo) throws DAOException {
        long inicio = System.currentTimeMillis();       
        
        long guardarArticulo = System.currentTimeMillis();
        ArticuloManager managerArticulo = ManagerFactory.getInstance().getArticuloManager();
        EstadoManager managerEstado = ManagerFactory.getInstance().getEstadoManager();
        ArticuloVO articulo =  managerArticulo.getArticulo(idArticulo);
        if(articulo.getEstadoArticulo().getIdEstado()==Constantes.ARTICULO_NO_ENVIADO){
            articulo.setEstadoArticulo(managerEstado.getEstado(Constantes.ARTICULO_PENDIENTE_REVISION));//PendienteRevision
        }
        else if(articulo.getEstadoArticulo().getIdEstado()==Constantes.ARTICULO_PENDIENTE_MODIFICACION){
            articulo.setEstadoArticulo(managerEstado.getEstado(Constantes.ARTICULO_EN_REVISION));//PendienteENRevision
        }
        
        managerArticulo.actualizarArticulo(articulo);
        System.out.println ("Tarda " + (System.currentTimeMillis()-guardarArticulo) + " milisegundos en guardar articulo");
        String datosAdicionales= "Titulo Articulo: "+articulo.getTituloArticulo();
        long tcorreo = System.currentTimeMillis();
        Correo correo = new Correo();
        correo.enviarCorreo(
                articulo.getAutor().getEmailUsuario(), 
                Constantes.TITULO_CORREO_ARTICULOENVIADO,
                Constantes.CUERPO_CORREO_ARTICULOENVIADO, datosAdicionales);
        System.out.println ("Tarda " + (System.currentTimeMillis()-tcorreo) + " milisegundos en guardar correo");
        System.out.println ("Tarda " + (System.currentTimeMillis()-inicio) + " milisegundos");
        return "redirect:administracion.htm";
    }
 

Esto me da los siguientes tiempos:
Tarda 82 milisegundos en guardar articulo
Tarda 2978 milisegundos en guardar correo
Tarda 3060 milisegundos

[chuidiang]

Hola:

Al darle a guardar haces varias cosas : base de datos y email. Imagino que la web se te bloquea desde que das al botón "guardar" hasta que puedes volver a trabajar con ella. ¿Qué te hace pensar que es problema de email? Hay dos tareas que haces ahí y suma el tiempo de las dos ¿no?.

Haz el guardado en base de datos y el envío de emails en un hilo separado, de forma que al dar al botón "guardar" lanzas el hilo y así la web se desbloquea inmediatamente.

Se bueno.

[rgf1987]

Supongo que no habras leido lo de los tiempos antes de contestar ;).

Sino el problema no esta al guardar en base de datos ya que solo me tarda 81 milisegundos como puedes ver.

Intentare utilizar hilos como me indicas, luego comentare ;)

Saludos!!

[rgf1987]

Bueno al final lo hice con hilos como me dices, sigue tardando, pero como se ejecuta en segundo plano no me da problemas : ), al final el codigo quedó así:

Creo el hilo:

Código Java:

Correo correo = new Correo(articulo.getAutor().getEmailUsuario(), 
                Constantes.TITULO_CORREO_ARTICULOENVIADO,
                Constantes.CUERPO_CORREO_ARTICULOENVIADO, datosAdicionales);
        Thread hiloCorreo = new Thread (correo);
        hiloCorreo.start();
 

Y la clase Correo quedaria asi implementando la interfaz Runnable:

Código Java:

public class Correo implements Runnable{
    Properties props = new Properties();
    String emailDestino;
    String subject;
    String body;
    String otherDates;
    public Correo(){}
    public Correo(String emailDestino, String subject, String body, String otherDates){
        this.emailDestino=emailDestino;
        this.subject=subject;
        this.body=body;
        this.otherDates=otherDates;
    }
    @Override
    public void run()
    {
        Correo obj = new Correo();
        String server = "smtp.gmail.com";
        String userName = "[email protected]";
        String password = "rootroot";
        String fromAddres = "todomotorrevista";
        String toAddres = emailDestino;
        String cc = "";
        String bcc = "";
        boolean htmlFormat = false;        
        
       
        obj.sendMail(server, userName, password, fromAddres, toAddres, cc, bcc,
                     htmlFormat, subject, body+otherDates);
             
        
    }
 
    private void sendMail(String server, String userName, String password, String fromAddress, String toAddress, String cc, String bcc, boolean htmlFormat, String subject, String body)
    {
    
        Properties properties = System.getProperties();
        properties.put("mail.smtps.host", server);
        properties.put("mail.smtps.auth", "true");
        Session ses  = Session.getInstance(properties);
 
        ses.setDebug(true);
 
        try{
        
            MimeMessage msg = new MimeMessage(ses);
    
            msg.setFrom(new InternetAddress(fromAddress));
    
            if (toAddress != null){
               msg.addRecipients(Message.RecipientType.TO, toAddress);
            }
    
            if (cc != null){
                msg.setRecipients(Message.RecipientType.CC,InternetAddress.parse(cc, false));
            }
    
            if (bcc != null){
                msg.setRecipients(Message.RecipientType.BCC,InternetAddress.parse(bcc, false));
            }
    
            if (htmlFormat){
                msg.setContent(body, "text/html");
            }
            else{
                msg.setContent(body, "text/plain");
            }
    
            msg.setSubject(subject);
            msg.saveChanges();
    
            Transport tr = ses.getTransport("smtps");
            tr.connect(server,userName, password);
            tr.sendMessage(msg, msg.getAllRecipients());
            tr.close();
        }
        
        catch(MessagingException e){
            e.printStackTrace();
        }  
        }
    
}
 
class MyPasswordAuthenticator extends Authenticator{
   String user;
   String pw;
 
   public MyPasswordAuthenticator (String username, String password)   {
      super();
      this.user = username;
      this.pw = password;
   }
   public PasswordAuthentication getPasswordAuthentication()   {
      return new PasswordAuthentication(user, pw);
   } 
    
}
 

En este caso... no haria falta sincronizacion, ¿o si?