MEDIA PLAYER COMPLET

Commentaire de Also know as le 23/06/2006 19:24:20

Salut,
Voici donc mes remarques par rapport à ton logiciel :
> Tout d'abord la bibliothèque :
- Faudrait revoir ta phrase si le fichier ini n'existe pas, d'une elle n'est pas bien écrite, et de deux on ne comprend pas trop cette phrase la première fois que l'on utilise ce logiciel
- Une fois qu’on à choisi un dossier avec des musiques, celui-ci ne peu pas se finir, pour ma part j'ai une erreur du style : "Violation d'accès à l'adresse XXXXXXXX dans le module "Project1.exe". Lecture de l'adresse XXXXXXXX.
- Si je clique du coup sur annuler, pareil il ne peut pas terminer l'opération car un problème intervient et un message du même style apparait
- Il devient donc impossible de quitter la fenêtre
- On ne peu sélectionner qu'une musique par une musique, il faudrait revoir globalement l'histoire de la bibliothèque
+ Le concept d'ajout de cette manière est bien faite et bien pensée
+ Si on ne tiens pas compte des erreurs ci-dessus, tout fonctionne correctement

> Point de vue de la PlayList
> J'ai eu des problèmes mais alors de tous les côtés
- Si je fais clic-droit ==> Nouveau ==> Nom de la playlist "Test" ou encore "ma playlist", le programme plante et s'arrête à cette ligne : "fichier.WriteString('playlist','nom',test);"
- Clic droit ==> Voir ==> Le programme plante à nouveau, normal vu qu'il n'existe pas de playlist (indice hors limite). Voici son message : "frame61.ComboBox1.Text:=listbox1.Items.Strings[listbox1.itemindex];"

> Sur le lecteur même
- On ne sait pas quelle chanson passe, je pense qu'il faudrait avoir la liste des chansons sélectionnées, à part ou dans la même fenêtre, mais visible dés le départ
- On voit que les boutons (Pause, lecture, etc) scintillent légèrement, j'ai testé avec le "DoubleBuffered" sur la form mais cela n'y change strictement rien.

J'aurais plein d'autre remarques à faire sur le lecteur, mais je n'ai aps envie que tu le prennes mal, car ce n'est pas mon but, bien au contraire.
Ensuite c'est dommage que tu ne commente pas de temps à autre ton code source, cela pourrait en valoir la peine pour les débutants comme moi.
Ce qui est bizarre c'est que tu utilises des Form, des Frames, de tous les côtés, mais alors ca par partout ! :D, on sait plus ou on est à la fin ^_^.

Donc voilà

Au final je ne pense pas que je vais la noter car je ne suis pas assez bon dans ce domaine pour donner mon point de vue.

@+

Commentaire de jlen100 le 23/06/2006 20:06:12

salut,
en plus des remarques ci dessus:
composants non renommés et pas de commentaires ce qui le code illisible

@+
jlen

Commentaire de Francky23012301 le 23/06/2006 20:20:27

Second post, je viens de voir ton source : le source est mal structuré. Tu as mal répartie les fonctions et procedures.

Je rejoins JLen : c'est mal codé et pas commenté. En plus d'erreurs flagrantes, tu aurais pu simplifier le source.

Désolé si on est durs mais quand on fait du réchauffer, en général les gens sont intransigeants

A+

Commentaire de jlen100 le 23/06/2006 20:22:14

faut dire à sa décharge :
Arrivée sur CodeS-SourceS le : 23/06/2006 16:46:56
Date création : 23/06/2006 16:51:46
autrement dit il 'a surement pas eu le temps de prendre connaissance des us et coutumes du site!!! ni même de lire ce qui existait déjà.


Bien venue quand même et corriges les imperfections

@+
jlen

Commentaire de Francky23012301 le 23/06/2006 21:03:44

Re,

Alors c'est pas bien de pointer du doigt les copains quand on ouvre "les hostilités" : ;).
En plus vu le nombres on aurait dit un listing : Mdrrrrr.

Sérieusement en ce qui concerne le MediaPlayer, il y a en a deja tellement sur CS que il me semblait important de le souligner. En plus je l'ai fais avec humour.

Pour le code : bon il y a un point positif : il y a eu du travail c'est flagrant vu la taille du code (en partant du principe que c'est un débutant). Donc oui continue  bien évidemment ... La prochaine fois que tu postes demandes aux membres avancés ce qu'ils en pensent et demande eventuellement une correction.

Pour conclure, continue à coder : tu me sembles bien partis pour etre un bon codeur

A+

Commentaire de delphinait le 24/06/2006 02:01:46

Bonjour , tres bon prog  bravo  , vraiment complet !!!

Commentaire de Matt 261 le 24/06/2006 14:43:16

Et allez, un de plus...

Commentaire de jlen100 le 27/06/2006 18:25:52

-->Michele58 tu pourrais traduire ton charrabia?
infonctionnable ça veut dire quoi?
Inutile d'écorcher la langue de Molière.
ce genre de remarques ne font pas avancer les choses si tu n'expliques pas ce qui ne te convient pas.
"Ce qui ce conçoit aisément s'exprime clairement"

@+
jlen

Commentaire de N_M_B le 05/07/2006 19:21:44

Bonjour ! bein moi je voulais juste te dire que ton lecteur me sera bien utile pour entendre de la zik quand mon pc commence a ralentire il est asser leger ! je te parle pas du code puisque je ne me suis pas encore lancé dans le multimedia mais tout ce que je peut te dire c'est continue !