0

Estou usando o seguinte código em meu Controller:

public ActionResult geraBpa()
{
    var caminho = System.Web.HttpContext.Current.Server.MapPath("~/Content");
    StreamWriter file = new StreamWriter($"{caminho}/BPA.txt");
    List<bpac> listaBpac = pegaBpac();

    int linhaTexto = 1;
    int linhaItem = 1;

    foreach (bpac linha in listaBpac)
    {
        file.WriteLine(
        "02" +
        linha.cnes +
        linha.cmp + //competencia
        linha.cbo +
        string.Format("{0:000}", linhaTexto) + string.Format("{0:00}", linhaItem) +
        linha.pa +
        "000" +
        string.Format("{0:000000}", linha.quant) +
        "EXT"
        );

        linhaItem++;
        if (linhaItem > 99)
        {
            linhaItem = 1;
            linhaTexto++;
        }
    }

    linhaTexto++;
    linhaItem = 1;

    byte[] fileBytes = System.IO.File.ReadAllBytes($"{caminho}/BPA.txt");
    string fileName = "myfile.ext";
    return File(fileBytes, MediaTypeNames.Application.Octet, fileName);
}

Estou recebendo erro nessa linha:

byte[] fileBytes = System.IO.File.ReadAllBytes($"{caminho}/BPA.txt");

O erro diz que o arquivo BPA.txt está aberto. Quando fui até a pasta, vi que o mesmo nem foi criado corretamente.

Alguma ajuda?

4

1 Resposta 1

4

É exatamente este o problema, o arquivo já está aberto quando é feita a tentativa de leitura.

Seu código abre um StreamWriter e não o fecha. Você pode usar o método Dispose ou colocar o código que usa a variável file dentro de um using. Você pode ler sobre o using em Qual a utilidade do using?

Eu seguiria pelo segundo caminho, o código ficaria assim

var caminho = System.Web.HttpContext.Current.Server.MapPath("~/Content");
using(StreamWriter file = new StreamWriter($"{caminho}/BPA.txt"))
{
    List<bpac> listaBpac = pegaBpac();

    int linhaTexto = 1;
    int linhaItem = 1;

    foreach (bpac linha in listaBpac)
    {
        file.WriteLine(
        "02" +
        linha.cnes +
        linha.cmp + //competencia
        linha.cbo +
        string.Format("{0:000}", linhaTexto) + string.Format("{0:00}", linhaItem) +
        linha.pa +
        "000" +
        string.Format("{0:000000}", linha.quant) +
        "EXT"
        );

        linhaItem++;
        if (linhaItem > 99)
        {
            linhaItem = 1;
            linhaTexto++;
        }
    }
}

linhaTexto++;
linhaItem = 1;

byte[] fileBytes = System.IO.File.ReadAllBytes($"{caminho}/BPA.txt");
string fileName = "myfile.ext";
return File(fileBytes, MediaTypeNames.Application.Octet, fileName);
4
  • Oi, o código funcionou, porém abre na tela ao invés de fazer o download. Alguma sugestão? Commented 22/08/2017 às 17:55
  • Sim. Algumas sugestões. Não acha melhor abrir outra pergunta? Pra ser sincero, acho importante até porque eu gostaria de ter esse assunto catalogado em algum lugar.
    – Jéf Bueno
    Commented 22/08/2017 às 17:55
  • Blz, abrirei sim. Commented 22/08/2017 às 17:56
  • pt.stackoverflow.com/questions/231227/… Commented 22/08/2017 às 17:58

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

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .