Discussion :

[tomtiiti]

Bonjour,

J'ai commencé à construire une macro mais la je bloque. Je vais essayer de faire simple.

Objectif : Créer à partir de la feuille "Donnees" une feuille pour chaque exploitant. Sur chaque feuille je veux créer trois TCD avec les données de la feuille "Donnees".

Environnement
Mes données pour créer le TCD sont dans la feuilles "Donnees", de A à AJ, avec un nombres de lignes différents par mois.
Il y à une feuille "Exploitants" avec la liste des exploitants variables selon le mois. (environ 15)

J'ai donc fais (en gros) une boucle qui dit que tant que la colonne où sont les exploitants n'est pas vide tu fais :
- Créer une feuille.
- Créer 3 TCD.

Mon problème : La feuille se crée, les 3 TCD aussi, c'est selon lorsque je passe à la seconde feuille, donc aux second exploitants, que le premier TCD se crée mais bloque lors de premier parametres. (Erreur 1004 : Impossible de lire la propriété PivotTables de la classe Worksheet)

J'ai été voir sur internet essayer plusieurs choses mais ça bloque.

Voici mon code :

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
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
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
Sub trere()
'###################
'CREATION TCD
'###################
Dim exploit
Dim fexploit
i = 3
n = 1
TCD = "1"
TCD2 = "2"
TCD3 = "3"
 
Sheets("Exploitants").Select
 
    While Range("A" & n) <> ""
        exploit = Range("A" & n)
        fexploit = Range("B" & n)
 
    Sheets.Add After:=Sheets(Sheets.Count)
   ' ActiveSheet.Name = "Feuil" & i
 
'#
'Tableau TCD numéro 1
'#
 
'Création du tableau:
 
 
ActiveWorkbook.PivotCaches.Add(SourceType:=xlDatabase, SourceData:= _
    [Donnees!A1].CurrentRegion.Address(, , xlR1C1, True)).CreatePivotTable _
    TableDestination:="Feuil" & i & "!R6C1", _
    TableName:=TCD
 
'Mise en forme du tableau :
With Worksheets("Feuil" & i).PivotTables(TCD).PivotFields( _
        "Exploit")
        .Orientation = xlPageField
        .Position = 1
     Worksheets("Feuil" & i).PivotTables(TCD).PivotFields("Exploit"). _
        CurrentPage = "/gfgfgf/fgfgft/gfgfgf/Nsso/Sspo/Mm/"
End With
With Worksheets("Feuil" & i).PivotTables(TCD).PivotFields( _
        "Maestro Label")
        .Orientation = xlPageField
        .Position = 1
End With
With Worksheets("Feuil" & i).PivotTables(TCD).PivotFields( _
        "APPLICATION")
        .Orientation = xlRowField
        .Position = 1
End With
With Worksheets("Feuil" & i).PivotTables(TCD).PivotFields("Object")
        .Orientation = xlRowField
        .Position = 2
End With
 
        Worksheets("Feuil" & i).PivotTables(TCD).AddDataField Worksheets("Feuil" & i).PivotTables(TCD). _
        PivotFields("Alarm Text"), "Nombre de Alarm Text", xlCount
 
 
Range("A7").Select
Selection.ShowDetail = False
 
TCD = TCD + 3
 
 
'#
'Tableau TCD numéro 2
'#
 
Range("E5").Select
'Création du tableau:
ActiveWorkbook.PivotCaches.Add(SourceType:=xlDatabase, SourceData:= _
    [Donnees!A1].CurrentRegion.Address(, , xlR1C1, True)).CreatePivotTable _
    TableDestination:="Feuil" & i & "!R6C5", _
    TableName:=TCD2
 
'Mise en forme du tableau :
With Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields( _
        "Exploit")
        .Orientation = xlPageField
        .Position = 1
     Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields("Exploit"). _
        CurrentPage = "/gfgfgf/fgfgft/gfgfgf/Nsso/Sspo/Mm/"
