16

ある場所から多数のファイル(大まかに言って100000を超える)を削除しようとしています。これにより、アクションはWebページから開始されます。明らかに私はただ使うことができました

string[] files = System.IO.Directory.GetFiles("path with files to delete");
foreach (var file in files) {
    IO.File.Delete(file);
}

Directory.GetFiles http://msdn.microsoft.com/en-us/library/wz42302f.aspx

この方法はすでに数回投稿されています: ディレクトリ内のすべてのファイルとフォルダを削除するにはどうすればよいですか?ファイル名に特定の単語が含まれている場合は、ディレクトリからファイル を 削除します

ただし、この方法の問題は、10万個のファイルがある場合、ループする前にすべてのファイルパスを最初に生成する必要があるため、パフォーマンスの問題になることです。

これに加えて、Webページがこれを実行しているメソッドからの応答を待っている場合は、少しごみのように見えることが想像できます。

私が持っていた1つの考えは、これを無秩序なWebサービス呼び出しにまとめることでした。それが完了すると、Webページへの応答が返され、それらが削除されたことを通知しますか?たぶん、deleteメソッドを別のスレッドに入れますか?または、別のバッチプロセスを使用して削除を実行することもできますか?

ディレクトリ内のファイルの数を数えようとすると、同様の問題が発生します-ディレクトリに多数のファイルが含まれている場合。

これは少しやり過ぎなのだろうか?つまり、これに対処するためのより簡単な方法はありますか?どんな助けでもいただければ幸いです。

4

10 に答える 10

12
  1. GetFiles非常に遅いです。
  2. Webサイトから呼び出す場合は、このトリックを実行する新しいスレッドをスローするだけです。
  3. 一致するファイルがまだあるかどうかを返すASP.NETAJAX呼び出しを使用して、基本的な進行状況の更新を行うことができます。

の高速Win32ラッピングの実装の下GetFilesで、新しいスレッドおよび次のようなAJAX関数と組み合わせて使用​​しますGetFilesUnmanaged(@"C:\myDir", "*.txt*).GetEnumerator().MoveNext()

使用法

Thread workerThread = new Thread(new ThreadStart((MethodInvoker)(()=>
{    
     foreach(var file in GetFilesUnmanaged(@"C:\myDir", "*.txt"))
          File.Delete(file);
})));
workerThread.Start();
//just go on with your normal requests, the directory will be cleaned while the user can just surf around

   public static IEnumerable<string> GetFilesUnmanaged(string directory, string filter)
        {
            return new FilesFinder(Path.Combine(directory, filter))
                .Where(f => (f.Attributes & FileAttributes.Normal) == FileAttributes.Normal
                    || (f.Attributes & FileAttributes.Archive) == FileAttributes.Archive)
                .Select(s => s.FileName);
        }
    }


public class FilesEnumerator : IEnumerator<FoundFileData>
{
    #region Interop imports

    private const int ERROR_FILE_NOT_FOUND = 2;
    private const int ERROR_NO_MORE_FILES = 18;

    [DllImport("kernel32.dll", SetLastError = true, CharSet = CharSet.Auto)]
    private static extern IntPtr FindFirstFile(string lpFileName, out WIN32_FIND_DATA lpFindFileData);

    [DllImport("kernel32.dll", SetLastError = true, CharSet = CharSet.Auto)]
    private static extern bool FindNextFile(SafeHandle hFindFile, out WIN32_FIND_DATA lpFindFileData);

    #endregion

    #region Data Members

    private readonly string _fileName;
    private SafeHandle _findHandle;
    private WIN32_FIND_DATA _win32FindData;

    #endregion

    public FilesEnumerator(string fileName)
    {
        _fileName = fileName;
        _findHandle = null;
        _win32FindData = new WIN32_FIND_DATA();
    }

    #region IEnumerator<FoundFileData> Members

    public FoundFileData Current
    {
        get
        {
            if (_findHandle == null)
                throw new InvalidOperationException("MoveNext() must be called first");

            return new FoundFileData(ref _win32FindData);
        }
    }

    object IEnumerator.Current
    {
        get { return Current; }
    }

    public bool MoveNext()
    {
        if (_findHandle == null)
        {
            _findHandle = new SafeFileHandle(FindFirstFile(_fileName, out _win32FindData), true);
            if (_findHandle.IsInvalid)
            {
                int lastError = Marshal.GetLastWin32Error();
                if (lastError == ERROR_FILE_NOT_FOUND)
                    return false;

                throw new Win32Exception(lastError);
            }
        }
        else
        {
            if (!FindNextFile(_findHandle, out _win32FindData))
            {
                int lastError = Marshal.GetLastWin32Error();
                if (lastError == ERROR_NO_MORE_FILES)
                    return false;

                throw new Win32Exception(lastError);
            }
        }

        return true;
    }

