[C#] SmtpClient Connect e Disconnect

[race2]

Salve,
in un invio massivo di E-mail tipo 300.000 tramite il Software che ho sviluppato, avrei la necessita di Connettere e Disconnetere ad ogni LOOP,

cioè dovrebbe funzionare cosi:


SmtpClient smtp = new SmtpClient("smtp.domino.it", 25);

foreach (DataRow emailDR in emailDt.Rows)
{
// *** Connect ***
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
// *** Disconnect ***
}


solo ceh non esistono i metodi "Connect e Disconnect" nella Classe "SmtpClient".

Come posso fare ???

Creo un Istanza di "SmtpClient"e la Distruggo ad ogni LOOP ???

[RaouL_BennetH]

Salve,
in un invio massivo di E-mail tipo 300.000 tramite il Software che ho sviluppato, avrei la necessita di Connettere e Disconnetere ad ogni LOOP,

cioè dovrebbe funzionare cosi:


SmtpClient smtp = new SmtpClient("smtp.domino.it", 25);

foreach (DataRow emailDR in emailDt.Rows)
{
// *** Connect ***
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
// *** Disconnect ***
}


solo ceh non esistono i metodi "Connect e Disconnect" nella Classe "SmtpClient".

Come posso fare ???

Creo un Istanza di "SmtpClient"e la Distruggo ad ogni LOOP ???

Ciao :)

Premetto che non so come funziona l'oggetto SmtpClient.

Volevo solo ricordarti che trovandoti in un ambiente managed, parlare di 'distruzione' di un oggetto non avrebbe molto senso.

L'unica cosa che mi viene in mente è che tu possa farti garantire una chiamata 'alla distruzione' ( :D ) mediante 'using' (purchè implementi IDisposable).


using(Classe c = new Classe())
{
blablabla;
}

[tomminno]

SmtpClient implementa IDisposable solo dal Framework 4.
Per il codice che hai postato potresti usare:

using (MailMessage msg= new MailMessage())
{
...
using (SmtpClient smtp = new SmtpClient("smtp.domino.it", 25))
{
foreach (DataRow emailDR in emailDt.Rows)
{
// *** Connect ***
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
// *** Disconnect ***
}
}
}


Se invece utilizzi il framework 3.5 non c'è modo di "chiudere" un SmtpClient in quanto è gestito automaticamente tramite un pool di connessioni. Pertanto dovresti togliere lo usign su SmtpClient altrimenti non ti compilerebbe.

Dimenticavo: Secondo me stai usando male il msg.To, ad ogni mail che invii la rimandi anche a tutti i destinatari precedentemente inseriti

[race2]

Scusami TOMMINO, il tuo metodo non crea un Istanza per ogni LOOP del foreach !!

Il tecnico del Server SMTP mi ha detto che nel LOG di Sistema si vede ceh la mia connessione rimane Aperta per tutte le ...mila... e-mail ceh invio, invece lui mi consiglia di Aprire e Chiudere la Connesione ad ogni Invio quindi ad ogni LOOP.

[tomminno]

Scusami TOMMINO, il tuo metodo non crea un Istanza per ogni LOOP del foreach !!

Il tecnico del Server SMTP mi ha detto che nel LOG di Sistema si vede ceh la mia connessione rimane Aperta per tutte le ...mila... e-mail ceh invio, invece lui mi consiglia di Aprire e Chiudere la Connesione ad ogni Invio quindi ad ogni LOOP.

Precisiamo: se stai usando il .Net 3.5 non hai speranza di farlo funzionare come chiedi in quanto essendo automaticamente gestito dal pool di connessioni ti rimane comunque aperta.

Se usi .Net 4 puoi semplicemente spostare lo using dentro il foreach e il gioco è fatto.
Anche se essendoci di mezzo un linguaggio managed non hai per niente la certezza che la tua connessione venga chiusa, che è esattamente quello che avviene con le connessioni al db.

Non hai specificato quale versione di C# stai usando

[race2]

Scusate la svista, uso il .NET 4.0

io ho fatto questo:


foreach (DataRow emailDR in emailDt.Rows)
{
SmtpClient smtp = new SmtpClient("out.alice.it", 25);
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
msg.To.Clear();
smtp.Dispose();
}


dici ceh il smtp.Dispose() possa pure fallire e non chiudere la connessione ???

con using() ottengo il solito risultato del mio Script ???

[tomminno]

Scusate la svista, uso il .NET 4.0

io ho fatto questo:


foreach (DataRow emailDR in emailDt.Rows)
{
SmtpClient smtp = new SmtpClient("out.alice.it", 25);
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
msg.To.Clear();
smtp.Dispose();
}


dici ceh il smtp.Dispose() possa pure fallire e non chiudere la connessione ???

Metti lo using per limitare lo scope e il dispose "automatico" dell'oggetto smtp, altrimenti se fallisce una qualunque operazione prima della chiamata la tua connessione rimane aperta.
Potenzialmente anche il Dispose potrebbe fallire, le operazioni eseguite sono tante: manda il comando quit, chiude la connessione tcp e il socket.

[race2]

ok, ho fatto come mi hai consigliato tomminno,

posso però alleviare un po le pene inviando multiple e-mail contemporanee.

dato il seguente Script come posso fare per inviare 10 e-mail contemporaneamente ad ogni apertura e chiusura di connessione ??


using (MailMessage msg= new MailMessage())
{
//code *********
foreach (DataRow emailDR in emailDt.Rows)
{
using (SmtpClient smtp = new SmtpClient("smtp.domino.it", 25))
{
msg.To.Add(new MailAddress(emailDR["sEmail"], emailDR["sNomeVis"]));
smtp.Send(msg);
}
}
}


- evitando di aggiungere nel campo TO, dato che si vedrebbero gli altri 9 destinatari.
- evitando di scrivere 10 volte using(){***//***}

anzi, non 10 volte ma vorrei dare possibilita di impostare da 1 a 30 con un settaggio.