romagny13
Messages postés687Date d'inscriptionlundi 10 janvier 2005StatutMembreDernière intervention27 août 2014
-
4 mars 2007 à 14:06
cs_Bidou
Messages postés5487Date d'inscriptiondimanche 4 août 2002StatutMembreDernière intervention20 juin 2013
-
5 mars 2007 à 13:22
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_Bidou
Messages postés5487Date d'inscriptiondimanche 4 août 2002StatutMembreDernière intervention20 juin 201361 5 mars 2007 à 13:22
Un goût de programmation?
Tu plaisantes j'espère ^^ (le seul qui pourrait être un goût de programmation c'est le !ok)
Pour le reste, c'est des grosses à très grosses erreurs de programmation.
Corriges vite!
Merci.
cs_Mimo24
Messages postés3Date d'inscriptiondimanche 15 mai 2005StatutMembreDernière intervention 5 mars 2007 5 mars 2007 à 12:31
Je suis d'accord avec Bidou pour le stream.......pour le reste ça reste juste un gout de programmation, mais merci quand même.
sebmafate voila le zip même en VB.NET l'interface et la classe si ça t'intéresse...
cs_Bidou
Messages postés5487Date d'inscriptiondimanche 4 août 2002StatutMembreDernière intervention20 juin 201361 5 mars 2007 à 11:04
Après un rapide coup d'oeil:
Si on enlève les Console.WriteLine, il ne reste pas grand chose (un snippet aurait-il suffit?)
En plus:
- Ne pas utiliser Parse mais TryParse (plus besoin de bloc try-catch)
- String.Format au lieu de tout ces '+' !!!!!!!!!!!!
- Le stream 'fichier' n'est jamais fermé
- ok==false => !ok
- _credit=float.Parse(Console.ReadLine().Replace(".",",").Trim()); keskeCkesa !? => System.Globalization.CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator
Dans l'état, je ne pense psa que cette source restera, merci donc de bien vouloir la mettre à jour!
sebmafate
Messages postés4936Date d'inscriptionlundi 17 février 2003StatutMembreDernière intervention14 février 201437 5 mars 2007 à 09:25
et puis... faire un zip de la source ;)
cs_Mimo24
Messages postés3Date d'inscriptiondimanche 15 mai 2005StatutMembreDernière intervention 5 mars 2007 4 mars 2007 à 15:01
tout à fait d'accord avec toi !!! merci du conseil
romagny13
Messages postés687Date d'inscriptionlundi 10 janvier 2005StatutMembreDernière intervention27 août 20143 4 mars 2007 à 14:26
oui
ou juste une fonction qui fait le calcul complet (donc il faut quelle recoive les parametres necessaire) , cette fonction effectuer le calcul (avec tous les tests) et retourne le resultat
imagine ainsi une personne qui aurait besoin de ce meme calcul il fait une recherche,tombe sur le code, et n'a plus qu'a le reutiliser !! merveilleux
++
cs_Mimo24
Messages postés3Date d'inscriptiondimanche 15 mai 2005StatutMembreDernière intervention 5 mars 2007 4 mars 2007 à 14:23
C'est vrai !! je pouvais créer une class, qui fait le boulot, et c'est ce que je vais faire, en ajoutant les autres types du crédits .... et biensur implémenter tout ça dans une interface
romagny13
Messages postés687Date d'inscriptionlundi 10 janvier 2005StatutMembreDernière intervention27 août 20143 4 mars 2007 à 14:06
Salut le pb de ton code c'est que c'est assez reutilisable tel quel, il vaudrait peut etre mieux creer des fonctions qui remplissent une tache et qui renvoient un resultat ainsi on peut tres facilement reprendre le code pour l'integrer dans un autre programme qui remplirait une tache similaire
++
5 mars 2007 à 13:22
Tu plaisantes j'espère ^^ (le seul qui pourrait être un goût de programmation c'est le !ok)
Pour le reste, c'est des grosses à très grosses erreurs de programmation.
Corriges vite!
Merci.
5 mars 2007 à 12:31
sebmafate voila le zip même en VB.NET l'interface et la classe si ça t'intéresse...
5 mars 2007 à 11:04
Si on enlève les Console.WriteLine, il ne reste pas grand chose (un snippet aurait-il suffit?)
En plus:
- Ne pas utiliser Parse mais TryParse (plus besoin de bloc try-catch)
- String.Format au lieu de tout ces '+' !!!!!!!!!!!!
- Le stream 'fichier' n'est jamais fermé
- ok==false => !ok
- _credit=float.Parse(Console.ReadLine().Replace(".",",").Trim()); keskeCkesa !? => System.Globalization.CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator
Dans l'état, je ne pense psa que cette source restera, merci donc de bien vouloir la mettre à jour!
5 mars 2007 à 09:25
4 mars 2007 à 15:01
4 mars 2007 à 14:26
ou juste une fonction qui fait le calcul complet (donc il faut quelle recoive les parametres necessaire) , cette fonction effectuer le calcul (avec tous les tests) et retourne le resultat
imagine ainsi une personne qui aurait besoin de ce meme calcul il fait une recherche,tombe sur le code, et n'a plus qu'a le reutiliser !! merveilleux
++
4 mars 2007 à 14:23
4 mars 2007 à 14:06
++