CALCULATRICE VBSCRIPT
nathansecret
Messages postés
63
Date d'inscription
mardi 11 novembre 2008
Statut
Membre
Dernière intervention
31 octobre 2011
-
24 juin 2010 à 12:53
nathansecret Messages postés 63 Date d'inscription mardi 11 novembre 2008 Statut Membre Dernière intervention 31 octobre 2011 - 25 juin 2010 à 11:39
nathansecret Messages postés 63 Date d'inscription mardi 11 novembre 2008 Statut Membre Dernière intervention 31 octobre 2011 - 25 juin 2010 à 11:39
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/51946-calculatrice-vbscript
https://codes-sources.commentcamarche.net/source/51946-calculatrice-vbscript
25 juin 2010 à 11:39
25 juin 2010 à 00:02
- Mets un "Option Explicit" en début de code et dimensionne tes variables globales juste après, ça t'évitera des bricoles (ou 20 minutes de perdues pour une variable non déclarée).
- "Bonjour et bienvenue" : c'est joli mais ça soule un peu ... Donc bye-bye la variable "Tour".
- Sois cohérent dans les noms de variables -> je choisis de tout mettre en anglais.
- N5 sert à quoi ? Bye-bye aussi.
- "Racine carréE" : faute d'orthographe
- "If Choix=5" : tu remplaces le bloc par "TwoNbr = Not(Choix=5)"
- "If Choix<>0 [...] If Choix=0" : tu peux mettre un "Else" au lieu de ré-effectuer la comparaison.
- "If Choix=1 [...] ElseIf Choix=2 [...]" : fais un "Select Case", tu gagneras de la place ...
- "Souhaitez-vous quitteR l'application" : faute d'orthographe ... encore.
- Vérifie si le choix de l'opérateur est correct (si l'utilisateur entre -5, ton programme ne le gère pas).
- N'affecte pas TwoNbr si l'utilisateur a choisi de quitter le programme.
- Si l'utilisateur demande le quotient d'un nombre avec zéro ou la racine d'un nombre négatif, ton programme se plante lamentablement.
- L'affichage du résultat est dans chaque "If" est inutile. Stocke l'opération dans un tableau et en fonction du choix de l'utilisateur, affiche la chaine correspondante.
Le code corrigé se trouve là : http://pastebin.com/HzXBwkfw
24 juin 2010 à 12:53