CASSE BRIQUE DEBUTANT

Commentaire de Bidou le 22/05/2007 21:15:52 administrateur CS

Salut,
Niveau performance, c'est vraiment pas le pied. Je n'ai pas regardé ton code en détail mais l'ai seulement survolé.
Je pense que tu as fait quelques grosses erreurs de conceptions: Il n'y pas vraiment de structure dans le programme.

Pourquoi ne pas faire un userControl, qui va s'occuper du dessin, et ensuite créer des objects Cercle, Rectangle etc. comme tu as fait qui vont être dessinés par ce Control?
En gros, les objects ne font pas ce qu'ils doivent faire dans ton code: ce n'est pas un cercle qui se dessine, c'est ton UserControl qui dessine un Cercle. C'est pas ta forme qui contient la logique pour faire bouger les éléments, c'est le UserControl.

Une fois la conception réalisée, tu pourras te concentrer sur les performances...

Commentaire de ptitdam17 le 23/05/2007 13:31:07

Salut sebmafate, pourrais tu faire une critique constructive stp ? :o)

Commentaire de ptitdam17 le 23/05/2007 13:43:13

T'inquietes pas ta petite blague était marrante, mais c'est juste pour avoir ton avis pour pouvoir améliorer l'application. Je suis débutant alors des conseils de pro me dérrange pas ;)

Commentaire de Servlax le 23/05/2007 22:09:03

Quelques commentaires, puisque j'ai moi aussi développé un casse-briques en tant qu'exercice il y a quelques mois lorsque je me formais à C# et à la plateforme .NET (mon passé est du C++) :

1. L'application est beaucoup trop lente, et cela n'est pas dû à la machine (je dispose très d'une forte configuration, à 4 Go de RAM). Le plus gros problème vient à mon avis du fait que le déplacement de la planche engendre une invalidation de la Form. Cette invalidation ne devrait pas avoir lieu au déplacement de la souris. Le déplacement de la planche devrait avoir lieu en même temps que le déplacement de la bille, par exemple.

2. Pourquoi ne pas utiliser les commentaires XML ?

3. Il serait préférable de mieux respecter les conventions de nommage .NET (nom des méthodes en PascalCase, etc.).

4. Il y a un problème d'architecture générale, et notamment trop de code dans la Form principale, ce qui rend la lecture malaisée. Comme BIDOU, je suggère la création d'un contrôle qui à chaque évènement du timer, ordonne aux objets (planche, bille, etc.) de se redessiner à leur nouvelle position. Toute la logique propre à ces objets et qui n'est pas partagée doit être encapsulée dans ces classes.

5. Au niveau du gameplay, je trouve que les objets sont dessinés beaucoup trop petits !

Pour l'instant, en effet, ça ne casse pas des briques, mais une fois ces points réglés, ça devrait être beaucoup mieux =)

Commentaire de Servlax le 24/05/2007 19:36:31

Pour faire un contrôle personnalisé, il suffit de créer une classe qui hérite de Control ou de l'un de ses enfants. De mémoire, ma version héritait d'un simple Panel.