19

What is a good regular expression that can validate a text string to make sure it is a valid Windows filename? (AKA not have \/:*?"<>| characters).

I'd like to use it like the following:

// Return true if string is invalid.
if (Regex.IsMatch(szFileName, "<your regex string>"))
{
    // Tell user to reformat their filename.
}

4 Answers 4

50

As answered already, GetInvalidFileNameChars should do it for you, and you don't even need the overhead of regular expressions:

if (proposedFilename.IndexOfAny(System.IO.Path.GetInvalidFileNameChars()) != -1)
{
  MessageBox.Show("The filename is invalid");
  return;
}
3
  • simple right parenthesy placement mishap in the if statement, but awesome thanks! Sep 19, 2008 at 6:47
  • 8
    So "C::::::::" is a valid filename?
    – Andrew
    Feb 18, 2010 at 20:25
  • Andrew: no, this is just a rough basic check for characters.. There are lots of other reasons accessing a file with proposedFilename may fail: file (doesn't) exist, read-only, path too long, directory doesn't exist, name is a dos device files (like 'nul' or 'prn') etc.
    – Isak Savo
    Mar 2, 2010 at 9:30
6

This isn't as simple as just checking whether the file name contains any of System.IO.Path.GetInvalidFileNameChars (as mentioned in a couple of other answers already).

For example what if somebody enters a name that contains no invalid chars but is 300 characters long (i.e. greater than MAX_PATH) - this won't work with any of the .NET file APIs, and only has limited support in the rest of windows using the \?\ path syntax. You need context as to how long the rest of the path is to determine how long the file name can be. You can find more information about this type of thing here.

Ultimately all your checks can reliably do is prove that a file name is not valid, or give you a reasonable estimate as to whether it is valid. It's virtually impossible to prove that the file name is valid without actually trying to use it. (And even then you have issues like what if it already exists? It may be a valid file name, but is it valid in your scenario to have a duplicate name?)

0

Why not using the System.IO.FileInfo class, together with the DirectoryInfo class you have a set of usefull methods.

0

Path.GetInvalidFileNameChars - Is not a good way. Try this:

if(@"C:\A.txt".IndexOfAny(System.IO.Path.GetInvalidFileNameChars()) != -1)
{
  MessageBox.Show("The filename is invalid");
  return;
}
2
  • It depends on what you want to achive. I came here from a google query and I want to check only for valid file names (not a full path) and I'm pretty sure you can't have a file called "c:\a.txt" inside a folder ;) Jun 18, 2010 at 11:20
  • Yes, for checking only name of the file Path.GetInvalidFileNameChars() acceptable, but Naming Conventions (msdn.microsoft.com/en-us/library/…) have a lot of interesting rules. Sep 13, 2010 at 8:26

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.