critique du code source de mon programme

**startout** · 10/08/2006, 16h18

Bonjour à tous,

Je m'arrête un peu dans mon petit projet pour vous soumettre une partie de mon code source.
Comme le dev n'est pas mon job, je souhaite que vous regardiez ce que j'ai pondu pour que vous me donniez des voies d'explorations sur ce que je pourrais amélioré, notamment aussi au niveau des commentaires.

Voici ce qui ce passe quand je clique

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
 
Private Sub Image2_Click()
Dim i, fin_client, deb_contrat, fin_contrat, deb_support, fin_support, nb_feuille As Integer
If chemin <> "" Then
 
'Ouverture de l'application excel
Set appExcel = CreateObject("Excel.Application")
 
Set wbExcel = appExcel.Workbooks.Open(chemin)
'wsExcel correspond à la première feuille du fichier
 
 
'si l'utilisateur souhaite importer l'en cours par client
If Check1.Value = 1 Then
    'Test pour savoir si les feuilles existe
    If feuille_existe("en cours client") = False Then
        Module1.ajout_feuille ("en cours client")
    End If
End If
 
'si l'utilisateur souhaite importer l'en cours client par contrat
If Check2.Value = 1 Then
    If feuille_existe("en cours client par contrat") = False Then
        Module1.ajout_feuille ("en cours client par contrat")
    End If
End If
 
'si l'utilisateur souhaite importer l'en cours client par support
If Check3.Value = 1 Then
    If feuille_existe("en cours client par support") = False Then
        Module1.ajout_feuille ("en cours client par support")
    End If
End If
 
 
'On  se positionne sur la dernière feuille (feuille tousclient)
nb_feuille = Sheets.Count
Set sheet = wbExcel.Worksheets(4)
 
Const deb_client = 1
Const cellule = 3
'parcours du tableau client
fin_client = parcours_tableau(deb_client, cellule)
deb_contrat = fin_client + 4
'parcours du tableau contrat
fin_contrat = parcours_tableau(deb_contrat, cellule)
deb_support = fin_contrat + 4
'parcours du tableau support
fin_support = parcours_tableau(deb_support, cellule)

Voici mon module

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
 
'Déclaration des variables
Public appExcel As Excel.Application 'Application Excel
Public wbExcel As Excel.Workbook 'Classeur Excel
Public wsExcel As Excel.Worksheet
'Feuille Excel
Public sheet As Excel.Worksheet
 
 
Public nomtemp As String
Public chemin_temp As String
Public chemin_temp_port As String
 
Public exldoc2 As Object
Public exlapp2 As Object
 
Private Declare Function shellexecute Lib "shell32.Dll" (ByVal hwnd As Long, ByVal lpoperation As String, ByVal lpfile As String, lpparameters As String, ByVal lpdirectory As String, ByVal nshowcmd As Long) As Long
 
Public Sub ajout_feuille(nom_feuille As String)
Dim nb As Integer
'calcul le nombre de feuille avant d'ajouter
nb = Sheets.Count
'ajout feuille
Sheets.Add
Sheets("Feuil" & nb).Select
Sheets("Feuil" & nb).Name = nom_feuille
 
End Sub
 
Function parcours_tableau(ByVal deb As Integer, ByVal cellule) As Integer
 
'parcours du tableau client
While (sheet.Cells(deb, cellule).Value <> "")
    deb = deb + 1
Wend
parcours_tableau = deb
End Function

J'hésite notamment à rajouter une boucle FOR pour mes trois check.
Est-ce rentable ? Est-ce justifié ?
Merci d'avance pour vos commentaires.

**SfJ5Rpw8** · 10/08/2006, 21h58

avec :

Code :

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

1
2
 
Dim i, fin_client, deb_contrat, fin_contrat, deb_support, fin_support, nb_feuille As Integer

il n'y as que nb_feuille qui est un integer les autres sont des variants...
il faut faire

Code :

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

1
2
3
4
5
6
7
 
Dim i As integer , fin_client As integer, deb_contrat As integer  ...
 
'..ou 
Dim i As integer 
Dim fin_client As integer
....

eu.... avant d'aller plus loin il fonctionne ton code ..? s'il y as des problémes utilise une discussion par problémes........

**startout** · 11/08/2006, 12h43

Oui mon code fonctionne. J'obtiens l'effet désiré.
Je suis simplement en train de l'optimiser.

En ce qui concerne fin_client As integer, deb_contrat, etc ce sont bien des entiers.
Je m'en sers pour garder en mémoire la ligne où commence le tableau client, la ligne où commence le tableau contrat, etc...

Sinon en ce qui concerne mes trois check que je pourrais intégrer dans une boucle.
Vous en pensez koi ?
J'ai vu en structure de données qu'il fallait éviter ce genre de structure. Une boucle qui se trouve dans une autre boucle. Complexité trop importante.
Merci de vos commentaires.