POO ET UTILISATION DE QUELQUES BOITES DE DIALOGUE (SAVE ET OPEN) ET DES USERCONTROLS

Commentaire de PCPT le 28/05/2009 09:28:03 administrateur CS

salut,
"go to zip" pourquoi pas, encore faut-il le joindre :)

Commentaire de Mayzz le 28/05/2009 13:19:36

Bonjour,

Ton code n'est pas mal, c'est un bon tuto pour débutant, cependant je trouve que tu as mal choisi l'exemple de programme.

Pour ce type de programme on utilise en générale une base de données et on évite ainsi de saisir au moins 50% du code qui compose ton application, si ce n'est plus...

Le petit reproche que j'aurais à faire à ta source est que si un nouveau se lançant dans le développement lit ta source, pour créer une application de données il suivra ton exemple qui n'est pas en effet le plus approprié.

Pour le reste, ton code est propre, un peu trop légèrement commenté, mais comme il est assez simple à comprendre cela compense.

Ce que je trouve simpa en revanche, c'est qu'il démontre l'utilisation des contrôles de base et de leurs propriétés, et la création d'une classe "métier".

Commentaire de Mayzz le 28/05/2009 14:58:32

Oui, oui biensur, je l'ai compris, d'ailleur ta source est très bien, c'est juste l'exemple qui n'est pas très approprié, il vaut mieux préciser au newbies que cette source est à but démonstartif, et que les applications de données ne se gère pas de cette manière (quoi que, pour une petite application c'est faisable, la preuve !), mais si ils ont l'idée de se lancer dans la création d'un projet de gestion d'une entreprise (comme c'est le cas pour la pluspart des nouveaux qui débute en vb ou qui font leurs études), avec cette méthode c'est perdu d'avance !

Par contre quand je répond au forum, il y a souvant des questions sur les contrôles de base de vb, je pourrais alors les redirigés vers ta source ;)

Commentaire de nhervagault le 28/05/2009 20:03:28 administrateur CS

Bonjour,

Je ne vois pas trop l'interet de la source.

1 --> Les boutons ont leur nom par défaut
2 --> En POO on n'utilise pas les modules (mais des classes)
3 --> Utilise les List<T> a la place de l'arraylist
4 --> utilise option strict et option explicit (pour ne pas avoir des conversions implicite cas de l'arraylist)
5 --> factorise le remplissage des champs car dans ce code tu as 5 fois la meme chose
6 --> evite d'utiliser des fonctions specifique vb (val --> int32.parse)

Voici quelques points pour améliorer ton programme.

Commentaire de kilvanox le 28/05/2009 22:35:16

oups j'ai oublié de précisé que l'interet de la source et purement educatif (sorte de tutoriel explicatif et démonstratif)
et je pense que les débutants ne seront pas d'accord avec toi ;)

cordialement,

Commentaire de kilvanox le 28/05/2009 23:44:12

pour le point 5 j'y avait pensé c'est juste que j'avais oublié de l'appliquer je le ferait tres prochainement

merci ;)

Commentaire de PCPT le 31/05/2009 16:55:15 administrateur CS

conseiller un débutant de ne pas nommer ses contrôles parce qu'il s'y retrouve mieux, c'est aussi l'empêcher de comprendre les sources "normalement" codées

qu'on hésite à renommer un bouton en "btnAction, Btn_Action, Cmd_Action", ok. mais plus généralement avoir 50 boutons et les laisser nommés par défaut ne peut que nuire à tout le monde (lecteur averti qui cherche lequel est où, lecteur débutant qui passe encore plus de temps à chercher tout en prenant aussi de mauvaises habitudes, et aussi l'auteur qui reste dans le 2e cas)


le framework et plus généralement dotnet (vb, c#, etc..) donc l'IDE par lui-même ont été fait de manière à accélérer le temps de RéALISATION de code
(reste à prendre le temps de comprendre les subtilités de quoi utiliser au lieu de quoi, même s'il y a toujours plusieurs manières de faire la même chose)

le temps gagné ne doit pas être re-perdu parce qu'on a pas ouvert son projet pendant 2 mois.
on lit par exemple "
sub cmdReadFile_Click(...)
dim reader as io.textreader = io.file.opentext(txtFile.text)"
, on sait tout de suite qu'on va récupérer le contenu d'un 'fichier' dont le 'chemin' est dans une 'zone de texte', ce par l'action 'click' d'un 'bouton'. on n'a pas besoin d'aller vérifier sur le formulaire où est le bouton command45 (nommer suffit ici à ne pas avoir besoin de connaître son emplacement), ni que c'est un chemin ou une chaine (on sait), ni que c'est une zone de texte, ni ou elle est, etc...

on doit pouvoir comprendre à la simple lecture sans même (pour les petits projets) avoir besoin de regarder l'interface


on peut comprendre éventuellement qu'un label NON utilisé par code (ex : label13.text = "machin") ne soit pas nommé, ce n'est qu'une étiquette graphique utilisée qu'en mode design, pour tout contrôle lié à du code il est nécessaire de se forcer à s'organiser (surtout comme le souligne nhervagault quand on est sujet à toucher à plusieurs langages)

on comprend bien sûr que çà ne se fasse pas en un claquement de doigts, mais refuser cet effort ne peut mener qu'à des confusions et, quand on souhaite partager ses sources, qu'à - au final - des échanges inutiles par nature
(cette intervention en haut de la liste évidemment :))