Strange Error: Junk value getting inserted into StringBuffer

Question

Hello everyone,
I have a method trying to send mails using SMTP server. This method calls the class 'EMail". In the constructor call, out of the many parameters, subject (StringBuffer) is one of them. If in the constructor call, if I pass the subject say like the one mentioned below:
StringBuffer subject=new StringBuffer("Vijay is a gud boy");
EMail email=new EMail(..,..,..,subject,...);
System.out.println("subject="+subject.toString());
email.send();
The subject gets printed very finely through the Sysout statement .
But when I get the mail, the subject of that mail is seen to be distorted.
For e.g: in the case of this particular subject: Vijayis a gud boy
If subject=" DUE OUT OF REVIEW" , it comes as "DUE OUT OF REVIEW".
Any idea, why this distortion?
Please pour in your suggestions.
Thanks in advance,
Ranjith

Answer 1

Hello,
PFB the code for EMail class:
import java.util.*;
import javax.mail.*;
import javax.mail.internet.MimeMessage;
public class EMail
    private String _smtpHost;
    private StringBuffer _subject;
    private StringBuffer _body;
    private Collection _recipients;
    private String _fromAddress ;
    private String _fromName ;
     *Construct an email access object
    public EMail()
        _recipients = new ArrayList(20);
     *Construct an email access object
     *@param host The smtp host
    public EMail( String host )
         this();
         System.out.println("in constructor");
        setSMTPHost( host ) ;
        System.out.println("host is set");
     *Construct an email access object
     *@param host The smtp host.
     *param recipients A list of recipients.
     *@param subject The subject of the email.
     *@param body The message body.
    public EMail( String host, Collection recipients, StringBuffer subject, StringBuffer body )
        setSMTPHost( host ) ;
        setSubject( subject ) ;
        setBody( body ) ;
        setRecipients( recipients ) ;
     *Construct an email access object
     *@param host The smtp host.
     *param recipients A list of recipients.
     *@param subject The subject of the email.
     *@param body The message body.
    public EMail( String host, Collection recipients, StringBuffer subject, StringBuffer body, String fromAddr, String fromName )
         //System.out.println("inside email constructor");
        setSMTPHost( host ) ;
        //System.out.println("host is set");
        setSubject( subject ) ;
        //System.out.println("subject is set");
        setBody(  body ) ;
        //System.out.println("body is set");
        setRecipients( recipients ) ;
        setFrom( fromAddr, fromName );
        //System.out.println("from address set");
     * Set the email smtp host.
     *@param host the SMTP server.
    public void setSMTPHost( String host )
        _smtpHost = host ;
     * Set the email subject.
     *@param subject The email subject.
    public void setSubject( StringBuffer subject )
        _subject = subject ;
     * Set the email message body.
     *@param body The email body.
    public void setBody( StringBuffer body )
        _body = body ;
     * Set the email from info.
     *@param body The email body.
    public void setFrom( String addr, String name )
        _fromAddress = addr ;
        _fromName = name ;
     * Add an emil recipient.
     *@param recipient The recipient to add.
    public void setRecipient( String recipient )
        _recipients.clear();
        try
            _recipients.add( new javax.mail.internet.InternetAddress( recipient ) );
        catch ( javax.mail.internet.AddressException AE )
     * Set the email recipient list.
     *@param recipients The email recipients list.
    public void setRecipients( Collection recipients )
        if( _recipients == null )
            _recipients = new ArrayList( recipients.size() );
        else
            _recipients.clear();
        Iterator it = recipients.iterator();
        try
            while( it.hasNext() )
                _recipients.add( new javax.mail.internet.InternetAddress( (String)it.next() ) );
        catch (javax.mail.internet.AddressException AE)
     *Send the current email.
    public void send()
        log ("EMail::send()" );
        try
            // set the smtp properties file
            Properties sysProps = System.getProperties();
            sysProps.put( "mail.smtp.host", _smtpHost );
            javax.mail.Session session = javax.mail.Session.getInstance( sysProps, null );
            javax.mail.internet.MimeMessage mm = new javax.mail.internet.MimeMessage( session );
            // set the from
            mm.setFrom( new javax.mail.internet.InternetAddress( _fromAddress, _fromName ) );
            //CLog.debug( "EMail::send() Num of recipients["+ _recipients.size()+"]" );
            if ( _recipients.size() > 0 )
                Iterator it = _recipients.iterator();
                while ( it.hasNext() )
                    javax.mail.internet.InternetAddress ia = (javax.mail.internet.InternetAddress) it.next() ;
                    mm.addRecipient( Message.RecipientType.TO, (ia) ) ;
                    ////CLog.debug( "EMail::send() Recipient["+ia.getAddress() + "]" );
                String sub1=_subject.toString();
                mm.setSubject( sub1 );
                mm.setContent( _body.toString(), "text/html" );
                javax.mail.Transport transport = session.getTransport( "smtp" );
                transport.send( mm );
        catch (javax.mail.internet.AddressException AE)
            ////CLog.error("EMail::send()",AE);
            log(" Error: " + AE.getMessage() );
        catch (javax.mail.NoSuchProviderException NSPE)
            ////CLog.error("EMail::send()",NSPE);
        catch (javax.mail.MessagingException ME)
            //CLog.error("EMail::send()",ME);
            log(" Error: " + ME.getMessage() );
        catch (Exception EX)
            //CLog.error("EMail::send()",EX);
            log(" Error: " + EX.getMessage() );
    }Hope this helps.
Thanks in advance,
Ranjith