cs_Warny
Messages postés473Date d'inscriptionmercredi 7 août 2002StatutMembreDernière intervention10 juin 2015 3 févr. 2004 à 12:06
Ton fichier utilise la dll scripting par l'intermédiaire de tes createobject.
Tu devrais lier la librairie pour plus d'efficacité.
Sinon, ton code est super mal présenté, il manque des commentaires et des nom de variables explicites. Il faut aussi que tu indentes (décalle) tes lignes pour plus de lisibilité.
Pour lire tes attributs, tu peux utiliser un masque :
ça fera des ligne de code genre
Dim attr as String
if qx and 1 then attr = attr & "+R"
if qx and 2 then attr = attr & "+H"
if qx and 4 then attr = attr & "+S"
if qx and 32 then attr = attr & "+A"
if qx and 64 then attr = attr & "+D"
attr = mid(attr,2)
soit 6 lignes où tu en mets 32 et où tu multiplie inutilement ton code
Elimine les Goto même dans les On Error (d'autres ne seront pas d'accord avec moi)
Utilise plutot cette forme :
On error resume next
instruction
If Err.number <> 0 then
traitement de l'erreur
eventuellement suivi d'un exit sub ou function
End If
3 févr. 2004 à 12:06
Tu devrais lier la librairie pour plus d'efficacité.
Sinon, ton code est super mal présenté, il manque des commentaires et des nom de variables explicites. Il faut aussi que tu indentes (décalle) tes lignes pour plus de lisibilité.
Pour lire tes attributs, tu peux utiliser un masque :
ça fera des ligne de code genre
Dim attr as String
if qx and 1 then attr = attr & "+R"
if qx and 2 then attr = attr & "+H"
if qx and 4 then attr = attr & "+S"
if qx and 32 then attr = attr & "+A"
if qx and 64 then attr = attr & "+D"
attr = mid(attr,2)
soit 6 lignes où tu en mets 32 et où tu multiplie inutilement ton code
Elimine les Goto même dans les On Error (d'autres ne seront pas d'accord avec moi)
Utilise plutot cette forme :
On error resume next
instruction
If Err.number <> 0 then
traitement de l'erreur
eventuellement suivi d'un exit sub ou function
End If