Optimiser une macro

Résolu
Bonjour,

Je souhaiterai optimiser une macro qui met un peu de temps.
Aujourd'hui je n'ai que 1100 lignes mais j'en aurai plus.

LgMax1 = Range("A" & Rows.Count).End(xlUp).Row
For LgAna = 2 To LgMax1
For LgAnab = 2 To LgMax1
' Un doublon est trouvé dans une ligne
If wsFicAna.Range("D" & LgAna).Value = _
wsFicAna.Range("D" & LgAnab).Value And LgAnab <> LgAna _
And wsFicAna.Range("D" & LgAna).Value <> "" Then

wsFicAna.Range("B" & LgAna).Value = "UP"

Exit For
End If
Next
Next

Merci

7 réponses

  1. Bonjour,

    Essayes cette Macros, elles transfère toutes les données de ton tableau dans une Table Virtuel et traite le tout virtuellement.

    Une fois tout les résultat obtenu

    elle vient agrémenterr ta colonne B sur la feuille

    LgMax1 = Cells(Rows.Count, 1).End(xlUp).Row
    CoMax1 = Cells(1, Columns.Count).End(xlToLeft).Column

    ReDim TBL_Original(LgMax1, ColMax1)

    For i = 2 To LgMax1
    For u = 1 To CoMax1
    TBL_Original(i, u) = wsFicAna.Cells(i, u).Value
    Next u
    Next i

    For LgAna = 2 To LgMax1
    For LgAnab = 2 To LgMax1
    ' Un doublon est trouvé dans une ligne
    If TBL_Original(LgAna, 4) = TBL_Original(LgAnab, 4) _
    And LgAnab <> LgAna _
    And TBL_Original(LgAna, 4) <> "" Then
    TBL_Original(LgAna, 2) = "UP"
    Exit For
    End If
    Next
    Next

    For i = 2 To LgMax1
    wsFicAna.Cells(i, 2).Value = TBL_Original(i, 2)
    Next i
    1
    1. Contributeur
      Bonjour,

      tu devrais déposer un fichier représentatif avec qcq données.

      Déposer le fichier xls (réduit au nécessaire et anonymisé, avec les explications et éventuellement le résultat attendu) sur cjoint.com et coller ici le lien fourni.

      eric

      0
      1. En fait je suis en train de travailler sur un projet de mise à jour de données.

        J'ai :
        - un premier fichier Anlyse_données_V7 en xls avec trois onglets : Accueil, AnalyseData, UpdateData et InsertData.
        - deux listes de données Export_SIG.csv (master : CSV1) et Export_CS.csv (slave : CSV2).

        Le but est :
        - d'ouvrir mon fichier analyse en premier où je vais lancer l'ouverture de mes deux fichiers csv.
        - de faire un mapping des deux fichiers csv
        - de copier toutes les lignes csv1 qui existent mais en doublons ou bien qui n'existent dans csv2 au niveau de l'onglet AnalyseData.
        - de rechercher des doublons dans cet onglet AnalyseData et à chaque fois que je trouve un doublons il me met "UP" de update à la colonne B ou laisser vide si pas de doublon donc à insérer.
        - de recopier les ligne avec "UP" dans l'onglet UpdateData, celles qui sont vides dans InsertData.

        ça fonctionne comme je souhaite. Mais un peu long alors qu'il n'y a que 1100 lignes

        Vous trouverai ci-dessous le lien des fichiers.

        http://cjoint.com/?3Gzm6b9FGIU

        Merci
        0
    2. Bonjour,

          Dim Plage As Range, Cel As Range
          Set Plage = wsFicAna.Range("D2:D" & wsFicAna.Range("A" & Rows.Count).End(xlUp).Row)
          Application.ScreenUpdating = False
          For Each Cel In Plage
              If Application.CountIf(Plage, Cel.Value) > 1 And Cel.Value <> "" Then Cel.Offset(0, -2).Value = "UP"
          Next Cel
          Set Plage = Nothing


      A+
      0
      1. Contributeur
        Bonjour,

        Lire et écrire des cellules est très lent, il faut que tu travailles en mémoire.
        Je n'ai pas trop le temps de faire ton fichier mais je t'ai fait un exemple.
        Si c'est toi qui a fait ton code je ne doute pas que tu comprendras très vite le principe :
        charger dans des variables toutes les données en une lecture, et travailler avec les variables.
        Pareil pour l'écriture : tout écrire en une fois.
        100 fois plus rapide au minimum.
        https://www.cjoint.com/c/CGzoFLCVLSZ

        eric
        0
        1. Merci à tous je vais tester et je vous tiendrai au courant.
          0
          1. Merci encore, ça fonctionne
            0
        2. Bonjour,

          Je reviens vers vous pour un petit coup de main.
          Car j'ai réussi à optimiser toutes les autres macro.
          Mais il y'a une que je n'arrive pas à faire. Elle ralentit considérablement le traitement de mes données.

          Merci d'avance

          LgMax = Application.WorksheetFunction.Max(LgLigA, LgLigB)
          CoMax = Cells(1, Columns.Count).End(xlToLeft).Column
          For lgLig = 2 To LgMax
          ' Colonnes : D à AO
          For lgCol = 2 To CoMax
          ' Une différence est trouvée dans une ligne
          If wsFicA.Cells(lgLig, lgCol).Value <> wsFicB.Cells(lgLig, lgCol).Value Then
          ' Affichage du nom du fichier en colonne A
          wsFicAna.Range("A" & lgLigDeb).Value = wbFicA.Name
          ' Copier la ligne du fichier A dans le fichier d'analyse
          wsFicA.Range("A" & lgLig & ":" & "AX" & lgLig).Copy _
          Destination:=wsFicAna.Range("C" & lgLigDeb)

          ' Affichage du nom du fichier en colonne A
          wsFicAna.Range("A" & lgLigDeb + 1).Value = wbFicB.Name
          ' Copier la ligne du fichier B dans le fichier d'analyse
          wsFicB.Range("A" & lgLig & ":" & "AX" & lgLig).Copy _
          Destination:=wsFicAna.Range("C" & lgLigDeb + 1)

          lgLigDeb = lgLigDeb + 2

          Exit For
          End If
          Next lgCol
          Next lgLig
          0
          1. Bonjour,

            Y'a-t-il quelqu'un qui pourra m'aider à optimiser cette macro?

            Merci d'avance
            0
            1. Contributeur
              Bonjour,

              le principe est toujours le même pour accélérer : mettre les données en mémoire.
              eric
              0
            2. Bonjour,

              J'ai essayé de mettre les données en mémoire mais ça ne fonctionne pas.
              Je suis débutant en VBA et c'est grace à votre aide que j'ai pu optimiser les autres macros avec l'aide de table en mémoire.
              Mais avec celle-là, je n'arrive pas du tout.

              Help!!
              0
            3. Contributeur
              sans fichiers réduits à l'essentiel moi non plus.
              eric
              0