End With
With Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields( _
        "Maestro Label")
        .Orientation = xlPageField
        .Position = 1
End With
With Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields( _
        "Superieur 6mins")
        .Orientation = xlPageField
        .Position = 1
    Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields( _
        "Maestro Label").CurrentPage = "Flux_Generique"
End With
With Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields( _
        "APPLICATION")
        .Orientation = xlRowField
        .Position = 1
End With
With Worksheets("Feuil" & i).PivotTables(TCD2).PivotFields("Object")
        .Orientation = xlRowField
        .Position = 2
End With
        Worksheets("Feuil" & i).PivotTables(TCD2).AddDataField Worksheets("Feuil" & i).PivotTables(TCD2). _
        PivotFields("Alarm Text Bis"), "Nombre de Alarm Text Bis", xlCount
 
        Worksheets("Feuil" & i).PivotTables(TCD2).AddDataField Worksheets("Feuil" & i).PivotTables(TCD2). _
        PivotFields("Oceane ID Bis"), "Nombre de Oceane ID Bis", xlCount
 
        Worksheets("Feuil" & i).PivotTables(TCD2).AddDataField Worksheets("Feuil" & i).PivotTables(TCD2). _
        PivotFields("Maestro Label Bis"), "Nombre de Maestro Label Bis", xlCount
With Worksheets("Feuil" & i).PivotTables(TCD2).DataPivotField
        .Orientation = xlColumnField
        .Position = 1
End With
 
TCD2 = TCD2 + 3
 
'#
'Tableau TCD numéro 3
'#
 
Range("K5").Select
'Création du tableau:
ActiveWorkbook.PivotCaches.Add(SourceType:=xlDatabase, SourceData:= _
    [Donnees!A1].CurrentRegion.Address(, , xlR1C1, True)).CreatePivotTable _
    TableDestination:="Feuil" & i & "!R6C11", _
    TableName:=TCD3
 
'Mise en forme du tableau :
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "Exploit")
        .Orientation = xlPageField
        .Position = 1
     Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Exploit"). _
        CurrentPage = "/gfgfgf/fgfgft/gfgfgf/Nsso/Sspo/Mm/"
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "Maestro Label")
        .Orientation = xlPageField
        .Position = 1
 
    Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Maestro Label").CurrentPage = "(All)"
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Maestro Label")
        .PivotItems("Flux_Generique").Visible = False
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "Superieur 6mins")
        .Orientation = xlPageField
        .Position = 1
 
    Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "Superieur 6mins").CurrentPage = "supérieur à 6 min"
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "PFS")
        .Orientation = xlRowField
        .Position = 1
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields( _
        "APPLICATION")
        .Orientation = xlRowField
        .Position = 2
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Object Class")
        .Orientation = xlRowField
        .Position = 3
End With
With Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Parameter Name")
        .Orientation = xlRowField
        .Position = 4
End With
        Worksheets("Feuil" & i).PivotTables(TCD3).AddDataField Worksheets("Feuil" & i).PivotTables(TCD3). _
        PivotFields("Alarm Text Bis"), "Nombre de Alarm Text Bis", xlCount
 
        Worksheets("Feuil" & i).PivotTables(TCD3).AddDataField Worksheets("Feuil" & i).PivotTables(TCD3). _
        PivotFields("Oceane ID Bis"), "Nombre de Oceane ID Bis", xlCount
 
With Worksheets("Feuil" & i).PivotTables(TCD3).DataPivotField
        .Orientation = xlColumnField
        .Position = 1
End With
    Worksheets("Feuil" & i).PivotTables(TCD3).CalculatedFields.Add _
        "Ratio", "='Alarm Text Bis' /'Oceane ID Bis'", True
    Worksheets("Feuil" & i).PivotTables(TCD3).PivotFields("Ratio"). _
        Orientation = xlDataField
 
Range("K7").Select
Selection.ShowDetail = False
 
