Capacité combobox

Résolu
Bonjour,

J'aurai besoin d'aide svp.
j'ai construit un formulaire avec une cascade de 3 combobox.
le problème c'est que quand je met ma liste d'alimentation des combox qui est assez importante (environ 4000) lignes, le formulaire ne s'ouvre pas et le fichier excel beug.
Par contre quand je met une centaine de ligne, il marche mais prend un laps de temps pour que le formulaire s'affiche.
Le formulaire marche parfaitement bien quand je met une liste de 10 lignes.

Si quelqu'un pourrait m'aider.

Merci par avance .

11 réponses

  1. Parfait ça marche nickel.

    Désolé du retard de ma réponse. Merci pour votre aide.

    Cordialement.
    0
    1. Re,

      Petite erreur c'est cette ligne qui provoque la recursivite et qui est pour moi inutile dans
      Private Sub Alim_Combo(CbxIndex As Integer, Optional Cible As Variant)
      
                  'Obj = Ws.Range("A" & J)
      


      D'ailleurs dans
       Private Sub ComboBox2_Change()
      .Value = Ws.Range("C" & J)
      est aussi "inutile"
      0
      1. Modérateur
        Cette récursivité est un problème facilement gérable.
        Mais, au vu du code "tortueux", le plus simple serait de tout changer...

        Par contre, pour en revenir sur ce seul aspect, j'aime bien utiliser :
        ComboBox.Value = "toto"
        If ComboBox.ListIndex = -1 Then ComboBox.AddItem "toto"

        pour éviter les doublons... C'est tout de même beaucoup moins "onéreux" que d'utiliser dictionary ou collection.
        0
      2. @pijakuRe,

        Je viens de voir qu'il ne faut pas supprimer la ligne dans Alim_combo a cause des doublons et d'accords avec le code pour eviter les dico. En effet vaudrait mieux changer

        Por le moment son fichier modifie (pas vraiment beau, mais ca marche) avec un drapeau pour les alim combo

        https://www.cjoint.com/c/GAAlBRgfROf
        0
    2. Modérateur
      Voici la procédure Alim_Combo :
      'Procédure pour alimenter les ComboBox
      Private Sub Alim_Combo(CbxIndex As Integer, Optional Cible As Variant)
          Dim J As Integer
          Dim Obj As Control
          
          'Définit le ComboBox à remplir
          Set Obj = Me.Controls("ComboBox" & CbxIndex)
          'Supprime les anciennes données
          Obj.Clear
          
          'alimente le Combobox initial (Combobox1)
          If CbxIndex = 1 Then
              'Boucle sur les lignes de la colonne A (à partir de la 2eme ligne)
              For J = 2 To NbLignes
                  Obj = Ws.Range("A" & J)
                  'Remplit le ComboBox sans doublons
                  If Obj.ListIndex = -1 Then Obj.AddItem Ws.Range("A" & J)
              Next J
          Else
              'Alimentation conditionnelle des autres Combobox en fonction de
              'ce qui est sélectionnée dans le contrôle précédent:
              '(La sélection du ComboBox1 définit le contenu du ComboBox2,
              'La sélection du ComboBox2 définit le contenu du ComboBox3 ?etc...)
              For J = 2 To NbLignes
                  If CStr(Ws.Range("A" & J).Offset(0, CbxIndex - 2)) = Cible Then
                      Obj = Ws.Range("A" & J).Offset(0, CbxIndex - 1)
                      If Obj.ListIndex = -1 Then Obj.AddItem Ws.Range("A" & J).Offset(0, CbxIndex - 1)
                  End If
              Next J
         End If
         
         'Enlève la sélection dans le ComboBox
         Obj.ListIndex = -1
      End Sub

      Cette procédure, un peu longuette, fonctionne toutefois très correctement sur 5000 lignes.
      Donc, tu as peut être un souci dans les données de ton fichier.

      Il y a d'autres méthodes de remplissage des combobox en cascade.
      Si tu veux, je peux t'en faire une, fiable, mais qui sera très compliquée pour toi la maintenir...
      0
      1. Salut a toi,

        Cette procédure, un peu longuette
        Oui, mais sauf erreur de ma part (apres tests quand meme), c'est long surtout pour Cb1 et parce que a chaque remplissage il y a recusivite en alternance par les evenements combobox1 et 2 change. C'est ce que je soupconne depuis le debut, entre autre.

        sute:
        C'est cette ligne qui pose probleme dans Alim_combo
        .Value = Ws.Range("C" & J)

        pas utile
        0
    3. Bonjour,

      Impressionnant quand même, j'ai crée un fichier exemple et cela marche nickel. par ailleurs dans mon fichier, ça ne marche pas.
      voici MON FICHIER exemple qui ressemble à mon fichier type.
      https://www.cjoint.com/c/GAAidfbbp5l
      Cordialement
      0
      1. Re,

        MON FICHIER exemple qui ressemble à mon fichier type.
        Doit y avoir une difference entre ces deux fichiers sinon ca marcherait aussi!

        Vous ne recuperez qu'UN seul "joueur", normal meme si c'est le meme pour le "groupe" ?

        Suite:
        Pige, sans doublons
        0
    4. Contributeur
      Depuis le début le problème se situe dans le remplissage de la comboBox.
      Je lui ai donné le code pour le remplissage de nombreuses lignes.
      Mais personne n'a l'air de s'en préoccuper!

      https://forums.commentcamarche.net/forum/affich-34305471-capacite-combobox#3
      0
      1. Bonjour cs_Le Pivert,

        Mais personne n'a l'air de s'en préoccuper!
        Si, mais ce qui manque dans le code que cette personne presente, c'est justement deux routines de remplissage combo ce qui nous permettrait de comparer avec les propositions qui ont ete faites et peut-etre comprendre ce qui plante excel.
        Ensuite, dans
        Private Sub ComboBox2_Change()

        le remplissage de la combobox3 peut etre long vu le code utilise, donc faut voir
        Je lui ai demande un fichier, mais par -10 c'est plus long arriver ....
        0
      2. "Voilà a toi de l'adapter à tes besoins "

        Bonjour Merci, j'ai vu votre réponse. je suis que débutant en Vba. donc je n'ai pas pu adapter votre code.

        Merci
        0
    5. Modérateur
      Bonjour tout le monde,

      1- Utilises plutôt, pour remplir ta ComboBox1, l'événement Activate de ton UserForm.
      Comme ceci :
      Option Explicit
      
      Dim Ws As Worksheet
      Dim NbLignes As Integer
      
      Private Sub UserForm_Initialize()
          'Définit la feuille contenant les données
          Set Ws = Worksheets("Feuil1")
          'Définit le nombre de lignes dans la colonne A
          NbLignes = Ws.Range("A5000").End(xlUp).Row
      End Sub
      Private Sub UserForm_Activate()
          'Remplissage du ComboBox1
          Alim_Combo 1
      End Sub


      2- Donne nous le code de ta procédure Alim_Combo

      3- As-tu des doublons dans la colonne des données que tu charges dans ta ComboBox1 ?
      0
      1. Bonjour pijaku;

        Merci d'abord pour ta réponse.
        1- J'ai utilisé ton code et ça marche toujours pas.
        2-tous le code que j'ai utilisé est dans le message d'avant.
        3-Oui j'ai doublons dans les 3 colonnes.

        Cordialement.
        0
      2. Modérateur
        @alberkrimotous le code que j'ai utilisé est dans le message d'avant
        NON. Pas celui de la procédure Alim_Combo...
        0
      3. @pijakuBonjour tout le monde,

        alberkrimo
        mon code est le suivant :
        Non, incomplet, il manque le code de deux procedures: Alim_Combo 1 et 2
        Et le fichier model avec TOUT votre code, c'est pour quand ?
        0
    6. Le beug c'est que le fichier excel s’arrête complétement et qu'il faut fermer tout.

      je suis débutant en Vba. et j'ai cherché pas mal sur la capacité des combobox. mais je ne sais toujours pas d'ou provient le problème

      Merci pour ta réponse.

      cordialement
      0
      1. Re,

        Y aurait moyen que vous mettiez votre fichier a dispo sans donnee confidentielle, mais des donnees quand meme et coherentes (vaudrait mieux), car pas moyen de vous aider

        Pour transmettre un fichier,
        Veillez a ce qu'il n'y ait PAS DE DONNEES CONFIDENTIELLES
        il faut passer par un site de pièce jointe tel que cjoint.com

        Allez sur ce site : https://www.cjoint.com/
        Clic sur parcourir,
        Cherche ton fichier,
        clic sur ouvrir,
        Clic sur "Créer le lien cjoint",
        Copier le lien,
        Revenir ici le coller dans une réponse...
        0
      2. @f894009En effet je ne peut mettre le fichier. c'est un fichier confidentiel.
        Désolé, j’apprécie votre proposition.

        Mais je ferai un exemple et je le mettrai sur le Ci joint.

        Merci encore une fois.
        0
      3. @alberkrimoRe,

        Combobox (excel2013): plus de 65000 item ("lignes")

        Le fichier, avec votre code, devra avoir 4000 lignes
        0
      4. @f894009mon fichier doit avoir a peu prés 4000 à 5000 lignes sur 3 colonnes, qui devront alimenter mes 3 combobox. mon code est le suivant :

        Option Explicit
        
        Dim Ws As Worksheet
        Dim NbLignes As Integer
        
        
        
        
        Private Sub Frame1_Click()
        
        End Sub
        
        Private Sub UserForm_Initialize()
        'Définit la feuille contenant les données
        Set Ws = Worksheets("Feuil1")
        'Définit le nombre de lignes dans la colonne A
        NbLignes = Ws.Range("A5000").End(xlUp).Row
        
        'Remplissage du ComboBox1
        Alim_Combo 1
        
        End Sub
        
        
        Private Sub ComboBox1_Change()
        'Remplissage Combo2
        Alim_Combo 2, ComboBox1.Value
        End Sub
        
        
        Private Sub ComboBox2_Change()
        Dim J As Long
        
        'Remplissage Combo3
        'Alim_Combo 3, ComboBox2.Value
        With Me.ComboBox3
        .Clear
        If Me.ComboBox2.ListIndex = -1 Then Exit Sub
        For J = 2 To NbLignes
        If CStr(Ws.Range("A" & J)) = Me.ComboBox1 And CStr(Ws.Range("B" & J)) = Me.ComboBox2 Then
        .Value = Ws.Range("C" & J)
        If .ListIndex = -1 Then .AddItem Ws.Range("C" & J)
        End If
        Next J
        If .ListCount = 1 Then .ListIndex = 0 Else .ListIndex = -1
        End With
        
        
        
        End Sub 
        Private Sub ComboBox3_Change()
        End sub
        

        Si cela peut vous éclaircir un petit peu.

        Merci

        cordialement.
        0
      5. @alberkrimoRe,

        mon code est le suivant :
        Ben oui, le meme que precedemment et que j'ai teste voir post 4.
        Il manque d'ailleur les codes de :
        Alim_Combo 1

        et
        Alim_Combo 2, ComboBox1.Value 
        0
    7. Bonjour a vous deux

      alberkrimo:
      Oui j'ai essayé, mais ça beug toujours.
      Oui, c'est quoi le beug ?

      J'ai teste votre code, la combobox3 ne peut avoir qu'un seul item vu la programmation: listindex passe a 0 a la premiere egalite de votre test

      .Value = Ws.Range("C" & J)
      If .ListIndex = -1 Then .AddItem Ws.Range("C" & J) 


      Pour la recherche des lignes, y a un peu plus rapide si vous avez 4000 lignes
      0
      1. Contributeur
        Voici la solution pour remplir une Combobox sans que cela bug:

        Option Explicit
        Private Sub UserForm_Initialize()
        Dim Cell As Range
            Dim Unique As New Collection
            Dim Valeur As Range
            Dim i As Integer
            Dim j As Integer
            For j = 1 To 1000
            Range("A" & j).Value = "item" & j 'rempli la colonne pour essai
            Next j
            'Récupère la derniere ligne non vide dans la colonne A
            i = Range("A65536").End(xlUp).Row
            On Error Resume Next
            'boucle sur les cellules de la colonne A
            For Each Cell In Range("A1:A" & i)
                'Stocke les données dans une collection
                '(La collection n'accepte que des données uniques et permet donc
                ' de filtrer facilement les doublons).
                Unique.Add Cell, CStr(Cell)
            Next Cell
            On Error GoTo 0
             'Boucle sur le contenu de la collection pour alimenter la ComboBox
            For Each Valeur In Unique
                Me.ComboBox1.AddItem Valeur
            Next Valeur
        End Sub
        


        Voilà a toi de l'adapter à tes besoins
        @+ Le Pivert
        0
        1. Oui j'ai essayé, mais ça beug toujours.

          Voici mon Code :
          Option Explicit
          
          Dim Ws As Worksheet
          Dim NbLignes As Integer
          
          
          
          
          Private Sub Frame1_Click()
          
          End Sub
          
          Private Sub UserForm_Initialize()
              'Définit la feuille contenant les données
              Set Ws = Worksheets("Feuil1")
              'Définit le nombre de lignes dans la colonne A
              NbLignes = Ws.Range("A5000").End(xlUp).Row
          
          'Remplissage du ComboBox1
              Alim_Combo 1
          
          End Sub
          
          
          Private Sub ComboBox1_Change()
              'Remplissage Combo2
              Alim_Combo 2, ComboBox1.Value
          End Sub
          
          
          Private Sub ComboBox2_Change()
          Dim J As Long
          
          'Remplissage Combo3
              'Alim_Combo 3, ComboBox2.Value
            With Me.ComboBox3
              .Clear
              If Me.ComboBox2.ListIndex = -1 Then Exit Sub
              For J = 2 To NbLignes
                If CStr(Ws.Range("A" & J)) = Me.ComboBox1 And CStr(Ws.Range("B" & J)) = Me.ComboBox2 Then
                  .Value = Ws.Range("C" & J)
                  If .ListIndex = -1 Then .AddItem Ws.Range("C" & J)
                End If
              Next J
              If .ListCount = 1 Then .ListIndex = 0 Else .ListIndex = -1
            End With
          
          End Sub


          MERCI POUR VOTRE Réponse
          0
          1. Contributeur
            Bonjour,

            Fais un essai comme ceci, tu verras qu'au dessus de 254 il y a dépassement de capacité!

              Dim i As Byte
                 For i = 1 To 254
                    ComboBox1.AddItem "Ligne" & i
                Next i

            0