Grosse macro en création. Problème d'édition des données dans un tableau filtré.

**Stanler** · 23/11/2015, 16h02

Bonjour à tous

Je suis en train de coder une nouvelle macro, mais je me heurte à un problème que j'ai toujours réussi à contourner jusqu'à présent.
Ma base de données comporte des lignes vides et des lignes pleines, sur certaines colonnes les lignes semblent remplies de façon aléatoire.
Je dois appliquer un filtre à la base de données pour ensuite attribuer la valeur "" ou "50" selon les cas.
J'arrive bien à filtrer avec le code suivant :

Code :

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

1
2
3
4
5
6
7
8
 
NomTable = "Table1"
Set Ws = Worksheets("J de P")
With Ws
    .ListObjects.Add(xlSrcRange, .Range("$A$1").CurrentRegion, , xlYes).Name = NomTable
    .ListObjects(NomTable).TableStyle = "TableStyleLight1"
End With
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X"

Jusque là, tout baigne. Le critère "X" peut parfois être totalement absent du tableau. J'ai donc ajouté un bloc If à mon code, comme suit :

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
 
Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If

J'ai remarqué que la saisie n'avait pas lieu, ma colonne "T" reste toujours avec la même valeur qu'avant le lancement de la macro. De même, la mise en forme n'est pas appliquée partout.
Et voici la suite du code s'il y a des infos dedans :

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
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
257
258
259
260
 
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X2"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select = "50"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X3"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = "50"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X4"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T1048576").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("A2:A" & Range("A65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1) = "50"
        Range("T2:T" & Range("T1048576").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X5"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = "50"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X6"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible) = "HP"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible) = "HP"
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X7"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible) = "Hors Plafond"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="x8"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible) = "HP"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X9"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible) = "HP"
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
        ActiveSheet.Range("Table1").AutoFilter Field:=13, Criteria1:="X10", Operator:=xlAnd
            ActiveSheet.Range("Table1").AutoFilter Field:=31, Criteria1:="Y1", Operator:=xlAnd
                Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
        For Each Cell In Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Cells
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Active Cell.FormulaR1C1 = "HP"
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
        'Quotité et Plafond Indemnitaires
    ActiveSheet.ListObjects("Table1").Range.AutoFilter Field:=13, Criteria1:="X11"
    Range("M1:M2").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible) = ""
        Range("T2:T" & Range("T65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Range("AA2:AA" & Range("AA65536").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
 
Worksheets("J de P").ListObjects("Table1").Unlist
Application.ScreenUpdating = True
End Sub

Je souhaiterai comprendre pourquoi ça ne fonctionne pas, même si je n'ai pas de message d'erreur. De même, je pense qu'il y a une façon plus courte de coder ça, mais je ne maîtrise pas la technique pour. J'aimerai justement me faire la main dessus.
Pouvez vous m'accompagner dans ce développement ?

**pasdechances** · 23/11/2015, 16h52

Bonjour,
enlève le filtre et essai, si le problème persiste prévient, si non c'est lier a un conflit avec les filtre il faudra donc que tu t'enregistre une macro pour pour enlever et mettre le filtre.

**Stanler** · 23/11/2015, 17h00

Bonsoir,

merci pour ton aide

Je viens d'essayer ce 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
 
Sub test()
    Columns("M").Copy Destination:=Sheets("NEPASSUPPR").Range("A1")
    If Not Sheets("NEPASSUPPR").Range("A2") = "" Then
        For Each Cell In Range("T2:T" & Range("T1048576").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1)
        Range("A2:A" & Range("A65536").End(xlUp).Row).SpecialCells(xlVisible).Cells(1, 1) = "X"
        Range("T2:T" & Range("T1048576").End(xlUp).Row).SpecialCells(xlVisible).Select
        With Selection.Interior
            .Pattern = xlSolid
            .PatternColorIndex = xlAutomatic
            .Color = 10092288
            .TintAndShade = 0
            .PatternTintAndShade = 0
        End With
        Next Cell
    Else
    End If
    Sheets("NEPASSUPPR").Cells.Delete Shift:=x1Up
    Sheets("NEPASSUPPR").Cells.ClearContents
End Sub

Rien n'a changé, la macro n'a pas rédigé "X" (X n'est pas une formule), seulement du texte. En effet, j'imagine qu'autant de filtres peuvent perturber Excel. Devrais-je désactiver .autofilter et le réactiver à chaque fois que je souhaite m'en servir ?

**pasdechances** · 23/11/2015, 17h17

non si sa ne marche pas sans, c'est que le problème vient d’ailleurs.
A je crois voir un truc, le range est incompatible avec les filtre, si tu veux faire du cell par cell évite le range et défini une plage réel.

ex : range(A1:A30) = cell(1,1) to cell(1,30)

et avec un petit algo tu peu faire la même.
par exemple tu place une double boucle pour bouger sur la colonne puis sur la ligne et tu met des condition de passage.
c'est une solution que j'avais trouver pour jouer avec mess TCD filtrer.

**Stanler** · 24/11/2015, 09h27

Bonjour,

En effet, ce n'était pas cohérent. J'ai enlevé le ".Cells(1, 1)"
Par contre, je ne maîtrise pas du tout les boucles, et je ne vois pas du tout ce que je dois faire quand tu me dis que je dois "[placer] une double boucle pour bouger sur la colonne puis sur la ligne et tu met des condition de passage"
Autrement, ne serait-il pas beaucoup plus simple de coder quelque chose comme :

Pour toutes les cellules dans la colonne M,
Si la valeur est égale à "Blablabla", alors écrire "X" dans la ligne correspondante de la colonne T

Avec mes connaissances en VBA, je pencherai pour un "For each" tel que (je n'ai pas testé):

Code :

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

1
2
3
4
5
Dim CGrade as Range
For each cells in Columns("M"), 
If CGrade = "Blablabla" then offset(2 ,0) = "X"
else
end if

**pasdechances** · 24/11/2015, 11h22

Désolé ^^,
quand je parle de double boucle c'est tout banal

on commence par boucler sur les ligne remplient(donc elle sont différentes de vide)

Code :

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

1
2
3
4
5
6
7
8
9
y = ton_initialisation
 
While Cells(y,1) <> "" 'tant que le contenu de la cellule n'est pas vide
    instruction 1 
    instruction 2
    ect ...
 
    y = y+1'on passe a la ligne suivante
wend

la double boucle correspond 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
16
17
18
x = ta_premiere_colone
 
while cells(y,x) <> "" 
    'ex de condition que tu peut utiliser : quand tu ne veux pas passer sur une colonne
    if x = 3 then x = x + 2 'je ne veux pas consulter les colonnes 3 et 4 je met donc x a 5
    ' tu peut réadapter cette condition pour les ligne (variable y)
    While Cells(y,1) <> "" 'tant que le contenu de la cellule n'est pas vide
        instruction 1 
        instruction 2
        ect ...
 
        y = y+1 'on passe a la ligne suivante
    wend
 
x = x +1 'je passe a la colonne suivante
y = ton_initialisation 'on revient au début de la colonne
 
wend

Grosse macro en création. Problème d'édition des données dans un tableau filtré. [XL-2010]

Macros et VBA Excel

Vue hybride

Discussions similaires

Partager

Partager