Skip to main content
se añadieron 41 caracteres en el cuerpo
Origen Enlace
FJSevilla
  • 62.1k
  • 7
  • 43
  • 68

No obstante, la concatenación de cadenas es muy ineficiente dada su inmutabilidad, además de generalmente muy poco legible, usa formateo de cadenas:

No obstante, la concatenación de cadenas es muy ineficiente dada su inmutabilidad, usa formateo de cadenas:

No obstante, la concatenación de cadenas es muy ineficiente dada su inmutabilidad, además de generalmente muy poco legible, usa formateo de cadenas:

Origen Enlace
FJSevilla
  • 62.1k
  • 7
  • 43
  • 68

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("...")