Discussion :

[69Pierre]

Bonjour.
J'ai une méthode sendMail qui envoi un mail qui est utilisée par une méthode sendMails qui envoie plusieurs mails.
Je souhaite que la méthode sendMails me renvoie le nombre de mails envoyés, le nombre de mails non remis et les adresses mails des destinataires qui n'ont pas reçus leurs mails (en plus d'envoyer les mails).

A ce niveau, il y a, je crois, 2 solutions :
La première, créer un objet et lui traverser les 2 méthodes pour avoir ses méthodes et attributs en sortie.
La seconde, moins couteuse, enlever les void et ajouer des return et des out à mes 2 méthodes.


J'ai donc un sendMail :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
public bool sendMail(String email, String sujet, String corps, SmtpClient client, out String mailsError, out int nbMailsTotal, out int nbMailsError)
        {
            ...
 
            return true;
        }

et sendMails sur le même modèle :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
public bool sendMail(String[] listeMailsString, String sujet, String corps, String mailsError, int nbMailsTotal, int nbMailsErro, out String mailsError, out int nbMailsTotal, out int nbMailsErro)
        {
            ...
           sendMail (...);
           ...
 
            return true;
        }

A part Visual Studio tousse dans les signatures et les return, est ce comme cela que je dois procéder.
Merci de votre aide.

[Noxen]

Tu peux te faire une classe de résultat personnalisée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
public SendMailsResult
{
  int TotalSent { get; }
  int TotalFailed { get { return _missedEmails.Length; } }
  IEnumerable<string> MissedEmails { get { return _missedEmails; } }
  private string[] _missedEmails;
  public SendMailsResult(int sent, IEnumerable<string> missed)
  {
    TotalSent = sent;
    _missedEmails = missed.ToArray();
  }
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public SendMailsResult SendMails(string[] listeMailsString, string sujet, string corps, SmtpClient client)
{
  int sent = 0;
  var missed = new List<string>();
  foreach(string mail in listeMailsString)
  {
    bool success = false;
    try { success = SendMail(mail, sujet, corps, client); }
    catch() { }
    if(success) ++send;
    else missed.Add(mail);
  }
  return new SendMailsResult(sent, missed);
}

[69Pierre]

Merci beaucoup Noxen pour cette réponse détaillée.
Je me demandais aussi si il était possible de retourner plusieurs valeurs avec une méthode (out) sans passer par un objet ou un tableau et comment les récupérer.
En fait oui, la méthode est là :
https://adrientorris.github.io/fr/cs...de-csharp.html