Skip to main content
adicionou 65 caracteres ao conteúdo
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

Primeiramente, chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if. Se o código não for "BANANA", a linha é escrita sem modificação.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

Primeiramente, chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

Primeiramente, chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if. Se o código não for "BANANA", a linha é escrita sem modificação.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

adicionou 65 caracteres ao conteúdo
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

Vamos por partes...

ChamarPrimeiramente, chamar strstr em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

Vamos por partes...

Chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

Primeiramente, chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.

Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

Vamos por partes...

Chamar str em str(linha[8:14]) é redundante e desnecessário, pois linha[8:14] já retorna uma string. Além disso, no arquivo de exemplo que você informou, este trecho corresponde a "ANANA " (mas vamos assumir que no arquivo original os índices sejam esses mesmo, é um pequeno ajuste que não interfere no restante).

Outro ponto é que você está lendo do arquivo ao mesmo tempo em que escreve nele. Se der algum erro no meio do caminho, o arquivo ficará corrompido, então o ideal é você primeiro escrever tudo em outro arquivo temporário, e só no final, se tudo der certo, mover o arquivo temporário para o original.

E como eu já disse em sua pergunta anterior, essa variável encontrado é desnecessária. Se quer fazer algo caso a string seja "BANANA", faça tudo dentro do primeiro if:

import shutil, tempfile

# lê do arquivo e escreve em outro arquivo temporário
with open('testando.txt', 'r') as arquivo, \
     tempfile.NamedTemporaryFile('w', delete=False) as out:
    for linha in arquivo:
        codigo = linha[8:14]
        
        if codigo == 'BANANA':
           print("ACHOU, CODIGO: " + codigo)
           linha = linha[:16] + "22" + linha[17:] # remontar a linha
        else:
           print("NÃO ACHOU, CÓDIGO: " + codigo)

        out.write(linha) # escreve no arquivo temporário

# move o arquivo temporário para o original
shutil.move(out.name, 'testando.txt')

Repare que o write fica fora do if, já que é algo que deve ser feito sempre. A única coisa que muda é que a linha é modificada caso caia no if.

Usei with para abrir os arquivos, pois isso garante que eles serão fechados ao final (mesmo em caso de erro, o que não acontece quando você chama close() diretamente - a não ser que ele esteja um bloco finally).

Eu também uso o módulo tempfile para criar o arquivo temporário e shutil.move para renomear o arquivo ao final.

Veja também que o replace que você fazia não é necessário, dá para atribuir a remontagem na própria variável linha.