Discussion :

[Pierre Fauconnier]

Salut.

J'ai un souci avec une gestion d'erreur.

Contexte:
J'ai deux classes, ExcelFiles et ExcelFile. ExcelFiles crée un objet ExcelFile et démarre une méthode sur cet objet. Cette méthode essaie d'ouvrir un fichier Excel. Je voudrais que si le fichier n'a pas répondu dans les 10 secondes, le code retourne une erreur à ExcelFiles pour pouvoir l'inscrire dans un fichier et passer au fichier suivant.

Le problème est que l'erreur générée dans ExcelFile ne remonte pas jusque ExcelFiles, VS s'arrête en debug sur la ligne qui génère l'erreur (throw new ExcelFileException(this.FileName;). J'ai pourtant l'impression de faire les choses correctement.

Code c# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
 
    class ExcelFiles
    {
        private ExcelFile file;
        private XL.Application app;
 
        public ExcelFiles()
        {
            app = new XL.Application();
            app.Visible = false;
        }
 
        public void Process(string fileName)
        {
            file = new ExcelFile(fileName);
            try
            {
                file.Open(app);
                //file.Close();
            }
            catch(ExcelFileException e)
            {
                // Ecriture de l'erreur dans la db
            }
        }
 
    public class ExcelFile
    {
        private XL.Workbook wb = null;
        private static System.Timers.Timer chrono;
 
        public ExcelFile(string fileName)
        {
            this.FileName = fileName;
            chrono = new System.Timers.Timer(10000);
            chrono.Elapsed += new System.Timers.ElapsedEventHandler(chronotick);
        }
 
        #region Properties
        public string FileName { get; private set; }
        #endregion
        /**
         * Ouvre le fichier renseigné dans wb
         * I app application Excel ouverte
         */
        public void Open(XL.Application app)
        {
            chrono.Start();
            wb = app.Workbooks.Open(this.FileName);
            chrono.Stop();
            swf.MessageBox.Show("Le fichier est ouvert");
 
        }
        /**
         * Ferme le fichier sans l'enregister
         */
        public void Close()
        {
            wb.Close(false);
        }
 
        /**
         * Se déclenche après le temps d'attente si le fichier n'a pas été ouvert
         * après le temps imparti à chrono
         */
        private void chronotick(object source, System.Timers.ElapsedEventArgs e)
        {
            chrono.Stop();
            throw new ExcelFileException(this.FileName); 
        }
    }
 
    public class ExcelFileException : Exception
    {
        public ExcelFileException(string fileName)
            : base("Problème à l'ouverture")
        {
            this.FileName = fileName;
        }
 
        public string FileName { get; private set; }
    }

[PixelJuice]

Bonjour,

Il est normal que l'exception ne soit pas catché car elle n'est pas impliquée dans le try catch, je m'explique :

Tu essaye de catché l'erreur quand tu lances ExcelFile.Open(), cette méthode lance le Timer et tu throw l'exception dans la méthode chronotick.

Cependant Timer.Start() ne relaye rien a chronotick() et vice-versa, celle-ci s’exécute a part et du coup l'exception est oubliée. Sans oublier aussi que la méthode qui est rattachée a l’événement Elapsed ne s’exécute pas forcément sur le même thread.

Du coup pour déterminer si la lecture n'a pas fonctionnée, soit tu passes par AppDomain.UnhandledException, soit un événement a toi, soit je sais pas.

[Pierre Fauconnier]

Merci pour cette première piste.

Je comprends bien le problème de la non levée de l'erreur.

Mais ta solution avec appdomain ne peut convenir car le but, c'est évidemment que l'application ne s'arrête pas. En fait, l'ouverture du fichier Excel est réalisée au sein d'une boucle (qui traite +/- 15.000 fichiers Excel), et mon souhait est que si Excel attend trop longtemps à l'ouverture d'un fichier, je puisse stopper l'ouverture de ce fichier, écrire l'erreur dans un log.txt et passer au suivant, le problème étant qu'Excel ne gère pas un "événement de temporisation" à l'ouverture.

Je pensais donc m'en sortir avec un timer, mais ça n'a pas l'air d'être la bonne piste. je continue à chercher une solution mais si tu as une autre idée, n'hésite pas