
Bramha,it would be better to check for email in contacts.What if your query returns a cotact that don't have any email address?So I modified query. Try this
private final List<Id> Contactids;
public List<Contact> Con;
public MassEmail(Apexpages.standardController Controller)
{
con=[select Id from Contact limit 250 where email!=null];
if(con!=null ||!con.isEmpty()){
for(Integer i=0;i<250;i++){
Contactids.add(con[i].Id);
System.debug('contact Ids are:'+Contactids);
}
}
}
public void sendEmailCustom()
{
Messaging.MassEmailMessage mail=new Messaging.MassEmailMessage();
if(Contactids!=null ||!Contactids.isEmpty()){
mail.setTargetObjectIds(Contactids);
Messaging.sendEmail(new Messaging.MassEmailMessage[]{mail});
}
}
Let me know if you have any issues.
Mark it as best answer if it works./Thanks.