- A la place d'un IF d=0 ... Utilises un SWITCH ( je trouve ça plus propre...)
- Renommes tes boutons pour leur mettre des noms "parlants" .. plutôt que :
Private Sub Button2_Click
J'aurais fait :
Private Sub RAZ_Click
- Plutôt que de faire une déclaration de tes variables sur une ligne, initialise les au moment de les utiliser
Dim a as Double = CDbl(TextBox1.Text)
Dim b as Double = CDbl(TextBox2.Text)
Dim c as Double = CDbl(TextBox3.Text)
Dim d as Double = CDbl((b * b) - 4 * (a * c))
- Il manque une déscription à ta source...
Expliques y qu'il faut créer une FORM.. y mettre 3 TextBox... 2 Boutons ..etc...
- Bien que bien écrite.. ta "source" est un peu légère...
Pour rappel.. il est précisé dans la charte du site qu'on ne dépose une source que si :
* Elle n'existe pas déjà dans de nombreux exemplaires sur le site
* Qu'elle ne doit pas contenir que quelques lignes de codes "basiques"...
Voir ici :
Vérifier avant de poster ma source si une source équivalente n'est pas déjà présente et si ma source apporte réellement quelque chose.
C'est très souvent le cas. Certaines sources (calculatrices, convertisseurs, msgbox, démineur, chronomètres, et autres routines plus que nombreuses) seront directement supprimées.
Ne postez pas une source ne faisant qu'utiliser une ou deux fonctions standard du langage.
Mettre un titre explicite et on décrit correctement la source.
Toute source sans description sera supprimée. De même qu'une source avec un titre ne correspondant pas.
Je ne vais pas supprimer ta source... mais rien ne dit qu'un autre modérateur ne le fera pas.
Mais bon.. pour un début.. je le redis.. c'est propre ... bien indenté..
Je dirais que pour poursuivre ton apprentissage du DOTNET.. l'idéal serait, même si pour un si "petit" programme il n'y a pas forcément d'utilité.. de te mettre à utiliser des CLASS;
Une class EQUATION par exemple... dans laquelle tu effectuerais tes calculs.. plutôt que de réaliser ton calcul directement dans le code de ta form....
17 nov. 2014 à 14:40
17 nov. 2014 à 14:29
N'oublies pas d'aller ajouter une description à ta source...
Modifié par jordane45 le 17/11/2014 à 14:28
merci beaucoup...
J'en prend bonne note
EDIT : Pas de langage SMS. Merci (je corrige ton message).
( Jordane)
14 nov. 2014 à 18:37
Le code est bien écrit ... mais (et oui il y a toujours un mais...) :
- J'ai édité ton message pour ajouter les balises de code.
Explications disponibles ici :
http://codes-sources.commentcamarche.net/faq/10686-le-nouveau-codes-sources-comment-ca-marche#balises-code
- A la place d'un IF d=0 ... Utilises un SWITCH ( je trouve ça plus propre...)
- Renommes tes boutons pour leur mettre des noms "parlants" .. plutôt que :
J'aurais fait :
- Plutôt que de faire une déclaration de tes variables sur une ligne, initialise les au moment de les utiliser
- Il manque une déscription à ta source...
Expliques y qu'il faut créer une FORM.. y mettre 3 TextBox... 2 Boutons ..etc...
- Bien que bien écrite.. ta "source" est un peu légère...
Pour rappel.. il est précisé dans la charte du site qu'on ne dépose une source que si :
* Elle n'existe pas déjà dans de nombreux exemplaires sur le site
* Qu'elle ne doit pas contenir que quelques lignes de codes "basiques"...
Voir ici :
Je ne vais pas supprimer ta source... mais rien ne dit qu'un autre modérateur ne le fera pas.
Mais bon.. pour un début.. je le redis.. c'est propre ... bien indenté..
Je dirais que pour poursuivre ton apprentissage du DOTNET.. l'idéal serait, même si pour un si "petit" programme il n'y a pas forcément d'utilité.. de te mettre à utiliser des CLASS;
Une class EQUATION par exemple... dans laquelle tu effectuerais tes calculs.. plutôt que de réaliser ton calcul directement dans le code de ta form....