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 :

Votre avis sur mon code en VBA ?


Sujet :

Macros et VBA Excel

  1. #1
    Membre averti
    Homme Profil pro
    Paramétreur de progiciels
    Inscrit en
    Octobre 2006
    Messages
    970
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Pas de Calais (Nord Pas de Calais)

    Informations professionnelles :
    Activité : Paramétreur de progiciels

    Informations forums :
    Inscription : Octobre 2006
    Messages : 970
    Points : 381
    Points
    381
    Par défaut Votre avis sur mon code en VBA ?
    Bonjour,

    Je dois parcourir une plage de cellules d'un fichier à la recherche d'une cellule équivalente dans un second fichier afin de retourner le résultat d'une autre cellule. C'est un genre de RECHERCHEV et RECHERCHEH mais en VBA.

    Voici mon code :
    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
    Sub LossTree()
        Dim chemin, fichier, onglet, chemin2, fichier2, onglet2 As String
        Dim colonne, ligne, ligne_debut, ligne_fin, colonne2, colonne2_debut, colonne2_fin, ligne2 As Integer
     
        'On désactive l'affichage et le calcul
        Application.ScreenUpdating = False
        Application.Calculation = xlCalculationManual
     
        'Initialisation des variables
        ligne_debut = 26
        ligne_fin = 1064
        colonne = Range("D1").Column
        chemin = "mon chemin"
        fichier = "mon fichier"
        onglet = "mon onglet"
        '--------------------------------------------------
        colonne2_debut = Range("BT1").Column
        colonne2_fin = Range("FF1").Column
        ligne2 = 10
        chemin2 = "mon chemin"
        fichier2 = "mon fichier"
        onglet2 = "mon onglet"
     
        'Ouverture du fichier
        Workbooks.Open (chemin2 & fichier2)
     
        'On se positionne sur le bon onglet du second fichier
        'Workbooks(fichier2).Worksheets(onglet2).Activate
     
        'On se positionne sur le bon onglet du premier fichier
        'Workbooks(fichier).Worksheets(onglet).Activate
     
        'On parcourt les lignes de la plage de cellules
        For ligne = ligne_debut To ligne_fin Step 2
            'On vérifie si la cellule est non vide
            If Workbooks(fichier).Worksheets(onglet).Cells(ligne, colonne).Value <> "" Then
                'On recherche la correspondance dans l'autre fichier
                For colonne2 = colonne2_debut To colonne2_fin
                    If Workbooks(fichier).Worksheets(onglet).Cells(ligne, colonne).Value = Workbooks(fichier2).Worksheets(onglet2).Cells(ligne2, colonne2).Value Then
                        Workbooks(fichier).Worksheets(onglet).Cells(ligne, Range("CR1").Column).Value = Workbooks(fichier2).Worksheets(onglet2).Cells(110, colonne2).Value
                    End If
                Next colonne2
            End If
        Next ligne
     
        'Fermeture du fichier
        Workbooks(fichier2).Close
     
        'On active l'affichage et le calcul
        Application.ScreenUpdating = True
        Application.Calculation = xlCalculationAutomatic
    End Sub
    Mon code vous semble-t-il optimisé, quel avis avez-vous dessus ?

    Merci,
    ZiP

  2. #2
    Rédacteur/Modérateur

    Avatar de Jean-Philippe André
    Homme Profil pro
    Développeur VBA/C#/VB.Net/Power Platform
    Inscrit en
    Juillet 2007
    Messages
    14 593
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 40
    Localisation : Canada

    Informations professionnelles :
    Activité : Développeur VBA/C#/VB.Net/Power Platform
    Secteur : Finance

    Informations forums :
    Inscription : Juillet 2007
    Messages : 14 593
    Points : 34 256
    Points
    34 256
    Par défaut
    salut,
    pourquoi cherches-tu a reinventer la roue ?
    Cycle de vie d'un bon programme :
    1/ ça fonctionne 2/ ça s'optimise 3/ ça se refactorise

    Pas de question technique par MP, je ne réponds pas

    Mes ouvrages :
    Apprendre à programmer avec Access 2016, Access 2019 et 2021

    Apprendre à programmer avec VBA Excel
    Prise en main de Dynamics 365 Business Central

    Pensez à consulter la FAQ Excel et la FAQ Access

    Derniers tutos
    Excel et les paramètres régionaux
    Les fichiers Excel binaires : xlsb,

    Autres tutos

  3. #3
    Membre averti
    Homme Profil pro
    Paramétreur de progiciels
    Inscrit en
    Octobre 2006
    Messages
    970
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Pas de Calais (Nord Pas de Calais)

    Informations professionnelles :
    Activité : Paramétreur de progiciels

    Informations forums :
    Inscription : Octobre 2006
    Messages : 970
    Points : 381
    Points
    381
    Par défaut
    Bonjour jpcheck,

    Ce n'est pas mon choix mais plutôt une demande qui m'a été faite afin de ne pas utiliser des formules dans les cellules mais une macro...

    Cordialement,
    ZiP

Discussions similaires

  1. Réponses: 0
    Dernier message: 25/02/2012, 16h00
  2. Votre avis sur mon code
    Par Schopenhauer dans le forum Fortran
    Réponses: 4
    Dernier message: 04/05/2011, 15h12
  3. Réponses: 3
    Dernier message: 01/09/2008, 14h43
  4. [FFT] Votre avis sur mon code
    Par deubelte dans le forum C++
    Réponses: 1
    Dernier message: 10/02/2007, 20h14
  5. [Code Prof]votre avis sur un code?
    Par granquet dans le forum Réseau
    Réponses: 6
    Dernier message: 11/04/2006, 20h41

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