'#
'INSERTION COULEUR RATIO
'#
 
    Worksheets("Feuil" & i).PivotTables(TCD3).PivotSelect _
        "'Somme de Ratio' 'exploit' 'supérieur à 6 min'", _
        xlDataAndLabel, True
    Selection.FormatConditions.Add Type:=xlCellValue, Operator:=xlLess, _
        Formula1:="=2,01"
    Selection.FormatConditions(Selection.FormatConditions.Count).SetFirstPriority
    With Selection.FormatConditions(1).Font
        .Color = -16752384
        .TintAndShade = 0
    End With
    With Selection.FormatConditions(1).Interior
        .PatternColorIndex = xlAutomatic
        .Color = 13561798
        .TintAndShade = 0
    End With
    Selection.FormatConditions(1).StopIfTrue = False
 
 
TCD3 = TCD3 + 3
ActiveSheet.Name = fexploit
i = i + 1
n = n + 1
Sheets("Exploitants").Select
Wend
 
End Sub

J'espere que j'ai été clair, si des personnes veulent bien m'aider j'en serais ravie.
Merci beaucoup par avance

[78chris]

Bonjour

De façon générale c'est une mauvaise idée de créer des TCD par VBA.

Un TCD doit être conçu et créé pour suivre l'évolution des données, pas refabriqué tous les mois.

De même il n'est pas forcément utile de créer un onglet par exploitant. Un seul onglet avec le choix de l'exploitant dans un segment connecté aux 3 TCD permet de visualiser d'un clic tel ou tel exploitant...

[tomtiiti]

Merci pour ta réponse, j'ai besoin de créer pour chaque exploitant car je veux ajouter des commentaires par la suite pour chaque feuilles.

Oui, mais apres je peux dans mes données source ajouté des données sur 3,4,5 ... mois pour fair une meilleure analyse.

[78chris]

RE

...Oui, mais apres je peux dans mes données source ajouté des données sur 3,4,5 ... mois pour fair une meilleure analyse.

Le TCD peut prendre en compte une source qui grandit... si correctement construit.

Ton code suppose un exploitant par ligne : cela se passera comment pour plusieurs mois ?

[tomtiiti]

Le but est que chaque mois faire une réunion avec l'exploitant et lui montré les chiffres.

Je voudrais faire 3 TCD pour un exploitants pour 1 feuille.

Le nombres d'exploitant est un copier collé des données puis j'ai fais une liste des exploitants et une boucle jusqu’à que la colonne dans la feuille "Exploitants" soit vide.

Et en faite mon code marche pour le premier exploitants, mais lorsque qu'il recommence la boucle il bloque ici :

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
'PI
'exploit = exploit = Range("A" & n)   =  c'est le nom de l'exploitant pour filtrer dans le TCD
'fexploit = Range("B" & n) = le nom de l'exploitant raccourcis pour nommer la feuille
'

ActiveWorkbook.PivotCaches.Add(SourceType:=xlDatabase, SourceData:= _
    [Donnees!A1].CurrentRegion.Address(, , xlR1C1, True)).CreatePivotTable _
    TableDestination:="Feuil" & i & "!R6C1", _
    TableName:=TCD

'Mise en forme du tableau :
With Worksheets("Feuil" & i).PivotTables(TCD).PivotFields( _
        "Exploit")  // Sa bloque ici, avec l'erreur cité plus haut
        .Orientation = xlPageField
        .Position = 1
     Worksheets("Feuil" & i).PivotTables(TCD).PivotFields("Exploit"). _
        CurrentPage = exploit

Désolé si je n'arrive pas à être claire et logique dans ce que je dit .... :/

Merci pour ton aide en tout cas

[mjpmjp]

bonjour,

ActiveWorkbook.PivotCaches.Add(SourceType:=xlDatabase, SourceData:= _
[Donnees!A1].CurrentRegion.Address(, , xlR1C1, True)).CreatePivotTable _
TableDestination:="Feuil" & i & "!R6C1", _
TableName:=TCD

