ENREGISTREUR DE NOMS
NHenry
Messages postés
15112
Date d'inscription
vendredi 14 mars 2003
Statut
Modérateur
Dernière intervention
13 avril 2024
-
1 sept. 2011 à 19:10
NHenry Messages postés 15112 Date d'inscription vendredi 14 mars 2003 Statut Modérateur Dernière intervention 13 avril 2024 - 17 sept. 2011 à 17:42
NHenry Messages postés 15112 Date d'inscription vendredi 14 mars 2003 Statut Modérateur Dernière intervention 13 avril 2024 - 17 sept. 2011 à 17:42
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/53543-enregistreur-de-noms
https://codes-sources.commentcamarche.net/source/53543-enregistreur-de-noms
17 sept. 2011 à 17:42
J'ai testé ta source, et voici mes commentaires :
- Actives Option Strict et corrige l'erreur.
- Une variable inutilisée (msg dans Form1.creernom)
- Dans l'onglet éRéférences" des propriétés du projet, retire l'"espace de nom importé" Microsoft.VisualBasic et corriges les erreurs.
- Remplace tous les MsgBox par MessageBox.Show
- Replace est un membre des String, merci de l'utiliser comme cela Chaine.Replace(...)
Sinon, mes remarques de mon premier message reste en attente.
Ta source n'a que très peu d'utilité si elle n'est pas réexploitable par d'autres personnes.
6 sept. 2011 à 21:56
6 sept. 2011 à 16:58
Maintenant, je me pose une question: si je tente d'enregistrer 2 fois le même nom dans la "base" il ce passe quoi (rien a ce propos dans les commentaires)? Et si je tente de lire un nom qui est en double?
Et une remarque: C'est normal si j'ai l'impression que supprimernom et creernom font l'inverse de ce qu'ils sont censé faire? Dans creernom on supprime quelques chose, et dans supprimernom on créer quelque chose... Je comprend plus rien.. C'est pour ça que je ne voyais pas la vérification des noms déjà existant.
Quelle bonne rigolade que de lire ce code source. On arrête pas le progrès.
5 sept. 2011 à 21:19
5 sept. 2011 à 21:05
5 sept. 2011 à 16:28
4 sept. 2011 à 18:28
1 sept. 2011 à 22:31
1 sept. 2011 à 19:10
Voici quelques remarques :
test.StartInfo.Arguments = "/C del bd.txt"
Tu peux supprimer un fichier, regardes dans System.Io.File
Ensuite, ton code est monobloc, essayes plutôt de faire une classe des gestion de la base de données, et peut être une classe pour chaque ligne.
Pour lire, écrire, ajouter les lignes, tu peux aussi utiliser :
System.Io.File.ReadAllLines, WriteAllLines et Append Text (je n'ai pas regardé si AppendLines existe aussi).
Dim i, j As System.Int32
C'est pareil que Integer, mais Integer est plus facilement lisible.
Ensuite, regarder la valeur de Text2 pour agir, pourquoi ne pas mettre plutôt des boutons d'option ?
Ensuite, quand tu codes, évites de conserver les noms par défaut de tes contrôles, le code est nettement moins compréhensible.
Je trouve qu'il y a beaucoup d'accès au fichier.
Dans ce cas, préfère utiliser une constante plutôt que de taper à chaque fois le nom du fichier, dans l'idéal, en passant par une base dans une classe, ton code sera réutilisable, plus facile à adapter et tu pourra ouvrir plusieurs fichier en même temps.