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...