econs
Messages postés4030Date d'inscriptionmardi 13 mai 2003StatutMembreDernière intervention23 décembre 200824 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és3246Date d'inscriptionlundi 25 avril 2005StatutMembreDernière intervention27 octobre 201241 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és4936Date d'inscriptionlundi 17 février 2003StatutMembreDernière intervention14 février 201437 15 oct. 2007 à 15:44
true ou... exception ;)
Lutinore
Messages postés3246Date d'inscriptionlundi 25 avril 2005StatutMembreDernière intervention27 octobre 201241 15 oct. 2007 à 15:37
Les méthodes BZipFile et BVerifyOutputPath renvoient toujours "true".
cs_Bidou
Messages postés5487Date d'inscriptiondimanche 4 août 2002StatutMembreDernière intervention20 juin 201361 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és1Date d'inscriptionvendredi 7 juillet 2006StatutMembreDernière intervention15 octobre 2007 15 oct. 2007 à 10:29
Merci pour les remarques!
Je prends note des commentaires XML.
sebmafate
Messages postés4936Date d'inscriptionlundi 17 février 2003StatutMembreDernière intervention14 février 201437 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.
20 oct. 2007 à 08:30
16 oct. 2007 à 00:11
15 oct. 2007 à 15:44
15 oct. 2007 à 15:37
15 oct. 2007 à 11:52
- 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.
15 oct. 2007 à 10:29
Je prends note des commentaires XML.
15 oct. 2007 à 10:10
- 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.