0

mi codigo intenta renombrar todos los archivos XML que se encuentran en una carpeta despues de obtener un dato dentro de este, pero al momento de hacerlo, renombra el primer archivo con el ultimo nombre que encuentra, alguna sugerencia? Gracias!

import os
from bs4 import BeautifulSoup

root = 'C:/Users/....'

fnames = [ arch.name for arch in os.scandir( root ) if(arch.is_file() ) ]

infiles = [open( root + fname, "r", encoding = "utf-8").read()for fname in fnames]

columnas = []

for contents in infiles:
    soup = BeautifulSoup(contents, 'xml')
    Emisores = soup.find('cfdi:Emisor')

    try:
        EmisorNombre = Emisores.get('Nombre')
    except:
        print("Something went wrong")

    print(EmisorNombre)


    for file in os.listdir(root):

         os.rename(os.path.join(root,file),
                   os.path.join(root,EmisorNombre+".xml"))
4
  • ¿Está correcta la identación? Pareciera que el os.rename sólo se ejecuta una vez, tras finalizar el ciclo for (y no dentro del ciclo)... Commented el 13 ene. 2020 a las 21:36
  • Disculpa, no, no esta correcta en un momento la corrijo. Commented el 13 ene. 2020 a las 21:39
  • Me parece que todavía no está correcta la identación. Tal cual está, el primer for te va a regresar el último emisor del contenido del último archivo. Luego de ahí va a recuperar el EmisorNombre (sólo uno). Con ese emisorNombre va a intentar renombrar cada uno de los archivos en la ruta, me parece que eso te va a generar error. Commented el 15 ene. 2020 a las 16:25
  • Tienes razon al parecer el ciclo for esta detenido y solo hace oda la operacion para uno solo, de hecho renombra correctamente el ultimo archivo pero ya no sigue haciendo el ciclo para los demas, alguna sugerencia? Commented el 15 ene. 2020 a las 17:17

1 respuesta 1

0

El problema está en el último for. Intentas renombrar todos los archivos con el mismo nombre del emisor.

Una alternativa sería evitar ese último ciclo colocando una referencia al nombre del archivo cuyo contenido se está procesando en el for externo. Algo como lo siguiente:

import os
from bs4 import BeautifulSoup

root = 'C:/Users/....'

fnames = [ arch.name for arch in os.scandir( root ) if(arch.is_file() ) ]

infiles = [open( root + fname, "r", encoding = "utf-8").read()for fname in fnames]

columnas = []
i=0 #Se agrega contador
for contents in infiles:
    soup = BeautifulSoup(contents, 'xml')
    Emisores = soup.find('cfdi:Emisor')

    try:
        EmisorNombre = Emisores.get('Nombre')
    except:
        print("Something went wrong")

    print(EmisorNombre)
    os.rename(os.path.join(root,fnames[i]),os.path.join(root,EmisorNombre+".xml"))
    i+=1 #Siguiente archivo. "i" se utiliza en la línea anterior...

Con ese código, te va a funcionar; sin embargo, es necesario mejorar el manejo de errores para los siguientes casos:

  • Más de un archivo tiene el mismo EmisorNombre. En este caso, habría archivos duplicados en su nombre y generaría error.
  • No se encuentra un EmisorNombre en el archivo.

Ambos casos se deben tratar de acuerdo a alguna regla que tengas definida.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.