    public void Reset()
    {
        if (_findHandle.IsInvalid)
            return;

        _findHandle.Close();
        _findHandle.SetHandleAsInvalid();
    }

    public void Dispose()
    {
        _findHandle.Dispose();
    }

    #endregion
}

public class FilesFinder : IEnumerable<FoundFileData>
{
    readonly string _fileName;
    public FilesFinder(string fileName)
    {
        _fileName = fileName;
    }

    public IEnumerator<FoundFileData> GetEnumerator()
    {
        return new FilesEnumerator(_fileName);
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return GetEnumerator();
    }
}

public class FoundFileData
{
    public string AlternateFileName;
    public FileAttributes Attributes;
    public DateTime CreationTime;
    public string FileName;
    public DateTime LastAccessTime;
    public DateTime LastWriteTime;
    public UInt64 Size;

    internal FoundFileData(ref WIN32_FIND_DATA win32FindData)
    {
        Attributes = (FileAttributes)win32FindData.dwFileAttributes;
        CreationTime = DateTime.FromFileTime((long)
                (((UInt64)win32FindData.ftCreationTime.dwHighDateTime << 32) +
                 (UInt64)win32FindData.ftCreationTime.dwLowDateTime));

        LastAccessTime = DateTime.FromFileTime((long)
                (((UInt64)win32FindData.ftLastAccessTime.dwHighDateTime << 32) +
                 (UInt64)win32FindData.ftLastAccessTime.dwLowDateTime));

        LastWriteTime = DateTime.FromFileTime((long)
                (((UInt64)win32FindData.ftLastWriteTime.dwHighDateTime << 32) +
                 (UInt64)win32FindData.ftLastWriteTime.dwLowDateTime));

        Size = ((UInt64)win32FindData.nFileSizeHigh << 32) + win32FindData.nFileSizeLow;
        FileName = win32FindData.cFileName;
        AlternateFileName = win32FindData.cAlternateFileName;
    }
}

/// <summary>
/// Safely wraps handles that need to be closed via FindClose() WIN32 method (obtained by FindFirstFile())
/// </summary>
public class SafeFindFileHandle : SafeHandleZeroOrMinusOneIsInvalid
{
    [DllImport("kernel32.dll", SetLastError = true)]
    private static extern bool FindClose(SafeHandle hFindFile);

    public SafeFindFileHandle(bool ownsHandle)
        : base(ownsHandle)
    {
    }

    protected override bool ReleaseHandle()
    {
        return FindClose(this);
    }
}

// The CharSet must match the CharSet of the corresponding PInvoke signature
[StructLayout(LayoutKind.Sequential, CharSet = CharSet.Auto)]
public struct WIN32_FIND_DATA
{
    public uint dwFileAttributes;
    public FILETIME ftCreationTime;
    public FILETIME ftLastAccessTime;
    public FILETIME ftLastWriteTime;
    public uint nFileSizeHigh;
    public uint nFileSizeLow;
    public uint dwReserved0;
    public uint dwReserved1;
    [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 260)]
    public string cFileName;
    [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 14)]
    public string cAlternateFileName;
}
于 2010-02-02T16:47:43.970 に答える
4

すべてのファイルを同じディレクトリに置くことができますか?

もしそうなら、Directory.Delete(string,bool)削除したいサブディレクトリを呼び出してみませんか?

削除したいファイルパスのリストがすでにある場合は、各ファイルを手動で削除するのではなく、一時ディレクトリに移動してから削除することで、実際にはより良い結果が得られる可能性があります。

乾杯、フロリアン

于 2010-02-02T17:28:06.773 に答える
2

ディレクトリに1000を超えるファイルがあることは、大きな問題です。

現在開発段階にある場合は、ファイルをランダムフォルダー(ルートフォルダー内)に配置し、そのフォルダー内のファイル数が1024未満であることを確認するアルゴを配置することを検討する必要があります。

何かのようなもの

public UserVolumeGenerator()
    {
        SetNumVolumes((short)100);
        SetNumSubVolumes((short)1000);
        SetVolumesRoot("/var/myproj/volumes");
    }

    public String GenerateVolume()
    {
        int volume = random.nextInt(GetNumVolumes());
        int subVolume = random.nextInt(GetNumSubVolumes());

        return Integer.toString(volume) + "/" + Integer.toString(subVolume);
    }

    private static final Random random = new Random(System.currentTimeMillis());

