XL 2019 Comment optimiser ce code ?

I

Ismagapi

XLDnaute Junior
Bonjour,

Je souhaiterai optimiser le temps de fonctionnement de cette macro comment puis je le faire ?

En sachant que dans ma logique, je vais chercher des feuilles dans d'autres classeurs, je les renomme, et ensuite je vais chercher des données afin de recopier les lignes qui contiennent ces données dans une autre feuille.

Merci au forumeurs qui voudront bien m'aider s'il vous plait.


VB: 
Sub CopierFeuilleDuClasseurFermé()
Application.ScreenUpdating = False
Application.DisplayAlerts = False
'USFWait.Show 0
  'USFWait.Repaint

Call efface

Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire ANZIN 1.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE ANZIN 1"
Columns("D:D").Delete Shift:=xlToLeft
''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire ANZIN 2.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE ANZIN 2"
'''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire CONDE 1.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE CONDE 1"
Range("A1").EntireColumn.Insert
Range("A1").EntireColumn.Insert
''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire CONDE 2.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE CONDE 2"
Range("A1").EntireColumn.Insert
Range("A1").EntireColumn.Insert
'''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire VALENCIENNES 1.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE VALENCIENNES 1"
Columns("E:E").Delete Shift:=xlToLeft
''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire VALENCIENNES 2.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE VALENCIENNES 2"
Columns("C:C").Delete Shift:=xlToLeft
'''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire ONNAING.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE ONNAING"
''''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire ST AMAND.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE ST AMAND"
'''''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire DENAIN BOUCHAIN 1.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE DENAIN BOUCHAIN 1"
''''''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire DENAIN BOUCHAIN 2.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE DENAIN BOUCHAIN 2"
'''''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire DENAIN LOURCHES 1.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE DENAIN LOURCHES 1"
''''''''''
Set classeurFermé = Workbooks.Open("C:\Users\salva\OneDrive\Bureau\Isa\Tableau gestionnaire DENAIN LOURCHES 2.xlsx", 0, True)
    classeurFermé.Sheets("ASE").Copy Before:=ThisWorkbook.Sheets(1)
    classeurFermé.Close SaveChanges:=False
ActiveSheet.Name = "ASE DENAIN LOURCHES 2"

Call CopyRows_Garde

Sheets("LISTE").Activate
Range("A:D,K:N,P:P,V:W,AC:AG,AJ:AN,BC:BD").Select
Range("BD1").Activate
Selection.Delete Shift:=xlToLeft

Sheets("LISTE").Activate
Range("A2:BD3000").Copy Worksheets("ASE").Range("A2")

Sheets("ASE").Activate
With Range("A2:BD3000")
    .HorizontalAlignment = xlHAlignCenter
    .VerticalAlignment = xlVAlignCenter
Call aa2
End With

Call retirer

Sheets("LANCEMENT").Select

'Unload USFWait


Application.DisplayAlerts = True
Application.ScreenUpdating = True

End Sub
Sub retirer()
    Dim ws As Worksheet
    For Each ws In Worksheets
    'Application.DisplayAlerts = False
        If ws.Name <> "ASE" And ws.Name <> "LANCEMENT" And ws.Name <> "AEMO, AEMO+TDC, AEMO-R" Then ws.Delete
    Next
    'Application.DisplayAlerts = True
End Sub

Public Sub CopyRows_Garde()
Dim xWs As Worksheet
Dim xCWs As Worksheet
Dim xRg As Range
Dim xStrName As String
Dim xRStr As String
Dim xAStr As String
Dim xPStr As String
Dim xRRg As Range
Dim xC As Integer
On Error Resume Next
'Application.DisplayAlerts = False
xStr = "LISTE"
xJtr = "LISTE AEMO"
xBtr = "AEMO, AEMO+TDC, AEMO-R"
xCtr = "LISTE ASE"
xRStr = "GARDE"
xAStr = "AP"
xPStr = "Pupille"
xMStr = "AP Mère/Enfant"
xDStr = "DAP"
xTStr = "Tutelle"
xCStr = ""
Set xCWs = ActiveWorkbook.Worksheets.Item(xStr)
If Not xCWs Is Nothing Then
    xCWs.Delete
End If
Set xCWs = ActiveWorkbook.Worksheets.Add
xCWs.Name = xStr
xC = 1
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xRStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xAStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xPStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xMStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xDStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xTStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
For Each xWs In ActiveWorkbook.Worksheets
    If xWs.Name <> "xStr" And xWs.Name <> "xJtr" And xWs.Name <> "xBtr" And xWs.Name <> "xCtr" Then
        Set xRg = xWs.Range("S:S")
        Set xRg = Intersect(xRg, xWs.UsedRange)
        For Each xRRg In xRg
            If xRRg.Value = xCStr Then
               xRRg.EntireRow.Copy
               xCWs.Cells(xC, 1).PasteSpecial xlPasteValuesAndNumberFormats
               xC = xC + 1
            End If
        Next xRRg
    End If
Next xWs
'Application.DisplayAlerts = True
End Sub
Sub efface()
Worksheets("ASE").Range("A2:BZ3000").Clear
End Sub
Sub aa2()
Dim i&
For i& = 2 To 2000 'ligne 2 à ligne 21
  With Application.WorksheetFunction
   Range("E" & i&) = .RoundDown(.YearFrac(Now, Range("C" & i&)), 0)
  End With
Next i&
End Sub
 
Dernière édition:
Trier par date Trier par votes
patricktoulon

patricktoulon

XLDnaute Barbatruc
Bonjour

1° met ton code dans une balise code (dans ton post que l'on se fasse pas un tour de rein de neurone) a tout lire
2° intéresse toi aux formule de liaisons + value=.value (à minima)
3° sinon tu a Adobd.connection
4° tu a aussi power query(c''est pas mon fort mais d'autre sont des pointures dans le domaines)

5° pour finir oublie les workbook.open blablabla c''est de la daube et c'est cela qui te ralenti le schmilblick
@+
 
Vote positif 0 Vote négatif
I

Ismagapi

XLDnaute Junior
Staple1600 à dit:
Bonjour le fil

Ma part du colibri ;)
VB: 
Sub aa2_bis()
With Cells(2, "E").Resize(Cells(Rows.Count, 3).End(3).Row - 1)
    .Formula = "=ROUND(YEARFRAC(NOW(),RC[-2]),0)"
    .Value = .Value
End With
End Sub
Cliquez pour agrandir...
Bonjour Staple1600, merci beaucoup de vous intéresser à mon code.
J'ai l'impression que la macro est lente pour executer le code Public Sub CopyRows_Garde.
J'ai un peu fait une usine à gaz pour faire une recherche de plusieurs mots dans une colonne S et copier la ligne s'il rencontre l'un de ses mots dans un autre onglet de récupération de données.
Cette boucle doit se faire pour chaque onglet ouvert précédemment en excluant 3 ou 4 feuilles pour ne pas faire de doublons.
Est ce que j'applique la bonne méthode ?
Mes connaissances en VBA reste limitées mais je peux me débrouiller.
Merci de votre patience ...
 
Vote positif 0 Vote négatif

Discussions similaires

A
  • Question
XL 2021 Comment créer un code VBA afin de création un chemin d'accès universel
Réponses
8
Affichages
1 K
AugusteP
A
J
  • Question
XL 2010 Récupérer les données de plusieurs fichiers pour un nom de feuille donné
Réponses
10
Affichages
2 K
JVOS
J
H
  • Question
XL 2010 svp comment modifier ce code vba??
Réponses
13
Affichages
1 K
job75
job75
G
  • Question
Comment optimiser ce code?
Réponses
2
Affichages
1 K
Grek
G
M
[REF] Wiki 1 de MichelXld (Généralités Excel)
Réponses
6
Affichages
59 K
Le_Phénomène
Le_Phénomène

Membres actuellement en ligne

7 membres et 82 invités.

Partager cette page

Tutos vidéos

Power Query : Ajouter des requêtes (Excel et Power BI)
Créez votre première fonction LAMBDA dans Excel en 3 étapes!
LAMBDA, l’arme secrète de Microsoft pour Excel !
Office Insider - Découvrez les nouveautés Excel avant tout le monde

Statistiques des forums

Discussions
314 588
Messages
2 110 988
Membres
111 002
dernier inscrit
Lolo73i