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 :

Macro lente a optimiser


Sujet :

Macros et VBA Excel

  1. #1
    Membre du Club
    Inscrit en
    Novembre 2007
    Messages
    107
    Détails du profil
    Informations forums :
    Inscription : Novembre 2007
    Messages : 107
    Points : 45
    Points
    45
    Par défaut Macro lente a optimiser
    Bonjour,

    J'ai écris une macro pour effectuer un certain nombre de calcules en se basant sur des parametres contenus sur des feuilles excel, le problème est que le temps de calcule est très long.

    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
     
    Sub DRP()
     
    p = Cells(9, 4)
     
    l = 4
     
    Date1 = Date
     
     
    'une boucle pour commencer à partir de la semaine en cour et sauvegarder l'historique
     
     
    Do While Date1 > Cells(5, l)
    l = l + 1
    Loop
     
    For k = 1 To 100
     
     
    ' on remarque que l'incrémentation commence dès l'index "l"
    If Cells(24 + k * 11, 2) < 26 Then
     
    For D = l To l + p - 1
     
    'condition de point de réappro
     
     
    If Cells(20 + k * 11, D) - Cells(23 + k * 11, D) < Cells(21 + k * 11, D) Then
     
     
    '><<<<<<<
     
    R = Cells(18 + k * 11, D)
     
    ' Ce bloc sert pour le calcul du besoin en fonction de la couverture cible ça rajoute à chaque fois la cellule de prévisions ayant l'index R+D jusqu'a R=0
     
    Do While R > 0
     
    Cells(19 + k * 11, D) = Cells(23 + k * 11, D + R - 1) + Cells(19 + k * 11, D)
     
    R = R - 1
     
    Loop
     
    '>>>>>>>
     
    ' CI après l'équation de stock : Besoin+Stock sécurité-STock début arrondi au multiple du standard palettes
     
    If (Cells(19 + k * 11, D) + Cells(22 + k * 11, D) - Cells(20 + k * 11, D)) > 0 Then
     
    S = Cells(19 + k * 11, D) + Cells(22 + k * 11, D) - Cells(20 + k * 11, D)
     
    Cells(19 + k * 11, D) = Application.WorksheetFunction.MRound(S, Cells(21 + k * 11, 2))
     
    Else
     
    Cells(19 + k * 11, D) = 0
     
     
    '>>>>>>>
     
     
    End If
    Else
     
    Cells(19 + k * 11, D) = 0
    End If
     
     
     'ce bloc rajoute une palette au besoin si le stock tombe en négatif
     
    If Cells(20 + k * 11, D) < 0 Then
     
    Cells(19 + k * 11, D) = Cells(19 + k * 11, D) + Cells(21 + k * 11, 2)
     
    End If
    '>>>>>>>>>>>>>>>>
    Cells(17, D) = Cells(17, D) + Cells(20 + k * 11, D) / Cells(21 + k * 11, 2)
    Cells(20, D) = Cells(20, D) + Cells(19 + k * 11, D) / Cells(21 + k * 11, 2)
     
    Next
     
     
    ' Calcul de la couverture glissante a chaque fois le stock est suppérieur aux prévs on compte 6 jours et on soustrait les prév d'une semaine
     
    For i = l To l + p
     
     
    x = Cells(20 + k * 11, i)
    J = i
     
    If x > 0 Then
     
    Do While x > Cells(23 + k * 11, J)
     
    x = x - Cells(23 + k * 11, J)
     
    Cells(24 + k * 11, i) = Cells(24 + k * 11, i) + 6
     
    J = J + 1
     
    Loop
     
     
    '>>>>>>>
     
    ' pour les semaines non totalement couverte on calcul on rajoute le # de jours couverts
     
    If Cells(23 + k * 11, J) * 6 > 0 Then
     
     
    Cells(24 + k * 11, i) = Cells(24 + k * 11, i) + x / Cells(23 + k * 11, J) * 6
     
    Else: Cells(24 + k * 11, i) = ND
     
    End If
    '>>>>>>>>>>>>>>>>>
     
     
    Else: Cells(24 + k * 11, i) = 0
     
     
     
    End If
    '
     
     
    Next
     
    End If
     
    Next
     
     
    End Sub

  2. #2
    Membre expérimenté Avatar de Transitoire
    Homme Profil pro
    Auditeur informatique
    Inscrit en
    Décembre 2017
    Messages
    724
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 69
    Localisation : France, Alpes Maritimes (Provence Alpes Côte d'Azur)

    Informations professionnelles :
    Activité : Auditeur informatique

    Informations forums :
    Inscription : Décembre 2017
    Messages : 724
    Points : 1 454
    Points
    1 454
    Par défaut
    Bonjour, essayez déja avec ça, ça empêche le rafraichissement de l'écran

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    Sub DRP()
    Application.ScreenUpdating = False
    '       votre code
    Application.ScreenUpdating = True
    End sub

    Cordialement
    On a deux vies, la deuxième commence quand on se rend compte qu'on n'en a qu'une.
    Confucius

  3. #3
    Responsable
    Office & Excel


    Homme Profil pro
    Formateur et développeur chez EXCELLEZ.net
    Inscrit en
    Novembre 2003
    Messages
    19 122
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : Belgique

    Informations professionnelles :
    Activité : Formateur et développeur chez EXCELLEZ.net
    Secteur : Enseignement

    Informations forums :
    Inscription : Novembre 2003
    Messages : 19 122
    Points : 55 921
    Points
    55 921
    Billets dans le blog
    131
    Par défaut
    Salut.

    Il est souvent plus intéressant de réfléchir en Excel plutôt qu'en VBA, et c'est valable pour le traitement que tu souhaites réaliser.

    Si tu devais le réaliser en Excel sans programmation, tu utiliserais une formule que tu recopierais sur la colonne à traiter. Cette démarche, tu peux l'appliquer en VBA.

    Dans l'exemple suivant, j'ai un tableau structuré de données et je souhaite, en VBA, majorer le prix pour les dates ultérieures à la date d'aujourd'hui en utilisant une plage nommée pour calculer la majoration.

    Nom : 2018-04-15_163250.png
Affichages : 170
Taille : 27,0 Ko


    En Excel, j'utiliserais la formule suivante, puis éventuellement, je réaliserais un collage spécial valeur pour figer les prix majorés:

    Nom : 2018-04-15_163336.png
Affichages : 128
Taille : 26,4 Ko


    En VBA, je vais réaliser la même opération, sans boucle et donc bien plus rapidement:
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
     
    Sub Majoration()
      Range("tableau1[prix majoré]").Formula = "=IF([@Date]>TODAY(),[@Prix]*(1+Majoration),[@Prix])" ' Formule en anglais sur toute la colonne
      Range("tableau1[prix majoré]").Copy ' Copie de la colonne
      Range("tableau1[prix majoré]").PasteSpecial xlPasteValues ' Collage spécial valeurs
      Application.CutCopyMode = False ' Restauration du mode "Prêt" d'Excel
    End Sub
    Voir ma signature: VBA pour Excel? Pensez D'ABORD en EXCEL avant de penser en VBA...
    "Plus les hommes seront éclairés, plus ils seront libres" (Voltaire)
    ---------------
    Mes billets de blog sur DVP
    Mes remarques et critiques sont purement techniques. Ne les prenez jamais pour des attaques personnelles...
    Pensez à utiliser les tableaux structurés. Ils vous simplifieront la vie, tant en Excel qu'en VBA ==> mon tuto
    Le VBA ne palliera jamais une mauvaise conception de classeur ou un manque de connaissances des outils natifs d'Excel...
    Ce ne sont pas des bonnes pratiques parce que ce sont les miennes, ce sont les miennes parce que ce sont des bonnes pratiques
    VBA pour Excel? Pensez D'ABORD en EXCEL avant de penser en VBA...
    ---------------

  4. #4
    Membre expert
    Profil pro
    Inscrit en
    Février 2007
    Messages
    2 267
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Février 2007
    Messages : 2 267
    Points : 3 663
    Points
    3 663
    Par défaut
    Bonjour,

    tant qu'on y est :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Range("tableau1[prix majoré]").Value=Range("tableau1[prix majoré]").Value
    à la place des 3 dernières lignes ;-)
    eric

  5. #5
    Responsable
    Office & Excel


    Homme Profil pro
    Formateur et développeur chez EXCELLEZ.net
    Inscrit en
    Novembre 2003
    Messages
    19 122
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : Belgique

    Informations professionnelles :
    Activité : Formateur et développeur chez EXCELLEZ.net
    Secteur : Enseignement

    Informations forums :
    Inscription : Novembre 2003
    Messages : 19 122
    Points : 55 921
    Points
    55 921
    Billets dans le blog
    131
    Par défaut
    Citation Envoyé par eriiic Voir le message
    [...]
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    Range("tableau1[prix majoré]").Value=Range("tableau1[prix majoré]").Value
    [...]
    Arf. J'y pense jamais...

    Merci Eric
    "Plus les hommes seront éclairés, plus ils seront libres" (Voltaire)
    ---------------
    Mes billets de blog sur DVP
    Mes remarques et critiques sont purement techniques. Ne les prenez jamais pour des attaques personnelles...
    Pensez à utiliser les tableaux structurés. Ils vous simplifieront la vie, tant en Excel qu'en VBA ==> mon tuto
    Le VBA ne palliera jamais une mauvaise conception de classeur ou un manque de connaissances des outils natifs d'Excel...
    Ce ne sont pas des bonnes pratiques parce que ce sont les miennes, ce sont les miennes parce que ce sont des bonnes pratiques
    VBA pour Excel? Pensez D'ABORD en EXCEL avant de penser en VBA...
    ---------------

Discussions similaires

  1. Optimisation d'une macro lente
    Par Zebulon777 dans le forum Macros et VBA Excel
    Réponses: 8
    Dernier message: 11/12/2012, 22h35
  2. Macro lente pour contrôle nombre caractere
    Par roidurif dans le forum Macros et VBA Excel
    Réponses: 7
    Dernier message: 23/04/2009, 20h43
  3. Très lent comment optimiser svp ?
    Par dev7 dans le forum PostgreSQL
    Réponses: 1
    Dernier message: 02/06/2006, 12h16
  4. [VBA-E]macro lente. étrange. étrange.
    Par zenix dans le forum Macros et VBA Excel
    Réponses: 5
    Dernier message: 22/04/2006, 11h59

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