Ceci est une page optimisée pour les mobiles. Cliquez sur ce texte pour afficher la vraie page.

XL 2019 Comment optimiser ce code ?

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:

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

Ismagapi

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

Staple1600

XLDnaute Barbatruc
Re

Comme je l'ai dit, j'ai fait la part du colibri
(C'est à dire que j'ai simplement simplifiée une seule procédure, la dernière, celle qui se nomme aa2
(Avec ma version, on évite de faire une boucle)
 
Les cookies sont requis pour utiliser ce site. Vous devez les accepter pour continuer à utiliser le site. En savoir plus…