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
.