Programme VBA super lent !

Version imprimable

Ce programme s'execute sur une feuille excel de 1500l et 26 colonnes et supprime des caractères speciaux dans 3 colonnes et il est super lent, pc qui rame comme pas possible qd je l execute et j'en vois pas la fin :bug:

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
Private Sub CommandButton1_Click()
    Dim Cell As Variant, i, j, k
    i = 1 ' SP
    j = 1 'FITCH
    k = 1 'MOODY
 
    Sheets("BBG").Select
    Do While Cells(1, i).Value <> "RTG_SP"
    i = i + 1
    Loop
    Cells(1, i).Select
    Range(Selection, Selection.End(xlDown)).Select
    For Each Cell In Selection
        Cell.Value = Replace(Cell.Value, "/*+", "")
        Cell.Value = Replace(Cell.Value, "/*-", "")
        Cell.Value = Replace(Cell.Value, "/*", "")
        Cell.Value = Replace(Cell.Value, "#N/A N.A.", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A N Ap", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A Sec", "NR")
        Cell.Value = Replace(Cell.Value, "e", "")
        Cell.Value = Replace(Cell.Value, "(P)", "")
        Cell.Value = Replace(Cell.Value, " ", "")
    Next Cell
 
    Do While Cells(1, j).Value <> "RTG_FITCH"
    j = j + 1
    Loop
    Cells(1, j).Select
    Range(Selection, Selection.End(xlDown)).Select
    For Each Cell In Selection
        Cell.Value = Replace(Cell.Value, "/*+", "")
        Cell.Value = Replace(Cell.Value, "/*-", "")
        Cell.Value = Replace(Cell.Value, "/*", "")
        Cell.Value = Replace(Cell.Value, "#N/A N.A.", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A N Ap", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A Sec", "NR")
        Cell.Value = Replace(Cell.Value, "e", "")
        Cell.Value = Replace(Cell.Value, "(P)", "")
        Cell.Value = Replace(Cell.Value, " ", "")
    Next Cell
    Do While Cells(1, k).Value <> "RTG_MOODY"
    k = k + 1
    Loop
    Cells(1, k).Select
    Range(Selection, Selection.End(xlDown)).Select
    For Each Cell In Selection
        Cell.Value = Replace(Cell.Value, "/*+", "")
        Cell.Value = Replace(Cell.Value, "/*-", "")
        Cell.Value = Replace(Cell.Value, "/*", "")
        Cell.Value = Replace(Cell.Value, "#N/A N.A.", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A N Ap", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A Sec", "NR")
        Cell.Value = Replace(Cell.Value, "e", "")
        Cell.Value = Replace(Cell.Value, "(P)", "")
        Cell.Value = Replace(Cell.Value, " ", "")
    Next Cell
End Sub

Qu est ce que je pourrais faire pour ameliorer ce code ?

03/12/2007, 12h10
ouskel'n'or

Citation:

Envoyé par solal57

Ce programme s'execute sur une feuille excel de 1500l et 26 colonnes et supprime des caractères speciaux dans 3 colonnes et il est super lent, pc qui rame comme pas possible qd je l execute et j'en vois pas la fin :bug:
Qu est ce que je pourrais faire pour ameliorer ce code ?

Tu supprimes toutes tes selections,
Ensuite en début de macro tu mets déjà

Code:

Application.screenUpdating = False

et en fin de macro

Code:

Application.screenUpdating = True

et tu nous dis ce que ça donne ;)
A+
03/12/2007, 12h14
Michel_M
Bonjour,

1/Essaies déjà de figer l'écran
application.screenupdating=false
m^me punition en fin de macro mais avec True

2/ Evite les "select" très très lents...
au lieu de:
Code:

1 2 Cells(1, i).Select Range(Selection, Selection.End(xlDown)).Select
essaies
derlig=cells(i,1).xl(down).row
for each cell in range(cells(1,i),cells(derling,i))
...

3/ pas trop le temps de fouiner mais il y a des boucles qui semblent inuile
03/12/2007, 12h16
Michel_M

