Beautiful Code

4 views

Skip to first unread message

Olivier Spinelli

unread,

Oct 15, 2011, 6:31:45 AM10/15/11

to civik...@googlegroups.com

Suite à un récent commit, une petite remarque :

if( extension.Substring( 0, 1 ) == "." ) //If the extension contains the "dot"
_path += extension;
else
_path += "." + extension;

Substring crée une nouvelle chaîne en mémoire. Il faut et il suffit d'écrire:

if( extension[0] == '.' ) //If the extension contains the "dot"
_path += extension;
else
_path += "." + extension;

Ce qui travaille directement au niveau des caractères: pas d'allocations mémoire superflues, un Spi content :-).
Dans la même veine, je voit souvent ça
if( s.EndsWith("/") )
qui peut être avantageusement remplacé par s.Length > 0 && s[s.Length-1] == '/'

Pour info: j'ai mis à jour le CK-Core version .Net 4.0 avec cette évolution du TemporaryFile.