Bonjour à tous,
J'expose ici à nouveau mes problèmes........... Peut être, j'espère, saurez vous m'aider

J'ai un code, qui fonctionne très bien mais très très répétitif, que je souhaiterais simplifier.
Voici le code :
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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
Dim Nb As Integer
 
Private Sub combobox1_change() 'Choix du type d'instruction
    ComboBox1.Visible = True
    CommandButton2.Visible = True
    Label2.Visible = True
 
    If ComboBox1 = "Marches dégradées" Then
        tot = Sheets("MD").Range("A2").End(xlDown).Row
        Randomize   'Initialiser le générateur de nombres aléatoires
        Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            While Sheets("MD").Cells(Nb, 2) <> ""
                Randomize   'Initialiser le générateur de nombres aléatoires
                Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            Wend
        Sheets("MD").Cells(Sheets("MD").Range("A1").End(xlDown).Row + 1, 3) = Nb
        Label2.Caption = Sheets("MD").Cells(Nb, 1)
        If Sheets("MD").Cells(Nb, 4).Value <> "" Then
            Label3.Visible = True
            Label4.Visible = True
            Label3.Caption = "Ancien n° d'avis : " & Sheets("MD").Cells(Nb, 4)
            Label4.Caption = Sheets("MD").Cells(Nb, 5)
            Me.Height = 200
        End If
    ElseIf ComboBox1 = "Instructions d'exploitation" Then
        tot = Sheets("IE").Range("A2").End(xlDown).Row
        Randomize   'Initialiser le générateur de nombres aléatoires
        Nb = Int(tot * Rnd) + 1  'Nombre aléatoire entier entre 4 et tot
            While Sheets("IE").Cells(Nb, 2) <> ""
                Randomize   'Initialiser le générateur de nombres aléatoires
                Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            Wend
        Sheets("IE").Cells(Sheets("IE").Range("A1").End(xlDown).Row + 1, 3) = Nb
        Label2.Caption = Sheets("IE").Cells(Nb, 1)
        If Sheets("IE").Cells(Nb, 4).Value <> "" Then
            Label3.Visible = True
            Label4.Visible = True
            Label3.Caption = "Ancien n° d'avis : " & Sheets("IE").Cells(Nb, 4)
            Label4.Caption = Sheets("IE").Cells(Nb, 5)
            Me.Height = 200
        End If
    ElseIf ComboBox1 = "Instructions de sécurité" Then
        tot = Sheets("IS").Range("A2").End(xlDown).Row
        Randomize   'Initialiser le générateur de nombres aléatoires
        Nb = Int(tot * Rnd) + 1  'Nombre aléatoire entier entre 4 et tot
            While Sheets("IS").Cells(Nb, 2) <> ""
                Randomize   'Initialiser le générateur de nombres aléatoires
                Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            Wend
        Sheets("IS").Cells(Sheets("IS").Range("A1").End(xlDown).Row + 1, 3) = Nb
        Label2.Caption = Sheets("IS").Cells(Nb, 1)
        If Sheets("IS").Cells(Nb, 4).Value <> "" Then
            Label3.Visible = True
            Label4.Visible = True
            Label3.Caption = "Ancien n° d'avis : " & Sheets("IS").Cells(Nb, 4)
            Label4.Caption = Sheets("IS").Cells(Nb, 5)
            Me.Height = 200
        End If
    End If
End Sub
Clairement, je fais exactement 3x la même chose en fonction du choix de mon combobox1.

J'ai donc essayé de simplifier de cette manière :
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
Dim Nb As Integer
Dim TypeInst As String
 
Private Sub combobox1_change() 'Choix du type d'instruction
    ComboBox1.Visible = True
    CommandButton2.Visible = True
    Label2.Visible = True
 
    If ComboBox1 = "Marches dégradées" Then
        TypeInst = "MD"
    ElseIf ComboBox1 = "Instructions d'exploitation" Then
        TypeInst = "IE"
    ElseIf ComboBox1 = "Instructions de sécurité" Then
        TypeInst = "IS"
    End If
 
        tot = Sheets(TypeInst).Range("A2").End(xlDown).Row
        Randomize   'Initialiser le générateur de nombres aléatoires
        Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            While Sheets(TypeInst).Cells(Nb, 2) <> ""
                Randomize   'Initialiser le générateur de nombres aléatoires
                Nb = Int(tot * Rnd) + 2  'Nombre aléatoire entier entre 2 et tot
            Wend
        Sheets(TypeInst).Cells(Sheets(TypeInst).Range("A1").End(xlDown).Row + 1, 3) = Nb
        Label2.Caption = Sheets(TypeInst).Cells(Nb, 1)
        If Sheets(TypeInst).Cells(Nb, 4).Value <> "" Then
            Label3.Visible = True
            Label4.Visible = True
            Label3.Caption = "Ancien n° d'avis : " & Sheets(TypeInst).Cells(Nb, 4)
            Label4.Caption = Sheets(TypeInst).Cells(Nb, 5)
            Me.Height = 200
        End If
End Sub
Mais j'ai ce message d'erreur qui s'affiche :
Nom : Sans titre.png Affichages : 127 Taille : 3,3 Ko

et le déboggeur qui me souligne la ligne 17 :
Code : Sélectionner tout - Visualiser dans une fenêtre à part
        tot = Sheets(TypeInst).Range("A2").End(xlDown).Row

Savez vous quel est le problème ou avez une solution plus efficace?

Merci d'avance.