Discussion :

[Aurel.89]

Bonjour à tous!

Je débute en VBA et je cherche à simplifier un code qui remplit une combobox en fonction d'une autre:

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
 
'Combobox9 en fonction de la 1
Private Sub ComboBox1_Change()
Dim matériaux As Range
 
Me.ComboBox9.Clear
With Sheets("Bibliothèque")
Set matériaux = .Columns(2).Find(Me.ComboBox1.Value)
 
If Not matériaux Is Nothing Then
For i = matériaux.Row To .Range("B65536").End(xlUp).Row
  If .Range("B" & i) = Me.ComboBox1.Value Or .Range("B" & i) = "" Then
    If .Range("C" & i) <> "" Then
    Me.ComboBox9.AddItem (.Range("C" & i).Value)
    End If
  Else
   Exit Sub
  End If
Next i
End If
End With
 
End Sub
 
'Combobox10 en fonction de la 2
Private Sub ComboBox2_Change()
Dim matériaux As Range
 
Me.ComboBox10.Clear
With Sheets("Bibliothèque")
Set matériaux = .Columns(2).Find(Me.ComboBox2.Value)
 
If Not matériaux Is Nothing Then
For i = matériaux.Row To .Range("B65536").End(xlUp).Row
  If .Range("B" & i) = Me.ComboBox2.Value Or .Range("B" & i) = "" Then
    If .Range("C" & i) <> "" Then
    Me.ComboBox10.AddItem (.Range("C" & i).Value)
    End If
  Else
   Exit Sub
  End If
Next i
End If
End With
End Sub

Ce code se répète une soixantaine de fois.
Y'a-t-il moyen de le généraliser pour une suite croissante de combobox de manière à faciliter la lecture et la saisie?

Merci d'avance!

Aurel ++

[sabzzz]

bonjour Aurel,

si c'est pour incrémenter le control,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Me.Controls("ComboBox" & x).AddItem

[Aurel.89]

Heuu

T'entends quoi par incrémenter?

[Didier Gonard]

bonjour,

Si tu souhaites faire appel à un ensemble de lignes de codes de façon récurrente, il ne faut effectivement surtout pas s'amuser à le dupliquer x fois, on alourdit le poids du classeur, on diminue la lisibilité et on sabote la maintenance.

La solution, à ton niveau, serait de faire une procédure indépendante qui contient ce sous ensemble de lignes de commandes et d'y faire appel quand tu en a besoin en passant les paramètres nécessaires...

voir ici pages 29 à 33 (pdf)

cordialement,

Didier

[sabzzz]

T'entends quoi par incrémenter?

pour reprendre en tes termes, "une suite croissante de combobox"

[Aurel.89]

Merci pour vos réponses! =)

Cependant j'avoue ne pas y arriver, de qu'elle façon démarrer la procédure?

[casefayere]

Bonjour à tou(te)s

essayes comme ça, une idée parmi d'autres

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Public matériaux As Range, combo As Control

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
Private Sub ComboBox1_Change()
Set combo = Me.ComboBox9
maj
End Sub

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
'Combobox10 en fonction de la 2
Private Sub ComboBox2_Change()
Set combo = Me.ComboBox2
maj
End Sub

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
Sub maj()
combo.Clear
 
With Sheets("Bibliothèque")
Set matériaux = .Columns(2).Find(combo.Value)
 
If Not matériaux Is Nothing Then
For i = matériaux.Row To .Range("B65536").End(xlUp).Row
  If .Range("B" & i) = combo.Value Or .Range("B" & i) = "" Then
    If .Range("C" & i) <> "" Then
    combo.AddItem (.Range("C" & i).Value)
    End If
  Else
   Exit Sub
  End If
Next i
End If
End With
 
End Sub

[Aurel.89]

Alors, résultats:

La 1 se remplie, la 9 reste vide
La 2 et la 10 restent vide

J'ai corrigé l'erreur dans le 3ème code: Set combo = Me.Combobox10


Mais avec ce code, il va falloir écrire chaque procédure "Combobox_change()", de qu'elle manière généraliser cette action à toutes les combo (1 à 8 dans mon exemple: qui influent sur 9 à 16 (dans l'ordre))

Merci