Si explicas algo más de lo que son "numero base" y "numero flotante" igual te podríamos ayudar más.
Algunas optimizaciones claras:
El bucle for
para los inputs
sobra:
y = int(input("introduce numero base :"))
j = int(input("introduce numero flotante :"))
Cuando abres un fichero, siempre hay que cerrarlo. Libera recursos:
file = open("XXpi.txt")
...
file.close()
El modo pythónico para abrir un fichero y no olvidarte cerrarlo es usando una estructura with
:
with open("XXpi.txt") as file:
...
Al salir del with
se cierra automáticamente el fichero.
Cuando compruebas si has llegado a una posición haces lo siguiente:
nlinea2 = [i+j]
for pos, l_num in enumerate(file):
if pos in nlinea2:
O sea, compruebas si pos
está en la lista nlinea2
. Esta comprobación es equivalente a hacer simplemente un chequeo de igualdad:
for pos, l_num in enumerate(file):
if pos == i + j:
Otra optimización:
k = int(str(a)+str(b))-int(str(d)+str(e))
if k == 0:
...
Este modo de comprobar si los dos números son iguales se puede dejar más simple como:
if str(a)+str(b) == str(d)+str(e):
...
Incluso mejor, ya que se tratan de cadenas de igual tamaño:
if (a, b) == (d, e):
...
Por otro lado, no es eficiente estar abriendo una y otra vez el fichero para recorrerlo
Sabiendo algo más, igual se podría usar alguna función de bajo nivel para acceder a la línea en concreto que se quiere leer, en lugar de leer todas las líneas del fichero.
Intentemos optimizar la lectura del fichero con la información que tenemos. Por lo que parece, se comprueba la línea y+j
con la j+j
, luego las siguientes, y+j+1
con j+j+1
, y+j+2
con j+j+2
, así hasta llegar al final del fichero. No sé cuánto valdrá y
y j
, pero supongamos que xmin = min(y+j, j+j)
y xmax = max(y+j, j+j)
. El número de líneas entre el máximo y el mínimo será numlineas = xmax - xmin + 1
. Seguramente, será un rango de líneas asumible para guardar en memoria en una caché para así evitar la segunda lectura del fichero.
from collections import deque
from itertools import islice
y = int(input("introduce numero base :"))
j = int(input("introduce numero flotante :"))
nlinea1 = y + j
nlinea2 = j + j
# tamaño para la caché
xmin = min(nlinea1, nlinea2)
xmax = max(nlinea1, nlinea2)
maxlen = xmax - xmin + 1
# caché
cache = deque(maxlen=maxlen)
with open("XXpi.txt") as file:
# saltamos las primeras líneas
next(islice(file, xmin, xmin), None)
# llenamos la caché
for i in range(maxlen):
cache.append(file.readline())
# comprobamos coincidencias
for pos, line in enumerate(file):
line1 = line[:3] + line[6:]
line2 = cache.popleft()
cache.append(line1) # se guarda la línea procesada
if line1 == line2:
print(line)
print("Línea 1", xmin+maxlen+pos)
print("Línea 2", xmin+pos)
El código lo he escrito sin comprobar nada, así que seguro que los índices están incorrectos. Basta con probar y hacer las correcciones correspondientes. Pero la idea sería algo así.
for
para pedir los datos 2. No abras el archivo en cada iteración, mejor léelo todo y guardalo en una lista, cada línea es un elemento de la lista, lo puedes hacer confile.readlines()
y esto lo debes hacer antes del ciclo.