Programme VBA super lent !

**solal57** · 03/12/2007, 11h47

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

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
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 ?

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

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

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 :

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

Application.screenUpdating = False

et en fin de macro

Code :

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

Application.screenUpdating = True

et tu nous dis ce que ça donne

A+

**Michel_M** · 03/12/2007, 12h14

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 :

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

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

**Michel_M** · 03/12/2007, 12h16

Et bing! une bosse!

Bonjour, Ouskelnord.

cordialement

**ouskel'n'or** · 03/12/2007, 12h25

Bonjour Michel, content que tu ré-apparaisses

**scully2501** · 03/12/2007, 12h27

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 :

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
 
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

**scully2501** · 03/12/2007, 12h34

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.

**solal57** · 03/12/2007, 13h38

Ok merci je vais tester ca

Je vous dirais si ca marche!

**solal57** · 03/12/2007, 16h29

Bizzare ca marche pas :/

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
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.

**scully2501** · 03/12/2007, 17h20

C'est normal que ca ne marche pas. Tu utilise derlig sans qu'il ne soit alimenter:

Code :

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

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 :

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

1
2
 
     derlig = Range(Cells(1,i), Cells(1, i + 2)).Select

**solal57** · 03/12/2007, 17h37

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

Programme VBA super lent !

Macros et VBA Excel

Discussions similaires

Partager

Partager