user1625324
user1625324

Reputation: 93

Sending email using java app

Why can't this code send email? There are no errors, it just doesn't send.

package tips.mails;

import java.util.Properties;


import javax.mail.Message;
import javax.mail.MessagingException;
import javax.mail.Session;
import javax.mail.Transport;
import javax.mail.Message.RecipientType;
import javax.mail.internet.AddressException;
import javax.mail.internet.InternetAddress;
import javax.mail.internet.MimeMessage;



public class SendMail {
private String from;
private String to;
private String subject;
private String text;

public SendMail(String from, String to, String subject, String text){
    this.from = from;
    this.to = to;
    this.subject = subject;
    this.text = text;
}

public void send(){

    Properties props = new Properties();
    props.put("mail.smtp.host", "smtp.gmail.com");
    props.put("mail.smtp.port", "465");

    Session mailSession = Session.getDefaultInstance(props);
    Message simpleMessage = new MimeMessage(mailSession);

    InternetAddress fromAddress = null;
    InternetAddress toAddress = null;
    try {
        fromAddress = new InternetAddress(from);
        toAddress = new InternetAddress(to);
    } catch (AddressException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }

    try {
        simpleMessage.setFrom(fromAddress);
        simpleMessage.setRecipient(RecipientType.TO, toAddress);
        simpleMessage.setSubject(subject);
        simpleMessage.setText(text);

        Transport.send(simpleMessage);
    } catch (MessagingException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
}

    public static void main(String args[])
    { 
        new SendMail("source", "dist","Subject", "Test Message!!!");
    }
}

Upvotes: 0

Views: 336

Answers (2)

Anand
Anand

Reputation: 21310

You forgot to call send. Try this

   new SendMail("source", "dist","Subject", "Test Message!!!").send();

Upvotes: 1

f_puras
f_puras

Reputation: 2505

You instantiate a SendMail object and do nothing with it. Maybe you should also execute your send() method.

Upvotes: 3

Related Questions