-Tu as surchargé la méthode SendMail pour ajouter l'argument "MSGBOXSUCCESS", c'est inutile, puisqu'en C# et il y a les arguments optionels.

ex:

public static void SendMail(string SMTP, string FROM, string TO, string SUBJECT, string BODY, string USERNAME, string PASSWORD, string MSGBOXSUCCESS = string.Empty)

Ça te permet d'appeler la fonction avec l'argument MSGBOXSUCCESS en option, donc si il n'est pas fourni, il sera égal a string.Empty (ou "").

Et puis au moment d'afficher la messagebox il suffit de check si l'argument a été fourni.

ex:

public static void SendMail(string SMTP, string FROM, string TO, string SUBJECT, string BODY, string USERNAME, string PASSWORD, string MSGBOXSUCCESS = string.Empty)
{
/*
...
*/

if(MSGBOXSUCCESS != string.Empty)
   MessageBox.Show(MSGBOXSUCCESS);

/*
    ...
*/
}

• Aucun usage de var

• Un block "using" aurait été préférable pour gérer les exceptions (et après le catch tu devrais mettre un finally pour t'assurer que SmtpServer.Close() est bien exécuté)

• Les noms des arguments pique les yeux

En espérant que ça t'aide