cs_alpha5
Messages postés74Date d'inscriptionjeudi 10 août 2006StatutMembreDernière intervention24 mars 2013 26 sept. 2011 à 07:58
Merci de vos commentaires.
us_30
Messages postés2065Date d'inscriptionlundi 11 avril 2005StatutMembreDernière intervention14 mars 201610 25 sept. 2011 à 17:38
Oupss, il faut aussi incrémenter b de deux en deux dans le cas de la formule. " b += 2 "
Amicalement,
Us.
us_30
Messages postés2065Date d'inscriptionlundi 11 avril 2005StatutMembreDernière intervention14 mars 201610 25 sept. 2011 à 17:36
Bonjour,
Il serait préférable de retirer le menu, et de mettre 2 boutons (au lieu d'un seul).
Un bouton "Affichage" et un bouton "Remise à zéro".
Le code dans le bouton serait alors :
Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click
'incrémente
b += 1 '<-- ligne à rajouter
'Affiche Nombres
=
Dans le code, il faut initialiser la variable "s" en toute rigueur. Ce qui donne :
'Affiche
TextBox1.Clear()
s = String.Empty '<-- ligne à rajouter For i 1 To n
La programmation de la fonction "position_dans_carre" n'est pas terrible. Tu emplois la variable "b" qui est globale. Ça marche, mais ce n'est pas recommandé.
Je te proposes d'employer une formule mathématique plutôt qu'un "select case". a partir de ton select case, j'ai déterminé une formule, ce qui donne :
Function position_dans_carre(ByVal nb As Integer) As Integer
position_dans_carre = CInt(-2 * nb + 12 + (1 / 3) * b - (1 / 3))
End Function
L'avantage est l'incrémentation te permet de connaitre plus de carré. Dans ta version tu avais la somme de 18,24,30... Ici tu as la somme pour 6,9,12,15,18, etc. Donc de 3 en 3, au lieu de 6 en 6.
@alpha5: Tu veux pas nous inventer un petit prog qui trouve les pixel mort d'un LCD tout seul? Et nommer tes variable et ihm (form1...)? Ah non, toi tu est un adepte du "je met des accent dans le noms de mes fonctions, et je nomme mes variable en une seule lettres."
Mais pourquoi?
1. Augmenter l’illisibilité du code?
2. Tu tente d'inventer une nouvelle mode? (il y en a beaucoup qui s'y essayent sur ce site pourtant...)
3. Nous faire péter un câble?
4. Nous prouver que tu ne sais pas programmer? (les gars, je sais programmer! regarder: printf('hello world mother fucker ça rox');
5. Tu code en braille, et tu na peut être pas vu les commentaires des autres personnes du coup (vu que tu est aveugle).
Bon, j'ai fini, me reste plus qu'a cliquer sur "j'approuve la charte de bonne conduite". voooiiilllaaa
NHenry
Messages postés15113Date d'inscriptionvendredi 14 mars 2003StatutModérateurDernière intervention22 avril 2024159 22 sept. 2011 à 12:54
26 sept. 2011 à 07:58
25 sept. 2011 à 17:38
Amicalement,
Us.
25 sept. 2011 à 17:36
Il serait préférable de retirer le menu, et de mettre 2 boutons (au lieu d'un seul).
Un bouton "Affichage" et un bouton "Remise à zéro".
Le code dans le bouton serait alors :
Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click
'incrémente
b += 1 '<-- ligne à rajouter
'Affiche Nombres
=
Dans le code, il faut initialiser la variable "s" en toute rigueur. Ce qui donne :
'Affiche
TextBox1.Clear()
s = String.Empty '<-- ligne à rajouter For i 1 To n
La programmation de la fonction "position_dans_carre" n'est pas terrible. Tu emplois la variable "b" qui est globale. Ça marche, mais ce n'est pas recommandé.
Je te proposes d'employer une formule mathématique plutôt qu'un "select case". a partir de ton select case, j'ai déterminé une formule, ce qui donne :
Function position_dans_carre(ByVal nb As Integer) As Integer
position_dans_carre = CInt(-2 * nb + 12 + (1 / 3) * b - (1 / 3))
End Function
L'avantage est l'incrémentation te permet de connaitre plus de carré. Dans ta version tu avais la somme de 18,24,30... Ici tu as la somme pour 6,9,12,15,18, etc. Donc de 3 en 3, au lieu de 6 en 6.
Amicalement,
Us.
22 sept. 2011 à 17:05
@alpha5: Tu veux pas nous inventer un petit prog qui trouve les pixel mort d'un LCD tout seul? Et nommer tes variable et ihm (form1...)? Ah non, toi tu est un adepte du "je met des accent dans le noms de mes fonctions, et je nomme mes variable en une seule lettres."
Mais pourquoi?
1. Augmenter l’illisibilité du code?
2. Tu tente d'inventer une nouvelle mode? (il y en a beaucoup qui s'y essayent sur ce site pourtant...)
3. Nous faire péter un câble?
4. Nous prouver que tu ne sais pas programmer? (les gars, je sais programmer! regarder: printf('hello world mother fucker ça rox');
5. Tu code en braille, et tu na peut être pas vu les commentaires des autres personnes du coup (vu que tu est aveugle).
Bon, j'ai fini, me reste plus qu'a cliquer sur "j'approuve la charte de bonne conduite". voooiiilllaaa
22 sept. 2011 à 12:54
Je vois que tu n'a pas appliqué les conseils prodigués de tes précédentes sources :
http://www.vbfrance.com/codes/CARRE-MAGIQUE-AVEC-CHIFFRES-LETTRES_53545.aspx
http://www.vbfrance.com/codes/NOMBRES-PRIMORDIAUX_53558.aspx