Discussion :

[shkyo]

Bonjour à tous,

Je développe une macro Excel en c# avec Visual Studio 2010, sous Windows 7 Pro x64 avec Excel 2010, à la base c'est assez simple:

J'ai un fichier cible déjà mis en forme, j'ouvre un autre fichier où je fais un tri par ordre croissant sur une zone donnée, puis je sélectionne et copy/colle des lignes d'un fichier à l'autre.
Tout se passe en mémoire, j'affiche Excel quand le traitement est fini ce qui gagne beaucoup de temps!

J'ai fais des dizaines d'essais durant la mise au point sans aucun problème particulier, mais depuis la semaine dernière impossible de réaliser le traitement car j'ai le message suivant:



J'ai déjà vérifié que je n'avais bien qu'un seul Excel en mémoire, mon PC a 16Go de ram dont 12 sont dispo, j'ai désinstallé/réinstallé tout le pack Office, mais rien n'y fait!

J'ai même compilé et transféré le programme de ma macro sur un autre PC qui a Office 2013 et il fait la même erreur!!!

Si quelqu'un a des idées ou des suggestions car je suis vraiment coincé...
Merci d'avance!

[parmi]

Bonjour,

Est-il possible qu'il y ait plusieurs instances d'Excel ouvertes en même temps ?

[clementmarcotte]

Bonjour,

Est-ce que tu utilises C# pour piloter Excel par automation, ou bien Est-ce que tu utilises C# pour insérer du code dans l'éditeur VBE pour le faire exécuter par Excel après ?

Tu peux toujours essayer de "passer" ton classeur dans VBA Code Cleaner de Rob Bovey:

http://www.appspro.com/Utilities/CodeCleaner.htm

Sinon, essaie avec Excel visible, au cas où...

Ou bien essaie d'ouvrir ton classeur en partant d'Excel lui-même, en dehors de Caiguisé; d'un coup qu'il te donnerait un autre message d'erreur.

[NVCfrm]

Bonjour,
et si tu montrais le code en question pour qu'on y voit clair. Surtout la partie relative au tri. (Je penses que ça doit se situer là).

[Invité]

Bonsoir,
Si tu n'as pas implémenter la méthode dispose sur ta classe, tu ne déchargeras pas la mémoire!

Vérifie les procès avec le gestionnaire de taches! Tu verras que les instances d'Excel sont toujours en mémoire

[shkyo]

Alors...

@clementmarcotte: Effectivement je pilote Excel en C# avec automation.

@parmi: Non quand je démarre, il n'y a aucun Excel en mémoire, je vérifie avec le gestionnaire de tâches avant de lancer mon programme.

Quand au code, j'ai fais plus de tests hier, et le message d'erreur est au moment où je veux ouvrir le 2ème classeur, donc bien avant le tri.
J'ouvre le 1er comme ça:

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
            try
            {
                _LesWorkBooks = _ApplicationXL.Workbooks;
                _NomFichierBase="C:\\Temp\\Diffusion de plan - trame.xlsx";
                //ouvre le fichier Excel de base situé dans C:\Temp
                _MonClasseurBase = _LesWorkBooks.Open(_NomFichierBase, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M);
 
                //Active la feuille 2 
                _MaFeuilleBase = (_Worksheet)_MonClasseurBase.Sheets["DP-Page 2"];
                _MaFeuilleBase.Activate();
            }
            catch (Exception FormBOpen)
            {
                MessageBox.Show(FormBOpen.Message);
            }

et j'essaie d'ouvrir le 2ème de cette façon:

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
            DialogResult result = opFileDialogParcourir.ShowDialog();
            if (result == DialogResult.OK)
            {
                _NomFichier = opFileDialogParcourir.FileName;
                TXTBPathFile.Text = _NomFichier.ToString();
            }
            if (TXTBPathFile.Text.Length!=0)
            {
                try
                {
                    _LesWorkBooks = _ApplicationXL.Workbooks;
                    //ouvrir le fichier Excel sélectionné
                    _MonClasseur = _LesWorkBooks.Open(_NomFichier, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M);

                    //Active la feuille 1 
                    _MaFeuille = (_Worksheet)_MonClasseur.Sheets[1];
                    _MaFeuille.Activate();

                }
                catch (Exception Parcourir)
                {
                    MessageBox.Show(Parcourir.Message);
                }
            }

Et c'est là (ligne en rouge) que l'erreur apparaît!!

[NVCfrm]

Bonjour,

Alors...
...
Et c'est là (ligne en rouge) que l'erreur apparaît!!

Jusqu'à cette ligne rien de méchante gourmandise en mémoire.
Le message d'erreur est la première piste ... indicative : Excel fait une action, qu'elle ne peut terminer faute de ressources.

