Macro qui fonctionne plus...

Ajt67 Messages postés 10 Statut Membre -  
yg_be Messages postés 23437 Date d'inscription   Statut Contributeur Dernière intervention   -
Bonsoir à tous,
Alors voilà, j'ai un classeur Excel qui me pose un problème... surtout depuis les versions Excel 2007 et 2013, j'ai une macro qui ne fonctionne plus. Le principe est simple, j'ai plusieurs onglet, un pour chaque lettre de l’alphabet et chaque onglet contient une liste de nom, un numéro respectif à chaque nom et d'autres informations. Ma macro me permettait, à partir d'une "messagebox" de rechercher le nom en ne connaissant que le numéro, ou rechercher le numéro en ne connaissant que le nom. Voilà la macro que j'utilisai :
"Private Sub CommandButton1_Click()
What = InputBox("Saisissez le nom du bateau à rechercher :")
If What = "" Then Exit Sub
For Each sht In Worksheets
sht.Activate
Set Found = sht.Cells.Find(What)
If Not Found Is Nothing Then
FirstAddress = Found.Address
Do
Found.Activate
Response = MsgBox("Continuer ?", vbYesNo + vbQuestion)
If Response = vbNo Then Exit Sub
Set Found = Cells.FindNext(After:=ActiveCell)
If Found.Address = FirstAddress Then Exit Do
Loop
End If
Next sht
MsgBox "Recherche terminée !"
End Sub"

Pourquoi cela ne fonctionne-t-il plus ?

Merci de votre aide

2 réponses

  1. f894009 Messages postés 17417 Date d'inscription   Statut Membre Dernière intervention   1 717
     
    Bonjour,

    Que vouliez vous faire si trouve car exit do ok, le for each continue………………………...
    0
    1. Ajt67 Messages postés 10 Statut Membre
       
      Bonjour, merci de votre réponse,
      Le principe du fonctionnement de ce bouton était d'afficher une boîte dans laquelle on pouvait soit saisir un numéro, soit un nom et ensuite le code cherchait dans tous les onglets si ce nom ou numéro existe déjà.

      Bonne journée.
      0
  2. yg_be Messages postés 23437 Date d'inscription   Statut Contributeur Dernière intervention   Ambassadeur 1 588
     
    bonjour,
    1° merci d'utiliser la coloration syntaxique quand tu postes du code
    2° "ne fonctionne plus": message d'erreur, comportement différent de celui que tu espères, ...?
    3° ajoute "option explicit" en début de module, et déclare toutes tes variables, par exemple:
    Option Explicit
    ' ...
    Private Sub CommandButton1_Click() 
    Dim What As String, sht As Worksheet, FOund As Range, FirstAddress As String, Response As Integer
    '...
    0
    1. Ajt67 Messages postés 10 Statut Membre
       
      Bonjour, merci de votre réponse.
      Je vais tacher de faire de mon mieux car je suis pas une "foudre" en code, loin de là car je n'ai pour ainsi dire aucune connaissance, ni en Anglais et ni en programmation. Je suis autodidacte, j'ai appris dans mon coin, en lisant et en essayant.
      0
    2. f894009 Messages postés 17417 Date d'inscription   Statut Membre Dernière intervention   1 717
       
      Bonjour,

      Vous ne repondez pas au 2° de Y_be, c'est peut-être la clé du problème.
      Personnellement, je vous informais que votre code, meme si Found est trouve, ne s'arrête que sur le dernier onglet..............
      0
    3. yg_be Messages postés 23437 Date d'inscription   Statut Contributeur Dernière intervention   1 588 > f894009 Messages postés 17417 Date d'inscription   Statut Membre Dernière intervention  
       
      le code parcourt toutes les occurrences dans toutes les feuilles du classeur, et, pour chaque occurrence, propose, via msgbox, d’arrêter ou de continuer.
      "exit sub" termine tout, si l'utilisateur en a décidé ainsi.
      "exit do" termine la boucle d'une feuille, pour passer à la feuille suivante quand on retombe sur la première occurrence trouvée précédemment dans la feuille.
      j'ai testé, tout me semble bien fonctionner.
      0
    4. f894009 Messages postés 17417 Date d'inscription   Statut Membre Dernière intervention   1 717
       
      Salut Y_Be,

      Ben, For Each sht In Worksheets Next sht, le exit Do est a l'interieur donc la recherche, meme si trouvee, va jusqu'au dernier onglet, sauf erreur de ma part!!
      0
    5. yg_be Messages postés 23437 Date d'inscription   Statut Contributeur Dernière intervention   1 588 > Ajt67 Messages postés 10 Statut Membre
       
      quelques mini changements:
      Option Explicit
      
      Private Sub CommandButton1_Click()
      Dim What As String, sht As Worksheet, FOund As Range, FirstAddress As String, Response As Integer
      What = InputBox("Saisissez le nom du bateau à rechercher :")
      If What = "" Then Exit Sub
      For Each sht In Worksheets
          Set FOund = sht.Cells.Find(What)
          If Not FOund Is Nothing Then
              sht.Activate
              FirstAddress = FOund.Address
              Do
                  FOund.Activate
                  Response = MsgBox("Continuer ?", vbYesNo + vbQuestion)
                  If Response = vbNo Then Exit Sub
                  Set FOund = sht.Cells.FindNext(After:=ActiveCell)
              Loop Until FOund.Address = FirstAddress
          End If
      Next sht
      MsgBox "Recherche terminée !"
      End Sub
      0