0

Estou desenvolvendo um sistema no qual me encontro no seguinte cenário: um programa precisa criar um arquivo parte a parte. Ele recebe bytes de um servidor e assim o arquivo é criado, ele deve funcionar desse modo pois é uma decisão de projeto.

O problema é que mais de uma thread pode acabar gravando esses bytes, logo creio que isso ocasionaria em uma condição de corrida. A maneira que lidei para contornar esse problema foi a implementação dessa classe:

internal class ConcurrentFile
{
    private readonly object _padlock = new object();
    public string Path { get; set; }

    public ConcurrentFile(string path)
    {
        Path = path;
    }

    public void CreateEmpty()
    {
        lock (_padlock)
        {
            File.Create(Path).Dispose();
        }
    }

    public void Write(byte[] data)
    {
        lock (_padlock)
        {
            using(var fileStream = new FileStream(Path, FileMode.Append, FileAccess.Write, FileShare.Write))
            {
                fileStream.Write(data, 0, data.Length);
                fileStream.Flush();
            }
        }
    }
}

Essa é uma solução problemática? Quais problemas poderei ter? Aliás, esse código é realmente seguro? Me ajudem a encontrar uma boa solução! Grato pela atenção dos senhores.

6
  • Vejo vários problemas nesse código. Na prática nem está obtendo as vantagens que espera ter, concorrência é muito difícil de fazer certa. Quer saber? Coloca no SQLite, provavelmente terá melhor performance e certamente mais robustez.
    – Maniero
    Commented 11/07/2018 às 6:28
  • @Maniero, fiz uns testes com o código e aparentemente o lock está controlando bem a concorrência. Você poderia explicar quais são os tipos de problemas encontrados?
    – Pedro
    Commented 11/07/2018 às 11:53
  • 2
    Se você fez teste e viu que está bom você é mais especialista em concorrência que eu, não tem como eu possa ajudá-lo. Eu acho que não testou tão bem quanto acha. Tem gente que é especialista nisso e não consegue testar totalmente.
    – Maniero
    Commented 11/07/2018 às 14:22
  • @Maniero, Quero saber quais foram os problemas que você encontrou... Sua ótica sobre o assunto pode ajudar na pergunta em questão.
    – Pedro
    Commented 11/07/2018 às 16:27
  • @Maniero Agradeço sua resposta, mas você poderia abrir uma resposta detalhando os problemas? Ter esse conhecimento seria muito útil pra mim, e acredito também que ajudará outras pessoas que podem cair nessa pergunta com problemas similares.
    – MGM
    Commented 12/07/2018 às 14:20

0

Você deve fazer log-in para responder a esta pergunta.

Pesquise outras perguntas com a tag .