ONERADIOPLAYER

Commentaire de Bestdoud le 22/12/2008 17:21:25

en socket j'ai déja essayé et le résultat final est pas génial. on a des coupure à cause des tampons. A part faire un tampon énorme,j'ai pas trouvé.

Et ca se voit toute de suite que j'utilise des dlls à en regarder le contenu plus haut.

l'avantage c'est qu'on peut ausi écouter les radio en ogg.

ici j'ai pas trop recherché à faire un super truc. j'ai juste repris un ancien addon et laissé que le minimum. j'ai fait ca pour la personne que tu as conseillé de me contacter.

De plus j'ai plus trop le temps de coder. tous mes projets sont un peu laissé à l'abandon.

merci quand même pour ton commentaire

Commentaire de uaip le 22/12/2008 22:04:12

Saloute
Même si on s'en fiche, on peut remplacer le goto par un break (dans la dernière boucle) ou même mettre directement le return.
C'est le seul "défaut" que j'ai trouvé à dire ^^
Sinon concernant cet éternel débat dll/mIRC c'est vrai que malgré l'augmentation de la qualité, ça enlève carrément du charme au codage mIRC (ce n'est en aucun cas une critique).

Commentaire de wims le 23/12/2008 12:51:35

Non, le break ne fait qu'arrété la boucle, il ne sort pas de la routine en cours (le return lui stop la routine en cours DONC stop la boucle).
Mais Uaip a raison ton code est mal fichu, actuelement tu retourne %x - 1 dans nimporte quel cas (je sais pas si c'est voulu).
Mettre un break ou le même return (dans tout les cas il faut laissé le return de fin) et viré tes gotos reviendrai exactement au même que ton code actuel, mais on peut faire plus propre encore :

alias -l OneRP.testtitreradio {
var %a $remove($1-,$chr(32))
var %x 2
while ($right($gettok(%a,%x,40),1) != $chr(41)) inc %x 1
return $calc(%x - 1)
}

mais je pense pas que l'optimisation t'interesse vraiment ici, sinon... :)
En tout cas les regex font grandement defaut dans ton code.

Commentaire de uaip le 23/12/2008 15:10:56

Remplacer le goto par un return ne veut pas dire enlever le return initial.
Ton goto amène à une seule instruction : un return. Donc au lieu de laisser mIRC se débrouiller à chercher l'étiquette du goto, un return est, à mon avis, plus efficace, un break aussi.
Mais comme je l'ai dit, c'était juste histoire d'avoir un truc à dire sur la source ^^ (sinon j'ai rien trouvé d'autre à dire, à part peut-être le $numtok() recalculé à chaque passage de la boucle) mais bon, ça aurait été plus important si les tampons étaient manipulés en mIRC, or là c'est une dll qui s'en charge. Donc je répète que mes petits commentaires sont à prendre à la légère, sinon ça reste relativement bien codé, puis c'est sympa d'avoir ressorti ça pour aider quelqu'un.

Commentaire de WorldDMT le 27/12/2008 21:34:07

salut
j'ai regardé vite fais dejà on a commenté mais j'ai vu quelque ptites truc là

on 1:start:{
  hmake OneRP 100
  var %a $dll($OneRP.Amp,PluginManager,load input $shortfn($scriptdirplugins\in_vorbis.dll))
  var %a $dll($OneRP.Amp,PluginManager,load input $shortfn($scriptdirplugins\in_mp3.dll))
  var %a $dll($OneRP.Amp,PluginManager,load output $shortfn($scriptdirplugins\out_ds.dll))
}

pourquoi un hmake puisqu'il n'y a pas de hload?
tu peus enlevé et utilisé hadd -m le "-m" crée la hash table

pourquoi un tas de variable locale utilise /dll tout simplement

@+

Commentaire de WorldDMT le 28/12/2008 12:06:04

ok je vois

bon a propos de l'alias OneRP.current je vois dans else

      else {
        if ($len($hget(OneRP,stream2)) == 0) {
        }
        hadd OneRP stream $remove(%a,$chr(40),$chr(41))
        hadd OneRP stream2 $remove(%a,$chr(40),$chr(41))
      }

ici la condition "if ($len($hget(OneRP,stream2)) == 0)" ça jou quel role?
puisque tu a une accolade fermé avant la cmd
si je ne me trompe pas si la condition est vraie ou fausse la commande hadd fontionnera tjr non?

au passage quand tu met dejà des "()" enleve les "{}" c'est plus rapide

exp:

if (x == y) { cmd } tu peu faire if (x == y) cmd

aussi pour

if ((x ==y) || (x == z)) { cmd }

=>

  if (x ==y) || (x == z) { cmd }

ou

  if ((x ==y) || (x == z)) cmd

c'est juste une proposition

Commentaire de WorldDMT le 29/12/2008 10:36:35

effectivement wims c'est pk j'ai mis "cmd" :) si non pour un vide on aura un msg d'erreur :)