Embora a solução apresentada possa gerar o resultado esperado, ele está longe de ser idiomático, embora acredito que tenha sido útil para comparar com o código apresentado na pergunta e auxiliado a identificar os erros. Assim sendo, proponho uma solução um pouco mais elegante para a linguagem em questão:
with open("numbers_1.txt") as numbers_1, open("numbers_2.txt") as numbers_2:
for pair in zip(numbers_1, numbers_2):
print(sum(map(int, pair)))
Veja funcionando no Repl.it
Com o with
abro os dois arquivos no modo leitura, instanciando os dois geradores responsáveis por cada arquivo, numbers_1
e numbers_2
, respectivamente. Após, itero com um laço for
sobre o resultado de zip()
, que faz a associação entre os elementos dos dois geradores: o n-ésimo termo de numbers_1
é associado ao n-ésimo termo de numbers_2
em uma tupla, onde n varia de 0 até o comprimento da sequência (menos um). Após, exibo o resultado da soma, através da função sum()
, que recebe um iterável responsável por converter os valores do par em valores inteiros.
De certa forma, o código ainda pode ser melhorado construindo um gerador a partir desta estrutura:
def sum_of_files(file_1, file_2):
with open(file_1) as numbers_1, open(file_2) as numbers_2:
for pair in zip(numbers_1, numbers_2):
yield sum(map(int, pair))
for s in sum_of_files('numbers_1.txt', 'numbers_2.txt'):
print(s)
Veja funcionando no Repl.it
A vantagem desta segunda solução é que você não precisa, em momento algum, manter a sequência de valores armazenado, por completo, na memória, o que otimiza muito caso seus arquivos cresçam esponencialmente, podendo chegar a milhares de linhas ou mais. A primeira solução teria a sequência final armazenada na memória (considerando que não fosse utilizada o print
) e a solução da outra resposta teria três vezes a sequência armazenada (demanda muito mais recursos).
É possível ainda melhorar o código, quebrando a limitação de apenas dois arquivos, permitindo quantos arquivos forem necessários:
def sum_of_files(*files):
streams = map(open, files)
for numbers in zip(*streams):
yield sum(map(int, numbers))
for stream in streams:
stream.close()
for s in sum_of_files('numbers_1.txt', 'numbers_2.txt', 'numbers_3.txt'):
print(s)
Veja funcionando no Repl.it
E ainda é possível melhorar quebrando a limitação de que os arquivos precisam ser do mesmo tamanho substituindo a função zip()
pela função itertools.zip_longest()
:
from itertools import zip_longest
def sum_of_files(*files):
streams = map(open, files)
for numbers in zip_longest(*streams, fillvalue=0):
yield sum(map(int, numbers))
for stream in streams:
stream.close()
for s in sum_of_files('numbers_1.txt', 'numbers_2.txt', 'numbers_3.txt'):
print(s)
Veja funcionando no Repl.it
Os arquivos menores, que não possuem a sequência completa de valores, terão seus números iguais a 0 devido ao parâmetro fillvalue
. Desta forma, você será capaz de efetuar a soma de quantos arquivos forem necessários, independente do tamanho de cada um deles, desde que possuam valores válidos, sem ter que armazenar em memória a sequência, sendo o código bastante eficiente mesmo para arquivos com milhões de linhas.