Probleme un macro sur excel

Fermé
kayankaiu - 27 oct. 2011 à 13:08
michel_m Messages postés 16602 Date d'inscription lundi 12 septembre 2005 Statut Contributeur Dernière intervention 16 décembre 2023 - 27 oct. 2011 à 17:19
Bonjour,

salut mes amis
voila j'ai un macro qui permet d'échanger un seule mot (qui se répète dans la même colonnes sur plusieurs classeur).
j ai essaye mais ca marche pas il me demande le répertoire mais lorsque j exécute ça ne change rien automatiquement sur tous les classeurs que j ai sur ce répertoire voila le code et si quelque 'un a une réponse n hésitez pas

merci pour vos réponses
Sub ModifierLibelle()

'déclaration des variables
Dim oShell As Object, strFileName As Object
Dim oFolder As Object
Dim oFolderItem As Object
Dim OldLibelle As String, NewLibelle As String, NomClasseur As String
Dim Trouve As Range

'empêche le rafraichissement de l'écran pour gagner en rapidité
Application.ScreenUpdating = False
'remplissage des variables :
'NomClasseur = comme son nom l'indique... le nom de ce classeur
'OldLibelle = comme son nom l'indique, l'ancien libellé (à remplacer)
'NewLibelle = comme son nom l'indique... le nouveau libellé
NomClasseur = Left(ThisWorkbook.Name, Len(ThisWorkbook.Name) - 4)
'on a inscrit l'ancien libellé en D5
OldLibelle = Sheets(1).Range("D5").Value
'on a inscrit le nouveau libellé en D6
NewLibelle = Sheets(1).Range("D6").Value
'test si les libellés (ancien et nouveau) sont bien remplis
If OldLibelle = "" Or NewLibelle = "" Then
'sinon, message à l'utilisateur
MsgBox "Saisie obligatoire en D5 et D6"
'et on ne fait plus rien, on sort de la procédure
Exit Sub
End If

'Ici, partie technique consistant à ouvrir une fenêtre
Set oShell = CreateObject("Shell.Application")
'permettant le choix du répertoire ou se situent les fichiers excel
Set oFolder = oShell.BrowseForFolder(&H0&, "Choisir un répertoire", &H1&, "c:\")
'en cas d'annulation par l'utilisateur
If oFolder Is Nothing Then
'message d'abandon
MsgBox "Abandon opérateur", vbCritical, "Annulation"
Else
'boucle sur tous les fichiers du répertoire
'ne tiens pas compte de l'extension donc sur TOUS les fichiers
For Each strFileName In oFolder.Items
'test si :
'1- le fichier est bien un fichier Excel
'oFolder.GetDetailsOf(strFileName, 2) = "Feuille de calcul Microsoft Excel"
'2- le fichier n'est pas celui qui contient la macro (le mien, ce classeur ci)
'And strFileName <> NomClasseur
If oFolder.GetDetailsOf(strFileName, 2) = "Feuille de calcul Microsoft Excel" And strFileName <> NomClasseur Then
'si c'est tout bon, on ouvre le classeur
Workbooks.Open oFolder.ParentFolder.ParseName(oFolder.Title).Path & "\" & strFileName
'début du bloc "dans le classeur qu'on vient d'ouvrir"
With Workbooks(strFileName & ".xls")
'bloc dans la feuille active (en même temps, il n'y a qu'une feuille par classeur...)
With ActiveSheet
'on cherche dans toutes les cellules l'ancien libellé
Set Trouve = .Cells.Find(OldLibelle)
's'il ne s'y trouve pas on fait rien
If Trouve Is Nothing Then
Else
's'il s'y trouve on remplace par le nouveau libellé dans la cellule
'(Trouve = cellule ou on l'a trouvé)
Trouve.Value = NewLibelle
End If
Set Trouve = Nothing
'fin du bloc feuille
End With
'on enregistre les modifs
.Save
'on ferme
.Close
'fin du bloc classeur
End With
End If
Next
End If
'on rétablit le rafraichissement d'écran
Application.ScreenUpdating = True
End Sub


A voir également:

1 réponse

