Haces la conversión en:
if os.path.isfile("C:/Program Files/logsearcher100/log" + "(" + str(i) + ")"):
^^^^^^
Pero no en:
namefinallog = "log" + "(" + i + ")"
^^^
ni en:
finallog.write("char(s): " + (placechar + i) + ":" + (placechar + len(search) + i))
^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
No obstante, la concatenación de cadenas es muy ineficiente dada su inmutabilidad, usa formateo de cadenas:
if os.path.isfile(f"C:/Program Files/logsearcher100/log({i})"):
namefinallog = f"log({i})"
finallog.write(f"{char(s)}: {placechar + i}:{placechar + len(search) + i}")
Unas cuantas observaciones más:
Tienes un ciclo infinito:
while alreadyfile == True: # <<<<<<<<<<<<<<<<<<<<
for i in range(charlengt_userconfig):
if os.path.isfile("C:/Program Files/logsearcher100/log" + "(" + str(i) + ")"):
alreadyfile = True # <<<<<<<<<<<<<<<<<<<< Por siempre jamás....
else:
namefinallog = "log" + "(" + i + ")"
No uses el operador ==
o !=
para saber si algo se evalúa como verdadero o falso, en vez de eso, usa simplemente if algo:
/ while algo:
o if not algo:
/ while not algo:
while alreadyfile == True:
debe ser simplemente:
while alreadyfile:
en vez de:
if usage == "run" and (alreadysetsearch == False or alreadysetfile == False or alreadysetperm == False):
simplemente:
if usage == "run" and (not alreadysetsearch or not alreadysetfile or not alreadysetperm):
Si se da el caso de quieres saber si un objeto es el objeto bool
True
, que no es lo mismo que querer saber si algo se evalúa como verdadero, usa el operador de identidad is
(if a is b
), dado que False
, True
(y None
) son singlentons.
Funcionamiento del operador is
No uses if
encadenados cuando sean excluyentes, es ineficiente, usa if-elif
:
if usage == "set search":
search = input("SEARCH: ")
alreadysetsearch = True
elif usage == "set file":
...
elif usage == "quit":
sys.exit()
Llamas a una función antes de definirla:
runintext(search,file,perm,finallog)
def runintext():
...
además la función que defines no acepta parámetros y tu al llamarla le pasas 4. Debe ser:
def runintext():
...
runintext()
No te dejes archivos abiertos por el camino sin cerrar (usefullfile
), no es tarea para que el GC se encargue, o los cierras explícitamente con close o mejor usa el administrador de contexto y with
, que lo cierra aunque existan excepciones de por medio:
with open("archivo.txt", "r") as file:
# hacer lo que sea con file
# Aquí el archivo se ha cerrado automáticamente
input
en Python 3 ya retorna una cadena, hacer:
variable_str = str(input("..."))
es redundante e ineficiente, simplemente
variable_str = input("...")