IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Macros et VBA Excel Discussion :

Code prend 1 heure pour afficher les resultats


Sujet :

Macros et VBA Excel

  1. #1
    Futur Membre du Club Avatar de Mateos_hiss
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2016
    Messages
    37
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Var (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant
    Secteur : High Tech - Matériel informatique

    Informations forums :
    Inscription : Mars 2016
    Messages : 37
    Points : 9
    Points
    9
    Par défaut Code prend 1 heure pour afficher les resultats
    Bonjour a tous ,

    je viens de terminer un code qui contient 13 sub routine je leur fait appel par un module voila le code prend 1 heure pour analyser 16000 lignes en gros je fait une comparaison ligne par ligne de 4 colonnes et autre 4 colonnes et j'affiche les resultats dans autres colonnes voila le ficher sur le quelle je travaille compar-test.xlsm et voila le code des 13 sub si vous pouvez m'aider a trouver une solution et merci :

    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
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    224
    225
    226
    227
    228
    229
    230
    231
    232
    233
    234
    235
    236
    237
    238
    239
    240
    241
    242
    243
    244
    245
    246
    247
    248
    249
    250
    251
    252
    253
    254
    255
    256
    257
    258
    259
    260
    261
    262
    263
    264
    265
    266
    267
    268
    269
    270
    271
    272
    273
    274
    275
    276
    277
    278
    279
    280
    281
    282
    283
    284
    285
    286
    287
    288
    289
    290
    291
    292
    293
    294
    295
    296
    297
    298
    299
    300
    301
    302
    303
    304
    305
    306
    307
    308
    309
    310
    311
    312
    313
    314
    315
    316
    317
    318
    319
    320
    321
    322
    323
    324
    325
    326
    327
    328
    329
    330
    331
    332
    333
    334
    335
    336
    337
    338
    339
    340
    341
    342
    343
    344
    345
    346
    347
    348
    349
    350
    351
    352
    353
    354
    355
    356
    357
    358
    359
    360
    361
    362
    363
    364
    365
    366
    367
    368
    369
    370
    371
    372
    373
    374
    375
    376
    377
    378
    379
    380
    381
    382
    383
    384
    385
    386
    Sub ComparisonRank4()
    Dim LastNRow, LastPRow, LastRRow, LastXRow, rwR, rwRf, rwX, rwXf As Long
     
    Application.ScreenUpdating = True
    Application.ScreenUpdating = False
    LastGRow = Cells(Rows.Count, "G").End(xlUp).Row
    Range("N3:Q" & LastGRow + 1).ClearContents
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
     
     
    'GOAL 1:  DETERMINE IF THERE WAS A CHANGE IN RANK
     
     
    'Scenario 1: Change in rank and features -
     
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
    Range("R1:AE" & LastGRow).ClearContents
    Call deleteR1dups
    Call deleteR2dups
    Call DeleteDupsInR1andR2
     
    'CHECK RANK
    For rwX = 3 To LastXRow
        For rwR = 3 To LastRRow
            LastNRow = Cells(Rows.Count, "N").End(xlUp).Row
            LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
            If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "AB") = Cells(rwR, "V") Then GoTo rwx_NEXT   'Same rank so skip
            If Cells(rwX, "X") <> Cells(rwR, "R") Then GoTo rwx_NEXT     'Not the same Site so skip
            If Cells(rwX, "X") = Cells(LastNRow, "N") Then GoTo rwx_NEXT  'Same Site as Previous Site in column N, so skip
            If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "AB") <> Cells(rwR, "V") Then   'Yes Rank has changed so continue
                Cells(LastPRow + 1, "N") = Cells(rwX, "X")
                Cells(LastPRow + 1, "O") = Cells(rwX, "Y")
                Cells(LastPRow + 1, "P") = "Rank"
                pc = Application.Round((Cells(rwX, "AA") / Cells(rwR, "U") - 1) * 100, 2)
                Cells(LastPRow + 1, "Q") = "New: " & Cells(rwX, "AB") & ", " & " Old: " & Cells(rwR, "V") & ", PNR % Change: " & pc
            End If
     
    'CHECK FEATURES
            If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "T") = Cells(rwR, "Z") Then
     
            Call FindNEWFeatures1
            Call FindOLDFeatures1
            Cells(LastPRow + 2, "P") = "Features"
            Cells(LastPRow + 2, "Q") = "New: " & Cells(LastPRow + 1, "AD") & "Old: " & Cells(LastPRow + 1, "AE")
            If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "T") <> Cells(rwR, "Z") Then GoTo rwx_NEXT
            End If
    rwx_NEXT:
        Next rwR
    Next rwX
     
    Range("AD3:AE" & LastXRow).ClearContents
    Application.ScreenUpdating = True
     
    End Sub
     
     
    Sub ComparisonFeatures4()
    Dim LastNRow, LastPRow, LastRRow, LastXRow, rwR, rwRf, rwX, rwXf As Long
    'Scenario 3: No change in rank, but change in features
    'If there is a change in features, show what was deleted and what was added...don't show any features if no features changed
     
    Application.ScreenUpdating = False
    Call CheckIfNew
    Call FindNEWFeatures
    Call CheckIfOld
    Call FindOLDFeatures
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
    Range("R1:AE" & LastXRow).ClearContents
    '****************************
     
    'Scenario 4: No change in rank OR features -        Do NOT show in results - do NOTHING
     
     
    'Delete R to V if Column R value =  existing values in column N
     
    LastNRow = Cells(Rows.Count, "N").End(xlUp).Row
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rwN = 3 To LastNRow
        For rwR1 = 3 To LastRRow
             If Cells(rwN, "N") <> "" And Cells(rwR1, "R") <> "" And Cells(rwN, "N") = Cells(rwR1, "R") Then
                  Range("R" & rwR1 & ":V" & rwR1).ClearContents
             End If
        Next rwR1
    Next rwN
     
    For rwN = 3 To LastNRow
        For rwX1 = 3 To LastXRow
             If Cells(rwN, "N") <> "" And Cells(rwX1, "X") <> "" And Cells(rwN, "N") = Cells(rwX1, "X") Then
                  Range("X" & rwX1 & ":AB" & rwX1).ClearContents
             End If
        Next rwX1
    Next rwN
    Application.ScreenUpdating = True
     
    End Sub
     
     
    Sub FeaturesNoRankChange4()
    Dim LastNRow, LastPRow, LastRRow, LastXRow, rwR, rwRf, rwX, rwXf As Long
    'List CHANGED FEATURES if rank unchanged
    Application.ScreenUpdating = False
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rwXf = 3 To LastXRow
        For rwRf = 3 To LastRRow
            LastNRow = Cells(Rows.Count, "N").End(xlUp).Row
            LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
     
            If Cells(rwRf, "R") = Cells(LastNRow, "N") Then GoTo rw2_NEXT
     
            If Cells(rwRf, "R") = "" Then GoTo rw2_NEXT
            If Cells(rwXf, "X") = "" Then GoTo rwr_NEXT
            If Cells(rwXf, "X") = Cells(rwRf, "R") Then
                Cells(LastPRow + 1, "N") = Cells(rwXf, "X")
                Cells(LastPRow + 1, "O") = Cells(rwXf, "Y")
                Cells(LastPRow + 1, "P") = "Features"
                Call FindNEWFeatures1
                Call FindOLDFeatures1
                Cells(LastPRow + 1, "Q") = "New: " & Cells(LastPRow + 1, "AD") & "Old: " & Cells(LastPRow + 1, "AE")
                If rwRf = LastRRow And rwXf = LastXRow Then GoTo PASSEM
            End If
     
    rw2_NEXT:
        Next rwRf
    rwr_NEXT:
    Next rwXf
     
    PASSEM:
    Range("R1:AE" & LastXRow).ClearContents
    Application.ScreenUpdating = True
    End Sub
     
     
    Sub deleteR1dups()
     
    Dim dic As Object, a As Range, i As Long
    Set a = Range("A1").CurrentRegion
    Set dic = CreateObject("scripting.dictionary")
     
        For i = 1 To a.Rows.Count
            dic((a(i, 1).Value & Chr(2) & a(i, 3).Value)) = a.Rows(i)
        Next i
     
    With Application
    Range("R1").Resize(dic.Count, 5) = .Transpose(.Transpose(dic.items))
    End With
     
    End Sub
     
     
    Sub deleteR2dups()
     
    Dim dic As Object, a As Range, i As Long
    Set a = Range("G1").CurrentRegion
    Set dic = CreateObject("scripting.dictionary")
     
        For i = 1 To a.Rows.Count
            dic((a(i, 1).Value & Chr(2) & a(i, 3).Value)) = a.Rows(i)
        Next i
     
    With Application
    Range("X1").Resize(dic.Count, 5) = .Transpose(.Transpose(dic.items))
    End With
     
    End Sub
     
     
    Sub DeleteR1dupsInR2()
    Dim rwR, rwX, LastRRow, LastXRow As Long
     
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
        For rwR = 3 To LastRRow
            For rwX = 3 To LastXRow
                If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "Z") = Cells(rwR, "T") Then
                    Range("X" & rwX & ":AB" & rwX).ClearContents
                End If
            Next rwX
        Next rwR
    End Sub
     
     
    Sub DeleteDupsInR1andR2()
    Dim rwR, rwX, LastRRow, LastXRow As Long
     
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
        For rwR = 3 To LastRRow
            For rwX = 3 To LastXRow
                If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "Z") = Cells(rwR, "T") And Cells(rwX, "v") = Cells(rwR, "ab") Then
                    Range("X" & rwX & ":AB" & rwX).ClearContents
                    Range("R" & rwR & ":V" & rwR).ClearContents
                End If
            Next rwX
        Next rwR
    End Sub
     
     
    Sub FindNEWFeatures1()
    'List New Features Scenario 1
    Dim Row As Integer
    Dim rw, LastXRow, LastORow, LastPRow As Long
    Dim Site As String
     
    On Error GoTo ErrHandler
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
    LastORow = Cells(Rows.Count, "O").End(xlUp).Row
    Site = Cells(LastORow, "O")
    Cells(LastPRow + 1, "AD").ClearContents
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rw = 3 To LastXRow
        If Cells(rw, "X") = Site Then
          Cells(LastPRow, "AD") = Cells(LastPRow, "AD") & Cells(rw, "Z") & ", "
        End If
    Next rw
     
    ErrHandler:
     
    Exit Sub
    End Sub
     
     
    Sub FindOLDFeatures1()
    'List OLD Features Scenario 1
    Dim rwR, LastRRow, LastPRow As Long
    Dim Site As String
     
    On Error GoTo ErrHandler
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
     
    Site = Cells(LastPRow, "O")
    Cells(LastPRow, "AE").ClearContents
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
     
    For rwR = 3 To LastRRow
        If Cells(rwR, "R") = Site Then
          Cells(LastPRow, "AE") = Cells(LastPRow, "AE") & Cells(rwR, "T") & ", "
        End If
    Next rwR
     
    ErrHandler:
     
    Exit Sub
     
    End Sub
     
     
    Sub CheckIfNew()
    Dim cnt As Integer
    Dim rw, rwX, LastPRow, LastRRow, LastXRow, LastNRow As Long
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rw = 3 To LastRRow
        cnt = 0
        For rwX = 3 To LastXRow
            If Cells(rw, "R") = Cells(rwX, "X") Then cnt = cnt + 1
        Next rwX
            If cnt >= 1 Then
                GoTo NOT_NEW
            Else
                LastNRow = Cells(Rows.Count, "N").End(xlUp).Row
                rwX = LastXRow
     
                If Cells(LastNRow, "N") = Cells(rwX, "X") Then GoTo NOT_NEW
     
                Cells(LastPRow + 1, "N") = Cells(rwX, "X")
                Cells(LastPRow + 1, "O") = Cells(rwX, "Y")
                Cells(LastPRow + 1, "P") = "New com"
                'Call FindNewFeatures
                'Cells(LastNRow + 1, "Q") = Cells(LastNRow + 1, "R")
                'Range("R" & LastNRow + 1).ClearContents
     
                '***************Inserts 'Last Rank' if Site is only in Release 1
                 'Cells(LastNRow + 1, "Q") = Cells(LastNRow + 1, "Q") & "Last Rank: " & Cells(rw, "E")
                '***************
             End If
     
    NOT_NEW:
    Next rw
    End Sub
     
     
    Sub FindNEWFeatures()
    Dim Row As Integer
    Dim rw, LastXRow, LastORow, LastPRow As Long
    Dim Site As String
     
    On Error GoTo ErrHandler
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
    LastORow = Cells(Rows.Count, "O").End(xlUp).Row
    Site = Cells(LastORow, "O")
    Cells(LastPRow, "AD") = ""
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rw = 3 To LastXRow
        If Cells(rw, "X") = Site Then
          Cells(LastPRow, "AD") = Cells(LastPRow, "AD") & Cells(rw, "Z") & ", "
        End If
    Next rw
     
    Cells(LastPRow, "Q") = "New Features: " & Cells(LastPRow, "AD") & " Rank: " & Cells(rw - 1, "AB")
     
    'Range("R" & LastPRow & ":S" & LastPRow).ClearContents
     
    ErrHandler:
     
    Exit Sub
    End Sub
     
     
    Sub CheckIfOld()
    Dim cnt As Integer
    Dim rwR, rwX, LastRRow, LastXRow, LastNRow As Long
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
    LastXRow = Cells(Rows.Count, "X").End(xlUp).Row
     
    For rwR = 3 To LastRRow
        cnt = 0
        For rwX = 3 To LastXRow
            If Cells(rwR, "R") = Cells(rwX, "X") Then cnt = cnt + 1
        Next rwX
            If cnt >= 1 Then
                GoTo NOT_DISCONTINUED
            Else
                rwX = LastXRow
                LastNRow = Cells(Rows.Count, "N").End(xlUp).Row
     
                If Cells(rwR, "R") = Cells(LastNRow, "N") Then GoTo NOT_DISCONTINUED
     
                Cells(LastNRow + 1, "N") = Cells(rwR, "R")
                Cells(LastNRow + 1, "O") = Cells(rwR, "S")
                Cells(LastNRow + 1, "P") = "Deleted com"
     
                'Call FindOLDFeatures2
                'Cells(LastNRow + 1, "Q") = Cells(LastNRow + 1, "R")
                'Range("R" & LastNRow + 1).ClearContents
     
                '***************Inserts 'Last Rank' if Site is only in Release 1
                ' Cells(LastNRow + 1, "Q") = Cells(LastNRow + 1, "Q") & "Last Rank: " & Cells(rw, "E")
                '***************
             End If
     
    NOT_DISCONTINUED:
    Next rwR
    End Sub
     
     
    Sub FindOLDFeatures()
    Dim rwR, LastRRow, LastPRow As Long
    Dim Site As String
     
    On Error GoTo ErrHandler
     
    LastPRow = Cells(Rows.Count, "P").End(xlUp).Row
     
    Site = Cells(LastPRow, "O")
    Cells(LastPRow, "AD").ClearContents
    LastRRow = Cells(Rows.Count, "R").End(xlUp).Row
     
    For rwR = 3 To LastRRow
        If Cells(rwR, "R") = Site Then
          Cells(LastPRow, "AD") = Cells(LastPRow, "AD") & Cells(rwR, "T") & ", "
        End If
    Next rwR
    rwR = LastRRow
    Cells(LastPRow, "Q") = "Old Features: " & Cells(LastPRow, "AD") & "Rank: " & Cells(rwR, "V")
     
    ErrHandler:
     
    Exit Sub
     
    End Sub

  2. #2
    Expert confirmé
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Mai 2013
    Messages
    3 617
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : Canada

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : Alimentation

    Informations forums :
    Inscription : Mai 2013
    Messages : 3 617
    Points : 5 912
    Points
    5 912
    Par défaut
    Bonjour,

    Je n'ai pas tout regardé le code...
    Premièrement, cette ligne n'a que rwXf qui est Long, les autres sont Variant puisque tu ne les a pas dimensionnées
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Dim LastNRow, LastPRow, LastRRow, LastXRow, rwR, rwRf, rwX, rwXf As Long
    équivaut à Exit For, de ce que j'ai vu.

    Pour accélérer, il te faudrait mettre les plages à vérifier dans des tableaux, boucler ces tableaux en mémoire et inscrire les résultats à la fin.
    Finalement, s'il y a des formules dans ta ou tes feuilles, met en mode calcul manuel avant de lancer la macro.
    MPi²

  3. #3
    Membre habitué
    Homme Profil pro
    Étudiant
    Inscrit en
    Janvier 2015
    Messages
    128
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Nord (Nord Pas de Calais)

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Janvier 2015
    Messages : 128
    Points : 188
    Points
    188
    Par défaut
    Bonjour,

    Dans ton code, il y a beaucoup de choses à dire

    Mais j'en citerai que 2:

    1) Quand tu fais
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Dim LastNRow, LastPRow, LastRRow, LastXRow, rwR, rwRf, rwX, rwXf As Long
    seul rwXf est de type Long, les autres sont de type Variant

    2) A chaque fois tu fais des accessions aux cellules
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "AB") = Cells(rwR, "V") Then GoTo rwx_NEXT   'Same rank so skip
            If Cells(rwX, "X") <> Cells(rwR, "R") Then GoTo rwx_NEXT     'Not the same Site so skip
            If Cells(rwX, "X") = Cells(LastNRow, "N") Then GoTo rwx_NEXT  'Same Site as Previous Site in column N, so skip
            If Cells(rwX, "X") = Cells(rwR, "R") And Cells(rwX, "AB") <> Cells(rwR, "V") Then   'Yes Rank has changed so continue
                Cells(LastPRow + 1, "N") = Cells(rwX, "X")
                Cells(LastPRow + 1, "O") = Cells(rwX, "Y")
                Cells(LastPRow + 1, "P") = "Rank"
                pc = Application.Round((Cells(rwX, "AA") / Cells(rwR, "U") - 1) * 100, 2)
                Cells(LastPRow + 1, "Q") = "New: " & Cells(rwX, "AB") & ", " & " Old: " & Cells(rwR, "V") & ", PNR % Change: " & pc
            End If
    C'est super long de faire ça.

    Regarde sur google: "comment maximiser le temps d'execution d'une macro", il y a déjà pleins de forum qui en parlent !

    Slooby

  4. #4
    Invité
    Invité(e)
    Par défaut
    Bonsoir,
    Quand on a un programme avec 13 sub pour gérer 16000 ligne Excel,c'est pas évidant de comprendre ce que fait le code et quel est son utilité

    Surtout que peut-être tu t'es fourvoyé et qu'une solution différente te ferait gagner un temps précieux.

    16000 lignes par exemple ce n'est pas quelques chose que on saisi en 1 heure. J'imagine que les données sont déjà là a l'ouverture du fichier.

    Sur des fichier volumineux je fais de requête Sql, mais peut-être que ce n'est pas possibles.

    Tu supprime des valeur, du moins c'est ce que tu dis dans ton message. Ça prend du temps. Ne peux tu pas faire un filtre élaboré qui envoi le résultat de ce que tu veux garder dans un onglet temporaire? Ca c'est rapide.

  5. #5
    Futur Membre du Club Avatar de Mateos_hiss
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2016
    Messages
    37
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Var (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant
    Secteur : High Tech - Matériel informatique

    Informations forums :
    Inscription : Mars 2016
    Messages : 37
    Points : 9
    Points
    9
    Par défaut
    Bonsoir rdurupt

    oui exactement je fais une extraction de SQL Server et je filtre en utilisant un code vba aussi le code en general il fait les taches suivantes : j'ai deux release a comparer les meme type de donnees le coresitecode et le site code referent a une communautees les features referent a des fonctionnalitees utilisees par ces communautees total_PNRs ce sont les reservations de chaque communautes et le rank id c est le classement je compare les deux release pour savoir les changement suivants j'affiche le coresite code et le sitecode a coté le changement et apres les details de ce changement .

    les changement que je traite sont les suivant :

    un changement de rank ==> details : le nouveau et l ancien rank et le poucentage de changement
    un changement de features ==> details : nouvelle features , et features supprimes
    une nouvelle communaute ==> detail : nouvelle features et son rank
    une communautes supprime ==> details : ancien features et ancien rank .
    un changement de rank et feature en meme temps ==> detail: un changement de rank ==> details : le nouveau et l ancien rank et le poucentage de changement un changement de features ==> details : nouvelle features , et features supprimes

  6. #6
    Nouveau membre du Club
    Homme Profil pro
    Étudiant
    Inscrit en
    Décembre 2014
    Messages
    45
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Décembre 2014
    Messages : 45
    Points : 34
    Points
    34
    Par défaut Quelques conseil
    Salut,
    Si je peux te donner quelques conseil supplémentaires par rapport aux autre déjà fait, j'ai vraiment jette un coup d’œil à ton code.

    1) Évite les déclarations dans les boucles du genre lastRow =. Tu cherches ta derniére ligne au tout début et ensuite tu rajoutes un compteur a chaque fois que tu ajoutes une ligne...

    2) Ne travaille pas sur tes cellules Excel, Utilises des tableaux dynamique c'est largement plus rapide:

    Chargement du tableau par rapport a ta feuil:

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    Dim data()
    data = ws.Range("A2:AD" & ligne).Value
    '--- et pour recoller
    ws.Range("A2:AB" & UBound(tableau)).Value = tableau
    3) Essaie de ne pas réfléchir aux étapes que tu ferais a la main, mais de faire du code... Avec les tableaux dynamiques tu perds la visions feuil excel.

    Bon courage, C'est largement possible,
    Aujourd'hui je pousse en base de donnée 4 millions de données en 2h30 dans un simple serveur Sql.

  7. #7
    Invité
    Invité(e)
    Par défaut
    Bonjour,
    Tu extrais des données d’une base de données Sql Serveur, OK.
    Dans ton poste, tu parles de supprimer des colonnes, la meilleur façon de supprimer des colonne, c’est de ne pas le sélectionner !
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="Select Champ1,Champ2 From Table
    tu parles également d’ajouter des colonnes.
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="Select Champ1,Champ2,0 as NewChampNum, '' as NewChampText, Now as NewChampDate From Table
    Tu parles de supprimer des valeurs, idem, il suffit de ne pas les sélectionner !
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="Select Champ1,Champ2,0 as NewChampNum, '' as NewChampText, Now as NewChampDate From Table where Champ1='toto'
    Je peux lier le résultat de cette requête à un autre table en faisant des requêtes imbriquées !
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="select * from (Select Champ1,Champ2,0 as NewChampNum, '' as NewChampText, Now as NewChampDate From Table where Champ1='toto') as frm inner join table2 on table2.id=frm.id

    Je ne dis pas qu’il est possible de résoudre entièrement ton problème Par Sql, mais tu peux sens doute qualifier l’ensemble de tes données avant le traitement par macro !

  8. #8
    Futur Membre du Club Avatar de Mateos_hiss
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2016
    Messages
    37
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Var (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant
    Secteur : High Tech - Matériel informatique

    Informations forums :
    Inscription : Mars 2016
    Messages : 37
    Points : 9
    Points
    9
    Par défaut
    Citation Envoyé par loic3484 Voir le message
    Salut,
    Si je peux te donner quelques conseil supplémentaires par rapport aux autre déjà fait, j'ai vraiment jette un coup d’œil à ton code.

    1) Évite les déclarations dans les boucles du genre lastRow =. Tu cherches ta derniére ligne au tout début et ensuite tu rajoutes un compteur a chaque fois que tu ajoutes une ligne...

    2) Ne travaille pas sur tes cellules Excel, Utilises des tableaux dynamique c'est largement plus rapide:

    Chargement du tableau par rapport a ta feuil:

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    Dim data()
    data = ws.Range("A2:AD" & ligne).Value
    '--- et pour recoller
    ws.Range("A2:AB" & UBound(tableau)).Value = tableau
    3) Essaie de ne pas réfléchir aux étapes que tu ferais a la main, mais de faire du code... Avec les tableaux dynamiques tu perds la visions feuil excel.

    Bon courage, C'est largement possible,
    Aujourd'hui je pousse en base de donnée 4 millions de données en 2h30 dans un simple serveur Sql.
    Bonjour loic 8434 ,
    oui je sais bien que je dois utilisé des tableaux ou bien les dictionnaire , mais je suis pas assez competant :/

  9. #9
    Invité
    Invité(e)
    Par défaut
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="select frm.Champ1,frm.Champ2, format(table2.ChampNum1 / table2.ChampNum2,'#%') as Pourcentage  from (Select Champ1,Champ2,From Table where Champ1='toto') as frm inner join table2 on table2.id=frm.id
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql=" select  frm2.Champ1,frm2.Champ2, format(frm2.ChampNum1 / tablfrm2e2.ChampNum2,'#%') as Pourcentage   from ( select frm.Champ1,frm.Champ2frm.Champ1,frm.Champ2, sum(table2.ChampNum1) as TotalNum1 , sum(table2.ChampNum2) as TotalNum2 from (Select Champ1,Champ2,From Table where Champ1='toto') as frm inner join table2 on table2.id=frm.id group by  frm.Champ1,frm.Champ2) as frm2

  10. #10
    Futur Membre du Club Avatar de Mateos_hiss
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2016
    Messages
    37
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Var (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant
    Secteur : High Tech - Matériel informatique

    Informations forums :
    Inscription : Mars 2016
    Messages : 37
    Points : 9
    Points
    9
    Par défaut
    Citation Envoyé par rdurupt Voir le message
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Sql="select Champ1,Champ2, format(table2.ChampNum1 / table2.ChampNum2,'#%') as Pourcentage  from (Select Champ1,Champ2,From Table where Champ1='toto') as frm inner join table2 on table2.id=frm.id
    Bonjour rdurupt,
    enfaite pour ma requette sql je fais que selecty et je laisse le filtrage a excel , mais pour l extraction je pense pas que c'est le probleme plutot le code vba enfaite j'ai commencé un petit peu avec les dictionnaires mais encors j'arrive pas a avoir le resultats exact de ce que je veux

  11. #11
    Nouveau membre du Club
    Homme Profil pro
    Étudiant
    Inscrit en
    Décembre 2014
    Messages
    45
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Décembre 2014
    Messages : 45
    Points : 34
    Points
    34
    Par défaut
    Citation Envoyé par Mateos_hiss Voir le message
    Bonjour loic 8434 ,
    oui je sais bien que je dois utilisé des tableaux ou bien les dictionnaire , mais je suis pas assez competant :/
    Essaie, Et on t'aidera justement,
    rdurupt dit vrai : Le plus léger n'est de sélectionner que ce que tu as besoin...

    Encore une fois essaie de ne pas faire les étapes manuelles dans ta tête, pour définir ton code.

  12. #12
    Invité
    Invité(e)
    Par défaut
    Moi par exemple, je traite un volume de données Excel supérieur, de plus repérées dans un centaine de fichiers et de répertoires. Là il ne s'agit pas de base de données mais purement de données Excel.

    J’importe toutes les données dans mon fichier Excel Via requête SQL Excel, et je qualifie mes données via SQL. je dois même éclater le contenue de certaines cellules en plusieurs ligne. Le traitement initial durait 3 heures, et mon intervention était justement d'optimiser le temps de traitement. Maintenant il dure 3 minutes. Je ne dis pas que c'est ce genre de solution qu'il te convient d'opter, mais seulement d'y penser!

    1. Import Dans Excel
    2. Sauvegarde du fichier
    3. Connexion ADODB sur ThisWorkbook
    4. Traitement sql
    5. Traitement VBA



    Il faut bien comprendre que le traitement de masse est plus rapide qu'un traitement vba ligne par ligne cellule par cellule.

    Un filtre élaboré par exemple filtrera plus vite qu'une suppression d lignes!
    Dernière modification par Invité ; 17/06/2016 à 10h38.

  13. #13
    Futur Membre du Club Avatar de Mateos_hiss
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2016
    Messages
    37
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Var (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Étudiant
    Secteur : High Tech - Matériel informatique

    Informations forums :
    Inscription : Mars 2016
    Messages : 37
    Points : 9
    Points
    9
    Par défaut
    merci rdurupt pour le conseil le sql et encore dur pour moi hahah mais je vais prendre en concideration , pour moi tout se passe sur excel pour le moment le code que j ai fornis il met pas une communaute en reference du coup a chaque fois qu il la trouve c est une nouvelle fois !! comment puis je faire poue eviter ca ?

Discussions similaires

  1. code php pour afficher les dernier article
    Par sousse92 dans le forum EDI, CMS, Outils, Scripts et API
    Réponses: 2
    Dernier message: 04/03/2013, 23h40
  2. [VxiR2] Difficultés pour afficher les bons resultat
    Par coolmek dans le forum Webi
    Réponses: 4
    Dernier message: 30/04/2012, 13h25
  3. [AC-2003] Code pour afficher les utilisateurs connectés à une BDD
    Par LionelDM dans le forum IHM
    Réponses: 2
    Dernier message: 07/01/2010, 17h04
  4. Afficher les resultat d'un code (matrice) dans l'interface
    Par Alucard9800XT dans le forum Interfaces Graphiques
    Réponses: 3
    Dernier message: 06/06/2007, 09h31
  5. afficher les resultats d'une requete mysql sur 3 colonnes
    Par harlock59 dans le forum Requêtes
    Réponses: 7
    Dernier message: 24/12/2005, 14h38

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo