Simplifier un code qui se répète

Version imprimable

Bonjour à tous!

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

Code:

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! :D

Aurel ++

14/04/2010, 15h21
sabzzz

bonjour Aurel,

si c'est pour incrémenter le control,

Code:

Me.Controls("ComboBox" & x).AddItem
14/04/2010, 15h54
Aurel.89

Heuu

T'entends quoi par incrémenter? :aie:
14/04/2010, 17h03
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
14/04/2010, 17h19
sabzzz

Citation:

T'entends quoi par incrémenter?

pour reprendre en tes termes, "une suite croissante de combobox"
15/04/2010, 10h13
Aurel.89

Merci pour vos réponses! =)

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

Bonjour à tou(te)s

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

Code:

Public matériaux As Range, combo As Control

Code:

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

Code:

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

Code:

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

15/04/2010, 10h56
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
16/04/2010, 12h36
Didier Gonard

Bonjour,

Peux-tu donner la règle relationnelle entre les N°s de tes combo ?

Citation:

Combobox9 en fonction de la 1

Citation:

'Combobox10 en fonction de la 2

Peux-t-on en déduire que le delta est toujours de 8 ?

Qu'en est-il de 3 à 8 ? faut-il s'en soucier etc..

cordialement,

Didier
16/04/2010, 14h34
Aurel.89

Alors la 9 se remplie en fonction de la 1
La 10 => 2
11 => 3
...
...
...
...
16 => 8

Le delta est toujours de 8

Bonjour,

vois cette approche (brouillon), je n'ai pas analysé ton code, juste construit une approche possible avec..

Code:

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
'Combobox109 en fonction de la 101
Private Sub ComboBox101_Change()
Dim strNameCboA As String
Dim strNameCboB As String
 
strNameCboA = Me.ActiveControl.Name
'le but ici, est de récupérer selon la syntaxe fournie le nombre affecté au combobox
' voir l'aide vba pour les différentes fonctions
strNameCboB = "combobox" & StrReverse(Val(StrReverse(strNameCboA))) + 8 'selon tes données
Call CallMe(strNameCboA, strNameCboB)
End Sub
 
Sub CallMe(strNameCboA, strNameCboB)
Dim matériaux As Range
Dim I As Long
 
Me.Controls(strNameCboB).Clear
With Sheets(1)
Set matériaux = .Columns(2).Find(Me.Controls(strNameCboA).Value)
 
If Not matériaux Is Nothing Then
For I = matériaux.Row To .Range("B65536").End(xlUp).Row
  If .Range("B" & I) = Me.Controls(strNameCboA).Value Or .Range("B" & I) = "" Then
    If .Range("C" & I) <> "" Then
    Me.Controls(strNameCboA).AddItem (.Range("C" & I).Value)
    End If
  Else
   Exit Sub
  End If
Next I
End If
End With
End Sub

cordialement,

Didier