C#サニタイズファイル名


174

最近、たくさんのMP3をさまざまな場所からリポジトリに移動しています。ID3タグを使用して新しいファイル名を作成していました(おかげで、TagLib-Sharp!)System.NotSupportedException

「指定されたパスの形式はサポートされていません。」

これはFile.Copy()またはによって生成されましたDirectory.CreateDirectory()

私のファイル名をサニタイズする必要があることを理解するのに長い時間はかかりませんでした。だから私は明白なことをしました:

public static string SanitizePath_(string path, char replaceChar)
{
    string dir = Path.GetDirectoryName(path);
    foreach (char c in Path.GetInvalidPathChars())
        dir = dir.Replace(c, replaceChar);

    string name = Path.GetFileName(path);
    foreach (char c in Path.GetInvalidFileNameChars())
        name = name.Replace(c, replaceChar);

    return dir + name;
}

驚いたことに、私は例外を受け続けました。「:」はPath.GetInvalidPathChars()パスのルートで有効であるため、のセットには含まれていないことが判明しました。それは理にかなっていると思いますが、これはかなり一般的な問題でなければなりません。パスをサニタイズする短いコードを誰かが持っていますか?私はこれを最も徹底的に考え出しましたが、おそらくやり過ぎだと感じています。

    // replaces invalid characters with replaceChar
    public static string SanitizePath(string path, char replaceChar)
    {
        // construct a list of characters that can't show up in filenames.
        // need to do this because ":" is not in InvalidPathChars
        if (_BadChars == null)
        {
            _BadChars = new List<char>(Path.GetInvalidFileNameChars());
            _BadChars.AddRange(Path.GetInvalidPathChars());
            _BadChars = Utility.GetUnique<char>(_BadChars);
        }

        // remove root
        string root = Path.GetPathRoot(path);
        path = path.Remove(0, root.Length);

        // split on the directory separator character. Need to do this
        // because the separator is not valid in a filename.
        List<string> parts = new List<string>(path.Split(new char[]{Path.DirectorySeparatorChar}));

        // check each part to make sure it is valid.
        for (int i = 0; i < parts.Count; i++)
        {
            string part = parts[i];
            foreach (char c in _BadChars)
            {
                part = part.Replace(c, replaceChar);
            }
            parts[i] = part;
        }

        return root + Utility.Join(parts, Path.DirectorySeparatorChar.ToString());
    }

この機能をより速く、よりバロックを少なくするための改善があれば、高く評価されます。


回答:


314

ファイル名をクリーンアップするには、次のようにします

private static string MakeValidFileName( string name )
{
   string invalidChars = System.Text.RegularExpressions.Regex.Escape( new string( System.IO.Path.GetInvalidFileNameChars() ) );
   string invalidRegStr = string.Format( @"([{0}]*\.+$)|([{0}]+)", invalidChars );

   return System.Text.RegularExpressions.Regex.Replace( name, invalidRegStr, "_" );
}

3
問題はファイル名ではなくパスに関するもので、これらの無効な文字は異なります。
Dour High Arch、

15
たぶん、しかし、同じ問題があったとき、このコードは確かに私を助けました:)
mmr

8
そして、もう1人の潜在的に素晴らしいSOユーザーが歩きます...この機能は素晴らしいです。Adrevdmに感謝...
ダンローゼンスターク2009

19
素晴らしい方法。その予約語がまだあなたを噛むことになることを忘れないでください、そしてあなたはあなたの頭を悩まされるでしょう。出典:Wikipediaファイル名の予約語
Spud

8
ピリオドは、ファイル名の最後にある場合は無効な文字であるためGetInvalidFileNameChars、含めません。ウィンドウで例外をスローするのではなく、それらを取り除くだけですが、ピリオドがそこにあると予期している場合、予期しない動作を引き起こす可能性があります。.文字列の末尾にある場合、無効な文字の1つと見なされるように、このケースを処理するように正規表現を変更しました。
スコットチェンバレン

120

より短い解決策:

var invalids = System.IO.Path.GetInvalidFileNameChars();
var newName = String.Join("_", origFileName.Split(invalids, StringSplitOptions.RemoveEmptyEntries) ).TrimEnd('.');

1
@PeterMajeed:行カウントがゼロから始まるTIL :-)
Gary McGill

これは、プラットフォームに基づいて異なる文字を返す可能性があるASP.NET Coreの場合は特に、上の回答よりも優れています。
アレクセイ

79

アンドレの素晴らしい答えに基づいていますが、予約語に関するスパッドのコメントを考慮して、私はこのバージョンを作りました:

