Sending mail in ASP.Net

Question

5.00/5 (1 vote)

See more:

Hi guys

I have a code to send email to my account. The mail is sent but problem is its sent twice while there are no loops. What might be the problem?
Below is the code:

C#

public void sendCancelAppointmentEmail()
{
    string footer = "If you have any questions about appointment       cancelation,please do not hesitate to call your clinic front desk";
    string subject = "Appointment Canceled";
    string body = "Hi" + " " + "Anele" + "\n\n" + footer;
    MailMessage message = new MailMessage();

    message.From = new MailAddress("Anele.Ngqandu@myaccount.co.za");
    message.To.Add(new MailAddress("Anele.Ngqandu@yahoo.co.za"));
    message.Subject = subject;
    message.Body = body;

    SmtpClient client = new SmtpClient();
    client.Credentials = new System.Net.NetworkCredential ("Anele@myaccount.co.za", "mypassword");
    client.Port = 00;
    client.Host = "mail.myaccount.co.za";
    client.EnableSsl = false;
    try
    {
        client.Send(message);
    }
    catch (Exception ex)
    {
        System.Windows.Forms.MessageBox.Show(ex.Message);
    }
}

Posted 15-Feb-12 5:39am

Anele Ngqandu

Updated 15-Feb-12 5:58am

Richard MacCutchan

v2

Add a Solution

Comments

ZurdoDev 15-Feb-12 11:59am

Likely you are either calling sendCancelAppointmentEmail twice or you have an issue with your SMTP server. The code looks fine.

[no name] 15-Feb-12 12:41pm

yes, the code looks fine.

Sergey Alexandrovich Kryukov 15-Feb-12 13:46pm

Agree.
--SA

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Answer 1 · 2012-02-15T07:54:00

You see: three people say that your code is fine, so the reason should be outside this code. By some reason, this method itself could be called twice.

Do the following: run it under debugger. Set a break point at the very beginning of the method sendCancelAppointmentEmail. When executions comes to the break point, open the Debug window "Call stack" to see where the call comes from.

Alternatively, use System.Diagnostics.EventLog to write log information to the system log. To look at the stack, the classes System.Diagnostics.StackTrace and System.Diagnostics.StackFrame to format the stack data as some text and save this diagnostic information to the system log which you can read later.

Please see MSDN help and code samples:
http://msdn.microsoft.com/en-us/library/system.diagnostics.stacktrace.aspx[^],
http://msdn.microsoft.com/en-us/library/system.diagnostics.stackframe.aspx[^],
http://msdn.microsoft.com/en-us/library/system.diagnostics.eventlog.aspx[^].

For the advanced usage of EventLog, please see my past solutions:
How to create event log under a folder[^],
MsBuild OutPut to the TextBox on the fly in Windows Application[^].

If you use these technique (which you will need to master sooner or later anyway, especially the Debugger), you will be able to find ends in no time.

Good luck,

—SA