Scalpweb
Messages postés1467Date d'inscriptionsamedi 13 mars 2004StatutMembreDernière intervention 5 mai 20104 16 juil. 2004 à 14:00
Oui, c vraiament pas mal. 8/10
daetips
Messages postés142Date d'inscriptionjeudi 10 juillet 2003StatutMembreDernière intervention10 novembre 2007 16 juil. 2004 à 13:55
C'est bon j'ais changé le code source
cs_Pingouin
Messages postés262Date d'inscriptionlundi 26 août 2002StatutMembreDernière intervention24 août 2005 16 juil. 2004 à 12:39
Pas mal pour l'idée mais c'est vrai que le code est perfectible. Je préfère celui de Renfield qui est nettement plus "propre". Mais bon c'est pas mal : 7/10
Renfield
Messages postés17287Date d'inscriptionmercredi 2 janvier 2002StatutModérateurDernière intervention27 septembre 202174 16 juil. 2004 à 07:50
très gentil a toi hhhp2004, mais quelques peu disproportionné.....
I et J sont déclarés en variables globales, c'est inutile.
les variables vertvbcyan et bleuvbcyan pourraient être des constantes.
vertvbcyan = (vbCyan And &HFF00FF00) / 256
vertvbcyan = (vbCyan And &HFF00FF00) / 256
Bleuvbcyan = (vbCyan And &HFFFF0000) / (65536)
Bleuvbcyan = (vbCyan And &HFFFF0000) / (65536)
pourquoi mettre ces lignes en double ?
RGB(0, vertvbcyan + I * 0, Bleuvbcyan + I * 0)
est-ce que ca ne ferait pas simplement :
RGB(0, vertvbcyan , Bleuvbcyan )
d'ailleurs, c'est amusant, puisque tu récupères.... VbCyan !!!
enfin, tu aurais pu utiliser un Timer ! car ralentir la boucle avec un :
For J = 1 To 60000 'le temps
Next
va donner des résultats inégaux selon le poste utilisé, et et de toute facon bien trop rapide !
voila une version quelques peu corrigée :
Option Explicit
Private Sub Commencer_Click()
Timer1.Enabled = True
End Sub
Private Sub Form_Load()
Timer1.Enabled = False
Timer1.Interval = 1
End Sub
Private Sub Timer1_Timer()
'# i conservera sa valeur à chaque itération de la procedure
Static i As Integer
barre.Line (i, 0)-(i, barre.ScaleHeight), vbCyan
i = i + 1
If i = barre.ScaleWidth Then
i = 0
barre.Cls
End If
End Sub
hhhp2004
Messages postés173Date d'inscriptionsamedi 25 octobre 2003StatutMembreDernière intervention13 octobre 2007 16 juil. 2004 à 05:26
pas mal du tout ...... 10/10 pour le style (et le code aussi)
16 juil. 2004 à 14:00
16 juil. 2004 à 13:55
16 juil. 2004 à 12:39
16 juil. 2004 à 07:50
I et J sont déclarés en variables globales, c'est inutile.
les variables vertvbcyan et bleuvbcyan pourraient être des constantes.
vertvbcyan = (vbCyan And &HFF00FF00) / 256
vertvbcyan = (vbCyan And &HFF00FF00) / 256
Bleuvbcyan = (vbCyan And &HFFFF0000) / (65536)
Bleuvbcyan = (vbCyan And &HFFFF0000) / (65536)
pourquoi mettre ces lignes en double ?
RGB(0, vertvbcyan + I * 0, Bleuvbcyan + I * 0)
est-ce que ca ne ferait pas simplement :
RGB(0, vertvbcyan , Bleuvbcyan )
d'ailleurs, c'est amusant, puisque tu récupères.... VbCyan !!!
enfin, tu aurais pu utiliser un Timer ! car ralentir la boucle avec un :
For J = 1 To 60000 'le temps
Next
va donner des résultats inégaux selon le poste utilisé, et et de toute facon bien trop rapide !
voila une version quelques peu corrigée :
Option Explicit
Private Sub Commencer_Click()
Timer1.Enabled = True
End Sub
Private Sub Form_Load()
Timer1.Enabled = False
Timer1.Interval = 1
End Sub
Private Sub Timer1_Timer()
'# i conservera sa valeur à chaque itération de la procedure
Static i As Integer
barre.Line (i, 0)-(i, barre.ScaleHeight), vbCyan
i = i + 1
If i = barre.ScaleWidth Then
i = 0
barre.Cls
End If
End Sub
16 juil. 2004 à 05:26