Kaste en exception

Du kan kaste en almindelig exception sådan her:
throw new System.Exception("Two identical keys");

og fange den i din GUI ved at lave en try-catch:
try
{
    // your business that maybe will throw an exception
}
catch(Exception exp)
{
    MessageBox.Show(exp.Message);
}

Tak for svar. Jeg prøver det i aften

Hej igen.
Det virker tilsyneladende, men er det en korrekt måde, jeg har gjort det på ?

Den kode hvor fejlen opstår ses lige nedfor.
Fejlen køre gennem controlleren og jeg fange den i den kode der står under den stiplede linie.

Er det korrekt fremgangsmåde ??

    public void Add(Kontakt k)
        {
           
            try
            {
                kontaktListe.Add(k.CprNr, k);
            }
            catch
            {
                throw new System.Exception("Two identical keys");
            }
        }
-------------------------------------------------------------------------

private void Btm_tilføjKontakt_Click(object sender, EventArgs e)
        {
           
            string cpr = this.textCpr.Text;
            string navn = this.textNavn.Text;
            string adr = this.textAdresse.Text;
            string tlf = this.textTelefon.Text;
            string email = this.textEmail.Text;
            try
            {
                con.KControl.OpretKontakt(cpr, navn, adr, tlf, email);
                MessageBox.Show("Kontakt kunne ikke oprettes", "Fejl", MessageBoxButtons.OK,
                    MessageBoxIcon.Asterisk);
            }
            catch{           
                MessageBox.Show("Kontakt kunne ikke oprettes","Fejl",MessageBoxButtons.OK,
                    MessageBoxIcon.Error);

            }

Og hvordan ved jeg, hvor fejlen stammer fra, når jeg fanger den.
Den kunne vel komme fra flere steder i koden ??

Jeg er kommet videre.
Vil nedenstående være en korrekt løsning?

public void Add(Kontakt k)
        {
           
            try
            {
                kontaktListe.Add(k.CprNr, k);
            }
            catch
            {
                throw new System.Exception("Two identical keys");
            }
        }

--------------------------------------------------------------------

  private void Btm_tilføjKontakt_Click(object sender, EventArgs e)
        {
           
            string cpr = this.textCpr.Text;
            string navn = this.textNavn.Text;
            string adr = this.textAdresse.Text;
            string tlf = this.textTelefon.Text;
            string email = this.textEmail.Text;
            try
            {
                con.KControl.OpretKontakt(cpr, navn, adr, tlf, email);

                MessageBox.Show("Kontakt blev oprettet", "Kontakt oprettelse", MessageBoxButtons.OK,
                    MessageBoxIcon.Asterisk);
            }
            catch(Exception excp ){
                MessageBox.Show("Kontakt kunne ikke oprettes", excp.Message ,MessageBoxButtons.OK,
                    MessageBoxIcon.Error);
            } 
           
           

        }

Der hvor du kaster din exception skal du være opmærksom på, at lige meget hvilken fejl der sker når du prøver at lave en Add, vil fejlen blive "Two identical keys". Dette er også tilfældet hvis kontaktListe eksempelvis er null.

Der hvor du fanger din exception ser umiddelbart fint ud.

Hvordan håndtere jeg så det ?
Det rigtigste ville vel være at fange fejlen, der hvor objektet blev oprettet.
Men hvis man nu antog at man ville fange den i følgende kode hvordan ville man så gøre ?

public void Add(Kontakt k)
        {
           
            try
            {
                kontaktListe.Add(k.CprNr, k);
            }
            catch
            {
                throw new System.Exception("Two identical keys");
            }
        }

En god løsning ville være at lave en custom execption, eksempelsvis ContactIdenticalKeysException, som bliver kastet når du laver en Add til kontaktlisten. Og du behøver egentlig ikke lave en try, catch udenom din kontaktListe.Add, medmindre at den kaster en exception som du vil ændre på. Hver gang der sker en try, catch koster det performance.

Så du kunne gøre noget med:

public void Add(Kontakt k)
{
    bool doesContactExists = false; //TODO
    if(doesContactExists)
        throw new ContactIdenticalKeysException();

    kontaktListe.Add(k.CprNr, k);
}

public abstract class ContactIdenticalKeysException: ApplicationException
{
    public ContactIdenticalKeysException()
        : base(){}
    public ContactIdenticalKeysException(string message)
        : base("Two identical keys"){}
}


Der hvor du så fanger den har du mulighed for at lave en bedre catch:

try
{
  ....
}
catch(ContactIdenticalKeysException iExp)
{
  MsgBox...
}
catch(Exception exp)
{
  MsgBox...
}

jeg er ikke helt med  på linien 

bool doesContactExists = false; //TODO
    if(doesContactExists)
        throw new ContactIdenticalKeysException();

Først sætter du doesContactExists til false
og lige efter, inden der er sket mere, tjekker du på, om den er true.
Kan den være andet end false?

Selvom du ikke har besvaret mit sidste spørgsmål, må du hellere få point.
Her kommmer de.

Michael