[c#] ¿Hay alguna manera de verificar si un archivo está en uso?


Answers

Puede sufrir una condición de carrera de subprocesos en la que existen ejemplos documentados de que esto se usa como vulnerabilidad de seguridad. Si comprueba que el archivo está disponible, pero luego intente utilizarlo, podría lanzarlo en ese punto, que un usuario malintencionado podría usar para forzar y explotar en su código.

Su mejor apuesta es una captura de try / finally que intente manejar el archivo.

try
{
   using (Stream stream = new FileStream("MyFilename.txt", FileMode.Open))
   {
        // File/Stream manipulating code here
   }
} catch {
  //check here why it failed and ask user to retry if the file is in use.
}
Question

Estoy escribiendo un programa en C # que necesita acceder repetidamente a 1 archivo de imagen. La mayoría de las veces funciona, pero si mi computadora funciona rápido, intentará acceder al archivo antes de que se haya guardado en el sistema de archivos y arrojará un error: "Archivo en uso por otro proceso" .

Me gustaría encontrar una forma de evitar esto, pero toda mi búsqueda en Google solo ha producido la creación de cheques mediante el manejo de excepciones. Esto va en contra de mi religión, entonces me preguntaba si alguien tiene una mejor manera de hacerlo.




Utilizo esta solución, pero tengo un intervalo de tiempo entre el momento en que compruebo el bloqueo de archivos con la función IsFileLocked y cuando abro el archivo. En este intervalo de tiempo, otro hilo puede abrir el archivo, por lo que obtendré IOException.

Entonces, agregué un código extra para esto. En mi caso quiero cargar XDocument:

        XDocument xDoc = null;

        while (xDoc == null)
        {
            while (IsFileBeingUsed(_interactionXMLPath))
            {
                Logger.WriteMessage(Logger.LogPrioritet.Warning, "Deserialize can not open XML file. is being used by another process. wait...");
                Thread.Sleep(100);
            }
            try
            {
                xDoc = XDocument.Load(_interactionXMLPath);
            }
            catch
            {
                Logger.WriteMessage(Logger.LogPrioritet.Error, "Load working!!!!!");
            }
        }

¿Qué piensas? ¿Puedo cambiar algo? Tal vez no tuve que usar la función IsFileBeingUsed en absoluto?

Gracias




Solo use la excepción como estaba previsto. Acepte que el archivo esté en uso y vuelva a intentarlo varias veces hasta que complete su acción. Este también es el más eficiente porque no desperdicia ningún ciclo verificando el estado antes de actuar.

Use la función a continuación, por ejemplo

TimeoutFileAction(() => { System.IO.File.etc...; return null; } );

Método reutilizable que expira después de 2 segundos

private T TimeoutFileAction<T>(Func<T> func)
{
    var started = DateTime.UtcNow;
    while ((DateTime.UtcNow - started).TotalMilliseconds < 2000)
    {
        try
        {
            return func();                    
        }
        catch (System.IO.IOException exception)
        {
            //ignore, or log somewhere if you want to
        }
    }
    return default(T);
}



En mi experiencia, generalmente quiere hacer esto, luego 'proteger' sus archivos para hacer algo sofisticado y luego usar los archivos 'protegidos'. Si solo tiene un archivo que desea usar de esta manera, puede usar el truco que se explica en la respuesta de Jeremy Thompson. Sin embargo, si intenta hacer esto en muchos archivos (por ejemplo, cuando está escribiendo un instalador), se sentirá bastante ofendido.

Una forma muy elegante de solucionar esto es usar el hecho de que su sistema de archivos no le permitirá cambiar el nombre de una carpeta si uno de los archivos está siendo utilizado. Mantenga la carpeta en el mismo sistema de archivos y funcionará como un amuleto.

Tenga en cuenta que debe ser consciente de las formas obvias en que esto puede ser explotado. Después de todo, los archivos no serán bloqueados. Además, tenga en cuenta que hay otras razones que pueden hacer que su operación Move falle. Obviamente, el manejo correcto de errores (MSDN) puede ayudar aquí.

var originalFolder = @"c:\myHugeCollectionOfFiles"; // your folder name here
var someFolder = Path.Combine(originalFolder, "..", Guid.NewGuid().ToString("N"));

try
{
    Directory.Move(originalFolder, someFolder);

    // Use files
}
catch // TODO: proper exception handling
{
    // Inform user, take action
}
finally
{
    Directory.Move(someFolder, originalFolder);
}

Para archivos individuales, me quedaría con la sugerencia de bloqueo publicada por Jeremy Thompson.




Me interesa ver si esto desencadena algún reflejo WTF. Tengo un proceso que crea y posteriormente inicia un documento PDF desde una aplicación de consola. Sin embargo, estaba lidiando con una fragilidad en la que si el usuario ejecutara el proceso varias veces, generando el mismo archivo sin antes cerrar el archivo generado previamente, la aplicación lanzaría una excepción y moriría. Esta fue una ocurrencia bastante frecuente porque los nombres de los archivos se basan en los números de las cotizaciones de venta.

En lugar de fallar de una manera tan desvergonzada, decidí confiar en el control de versiones de archivo auto-incrementado:

private static string WriteFileToDisk(byte[] data, string fileName, int version = 0)
{
    try
    {
        var versionExtension = version > 0 ? $"_{version:000}" : string.Empty;
        var filePath = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, $"{fileName}{versionExtension}.pdf");
        using (var writer = new FileStream(filePath, FileMode.Create))
        {
            writer.Write(data, 0, data.Length);
        }
        return filePath;
    }
    catch (IOException)
    {
        return WriteFileToDisk(data, fileName, ++version);
    }
}

