Discussion :

[GLDavid]

Bonjour CSharpers

Voici mon problème. Je veux lire un document Word pour récupérer certaines informations. Pour ce faire, j'utilise l'interopérabilité. Je teste, Ok, ça marche.
Maintenant, je veux pouvoir utiliser ces données récupérées pour écrire un fichier Excel, toujours avec l'interopérabilité. Voici le code de la classe principale:

Code :

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
 
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using StudyPlanReader;
using DmpkFilesGenerator;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            string cwd = @"C:\Documents and Settings\gldavid\Desktop";
            string file = @"C:\Documents and Settings\gldavid\My Documents\Documents\MyDoc.doc";;
            StudyPlanParser target = new StudyPlanParser(file);
            try
            {
                Console.WriteLine("Reading Word");
                target.Open();
                target.Parse();
                string sn = target.StudyNumber;
                List<String>clots = target.CompoundLots;
                target.Close();
                Console.WriteLine("Creating Excel");
                CalculationsSummaryCreator csc = new CalculationsSummaryCreator(sn, clots, cwd);
                csc.CreateFile();
                Console.WriteLine("OK...");
                Console.ReadKey();
            }
            catch (Exception e)
            {
                target.Close();
                Console.WriteLine("{0}", e.Message);
                Console.WriteLine("{0}", e.Source);
                Console.WriteLine("{0}", e.StackTrace);
                Console.WriteLine("...");
                Console.ReadKey();
            }
        }
    }
}

Le problème est qu'en cours d'exécution, j'ai l'erreur suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
System.Runtime.InteropServices.COMException was unhandled
  Message=The remote procedure call failed and did not execute. (Exception from HRESULT: 0x800706BF)
  Source=StudyPlanReader
  ErrorCode=-2147467259
  StackTrace:
       at StudyPlanReader.AWordReader.Close() in C:\Documents and Settings\dbourgais\my documents\visual studio 2010\Projects\DmpkExcelCalculationsLib\StudyPlanReader\AWordReader.cs:line 74
       at ConsoleApplication1.Program.Main(String[] args) in C:\Documents and Settings\dbourgais\my documents\visual studio 2010\Projects\DmpkExcelCalculationsLib\ConsoleApplication1\Program.cs:line 33
       at System.AppDomain._nExecuteAssembly(Assembly assembly, String[] args)
       at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence assemblySecurity, String[] args)
       at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()
       at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
       at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
       at System.Threading.ThreadHelper.ThreadStart()
  InnerException:

J'ai oublié quelque chose ?
Merci d'avance de votre aide.

@++

[micka132]

Peut etre qu'il faudrait mettre le code de target.Close();
A priori si celui-ci pose problème, tu le rappelles dans le catch, et à ce moment là il ne sera plus catch!

[GLDavid]

Bonjour

Voici le code mon catch:

Code :

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
 
public void Close()
        {
            try
            {
                if (this.document != null)
                {
                    this.document.Close(missing, missing, missing);
                }
                if (this.word != null)
                {
                    this.word.Quit(missing, missing, missing);
                }
            }
            catch (Exception e)
            {
                throw new IOException(e.Message);
            }
        }

J'ai "résolu" mon problème de la manière suivante toutefois:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Console.WriteLine("Reading Word");
                target.Open();
                target.Parse();
                string sn = target.StudyNumber;
                List<String>clots = target.CompoundLots;
                target.Close();
                Thread.Sleep(1000);
                Console.WriteLine("Creating Excel");
                CalculationsSummaryCreator csc = new CalculationsSummaryCreator(sn, clots, cwd);
                csc.CreateFile();
                Thread.Sleep(1000);
                Console.WriteLine("OK...");
                Console.ReadKey();

Bon, franchement, j'aime pas mettre des Sleep dans mon code...
Si quelqu'un a une solution plus raffinée, je prends volontiers,

Merci,

@++

[micka132]

On ne connais pas les types de document et word, je suppose que document c'est un workbook et word est excel ?

Je connais pas trop mais souvent ce que je vois c'est ca :
Marshal.FinalReleaseComObject(document);
apres le close( et aussi apres le quit).
Regarde si ca change quelque chose!

[sisqo60]

Bonjour,

Le code de la méthode Close est l'effet, cela produit l'erreur, mais ce qu'il faut connaître pour comprendre c'est le code de la classe StudyPlanParser .

GLDavid : ça fait du bien d'entendre que les Thread.Sleep sont moche, car c'est le cas. Beaucoup de gens se satisfont de ce genre de solutions. J'aime cette démarche.

[GLDavid]

Bonjour à tous

Merci de vos réponses. De mon côté, j'ai décidé d'attraper le COMException...et de ne rien en faire:

Code :

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
 
        public void Close()
        {
            try
            {
                if (this.document != null)
                {
                    this.document.Close(missing, missing, missing);
                }
                if (this.word != null)
                {
                    this.word.Quit(missing, missing, missing);
                }
            }
            catch (COMException) { }
            catch (Exception e)
            {
                throw new IOException(e.Message);
            }
        }

J'enlève mes Thread.Sleep.
A l'exécution, OK, ça marche.
Néanmoins, je me demande si c'est la bonne procédure à suivre.
Qu'en pensez-vous ?

Merci d'avance,

@++