SUBTITLE REVERSE .SRT

Commentaire de Bidou le 18/08/2008 08:09:21 administrateur CS

Pas testé, mais en regardant le code, on reste un peu sur sa faim (je ne parle pas de la traduction)...
Le plus gros problème, c'est qu'absolument rien n'est modularisé, en gros on nous sert un code avec tout dans le main => inutilisable.

Y'a d'autres problèmes +/- gênant dans le code, comme par exemple le parsing de l'HTML. Tu aurais probablement mieux fait d'utiliser les Regex. Autres choses en vrac juste en jettant en rapide coup d'oeil:

- Les concaténations dans une boucle se font avec un StringBuilder!
- numline = numline + 1 => numline++; i = i + 2 => i += 2;

- catch (Exception e)
{
   throw e; // Utile ?
}

- StreamReader _Reader n'est jamais fermé
- FileStream fs n'est jamais fermé

try
{
   if (!File.Exists(strFileName))
   {

- Pourquoi le if est dans le try?
- etc etc...

Une mise à jour pourrait être la bienvenue :)

Une remarque positive quand même pour finir (ben oui je suis pas seulement méchant) : L'idée est plutôt bonne et utile (même si ça reste une traduction google...)

Commentaire de Bidou le 18/08/2008 17:58:38 administrateur CS

TMCUH> Si les regex ne fonctionnaient pas, c'est sûrement parce que tu les avais mal programmées... non?

Commentaire de Zaltez le 19/08/2008 00:06:41

Tes remarques sont justifiées Bidou, je viens de mettre à jour la source en conséquence :)
Finalement j'ai utiliser les "regular expression" mais je ne suis pas sur que cela apporte grand chose au niveau du code comme au niveau des performances ;)

Mais je ne suis  pas arriver à passer par mon delégé depuis ma classe GoogleTranslate pour mettre à jour la progressbar. La methode Invoke() ne marche vu que je ne suis pas sur ma form! Je sais pas si vous avez une idée? Pour l'instant c'est une solution temporaire avec un timer et des variables globales.

Commentaire de Zaltez le 19/08/2008 16:38:50

Salut,
Le backgroundWorker me premettrait de mettre plus facilement a jour ma progressBar? J'ai du mal à comprendre tu pourrais m'éclairer stp...

Commentaire de Bidou le 19/08/2008 22:28:31 administrateur CS

Il faudrait faire des testes pour savoir si c'est plus lent ou pas dans ce cas ci, mais je dirais que globalement, on a meilleur temps d'utiliser les expressions régulières car:

- Il faut (beaucoup) moins de code pour arriver au même résultat;
- Probablement moins de temps aussi pour coder (pour autant qu'on maîtrise la chose, ce qui n'est pas très simple au début je trouve...)
- Les temps de réponses sont globalement plus rapide. A ce niveau, il faut bien gérer l'option compilée de la Regex (à utiliser seulement pour les gros textes, sinon, le temps de compilation est beaucoup trop important par rapport au temps total). Pour les petits textes cependant, des substrings et autres sont peut-être plus rapide...