/// <summary>
/// Strip illegal chars and reserved words from a candidate filename (should not include the directory path)
/// </summary>
/// <remarks>
/// http://stackoverflow.com/questions/309485/c-sharp-sanitize-file-name
/// </remarks>
public static string CoerceValidFileName(string filename)
{
    var invalidChars = Regex.Escape(new string(Path.GetInvalidFileNameChars()));
    var invalidReStr = string.Format(@"[{0}]+", invalidChars);

    var reservedWords = new []
    {
        "CON", "PRN", "AUX", "CLOCK$", "NUL", "COM0", "COM1", "COM2", "COM3", "COM4",
        "COM5", "COM6", "COM7", "COM8", "COM9", "LPT0", "LPT1", "LPT2", "LPT3", "LPT4",
        "LPT5", "LPT6", "LPT7", "LPT8", "LPT9"
    };

    var sanitisedNamePart = Regex.Replace(filename, invalidReStr, "_");
    foreach (var reservedWord in reservedWords)
    {
        var reservedWordPattern = string.Format("^{0}\\.", reservedWord);
        sanitisedNamePart = Regex.Replace(sanitisedNamePart, reservedWordPattern, "_reservedWord_.", RegexOptions.IgnoreCase);
    }

    return sanitisedNamePart;
}

そして、これらは私のユニットテストです

[Test]
public void CoerceValidFileName_SimpleValid()
{
    var filename = @"thisIsValid.txt";
    var result = PathHelper.CoerceValidFileName(filename);
    Assert.AreEqual(filename, result);
}

[Test]
public void CoerceValidFileName_SimpleInvalid()
{
    var filename = @"thisIsNotValid\3\\_3.txt";
    var result = PathHelper.CoerceValidFileName(filename);
    Assert.AreEqual("thisIsNotValid_3__3.txt", result);
}

[Test]
public void CoerceValidFileName_InvalidExtension()
{
    var filename = @"thisIsNotValid.t\xt";
    var result = PathHelper.CoerceValidFileName(filename);
    Assert.AreEqual("thisIsNotValid.t_xt", result);
}

[Test]
public void CoerceValidFileName_KeywordInvalid()
{
    var filename = "aUx.txt";
    var result = PathHelper.CoerceValidFileName(filename);
    Assert.AreEqual("_reservedWord_.txt", result);
}

[Test]
public void CoerceValidFileName_KeywordValid()
{
    var filename = "auxillary.txt";
    var result = PathHelper.CoerceValidFileName(filename);
    Assert.AreEqual("auxillary.txt", result);
}

1
これは、少なくとも質問のファイル名部分に対する非常に完全な回答であり、より多くの賛成票を投じる価値があります。
ブライアンマッケイ2017

2
メソッドがこの方向に進んでいるように見えるため、マイナーな提案:thisキーワードを追加すると、便利な拡張メソッドになります。public static String CoerceValidFileName(this String filename)
Ryan McArthur

2
小さなバグ:このメソッドは、ファイル拡張子のない予約語(例:)を変更しません。COM1これも許可されていません。修正"^{0}(\\.|$)""_reservedWord_$1"


4

私が使用しているSystem.IO.Path.GetInvalidFileNameChars() 無効な文字をチェックするための方法を、私は何の問題を持っていないしました。

次のコードを使用しています。

foreach( char invalidchar in System.IO.Path.GetInvalidFileNameChars())
{
    filename = filename.Replace(invalidchar, '_');
}

3

単に文字をアンダースコアに置き換えるだけでなく、何らかの方法で文字を保持したかったのです。

私が考えた方法の1つは、通常の文字として使用される可能性が低い(私の状況では)類似した文字に文字を置き換えることです。だから私は無効な文字のリストを取り、そっくりさんを見つけました。

以下は、look-a-likeでエンコードおよびデコードする関数です。

このコードには、すべてのSystem.IO.Path.GetInvalidFileNameChars()文字の完全なリストは含まれていません。したがって、残りの文字のアンダースコア置換を拡張または利用するかどうかは、あなた次第です。

