Envoi SMTP est le verrouillage de mes fichiers - c#

J'ai une fonction c'est l'envoi de messages ( beaucoup) et de leurs pièces jointes.

Essentiellement, il parcourt une structure de répertoire et crée des e-mails à partir d'un fichier de structure par exemple

 c:\emails\message01
                \attachments
 c:\emails\message02
                \attachments

La création des messages s'effectue à l'aide .net c#, norme des choses.

Après tous les messages sont créés... j'ai une autre fonction qui s'exécute directement après que des copies du dossier de messages vers un autre emplacement.

Problème est - fichiers sont verrouillés...

Note: je ne suis pas déplacer les fichiers, juste à les copier....

Des suggestions sur la façon de copier des fichiers verrouillés à l'aide de c#?

Mise à jour

J'ai cette ajouter des pièces jointes méthode

    private void AddAttachments(MailMessage mail)
    {
        string attachmentDirectoryPath = "c:\messages\message1";
        DirectoryInfo attachmentDirectory = new DirectoryInfo(attachmentDirectoryPath);
        FileInfo[] attachments = attachmentDirectory.GetFiles();
        foreach (FileInfo attachment in attachments)
        {
            mail.Attachments.Add(new Attachment(attachment.FullName));
        }
    }
InformationsquelleAutor JL. | 2009-08-18
  1. 52

    Comment êtes-vous de lire les fichiers pour créer le message e-mail? Ils devraient être ouvert en lecture seule, avec un FileShare ensemble de FileShare.ReadWrite... alors ils ne devraient pas être verrouillé. Si vous utilisez un FileStream vous devez également placer votre logique dans la using mot-clé, de sorte que les ressources soient éliminés correctement.

    Mise à jour:

    Je crois que l'élimination du courrier électronique lui-même va se fermer ressources et de déverrouiller les fichiers:

    using (var mail = new MailMessage())
{
    AddAttachments(mail);
}
//File copy code should work here
    • Est-il quelque chose que je pourrais faire pour le code ci-dessus, pour le faire céder des biens, je pensais vraiment que le garbage collector prendrait soin d'elle, peut-être que je me trompe?
    • Merci John - je crois que votre réponse d'éliminer le courrier de l'objet lui-même est le meilleur scénario, comme l'a souligné Luc
    • John, je crois que je vous aime 🙂
    • Moi aussi je vous aime pour cela, elle a passé tout l'après-midi à la recherche de cette solution!
    • Merci beaucoup pour sauver ma vie
    • merci pour le sauver John. Cela fonctionne parfaitement.

    InformationsquelleAutor John Rasch
  2. 16

    haine de répondre à mon propre post, mais oui pour la prochaine pauvre mec qui a ce problème voici la solution:

    APRÈS AVOIR ENVOYER LE MESSAGE

            //Send the mail
        client.Send(message);

        //Clean up attachments
        foreach (Attachment attachment in message.Attachments)
        {
            attachment.Dispose();
        }

    Disposer les pièces jointes... efface la serrure, et les messages seront toujours envoyés avec les pièces jointes. Disposer NE supprime PAS les fichiers, efface simplement les pièces jointes 🙂

    • Jean de Rasch a la bonne réponse: Vous devez disposer de la MailMessage lui-même (de préférence en l'enveloppant dans un using bloc). Le Dispose méthode de la MailMessage dispose également toutes les pièces jointes.
    InformationsquelleAutor JL.
  3. 3

    Êtes-vous de fermer les fichiers après avoir fini de les lire? Si vous les ouvrez pour la lecture, mais ne pas les fermer lorsque vous avez terminé, il doit conserver un verrou sur elle jusqu'à ce que le programme s'arrête et se ferme automatiquement tous les fichiers.

    • Merci pour la réponse, pas vraiment d'ouvrir les fichiers, tout en ajoutant, comme dans le code ci-dessus
    InformationsquelleAutor matthock
  4. 2
        MailMessage email = new MailMessage();

    email.From = txtFrom.Text;
    email.To = txtToEmail.Text;
    email.Subject = txtMSubject.Text; 
    email.Body = txtBody.Text;

    SmtpClient mailClient = new SmtpClient();
    mailClient.Host = "smtp.emailAddress";
    mailClient.Port = 2525;
    mailClient.Send(email );
    email.Dispose();

    //After Disposing the email object you can call file delete

    if (filePath != "")
    {
      if (System.IO.File.Exists(filePath))
      {
        System.IO.File.Delete(filePath); 
      }
    }
    • C'est la réponse correcte et devrait fonctionner avec ASync envoyer aussi si vous passez l'objet MailMessage dans les paramètres de l'utilisateur
    InformationsquelleAutor dnxit
  5. 0

    Je vois cela beaucoup lors de l'envoi de pièces jointes. J'ai l'habitude d'utiliser quelque chose comme ce qui suit:

    Dans le code qui déplace les fichiers dans un autre emplacement, vous pouvez utiliser le modèle suivant:

    L'intérieur de la boucle pour parcourir les fichiers

    bool FileOk = false;
while (!FileOk)
{
   try
   {
      //code to move the file
      FileOk = true;
   }
   catch(Exception)
   {
      //do nothing or write some code to pause the thread for a few seconds.
   }

}
    • Ne veux pas vraiment aller dans cette voie, pour la raison qu'un autre processus comme le service d'indexation pourrait verrouiller le fichier, de créer un lien mort dans mon application, mais merci pour la suggestion.
    InformationsquelleAutor David