Vérification Macro Terminée

Version imprimable

Bonjour,

J'ai terminé ma macro (grâce aussi à certains d'entre vous, merci beaucoup en tt cas)
Mais avant de l'a présenter à mon responsable, j'aimerais que vous me disiez ce que vous en pensez, si il y a des trucs inutiles, ou à améliorer, mais dans l'ensemble elle fonctionne très bien, hormis des cases ou la police changent, et où les lignes des tableaux partent

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
 
 
Option Explicit
 
 
' Modèle de sécurité
 
Sub Macro1()
Dim data As Worksheet
Dim i As Integer
Set data = ThisWorkbook.Sheets("Export")
i = 1
 
Do While data.Cells(i, 12) <> ""
 
If data.Cells(i, 12) = "RESTREINT" Then
data.Cells(i, 11) = "00 Destinataires"
ElseIf data.Cells(i, 12) = "INTERNE" Then
data.Cells(i, 11) = "03-Tous les utilisateurs EDF de l'ECM + non EDF autorisés par UNIE"
End If
i = i + 1
 
Loop
 
' Export SDIN
'
 
' Entité Emettrice Principale
 
   Sheets("Feuil2").Select
    Range("N2:N1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("B2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Référence Principale
 
   Sheets("Feuil2").Select
    Range("S2:S1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("C2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Indice Référence Principale
 
   Sheets("Feuil2").Select
    Range("T2:T1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("D2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Titre
 
   Sheets("Feuil2").Select
    Range("R2:R1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("E2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Rédacteurs
 
   Sheets("Feuil2").Select
    Range("Q2:Q1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("F2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Nature du Document
 
 
    Sheets("Export").Select
    Range("G3:G1002") = "NOTE"
 
' Catégorie du document
 
   Sheets("Export").Select
    Range("H3:H1002") = "MULTI-CATEGORIES"
 
' Code du produit type
 
   Sheets("Export").Select
    Range("I3:I1002") = "NT"
 
' Entité Documentaire
 
   Sheets("Export").Select
    Range("J3:J1002").FormulaR1C1 = "ING/UNIE"
 
 
' Accessibilité
 
   Sheets("Feuil2").Select
    Range("X2:X1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("L2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Entité Fonctionnelle Responsable (EFR)
 
   Sheets("Feuil2").Select
    Range("I2:I1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("M2").Select
    ActiveCell.Offset(1, 0).Range("A1").Select
    Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    :=False, Transpose:=False
 
' Entité Fonctionnelle Responsable (EFR)
 
Dim feuille1 As Worksheet
Dim feuille2 As Worksheet
Dim Yfeuille1 As Integer
Dim Yfeuille2 As Integer
 
 
Set feuille1 = Workbooks(1).Worksheets("Feuil2")
Set feuille2 = Workbooks(1).Worksheets("Export")
Yfeuille1 = 2
Yfeuille2 = 3
While feuille1.Cells(Yfeuille1, 9).Value <> ""
feuille2.Cells(Yfeuille2, 13).Value = "ING/UNIE/" & feuille1.Cells(Yfeuille1, 9).Value
Yfeuille1 = Yfeuille1 + 1
Yfeuille2 = Yfeuille2 + 1
Wend
 
 
' Pays de l'Entité Documentaire Émettrice
 
   Sheets("Export").Select
    Range("N3:N1002") = "FRA"
 
' Etat du cycle de vie
 
   Sheets("Export").Select
    Range("O3:O1002") = "APPROUVE"
 
' Format
 
    Sheets("Feuil2").Select
    Range("AL2:AL1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("P3: P1001").Select
    ActiveSheet.Paste
 
 
' Durée de conservation
 
  Sheets("Export").Select
    Range("S3:S1002") = "100"
 
' Lieu de rangement
 
    Sheets("Export").Select
    Range("T3:T1002") = "UNIE-LOCAL"
 
' Type d'envoi du document à archiver
 
   Sheets("Export").Select
    Range("Y3:Y1002") = "ELECTRONIQUE"
 
'Relève de l'archive intermédiaire
 
     ThisWorkbook.Sheets("Export").Range("DB3:DB1002").FormulaLocal = "TRUE"
 
 
'Numéro de contenant temporaire (bure-local)
 
    Sheets("Feuil2").Select
    Range("G2:G1000").Select
    Range(Selection, Selection.End(xlDown)).Select
    Selection.Copy
 
    Sheets("Export").Select
    Range("V3:V1001").Select
    ActiveSheet.Paste
 
 
 
' Palier
 
   'Sheets("Feuil2").Select
    'Range("W2:W1000").Select
    'Range(Selection, Selection.End(xlDown)).Select
    'Selection.Copy
 
    'Sheets("Export").Select
    'Range("W2").Select
    'ActiveCell.Offset(1, 0).Range("A1").Select
    'Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
    ':=False, Transpose:=False
 
' Présence de couleur
 
Sheets("Feuil2").Select
Range("AA2:AA1000").Select
Range(Selection, Selection.End(xlDown)).Select
Selection.Copy
 
Sheets("Export").Select
Range("X2").Select
ActiveCell.Offset(1, 0).Range("A1").Select
Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
 
Dim sh As Worksheet
Dim Y As Integer
Dim X As Integer
 
Set sh = ActiveSheet
Y = 3
X = 24
While sh.Cells(Y, X).Value <> ""
If sh.Cells(Y, X).Value = "N&B" Then sh.Cells(Y, X).FormulaLocal = "FALSE"
If sh.Cells(Y, X).Value = "couleur" Then sh.Cells(Y, X).FormulaLocal = "TRUE"
Y = Y + 1
Wend
 
 
End Sub

Donc voilà, si quelqu'un peut y jeter un oeil, même partiellement ce serait déjà très gentil de votre part !

16/12/2014, 10h55
Kimy_Ire
Salut EnzoExcel,

Je vais peut-être te démoraliser, mais pour moi, utiliser les "select" est à proscrire.
Il y a toujours un moyen de ne pas les utiliser.

Les deux cas d'utilisation sont :
1. pour les macros simples (quelques lignes)
2. pour faire des vérifications en "live" - lors d'un débugage, par exemple
Bref, sans le fonctionnel de ton fichier, peu de personnes seront en mesure de "vérifier" ton code comme tu dis.

N’hésite pas à poster un fichier avec des données d'exemple, ça sera plus simple.

Cordialement,
Kimy

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
Option Explicit
' Modèle de sécurité
Sub Macro1()
Dim data As Worksheet
Dim i As Integer
Set data = ThisWorkbook.Sheets("Export")
i = 1
Do While data.Cells(i, 12) <> ""
    Select Case data.Cells(i, 12)
        Case "RESTREINT"
            data.Cells(i, 11) = "00 Destinataires"
        Case "INTERNE"
            data.Cells(i, 11) = "03-Tous les utilisateurs EDF de l'ECM + non EDF autorisés par UNIE"
        End Select
    i = i + 1
Loop
' Export SDIN
' Entité Emettrice Principale
Sheets("Feuil2").Range(Sheets("Feuil2").Range("N2"), Sheets("Feuil2").Range("N" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("B2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Référence Principale
Sheets("Feuil2").Range(Sheets("Feuil2").Range("S2"), Sheets("Feuil2").Range("S" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("C2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Indice Référence Principale
Sheets("Feuil2").Range(Sheets("Feuil2").Range("T2"), Sheets("Feuil2").Range("T" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("D2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Titre
Sheets("Feuil2").Range(Sheets("Feuil2").Range("R2"), Sheets("Feuil2").Range("R" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("E2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Rédacteurs
Sheets("Feuil2").Range(Sheets("Feuil2").Range("Q2"), Sheets("Feuil2").Range("Q" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("F2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Nature du Document
Sheets("Export").Range("G3:G1002") = "NOTE"
' Catégorie du document
Sheets("Export").Range("H3:H1002") = "MULTI-CATEGORIES"
' Code du produit type
Sheets("Export").Range("I3:I1002") = "NT"
' Entité Documentaire
Sheets("Export").Range("J3:J1002").FormulaR1C1 = "ING/UNIE"
' Accessibilité
Sheets("Feuil2").Range(Sheets("Feuil2").Range("X2"), Sheets("Feuil2").Range("X" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("L2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Entité Fonctionnelle Responsable (EFR)
Sheets("Feuil2").Range(Sheets("Feuil2").Range("I2"), Sheets("Feuil2").Range("I" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("M2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
' Entité Fonctionnelle Responsable (EFR)
Dim feuille1 As Worksheet
Dim feuille2 As Worksheet
Dim Yfeuille1 As Integer
Dim Yfeuille2 As Integer
Set feuille1 = Workbooks(1).Worksheets("Feuil2")
Set feuille2 = Workbooks(1).Worksheets("Export")
Yfeuille1 = 2
Yfeuille2 = 3
While feuille1.Cells(Yfeuille1, 9).Value <> ""
    feuille2.Cells(Yfeuille2, 13).Value = "ING/UNIE/" & feuille1.Cells(Yfeuille1, 9).Value
    Yfeuille1 = Yfeuille1 + 1
    Yfeuille2 = Yfeuille2 + 1
Wend
' Pays de l'Entité Documentaire Émettrice
Sheets("Export").Range("N3:N1002") = "FRA"
' Etat du cycle de vie
Sheets("Export").Range("O3:O1002") = "APPROUVE"
' Format
Sheets("Feuil2").Range("AL2:AL1000").Range(Selection, Selection.End(xlDown)).Copy
Sheets("Export").Range("P3: P1001").ActiveSheet.Paste
' Durée de conservation
Sheets("Export").Range("S3:S1002") = "100"
' Lieu de rangement
Sheets("Export").Range("T3:T1002") = "UNIE-LOCAL"
' Type d'envoi du document à archiver
Sheets("Export").Range("Y3:Y1002") = "ELECTRONIQUE"
'Relève de l'archive intermédiaire
ThisWorkbook.Sheets("Export").Range("DB3:DB1002").FormulaLocal = "TRUE"
'Numéro de contenant temporaire (bure-local)
Sheets("Feuil2").Range("G2:G1000").Range(Selection, Selection.End(xlDown)).Copy
Sheets("Export").Range("V3:V1001").ActiveSheet.Paste
Sheets("Feuil2").Range(Sheets("Feuil2").Range("AA2"), Sheets("Feuil2").Range("AA" & Cells.Rows.Count).End(xlUp)).Copy
Sheets("Export").Range("D2").Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
Sheets("Export").Range("X2").ActiveCell.Offset(1, 0).Range("A1").PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
:=False, Transpose:=False
Dim sh As Worksheet
Dim Y As Integer
Dim X As Integer
Set sh = ActiveSheet
Y = 3
X = 24
While sh.Cells(Y, X).Value <> ""
    If sh.Cells(Y, X).Value = "N&B" Then sh.Cells(Y, X).FormulaLocal = "FALSE"
    If sh.Cells(Y, X).Value = "couleur" Then sh.Cells(Y, X).FormulaLocal = "TRUE"
    Y = Y + 1
Wend
End Sub