private static Dictionary<string, string> EncodeMapping()
{
    //-- Following characters are invalid for windows file and folder names.
    //-- \/:*?"<>|
    Dictionary<string, string> dic = new Dictionary<string, string>();
    dic.Add(@"\", "Ì"); // U+OOCC
    dic.Add("/", "Í"); // U+OOCD
    dic.Add(":", "¦"); // U+00A6
    dic.Add("*", "¤"); // U+00A4
    dic.Add("?", "¿"); // U+00BF
    dic.Add(@"""", "ˮ"); // U+02EE
    dic.Add("<", "«"); // U+00AB
    dic.Add(">", "»"); // U+00BB
    dic.Add("|", "│"); // U+2502
    return dic;
}

public static string Escape(string name)
{
    foreach (KeyValuePair<string, string> replace in EncodeMapping())
    {
        name = name.Replace(replace.Key, replace.Value);
    }

    //-- handle dot at the end
    if (name.EndsWith(".")) name = name.CropRight(1) + "°";

    return name;
}

public static string UnEscape(string name)
{
    foreach (KeyValuePair<string, string> replace in EncodeMapping())
    {
        name = name.Replace(replace.Value, replace.Key);
    }

    //-- handle dot at the end
    if (name.EndsWith("°")) name = name.CropRight(1) + ".";

    return name;
}

あなた自身のそっくりさんを選択できます。Windowsでキャラクターマップアプリを使って私のものを選択しました%windir%\system32\charmap.exe

発見を通じて調整を行うので、このコードを更新します。


全幅形式 !"#$%&'()*+,-./:;<=>?@{|}~や、/SOLIDUSや `⁄` FRACTION SLASHのような他の形式のように、それらに類似している多くの文字があることに注意してください。問題なくファイル名に直接使用できます
phuclv

2

問題は、まず最初Path.GetDirectoryNameに不正な文字列を呼び出すことです。これにファイル名以外の文字が含まれている場合、.Netは文字列のどの部分がディレクトリであり、スローされているかを判断できません。文字列の比較を行う必要があります。

パス全体ではなく、ファイル名だけが悪いと仮定して、これを試してください:

public static string SanitizePath(string path, char replaceChar)
{
    int filenamePos = path.LastIndexOf(Path.DirectorySeparatorChar) + 1;
    var sb = new System.Text.StringBuilder();
    sb.Append(path.Substring(0, filenamePos));
    for (int i = filenamePos; i < path.Length; i++)
    {
        char filenameChar = path[i];
        foreach (char c in Path.GetInvalidFileNameChars())
            if (filenameChar.Equals(c))
            {
                filenameChar = replaceChar;
                break;
            }

        sb.Append(filenameChar);
    }

    return sb.ToString();
}

2

私は過去にこれで成功しました。

素晴らしく、短くて静的です:-)

    public static string returnSafeString(string s)
    {
        foreach (char character in Path.GetInvalidFileNameChars())
        {
            s = s.Replace(character.ToString(),string.Empty);
        }

        foreach (char character in Path.GetInvalidPathChars())
        {
            s = s.Replace(character.ToString(), string.Empty);
        }

        return (s);
    }

2

ここには多くの実用的な解決策があります。完全を期すために、正規表現を使用せずにLINQを使用するアプローチを次に示します。

var invalids = Path.GetInvalidFileNameChars();
filename = invalids.Aggregate(filename, (current, c) => current.Replace(c, '_'));

また、それは非常に短い解決策です;)


1
私は1つのライナーが大好きです:)
ラリー

1

Andreのコードに基づく効率的な遅延読み込み拡張メソッドを次に示します。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace LT
{
    public static class Utility
    {
        static string invalidRegStr;

        public static string MakeValidFileName(this string name)
        {
            if (invalidRegStr == null)
            {
                var invalidChars = System.Text.RegularExpressions.Regex.Escape(new string(System.IO.Path.GetInvalidFileNameChars()));
                invalidRegStr = string.Format(@"([{0}]*\.+$)|([{0}]+)", invalidChars);
            }

            return System.Text.RegularExpressions.Regex.Replace(name, invalidRegStr, "_");
        }
    }
}

0

ディレクトリとファイル名を一緒に追加して、それらを個別にサニタイズするのではなくサニタイズすると、コードはよりクリーンになります。:のサニタイズについては、文字列の2番目の文字をそのまま使用します。「replacechar」と等しい場合は、コロンに置き換えます。このアプリはご自分で使用するため、このようなソリューションで十分です。


-1
using System;
using System.IO;
using System.Linq;
using System.Text;

public class Program
{
    public static void Main()
    {
        try
        {
            var badString = "ABC\\DEF/GHI<JKL>MNO:PQR\"STU\tVWX|YZA*BCD?EFG";
            Console.WriteLine(badString);
            Console.WriteLine(SanitizeFileName(badString, '.'));
            Console.WriteLine(SanitizeFileName(badString));
        }
        catch (Exception ex)
        {
            Console.WriteLine(ex.ToString());
        }
    }

    private static string SanitizeFileName(string fileName, char? replacement = null)
    {
        if (fileName == null) { return null; }
        if (fileName.Length == 0) { return ""; }

        var sb = new StringBuilder();
        var badChars = Path.GetInvalidFileNameChars().ToList();

        foreach (var @char in fileName)
        {
            if (badChars.Contains(@char)) 
            {
                if (replacement.HasValue)
                {
                    sb.Append(replacement.Value);
                }
                continue; 
            }
            sb.Append(@char);
        }
        return sb.ToString();
    }
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.