[How Do I:] Create a Reusable Component for Sending Email to a Distribution List

Good job Chris Pels

Good job Chris Pels

Thanks for a great tutorial Chris.

Chris, YOU ROCK.

I was about to start woring on similar stuff for my CMS.

THANKS a bunch !!

Chris, i must really commend your tutorials. I have benefitted intensely from them. I have been following your series on emailing in .netand i like the way you carefully explain. God bless you and I would appreciate a tutorial also on Instant messenging and chatting. Thanks a lot

Thanks Chris Pels

chris you're the man..

I am getting "System.InvalidOperationException: An asynchronous call is already in progress. It must be completed or canceled before you can call this method." error. Did anybody else experience the same error?

Also, shouldn't SendMessage(mailMessage) call be outside of the For loop.  When I move it out I get no error message but email is only being sent to the last person in the table probably because of the mailMessage.To.Clear. When I remove To.Clear and move SendMessage(mailMessage) out it works but I get a list of addresses in the To field and I really don't want people on the dist list to see that. Is there a way to hide that list?  I want people to only see their own email address in the To field.

I get the same exception, and am pretty sure this code is flawed.  Since the demo code swallows exceptions, he's not seeing it, but MS doesn't let you use SendAsync this way.

Yeah, pretty sure this code is bugged. The InvalidOperationException gets thrown when sending multiple emails. 'An asynchronous call is already in progress. It must be completed or canceled before you can call this method.' How can this be fixed?

Thanks..

I've just been researching this same thing (which is how I ended up here). According to the MSDN page http://msdn.microsoft.com/en-us/library/x5x13z6h(VS.80).aspx trying to use SendAsync as a way of rapidly cycling through multiple emails doesn't appear to be what they had in mind: "After calling SendAsync, you must wait for the e-mail transmission to complete before attempting to send another e-mail message using Send or SendAsync."

Basically, it looks like SendAsync is meant to allow you to send a single email and process code that comes after it, rather than to allow you to send the next without waiting for the first to complete. I did see one forum that said that if you created a new instance of smtpclient for each email, you could process faster. I can't currently confirm whether this works, however.

excellent job

Hello

Thanks for this great article . I followed the steps but I get an exception says :

ex = {"An asynchronous call is already in progress. It must be completed or canceled before you can call this method."}

I don't know why this happen . can someone help me with it

Regards

Tamer Kamal

Hello

Thanks for this great article . I followed the steps but I get an exception says :

ex = {"An asynchronous call is already in progress. It must be completed or canceled before you can call this method."}

I don't know why this happen . can someone help me with it

Regards

Hi Chris Pels and everyone,

This code is good for a sample, but will never work in practice.

1) Because in the loop you are sending emails asynchronously. That is when you get to the next reciepient in the loop, you are not done with the previous. The prevois email is being sent and being used. That is why you cannot just clear the To property.

2) You cannot return the sentEmails property with a correct number of sent emails, because when you exit the method the emails are still being sent and the OnSendCompleted Eventhandler will continue to fire long after you have left the method.

I could propose, Chris, you fixed the issues and come out with a new video, if this is possible. Because there is a lot of wrong in this video, that will confuse the folks out there trying to get started with such an application.

Regards

I agree with flybentley. I wasted a lot of time. Next time I'll read the posts first.