CLASSE PERMETTANT DE ZIPPER DE(S) FICHIER(S) AVEC (OU SANS) LEUR ARBORESCENCE

sebmafate Messages postés 4936 Date d'inscription lundi 17 février 2003 Statut Membre Dernière intervention 14 février 2014 - 15 oct. 2007 à 10:10
econs Messages postés 4030 Date d'inscription mardi 13 mai 2003 Statut Membre Dernière intervention 23 décembre 2008 - 20 oct. 2007 à 08:30
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/44390-classe-permettant-de-zipper-de-s-fichier-s-avec-ou-sans-leur-arborescence

econs Messages postés 4030 Date d'inscription mardi 13 mai 2003 Statut Membre Dernière intervention 23 décembre 2008 24
20 oct. 2007 à 08:30
J'ai comme l'impression que le Do de BZipFile ne sert à rien. On en sort si 'flag' est false, mais comme il n'est jamais mis à true, et bien on sort directement après le premier passage.
Lutinore Messages postés 3246 Date d'inscription lundi 25 avril 2005 Statut Membre Dernière intervention 27 octobre 2012 41
16 oct. 2007 à 00:11
Peu importe qu'elles lancent des exceptions ou pas.. elles ne renverront jamais "false", la valeur de retour est donc inutile, elles devraient être marquées "void".
sebmafate Messages postés 4936 Date d'inscription lundi 17 février 2003 Statut Membre Dernière intervention 14 février 2014 37
15 oct. 2007 à 15:44
true ou... exception ;)
Lutinore Messages postés 3246 Date d'inscription lundi 25 avril 2005 Statut Membre Dernière intervention 27 octobre 2012 41
15 oct. 2007 à 15:37
Les méthodes BZipFile et BVerifyOutputPath renvoient toujours "true".
cs_Bidou Messages postés 5487 Date d'inscription dimanche 4 août 2002 Statut Membre Dernière intervention 20 juin 2013 61
15 oct. 2007 à 11:52
Après avoir passé en revue en 2min, j'ajouterais:

- Pourquoi les variables de classes sont publiques ?????
- public ArrayList c_alFiles = new ArrayList(); // Ca c'est du framework 1, voire les Generics
- (this.c_iPosDebCheminRacine) + @"" + sFileName; // Voire la class Path pour concaténer des path
- String.Format jamais utilisé ???
- Even. utiliser un using pour les FileStream...

Rien d'autre au premier coup d'oeil.
Mitch Buchannon Messages postés 1 Date d'inscription vendredi 7 juillet 2006 Statut Membre Dernière intervention 15 octobre 2007
15 oct. 2007 à 10:29
Merci pour les remarques!
Je prends note des commentaires XML.
sebmafate Messages postés 4936 Date d'inscription lundi 17 février 2003 Statut Membre Dernière intervention 14 février 2014 37
15 oct. 2007 à 10:10
Juste quelques remarques rapides sur ton code...
- Pourquoi n'utilises-tu pas les commentaires XML pour tes méthodes ?
- En .net on utilise la syntaxe Camel : toutes les méthodes et les propriétés doivent avoir une majuscule

une erreur ici : (ligne 287)
//Close the outputstream if it's null
if (baseOutputStream == null)
{
baseOutputStream.Close();
}

si baseOuputStream est nul alors l'appel de la méthode Close() soulèvera une exception.

c'est juste une lecture en diagonale.
Rejoignez-nous