DECODEUR/ENCODEUR DE PLAYLIST AU FORMAT PLS/M3U

Commentaire de BananaTree le 17/10/2009 11:19:27

slt,

j'ai pas téléchargé, mais juste jeté un rapide coup d'oeil sur l'as (comme d'hab lol)

je penses que tu devrais virer le loader et définir l'action de l'event complete en fonction "trim".
ça te permetrais de renomer ta classe en "???Parser.as" et la rendrait plus souple d'utilisation (utilisation avec massapi, ou des systèmes de loader intégrée comme ceux du spas 3.0)

beau travail...

++
pascal

Commentaire de BananaTree le 17/10/2009 13:48:36

en as3 une classe statique peut être un event dispatcher, mais ce n'est pas le sujet.

ta classe actuelle playlist.as est une classe utilitaire. il est don normale qu'elle fournisse des methodes de classe qui permettent de traiter telle ou telle action.
par contre, le chargement d'un fichier n'a rien à voir dans cette logique. cela génère forcement des doublons de code, puisque l'ensemble des dev as3 utilisent leurs propres gestionaires de chargement.

je te conseilles (c nest qu'un conseil ;) ) de transformer "PlayList.as" en "PlayListParser.as", d'en supprimer toute la logiquee de gestion de chargement. crée ensuit ta classe PlayList.as qui utilisera les methodes de "PlayListParser.as" et la gestion du chargement.

en faisant cela, tu seras bien plus en phase avec les règle de l'orienté objet. et ça évitera aux developpeurs de modifier tes classes à chaque fois qu'une nouvelle mise à jour est dispo.

pascal

Commentaire de BananaTree le 17/10/2009 19:01:30

slt,

en fait j'ai pas trés vite (20secondes maxi), je parlais de cette fonction (donc aucun xml effectivement)

private static function loadComplete(evt:Event):void {
var re:RegExp = /\n/g; //Efface les doubles sauts ...
loader.data = loader.data.replace(re, ""); // ... de ligne (Pc uniquement)
tab = loader.data.split("\r"); //place les lignes dans un tableau (indépendants des types de playList)
}

merci d'avoir corrigé gremlins7 ;)
++
pascal

Commentaire de dadoo1 le 18/10/2009 15:12:19

Format de fichier inattendu ! super on a déjà changer de version de flash ? c'est la combien j'en suis a la 8 :(

Commentaire de gremlins7 le 19/10/2009 10:59:37

Ah tiens, encore un spéléologue...

Commentaire de pegase31 le 19/10/2009 12:08:08 administrateur CS

Le fichier .as est tout à fait ouvrable avec n'importe quel notepad, cependant, le code est en as3 .. et flash 8 ne fait que de l'as2.

Peg'

Commentaire de top30 le 22/10/2009 22:50:45

Ben moi, aprés 2 whisky-coca et le plaisir de voir lire une bonne source je te dirais:
Une interface déterminant ce qu'est une Playlist"...
- IPlaylist.as
     get length():uint;
     getSongURLAt( index:uint ):String ;
     getSongTitleAt( index:uint ):String ;
     getSongDurationAt( index:uint ):int;
     addSong( url:String, title:String="", duration:Number=0 ):Boolean;
     removeSong( url:String ):Boolean;
     removeSongAt( index:uint ):Boolean;
     moveSong( from:uint, to:uint ):Boolean ;
     encode(data:XML):String;
     decode(data:String):XML;


La classe base contenant les playlist :
- AbstractPlaylist.as
     get length():uint;
     getSongURLAt( index:uint ):String ;
     getSongTitleAt( index:uint ):String ;
     getSongDurationAt( index:uint ):String ;
     addSong( url:String, title:String="", duration:Number=0 ):void;
     removeSong( url:String ):void;
     removeSongAt( index:uint ):void;
     moveSong( from:uint, to:uint ):Boolean ;
     encode(data:XML):String; (JETANT UNE ERREUR)
     decode(data:String):XML; (JETANT UNE ERREUR)

--> PlaylistPLS.as
    extends AbstractPlayList
    implements IPlaylist
    --> encode(data:XML):String ; (EFFECTUANT l'ENCODAGE ET SUPPRIMANT l'ERREUR)
    --> decode(data:String):XML ; (EFFECTUANT l'DECODAGE ET SUPPRIMANT l'ERREUR)

--> PlaylistM3U.as
    extends AbstractPlayList
    implements IPlaylist
    --> encode(data:XML):String ; (EFFECTUANT l'ENCODAGE ET SUPPRIMANT l'ERREUR)
    --> decode(data:String):XML ; (EFFECTUANT l'DECODAGE ET SUPPRIMANT l'ERREUR)

A part ca, j'ai rien compris au code...
AH AH AH AH ! C'est vrai en plus...

Commentaire de top30 le 22/10/2009 22:59:27

Je profite qu'il faille un commentaire pour pouvoir noter (j'ai oublier) pour te préciser que j'ai inverser le concept pour l'interface "playlist":
--> encode(data:String):XML ; (EFFECTUANT l'ENCODAGE ET SUPPRIMANT l'ERREUR)
--> decode(data:XML):String ; (EFFECTUANT l'DECODAGE ET SUPPRIMANT l'ERREUR)

J'ai mis 9.5 (9 avec la souris, et 0.5 avec tout mon respect..)

Commentaire de top30 le 23/10/2009 15:49:13

¿Pourquoi si (selon moi) elle ne peut pas fonctionner seule?
Il lui faut absolument une sous classe qui surécrirait les fonctions "encode" et "decode".
Je serais d'accord avec toi, si, les instances de superclasse "PlaylistBase" pouvaient fonctionner seules, hors dans le schémas proposé, ce n'est pas le cas !

Dis nous vite pourquoi "base", jai hâte de me coucher moins bête...

Commentaire de BananaTree le 23/10/2009 19:39:25

+1 à gremlins7, c vrai que je chipote ;)
(d'autant que je factorise de cette façon depuis SPAS 2.0...)

mais je pense que ça mérite un peu + que 7 vu que ça sort largement du train train des codes à 2 balles qu'on à l'habitude de subir...

++ et bon codes

Commentaire de top30 le 24/10/2009 19:18:57

Aprés une certaine réflexion je suis d'accord avec Peg'.
Une seule classe est bien suffisante. Une classe instanciable comme décrite avant avec en plus 4 méthodes statique :
- PlayList.encodeM3U(xml):String
- PlayList.decodeM3U(string):XML
- PlayList.encodePLS(xml):String
- PlayList.decodeM3U(string):XML

J'ai changé le nom du zip précedent.
http://hopemultimedias.com/lab/pegase31-classUnic.zip (la nouvelle version)
http://hopemultimedias.com/lab/pegase31-multiClasses.zip (l'ancienne version)