michel_m Messages postés 16602 Date d'inscription lundi 12 septembre 2005 Statut Contributeur Dernière intervention 16 décembre 2023 3 313
27 oct. 2011 à 15:47
Bonjour,

essaies cette macro
détail: il vaut mieux regrouper les sorties de macro comme les erreurs à la fin du code d'où les "goto" bannis hors des gestions d'erreur

à l'avenir pense à mettre tes codes entre les balises <>(onglet en haut du message)

Sub ModifierLibelle()
'déclaration des variables
Dim ObjShell As Object, strFileName As Object
Dim ObjFolder As Object
Dim OldLibelle As String, NewLibelle As String, NomClasseur As String
Dim Chemin As String, Trouve As String

'empêche le rafraichissement de l'écran pour gagner en rapidité
Application.ScreenUpdating = False
'remplissage des variables :
'NomClasseur = comme son nom l'indique... le nom de ce classeur
'OldLibelle = comme son nom l'indique, l'ancien libellé (à remplacer)
'NewLibelle = comme son nom l'indique... le nouveau libellé
NomClasseur = ThisWorkbook.Name

'on a inscrit l'ancien libellé en D5
OldLibelle = Sheets(1).Range("D5").Value
'on a inscrit le nouveau libellé en D6
NewLibelle = Sheets(1).Range("D6").Value
'test si les libellés (ancien et nouveau) sont bien remplis
If OldLibelle = "" Or NewLibelle = "" Then GoTo saisie

'Ici, partie technique consistant à ouvrir une fenêd'images:"
     Set ObjShell = CreateObject("Shell.Application")
     Set ObjFolder = ObjShell.BrowseForFolder(&H0&, "choisir un dossier", 1)
    'Set objFolder = objShell.BrowseForFolder(&H0&, "Choisir un répertoire", &H1&, "c:\") ton classeur
    If ObjFolder Is Nothing Then GoTo abandon 'Si on sort sans sélection
    Chemin = ObjFolder.ParentFolder.ParseName(ObjFolder.Title).Path & ""
     ChDir Chemin

'parcours du dossier
fich = Dir("*.xls")
While fich <> ""
     If fich <> NomClasseur Then
          Workbooks.Open fich
               Trouve = ActiveSheet.Cells.Find(OldLibelle, , xlValues).Address
               If Trouve <> "" Then
                    Range(Trouve) = NewLibelle
                    With ActiveWorkbook
                         .Save
                         .Close
                    End With
               End If
               
     End If
     fich = Dir
Wend

Exit Sub
'-----------------Gestion des erreurs
saisie:
     MsgBox "Saisie obligatoire en D5 et D6", vbCritical
     Exit Sub
abandon:
     MsgBox "Abandon opérateur", vbCritical, "Annulation"

End Sub
0
bonjour
ça exécute mais il ne remplace pas (il me demande le chemin du dossier )
mais il ne remplace pas le remplacement se fait sur la même colonne dans chaque classeur)
merci une 2eme fois pour la réponse
0
michel_m Messages postés 16602 Date d'inscription lundi 12 septembre 2005 Statut Contributeur Dernière intervention 16 décembre 2023 3 313
27 oct. 2011 à 16:46
Curieux, j'avais essayé chez moi et ça marchait.....
0
voila la situation plus detaille j'ai des classeurs (100) et je veux automatise l outil rechercher replacer par un macro en donnant le mot que je veux changer et le remplacant dans ces 100 classeur tous les classeur sont regroupes dans un seul repertoire avec nom 20111028 20111027.......la colonne cible est G dans tous ces feuilles

merci encore de votre aide
0
michel_m Messages postés 16602 Date d'inscription lundi 12 septembre 2005 Statut Contributeur Dernière intervention 16 décembre 2023 3 313
27 oct. 2011 à 17:04
c'est un autre problème que celui que tu avais posé....

Tu es sûr que c'est définitif?
0
oui tu peux m aider n'est ce pas??
je compte bien sur toi

donc au lieu d ouvrir chaque classeur et changerpar 'rechercher replacer' pour chacune de ces 100 claseurs je veux l automatise pour le faire pour les 100 classeurs.
merci
0