Probablemente, se le puede dar más cuidado al bloque catch para asegurar que estoy capturando las IOException (s) correctas. Probablemente también borre el almacenamiento de la aplicación en el inicio ya que estos archivos están destinados a ser temporales de todos modos.

Me doy cuenta de que esto va más allá del alcance de la pregunta del OP de simplemente verificar si el archivo está en uso, pero este fue realmente el problema que estaba tratando de resolver cuando llegué aquí, así que tal vez sea útil para otra persona.




Las respuestas aceptadas anteriormente tienen un problema donde si el archivo se ha abierto para escribir con un modo FileShare.Read o si el archivo tiene un atributo de solo lectura, el código no funcionará. Esta solución modificada funciona de la manera más confiable, con dos cosas a tener en cuenta (también es cierto para la solución aceptada):

  1. No funcionará para archivos que se han abierto con un modo de compartir escritura
  2. Esto no tiene en cuenta los problemas de subprocesamiento, por lo que deberá bloquearlo o manejar los problemas de subprocesamiento por separado.

Teniendo en cuenta lo anterior, esto comprueba si el archivo está bloqueado para escritura o bloqueado para evitar la lectura :

public static bool FileLocked(string FileName)
{
    FileStream fs = null;

    try
    {
        // NOTE: This doesn't handle situations where file is opened for writing by another process but put into write shared mode, it will not throw an exception and won't show it as write locked
        fs = File.Open(FileName, FileMode.Open, FileAccess.ReadWrite, FileShare.None); // If we can't open file for reading and writing then it's locked by another process for writing
    }
    catch (UnauthorizedAccessException) // https://msdn.microsoft.com/en-us/library/y973b725(v=vs.110).aspx
    {
        // This is because the file is Read-Only and we tried to open in ReadWrite mode, now try to open in Read only mode
        try
        {
            fs = File.Open(FileName, FileMode.Open, FileAccess.Read, FileShare.None);
        }
        catch (Exception)
        {
            return true; // This file has been locked, we can't even open it to read
        }
    }
    catch (Exception)
    {
        return true; // This file has been locked
    }
    finally
    {
        if (fs != null)
            fs.Close();
    }
    return false;
}



Aquí hay una versión Powershell de la respuesta aceptada.

function IsFileLocked($filename) {

    $result = $false

    $fileinfo = [System.IO.FileInfo] (gi $filename).fullname

    try {
        $stream = $fileInfo.Open([System.IO.FileMode]"Open",[System.IO.FileAccess]"ReadWrite",[System.IO.FileShare]"None")
        $stream.Dispose()
    } catch [System.IO.IOException] {
        $result = $true
    }

    $result
}



la única forma que conozco es usar la API de bloqueo exclusivo de Win32 que no es demasiado rápida, pero existen ejemplos.

La mayoría de las personas, para una solución simple a esto, simplemente para probar / atrapar / dormir bucles.




Related