Revisions to How to make a valid Windows filename from an arbitrary string?

Added code format and, since the system felt this beautiful answer needed a beautiful form pushing me to edit further, added strong caption to edit

Source Link

edit approved Oct 2, 2012 at 15:16

Gabber

5.5k
7
37
49

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Edit: since GetInvalidFileNameChars()Edit:

Since GetInvalidFileNameChars() will return 10 or 15 chars, it's better to use a StringBuilderStringBuilder instead of a simple string; the original version will take longer and consume more memory.

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Edit: since GetInvalidFileNameChars() will return 10 or 15 chars, it's better to use a StringBuilder instead of a simple string; the original version will take longer and consume more memory.

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Edit:

Since GetInvalidFileNameChars() will return 10 or 15 chars, it's better to use a StringBuilder instead of a simple string; the original version will take longer and consume more memory.

added comments regarding performance

Source Link

edited Sep 15, 2010 at 16:08

Diego Jancic

7.5k
7
55
81

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Edit: since GetInvalidFileNameChars() will return 10 or 15 chars, it's better to use a StringBuilder instead of a simple string; the original version will take longer and consume more memory.

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

Edit: since GetInvalidFileNameChars() will return 10 or 15 chars, it's better to use a StringBuilder instead of a simple string; the original version will take longer and consume more memory.

Fixed string "_" to be a char '_'

Source Link

edited Dec 8, 2009 at 16:52

Daniel Fortunov

44.6k
28
84
107

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, "_"'_');
}

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, "_");
}

Try something like this:

string fileName = "something";
foreach (char c in System.IO.Path.GetInvalidFileNameChars())
{
   fileName = fileName.Replace(c, '_');
}

added 11 characters in body

Source Link

edited Mar 6, 2009 at 22:17

JaredPar

759.2k
152
1.3k
1.5k

Loading

Source Link

answered Mar 6, 2009 at 22:09

Diego Jancic

7.5k
7
55
81

Loading

Collectives™ on Stack Overflow

Return to Answer