これを行うときは、ファイルを作成するたびに、そのファイルをHashMapまたはリスト(パス)に同時に追加することも確認してください。JSON.netなどを使用してこれを定期的にファイルシステムにシリアル化します(整合性のため、サービスが失敗した場合でも、シリアル化されたフォームからファイルリストを取得できます)。

ファイルをクリーンアップしたり、ファイル間でクエリを実行したりする場合は、最初にこのHashMapまたはリストを検索してから、ファイルを操作します。これはより良いですSystem.IO.Directory.GetFiles

于 2010-02-03T06:21:23.597 に答える
2

バックエンドでそれをスピードアップするためのいくつかの改善:

  • 使用Directory.EnumerateFiles(..):これは、すべてのファイルが取得された後、待たずにファイルを反復処理します。

  • 使用Parallel.Foreach(..):これはファイルを同時に削除します。

高速である必要がありますが、HTTPリクエストは多数のファイルでタイムアウトになるため、バックエンドプロセスは別のワーカースレッドで実行し、終了後に結果をWebクライアントに通知する必要があります。

于 2018-04-11T11:26:40.867 に答える
1

別のスレッドでそれを行うか、別のアプリケーション(おそらくWindowsサービス)がそのキューにサブスクライブされてコマンドを実行するキュー(おそらくMSMQ ?)にメッセージを投稿します(つまり、 "Delete e:\ dir * .txt")それ自身のプロセスで。

メッセージには、おそらくフォルダ名だけが含まれているはずです。NServiceBusやトランザクションキューなどを使用している場合は、メッセージを投稿して、メッセージが正常に投稿されている限りすぐに戻ることができます。メッセージの実際の処理に問題がある場合は、再試行し、最終的にはエラーキューに移動し、監視してメンテナンスを実行できます。

于 2010-02-02T16:46:35.793 に答える
0

ワークアウトをワーカースレッドで起動してから、ユーザーに応答を返します。

アプリケーション変数にフラグを立てて、同じ作業を行う複数のスレッドの実行を停止するために「大きな削除ジョブ」を実行していることを示します。次に、必要に応じて、これまでに削除されたファイルの数の進捗状況の更新を提供できる別のページをポーリングできますか?

単なるクエリですが、なぜこれほど多くのファイルがあるのでしょうか。

于 2010-02-02T16:47:42.137 に答える
0

aspxコードビハインドで単純なajaxwebmethodを作成し、javascriptで呼び出すことができます。

于 2010-02-02T16:48:07.057 に答える
0

最良の選択(imho)は、ファイルを削除/カウントする別のプロセスを作成し、ポーリングによって進行状況を確認することです。そうしないと、ブラウザーのタイムアウトで問題が発生する可能性があります。

于 2010-02-02T16:48:23.247 に答える
0

わお。他のサービスやエンティティが削除を処理することで、あなたは間違いなく正しい方向に進んでいると思います。そうすることで、削除のプロセスを追跡し、非同期javascriptを使用して結果をユーザーに表示するためのメソッドを提供することもできます。

他の人が言っているように、これを別のプロセスに入れることは素晴らしいアイデアです。このような長時間実行されるジョブを使用してIISがリソースを占有することは望ましくありません。そうするもう1つの理由は、セキュリティです。ディスクからファイルを削除する機能をワークプロセスに与えたくない場合があります。

于 2010-02-02T16:48:39.143 に答える
0

私はそれが古いスレッドであることを知っていますが、Jan Jongboomの回答に加えて、非常にパフォーマンスが高く、より普遍的な同様のソリューションを提案します。私のソリューションは、長いファイル名(> 255文字)をサポートするDFSのディレクトリ構造をすばやく削除するように構築されました。最初の違いは、DLLインポート宣言にあります。

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
static extern IntPtr FindFirstFile(string lpFileName, ref WIN32_FIND_DATA lpFindFileData);

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
static extern bool FindNextFile(IntPtr hDindFile, ref WIN32_FIND_DATA lpFindFileData);

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
[return: MashalAs(UnmanagedType.Bool]
static extern bool DeleteFile(string lpFileName)

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
[return: MashalAs(UnmanagedType.Bool]
static extern bool DeleteDirectory(string lpPathName)

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
static extern bool FindClose(IntPtr hFindFile);

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLAstError = true)]
static extern uint GetFileAttributes(string lpFileName);

[DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLAstError = true)]
static extern bool SetFileAttributes(string lpFileName, uint dwFileAttributes);

WIN32_FIND_DATA構造もわずかに異なります。

    [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode), Serializable, BestFitMapping(false)]
    internal struct WIN32_FIND_DATA
    {
        internal FileAttributes dwFileAttributes;
        internal FILETIME ftCreationTime;
        internal FILETIME ftLastAccessTime;
        internal FILETIME ftLastWriteTime;
        internal int nFileSizeHigh;
        internal int nFileSizeLow;
        internal int dwReserved0;
        internal int dwReserved1;
        [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 260)]
        internal string cFileName;
        [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 14)]
        internal string cAlternative;
    }

長いパスを使用するには、パスを次のように準備する必要があります。

public void RemoveDirectory(string directoryPath)
{
    var path = @"\\?\UNC\" + directoryPath.Trim(@" \/".ToCharArray());
    SearchAndDelete(path);
}

主な方法は次のとおりです。

private void SearchAndDelete(string path)
{
    var fd = new WIN32_FIND_DATA();
    var found = false;
    var handle = IntPtr.Zero;
    var invalidHandle = new IntPtr(-1);
    var fileAttributeDir = 0x00000010;
    var filesToRemove = new List<string>();
    try
    {
        handle = FindFirsFile(path + @"\*", ref fd);
        if (handle == invalidHandle) return;
        do
        {
            var current = fd.cFileName;
            if (((int)fd.dwFileAttributes & fileAttributeDir) != 0)
            {
                if (current != "." && current != "..")
                {
                    var newPath = Path.Combine(path, current);
                    SearchAndDelete(newPath);
                }
            }
            else
            {
                filesToRemove.Add(Path.Combine(path, current));
            }
            found = FindNextFile(handle, ref fd);
        } while (found);
    }
    finally
    {
        FindClose(handle);
    }
    try
    {
        object lockSource = new Object();
        var exceptions = new List<Exception>();
        Parallel.ForEach(filesToRemove, file, =>
        {
            var attrs = GetFileAttributes(file);
            attrs &= ~(uint)0x00000002; // hidden
            attrs &= ~(uint)0x00000001; // read-only
            SetFileAttributes(file, attrs);
            if (!DeleteFile(file))
            {
                var msg = string.Format("Cannot remove file {0}.{1}{2}", file.Replace(@"\\?\UNC", @"\"), Environment.NewLine, new Win32Exception(Marshal.GetLastWin32Error()).Message);
                lock(lockSource)
                {
                    exceptions.Add(new Exceptions(msg));
                }
            }
        });
        if (exceptions.Any())
        {
            throw new AggregateException(exceptions);
        }
    }
    var dirAttr = GetFileAttributes(path);
    dirAttr &= ~(uint)0x00000002; // hidden
    dirAttr &= ~(uint)0x00000001; // read-only
    SetfileAttributtes(path, dirAttr);
    if (!RemoveDirectory(path))
    {
        throw new Exception(new Win32Exception(Marshal.GetLAstWin32Error()));
    }
}

もちろん、さらに進んで、そのメソッドの外部の別のリストにディレクトリを格納し、後で次のような別のメソッドでそれらを削除することもできます。

private void DeleteDirectoryTree(List<string> directories)
{
        // group directories by depth level and order it by level descending
        var data = directories.GroupBy(d => d.Split('\\'),
            d => d,
            (key, dirs) => new
            {
                Level = key,
                Directories = dirs.ToList()
            }).OrderByDescending(l => l.Level);
        var exceptions = new List<Exception>();
        var lockSource = new Object();
        foreach (var level in data)
        {
            Parallel.ForEach(level.Directories, dir =>
            {
                var attrs = GetFileAttributes(dir);
                attrs &= ~(uint)0x00000002; // hidden
                attrs &= ~(uint)0x00000001; // read-only
                SetFileAttributes(dir, attrs);
                if (!RemoveDirectory(dir))
                {
                    var msg = string.Format("Cannot remove directory {0}.{1}{2}", dir.Replace(@"\\?\UNC\", string.Empty), Environment.NewLine, new Win32Exception(Marshal.GetLastWin32Error()).Message);
                    lock (lockSource)
                    {
                        exceptions.Add(new Exception(msg));
                    }
                }
            });
        }
        if (exceptions.Any())
        {
            throw new AggregateException(exceptions);
        }
}
于 2016-03-09T16:35:15.780 に答える