Essaies de reproduire manuellement l'action effectué. Que contient le classeur. Des formules, connexions de données, du vba ... quelles modifications récentes ?

[shkyo]

Non ce classeur est un export de données de Catia, il y a juste la feuil1 avec 11 colonnes avec 100 à 500 lignes selon les cas, contenant seulement des valeurs, il n'y a aucune formule!!!
Les cellules sont soient "standard", soient "texte", pas de protection d'aucune sorte...

Mais est-ce que cela ne vient pas que je lance un seul Excel?

Code :

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

1
2
                _ApplicationXL = new Microsoft.Office.Interop.Excel.Application();
                _ApplicationXL.Visible = true;

Ne faudrait-il pas une instance par classeur?

[NVCfrm]

...
Ne faudrait-il pas une instance par classeur?

Possible, mais je ne crois pas. Il n'est pas nécessaire d'avoir plus d'une instance, sauf traitements particuliers.

Catia. Hum ... (je ne connais pas cet outil de CAO. C'est pas mon domaine.)
Mais c'est déjà une piste intéressante pour situer l'origine de ton problème. Le code d'exportation du fichier demande peut-être un traitement titanesque.

Que donne la reproduction manuelle de ton code d'ouverture des 2 fichiers à partir d'Excel ?

[shkyo]

Non, c'est juste un export d'infos toutes bêtes (nom, référence, valeurs numériques, etc...), il peut le faire dans un fichier texte, ou de l'Excel basique

Quand je reproduis la manipulation "à la main" que devrait faire mon code, avec les même fichiers bien sûr, tout marche impec, une seule instance Excel en mémoire et aucune erreur!!

Je viens d'essayer en lançant 2 instances d'Excel, je n'ai plus l'erreur des ressources insuffisantes, par contre quand je fais mes "copier/coller" entre les 2 classeurs, la moitié des "coller" se fait sous forme d'image et non plus de valeurs...

Peut-être que tout laisser par défaut à l'ouverture:

Code :

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

_MonClasseurBase = _LesWorkBooks.Open(_NomFichierBase, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M);

n'est pas une bonne idée? Mais dans ce cas, quel est le bon paramètre à changer...

[Many31]

Qui dit export CATIA dit surement extension csv et non xls?

[shkyo]

Qui dit export CATIA dit surement extension csv et non xls?

Oui, mais le soucis ne vient pas de là, les fichiers sont propres, je les ouverts et utilisés sur plusieurs pc sans aucun problème...

C'est plutôt le fait que je n'arrive pas à ouvrir deux classeurs en même temps dans une seule instance Excel avec mon code en C#, pourtant cela ne semble pas si exotique que ça!

Au fait, j'ai épluché la doc de la méthode Workbooks.Open et les réglages par défaut sont très bien dans mon cas, normalement rien de bloquant de ce côté-là...

[Many31]

Sauf que j'ai déjà eu des soucis de mémoire justement à cause du CSV :/ ca fait un bail j'ai pas trop souvenir du problème ni du contournement mais avez vous essayé le même code en ciblant des fichiers excel basiques (juste pour tester, de vrais xls pas une extension particulière)

Il me semble qu'à lépoque je comparais 2 fichiers CSV et que j'avais du passer par un tableau variable pour "charger" temporairement le premier CSV avant de libérer la mémoire, charger le second dauns un autre tableau variable pour enfin comparer le tout...
Plus souvenir exacte, mais il me semble aussi que j'avais du remettre Application.screenupdating/enableevents ou Application.visible à vrai le temps de l'ouverture du fichier etc.

Bref pour moi c'est le CSV qui est à mettre en cause.

[shkyo]

ça fonctionne enfin!!! En Excel 2010 seulement...

Bon alors j'ai fais deux choses, premièrement une modif du code d'ouverture des classeurs:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
            try
            {
                _NomFichierBase = "C:\\Temp\\Diffusion de plan - trame.xlsx";
                //ouvre le fichier Excel de base situé dans C:\Temp
                _MonClasseurBase = _ApplicationXL.Workbooks.Open(_NomFichierBase, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M);
                //Active la feuille 2 
                _MaFeuilleBase = (_Worksheet)_MonClasseurBase.Sheets["DP-Page 2"];
                _MaFeuilleBase.Activate();
            }
            catch (Exception FormBOpen)
            {
                MessageBox.Show(FormBOpen.Message);
            }

et

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
                try
                {
                    //ouvrir le fichier Excel sélectionné
                    _MonClasseur = _ApplicationXL.Workbooks.Open(_NomFichier, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M, _M);
                    //Active la feuille 1 
                    _MaFeuille = (_Worksheet)_MonClasseur.Sheets[1];
                    _MaFeuille.Activate();
                }
                catch (Exception Parcourir)
                {
                    MessageBox.Show(Parcourir.Message);
                }

Ensuite, un truc tout bête concernant le deuxième classeur issu de l'export que je tentais d'ouvrir, c'était un .xls, il l'ouvrait (ou essayait en tout cas) en mode de compatibilité...
Je l'ai donc ouvert à part, converti en .xlsx, et après zou plus aucune erreur!!

MAIS, car il y a un MAIS, quand j'essaye mon programme sur un Excel 2013, ben ça me refait la même erreur "pas assez de mémoire" bla bla bla...

Sniff... Des suggestions??

[parmi]

Excel 2013 est très capricieux...

On est en train de le tester au bureau et ça plante souvent pour différentes raisons.
Une des raisons est qu'il se sert de 2 gb de mémoire max pour faire ses traitements.
S'il y a beaucoup de formules, ça plante ou gèle.

Il y a des problèmes au niveau graphique aussi ou encore des problèmes de compatibilité avec des applications externes.

Le web est rempli de problèmes là-dessus...

[shkyo]

Je viens de regarder un truc, mon pc de dév utilise Visual Studio 2010 (complet pas express) avec Office pro 2010, du coup la dll "microsoft.office.interop.excel.dll" est en version office12 alors que Excel 2013 c'est plutôt office15...

C'est peut-être de ce côté-là qu'il faut chercher, qu'en pensez-vous??

[Invité]

Bonjour,
test ce code il te permet d'utiliser la version d'Excel de ta machine!

Code Vb.net :

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
import System
Import System.Collections.Generic
Import System.Linq
Import System.Text
Import System.Runtime.InteropServices
 
Dim crObj as CreaObj=new CreaObj
 
 dim ExcelApp as object=crObj.CreateObject("Excel.Application")
 
 
 
 
    public class CreaObj
        Public Function CreateObject(ByVal app As String) As Object
        Dim AppType As Object = Type.GetTypeFromProgID(app)
        Dim ApplInst As Object = Activator.CreateInstance(AppType)
        Return ApplInst
       End Function
End class

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Runtime.InteropServices;
 
namespace WindowsFormsApplication1
{
     CreatGetObject crObj=new CreatGetObject();
    dynamic ExcelApp =crObj.CreateObject("Excel.Application");
    public class CreatGetObject
    {
 
        public dynamic GetObject(String App)
        {
 
            try
            {
                return (dynamic)Marshal.GetActiveObject(App);
            }
            catch
            {
                return (dynamic)CreateObject(App);
            }
 
        }
        public dynamic CreateObject(string app)
        {
            dynamic AppType = Type.GetTypeFromProgID(app);
            dynamic ApplInst = Activator.CreateInstance(AppType);
            return ApplInst;
        }
    }
}

[shkyo]

Bonjour,
test ce code il te permet d'utiliser la version d'Excel de ta machine!
....

Merci de ta réponse, mais je suis désolé car n'importe où que je mette ton code en C#, VS n'en veut pas... Ceci dit je ne suis pas un pro du C#...

Si je le met juste après ça:

Code :

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

1
2
3
namespace NomenclatureEmbout
{
...

il n'en veut pas, après ça:

Code :

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

1
2
public partial class FormBase : Form
    {...

non plus et après ça:

Code :

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

1
2
public FormBase()
        {...

pas mieux...

En plus, si j'utilise "using System.Runtime.InteropServices;" au lieu de "using Microsoft.Office.Interop.Excel;", je n'ai plus accès aux interfaces _Workbook, _Worksheet, Range, etc...

J'ai loupé un truc?

[Invité]

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Runtime.InteropServices;
 
namespace NomenclatureEmbout
{
    public class CreatGetObject
    { 
        public dynamic GetObject(String App)
        {
            try
            {
                return (dynamic)Marshal.GetActiveObject(App);
            }
            catch
            {
                return (dynamic)CreateObject(App);
            }
        }
        public dynamic CreateObject(string app)
        {
            dynamic AppType = Type.GetTypeFromProgID(app);
            dynamic ApplInst = Activator.CreateInstance(AppType);
            return ApplInst;
        }
    }
 
public partial class FormBase : Form
    { dynamic AppType=Null
      Dim crObj as CreaObj=Null;
      public FormBase()
        {
            crObj =new CreaObj;
           ExcelApp =crObj.CreateObject("Excel.Application");
        }
     }
}

[shkyo]

mon code se lance, mais par contre l'erreur du manque de mémoire se reproduit de nouveau...

mais bon, ce n'est pas trop grave, on a beaucoup plus de Office 2010 que des 2013 à la boite, d'ici à ce qu'on migre, j'aurais trouvé une solution...