NAVIGATEUR WEB MODIFIÉ

Commentaire de FREMYCOMPANY le 09/05/2008 20:37:01

J'ai noté 4 :

Positifs :
* Code (généralement) assez explicite
* Bon exemple de l'utilisation du COM managé
* Code fonctionnel (apparemment en tout cas)

Négatifs :
* Un contrôle managé (WebBrowser) existe déjà, et il permet d'appeler les véritables objets ActiveX qui sont derrière, ce qui permet à la fois simplicité (interface managée) et finesse (objet COM natif).
* Ton code montre la page d'accueil de FireFox, mais il s'agit du navigateur Trident d'IE, ce qui est assez trompeur.
* Des tas de navigateurs basé sur IE existent déjà, donc le controle navigateur ne sera jamais utilisé comme "navigateur" à part entière.
* Noms des formulaires et des modules peu clairs et peu de commentaires "intéressants"
* Question de gout personnel, mais bon, ca a peut-être joué, mais ton code a des variables francaise et des variables anglaise, ce qui crée un peu la confusion. Mieux vaut choisir (moi je préfère le tout anglais).

Commentaire de DrDoom544 le 10/05/2008 00:38:19

Je vais répondre aux points négatifs de FREMYCOMPANY :

* Un contrôle managé (WebBrowser) existe déjà, et il permet d'appeler les véritables objets ActiveX qui sont derrière, ce qui permet à la fois simplicité (interface managée) et finesse (objet COM natif).

Rep : Si je me trompe pas, je les ai utilisé pour les options Copier, Coller, Supprimer, etc. Si tu dit ça pour les formulaires "Favoris" et "Historique", je ne savais pas qu'on pouvais aussi appeler ces options. Les coder m'a aussi permis de travailler avec des DataTables.

* Ton code montre la page d'accueil de FireFox, mais il s'agit du navigateur Trident d'IE, ce qui est assez trompeur.

Rep : Et pourquoi se serait négatif ? Il aurait pu avoir n'importe quoi d'autre; Google, MSN, Yahoo, etc. En plus, c'est l'auteur du code qui avait mit cette page alors, je jugeais inutile de le modifier.

* Des tas de navigateurs basé sur IE existent déjà, donc le controle navigateur ne sera jamais utilisé comme "navigateur" à part entière.

Rep : J'avoue que le contrôle "WebBrowser" n'est pas là pour faire un navigateur, mais plutôt l'utiliser pour faire afficher des pages web dans nos programmes mais rien n'empêche de l'utiliser pour coder un navigateur. Mon bût n'était pas de faire un navigateur qui révolutionnera le monde des navigateur web mais d'apprendre.

* Noms des formulaires et des modules peu clairs et peu de commentaires "intéressants"

Rep : Pour les formulaires, si c'est celui des noms de fichier, j'avoue que c'est mauvais de laisser form1.vb. Je voulais les changer et sa ma oublier, je vais le rectifier. Pour les commentaires, j'avoue que j'ai beaucoup commenté des ligne pour rien, mais c'etait pour mon prof qui voulais ça hyper-commenté. Je n'avais pas beaucoup de choix. Aussi j'ai toujours commenté pour me comprendre et j'ai tendance à oublier les autre qui passerons après moi.

* Question de gout personnel, mais bon, ca a peut-être joué, mais ton code a des variables francaise et des variables anglaise, ce qui crée un peu la confusion. Mieux vaut choisir (moi je préfère le tout anglais).
Rep : J'ai été habituer a coder en français, les variables anglaises sont de l'auteur du code de base. J'avoue que j'aurais pu continuer en anglais, mais comme à la base il avait déjà du français (Ecrire et Lire par exemple) et de l'anglais (BackOn et FwdOn), j'ai choisi le français.

Voilà j'espère que mes réponses sont pertinente.

Commentaire de Arnotic le 12/05/2008 22:28:49 administrateur CS

Le mérite d'avoir était fait. En plus l'auteur est humble et conscient des choses. Je pense que ça était pour lui trés positif de coder cette source pour apprendre.

Et je pense sincèrement que l'auteur à un bon esprit :) Alors encouragement !!! :) Faut pas démoraliser les troupes qui font des efforts et receptives aux critiques de façon si objective !! :)

Commentaire de DrDoom544 le 15/05/2008 16:06:52

Oui, je le sais mais mes connaissances en vb.net ne sont pas assez au point pour permettre de le faire... Je vais essayer de trouver une solution prochainement.