[XL-2010] Procédure trop longue VBA

Procédure trop longue VBA

Bonjour à tous,

Voilà je dois extraire des données depuis une base excel.
Il y a dans ma feuille de données (sheets(8)) une multitude de lignes reprenant des données de production (une ligne par produit et par jour) avec des colonnes qui correspondent aux litrages, aux poids et au nombre de produits fabriqués.
Or ce que je cherche à faire c'est une nouvelle base reprenant par produit, mais cette fois-ci par mois, le total des litrages, poids et nombres produits.
J'ai pour ce faire créer un code, et je précise que celui-ci fonctionne lorsque je le teste avec seulement quelques types de produits, mais dès lors que je le complète pour qu'il fonctionne avec le reste, la procédure devient trop longue.
C'est là que j'ai besoin d'aide.
Je pense qu'il serait judicieux de créer une sorte de boucle pour faire mois après mois, mais je ne sais comment y parvenir.

Voici mon code (pour les seuls mois de Janvier et Février).

Code:

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
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
Sub Bilan_mensuel_TG()
 
        Application.ScreenUpdating = False
 
        Dim anencourt As Integer
        Dim anavant As Integer
        Dim litrage As Long
        Dim nbligbaseprod As Long
        Dim nbFrom As Long
        Dim pdsFrom As Long
        Dim nbligBilan As Long
 
        fa = "Produit A"
        fg = "Produit G"
        fm = "Produit M"
        fw = "Produit W"
 
        anencourt = Sheets("Accueil").Cells(7, 3).Value
        anavant = anencourt - 1
 
                    '/////TG\\\\\
                    s = "TG"
                    nbligbaseprod = Sheets(8).Range("A1").End(xlDown).Row
 
 
                    ' Mois de Janvier
                    m = "Janvier"
 
                    Sheets(8).Select
                    p = fa
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fg
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fm
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fw
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    ' Mois de Février
                    m = "Février"
 
                    Sheets(8).Select
                    p = fa
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fg
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fm
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
 
 
                    Sheets(8).Select
                    p = fw
                    litrage = 0
                    nbFrom = 0
                    pdsFrom = 0
                    nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value
                    Next i
                    For i = 2 To nbligbaseprod
                    If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value
                    Next i
 
                    Sheets("BILAN").Select
                    If litrage <> 0 Then
                    Cells(nbligBilan + 1, 1).Value = anavant
                    Cells(nbligBilan + 1, 2).Value = m
                    Cells(nbligBilan + 1, 3).Value = s
                    Cells(nbligBilan + 1, 4).Value = p
                    Cells(nbligBilan + 1, 5).Value = litrage
                    Cells(nbligBilan + 1, 6).Value = nbFrom
                    Cells(nbligBilan + 1, 7).Value = pdsFrom
                    End If
(...)
 
End Sub

Merci d'avance pour votre aide !
Lou

Bonjour,

la simplification la plus rapide d'après ce que je vois dans ton code, c'est d'intégrer tes 12 traitements mensuels dans un seule boucle
car j'ai l'impression que quelque soit le mois, tu fais la même chose n'est-ce pas ?

si tel est le cas (je n'ai pas décortiqué ligne par ligne ton code) il suffit d'utiliser la variable m qu'on va modifier à chaque tour de boucle en y mettant le nom du mois

pour se faire, on construit un tableau contenant tes 12 mois

on pourrait se passer de la variable m en utilisant directement la valeur du tableau, à toi de voir
Code:

1 2 3 4 5 6 7 8 9 10 Dim TabMois() Dim m As String Dim i As Integer TabMois = Array("Janvier", "Fevrier", "Mars") 'etc... mettre toutes tes valeurs For i = LBound(TabMois) To UBound(TabMois) m = TabMois(i) ' le traitement à faire Next i

Re,

Tout d'abord merci pour ta réponse ! :D
Oui, le traitement est bien le même pour tous les mois, ta solution m'a l'air parfaite.

Merci pour cette technique, ça fonctionne impec !

Bonjour,
Pour information, Les procédure trop long peuvent très souvent être découpées en procédure.
Pour exemple tu as 12 mois mais ta sub est trop longue pour le gérer mais elle accepte 1 mois sen problème.

Tu fais un sub pour 1 mois en lui passant en paramètres le informations dont ton code a besoin!

Code:

sub GestionMois (Feuille as WorkSeet, Periode as DatTime)

Mai je suis sur que tu peux découper également la sub mois en sub ou Functon!

rdurupt a tout à fait raison

d'ailleurs, j'ai regardé un peu plus en détail ton code, et y'a des choses qui peuvent être regroupées pour économiser des lignes (certes) mais (surtout) du temps d'exécution

par exemple, ce bloc :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 Sheets(8).Select p = fa litrage = 0 nbFrom = 0 pdsFrom = 0 nbligBilan = Sheets("BILAN").Range("A1").End(xlDown).Row For i = 2 To nbligbaseprod If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value Next i For i = 2 To nbligbaseprod If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then nbFrom = nbFrom + Cells(i, 10).Value Next i For i = 2 To nbligbaseprod If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then pdsFrom = pdsFrom + Cells(i, 11).Value Next i
tu effectues 3 fois la même boucle, pour vérifier 3 fois les mêmes conditions

ça peut donc se simplifier ainsi :
Code:

1 2 3 4 5 6 7 For i = 2 To nbligbaseprod If Cells(i, 3).Value = anavant And Cells(i, 4) = m And Cells(i, 7) = p Then litrage = litrage + Cells(i, 9).Value nbFrom = nbFrom + Cells(i, 10).Value pdsFrom Cells(i, 11).Value End If Next i
Plutôt que de sélectionner une feuille et d'ensuite travailler sur la feuille active, tu peux encore accélérer le traitement en travaillant comme tel :
Code:

1 2 3 4 5 6 7 8 9 With Sheets(8) For i = 2 To nbligbaseprod If .Cells(i, 3).Value = anavant And .Cells(i, 4) = m And .Cells(i, 7) = p Then litrage = litrage + .Cells(i, 9).Value nbFrom = nbFrom + .Cells(i, 10).Value pdsFrom .Cells(i, 11).Value End If Next i End With
le bloc With / End With permet d'écrire une référence, et de ne pas avoir à la recopier dans l'ensemble du bloc.
il suffit donc de rajouter un point devant chaque Cells() et c'est comme si tu écrivais "Sheets(8).Cells()"

On peut pousser le vice en utilisant une seule variable tableau qui contiendrait l'ensemble des variables que tu incrémentes et un autre tableau contenant les variables de références... m'enfin le code devient moins lisible quand on a pas l'habitude d'utiliser des tableaux

Merci pour toutes vos réponses ! Je vais m'affairer à alléger tout ça.