Et bing! une bosse!

Bonjour, Ouskelnord.

cordialement
03/12/2007, 12h25
ouskel'n'or

Bonjour Michel, content que tu ré-apparaisses ;)

Tu peux rassembler tes 3 boucles en 1 seules en mettant des OR ou AND.
J'ai rajouter les modifications données par ouskel'n'or et Michel_M

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
 
Private Sub CommandButton1_Click()
    Dim Cell As Variant, i
    Dim valeur As String
    i = 1
 
    Application.ScreenUpdating = False
 
    Sheets("BBG").Select
 
    valeur = Cells(1, i).Value
 
    Do While valeur <> "RTG_SP" Or valeur <> "RTG_FITCH" Or valeur <> "RTG_MOODY"
    i = i + 1
    Loop
 
    derlig = Cells(i, 1).xl(down).Row
    For Each Cell In Range(Cells(1, i), Cells(derling, i))
        Cell.Value = Replace(Cell.Value, "/*+", "")
        Cell.Value = Replace(Cell.Value, "/*-", "")
        Cell.Value = Replace(Cell.Value, "/*", "")
        Cell.Value = Replace(Cell.Value, "#N/A N.A.", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A N Ap", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A Sec", "NR")
        Cell.Value = Replace(Cell.Value, "e", "")
        Cell.Value = Replace(Cell.Value, "(P)", "")
        Cell.Value = Replace(Cell.Value, " ", "")
    Next Cell
 
    Application.ScreenUpdating = True
 
End Sub

Je n'ai pas tester

03/12/2007, 12h34
scully2501

Ma boucle n'ai pas bonne. Je n'avais pas bien lu tes commentaires sur le fait que tes 3 valeurs sont sur 3 colonnes différentes mais tu peux quand même améliorer tes boucles et créer une fonction pour remplacer les caractere

Je viens de remarquer qu'il n'y a pas de fin préciser donc la boucle risque de tourner en boucle indéfiniment.

Rajoute une condition avec une fin genre quand que fin de ligne <> de 1500.
03/12/2007, 13h38
solal57

Ok merci je vais tester ca ;)
Je vous dirais si ca marche!

Bizzare ca marche pas :/

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
Private Sub CommandButton1_Click()
    Dim Cell As Variant, i, derlig
   i = 1
 
    Application.ScreenUpdating = False
 
    Sheets("BBG").Select
 
    Do While Cells(1, i).Value <> "RTG_SP"
    i = i + 1
    Loop
 
    Cells(1,i).select
    derlig = Range(Cells(1,i), Cells(derlig, i + 2)).Select
    Range(Cells(1, i), Cells(derlig, i + 2)).Select
    For Each Cell In Selection
        Cell.Value = Replace(Cell.Value, "/*+", "")
        Cell.Value = Replace(Cell.Value, "/*-", "")
        Cell.Value = Replace(Cell.Value, "/*", "")
        Cell.Value = Replace(Cell.Value, "#N/A N.A.", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A N Ap", "NR")
        Cell.Value = Replace(Cell.Value, "#N/A Sec", "NR")
        Cell.Value = Replace(Cell.Value, "e", "")
        Cell.Value = Replace(Cell.Value, "(P)", "")
        Cell.Value = Replace(Cell.Value, " ", "")
    Next Cell
 
    Application.ScreenUpdating = True
 
End Sub

j'ai mis i+2 car les 3 colonnes sont tjrs les unes apres les autres.

03/12/2007, 17h20
scully2501
C'est normal que ca ne marche pas. Tu utilise derlig sans qu'il ne soit alimenter:
Code:

1 2 derlig = Range(Cells(1,i), Cells(derlig, i + 2)).Select
en claire derlig = derlig donc ca plante.

Ca ne serait pas plutot ainsi:
Code:

1 2 derlig = Range(Cells(1,i), Cells(1, i + 2)).Select
03/12/2007, 17h37
solal57

Exact, j'avais pas fait gaffe ! Merci !
Bon la ca tourne depuis un moment wait & see !