pour les noms de tes onglet et TCD , tu devrais ajouter le nom (sans espace) de l'exploitant

@+JP

[RyuAutodidacte]

Bonjour,

Un TCD doit être conçu et créé pour suivre l'évolution des données, pas refabriqué tous les mois.
De même il n'est pas forcément utile de créer un onglet par exploitant. Un seul onglet avec le choix de l'exploitant dans un segment connecté aux 3 TCD permet de visualiser d'un clic tel ou tel exploitant...

Le TCD peut prendre en compte une source qui grandit... si correctement construit.

Je suis complètement d'accord avec @78chris et suivrait ses conseils

Sur le TCD les 2 filtres principaux seraient l'exploitant et le mois
Ne seraient t-il pas mieux de faire un code VBA bouclant plutôt sur les filtres principaux (exploitant et mois)
afin de récupérer le report des données du TCD par exploitant/mois et enregistrer dans un nouveau classeur Excel
qui lui sera enregistré automatiquement et rangé dans le dossier \année\mois\NomExploitant ?

Cela évitera de créer trop de feuilles sur un même fichier (ce qui alourdi grandement et risque de poser problème par la suite)
et de faire simplement l'exploitation des données exploitants au fur et à mesure des mois …

C'est une autre façon de faire … à voir …

[78chris]

Bonjour

Plusieurs remarques :
TCD qui vaut "1" au départ devient numérique avec TCD=TCD+1.

Tu ne peux créer un TCD dont le nom est un nombre : il est étonnant que la création du TCD du second exploitant ne plante pas.

Ton code crée autant de cache que de TCD : ton classeur va peser inutilement un poids considérable et tu ne pourras synchroniser les TCD d'un exploitant par un segment, ce qui est dommage.

Le principe est de créer un Cache ( cela se fait avec la création du premier TCD) puis de créer chaque autre TCD à partir du même cache.

Il faudrait un code de ce type

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
44
45
46
47
48
49
50
51
52
53
54
55
Dim Moncache As PivotCache, NumTCD As Long, i As Long, Col As Long, TCD As String, TypTCD As Byte
NumTCD = 1
i = 1
Col = 1
    
Sheets("Exploitants").Select
 
    While Range("A" & N) <> ""
        exploit = Range("A" & N)
        fexploit = Range("B" & N)
        If NumTCD Mod 3 = 1 Then
            Sheets.Add After:=Sheets(Sheets.Count)
            ActiveSheet.Name = "Feuil" & i
            i = i + 1
        End If
        
        'Création du tableau:
        
        If NumTCD = 1 Then
            ActiveWorkbook.PivotCaches.Create(SourceType:=xlDatabase, SourceData:= _
                Worksheets("Données").Range("A1").CurrentRegion).CreatePivotTable _
                TableDestination:="Feuil1!R6C1", TableName:="TCD1"
            
            Set Moncache = ActiveWorkbook.Worksheets("Feuil" & i - 1).PivotTables("TCD" & NumTCD).PivotCache
        Else
            TCD = "TCD" & NumTCD
            Moncache.CreatePivotTable TableDestination:="Feuil" & i - 1 & "!R6C" & Col, TableName:=TCD
        End If
        
         Select Case NumTCD Mod 3
            Case 0
                 Col = 1
                 TypTCD = 3
            Case 2
                Col = Col + 5
                TypTCD = 2
            Case 1
                Col = Col + 4
                TypTCD = 1  
         End Select        

         'Mise en forme du tableau :
        Select Case TypTCD
            Case 1
                'Mise en forme du TCD de type 1
            Case 2
                'Mise en forme du TCD de type 2
            Case 3
                'Mise en forme du TCD de type 3
        End Select

        NumTCD = NumTCD + 1    
    Wend

End Sub

EDIT petite modif sur le select case et l'ordre du NumTCD = NumTCD + 1