Оцените код для парсинга?

Question

[Twelfth Doctor]

def parsestructure(path, userid, date):
        with open(f'/backups/{userid}/{date}') as structfile:
            content = structfile.readlines()
            parsed = []
            for line in content:
                item = line.split(' ')
                fullname = item[0]
                if fullname.startswith(path):
                    name = fullname.replace(path, '')
                    namelst = name.split('/')
                    if item[1] == 'dir':
                        isdir = True
                        isfile = False
                        name = namelst[0]
                    else:
                        isdir = False
                        isfile = True
                    pdict = {'path': path, 'name': name, 'is_dir': isdir, 'is_file': isfile}
                    parsed.append(pdict)

Код предназначен для парсинга такого файла

/home/user/vasya/file.html file
/home/user/vasya dir
<code>

Comments

[Twelfth Doctor]

Pavel Denisov, а как можно сделать лучше?

[Twelfth Doctor]

Pavel Denisov, задача - парсить файл с путями, определить что есть файл, а что- папка, получить путь до файла/папки

[Bjornie]

Pavel Denisov, про namelst соглашусь, а pdict почему лишний? чем в данном случае плохо создание переменной и аккуратное её подстановка в метод append? Что меняет перенос словаря прямо в скобки, кроме минус одной строки кода?
Реально интересно.

[Twelfth Doctor]

Pavel Denisov, namelst не лишний. Если есть путь:
/home/vasya/dir/subdir/img/...
Мне же нужно получить то, что лежит в папке /home/vasya/dir(из файла)

Answer 1

[longclaps]

О, какой прекрасный код! Неделя только еще началась, а вы уже так нас порадовали!

def parsestructure(path, userid, date):
    with open(f'/backups/{userid}/{date}') as structfile:
        parsed = []
        for line in structfile:
            item = line.split()  # https://docs.python.org/3/library/stdtypes.html#str.split
            fullname = item[0]
            if fullname.startswith(path):
                name = fullname.replace(path, '')  # для этого существует
                namelst = name.split('/')          # модуль os.path
                isdir = item[1] == 'dir'
                isfile = not isdir  # нафиг вообще нужно
                if isdir:
                    name = namelst[0]
                pdict = {'path': path, 'name': name, 'is_dir': isdir, 'is_file': isfile}
                parsed.append(pdict)

Comments

[Twelfth Doctor]

Совсем ужасный код?

[longclaps]

def parsestructure(path, userid, date):
    with open(f'/backups/{userid}/{date}') as structfile:
        parsed = []
        for line in structfile.readlines():
            item = line.split()  # https://docs.python.org/3/library/stdtypes.html#str.split
            fullname = item[0]
            if fullname.startswith(path):
                name = fullname.replace(path, '')  # для этого существует
                namelst = name.split('/')          # модуль os.path
                isdir = item[1] == 'dir'
                isfile = not isdir  # нафиг вообще нужно
                if isdir:
                    name = namelst[0]
                pdict = {'path': path, 'name': name, 'is_dir': isdir, 'is_file': isfile}
                parsed.append(pdict)

[Алексей Черемисин]

Ну, хорошие мальчики уже пересели с os.path на pathlib :-)

# python 3
from pathlib import Path

..
p = Path(item)
try: # вместо if
  rel = p.relative_to(path) # выбрасывает exception, если путь не соотвествует /home/user
  name = rel.parts[0]
  isdir = p.is_dir()
  pdict = {'path': path, 'name': name, 'is_dir': isdir}
except Exception:
  pass

И да, этот код будет обрабатывать и такое (vasya так им и останется)
/home/user/vasya/file.html
/home/user/vasya/dir/dir/dir/file.html
/home/user/vasya/dir/dir/dir

[sim3x]

longclaps, если не сложно, замените / дополните хохму в ответе кодом из коментов, а то могут случайно пометить как "шутку" и случайно удалить ответ

[Twelfth Doctor]

Алексей Черемисин, а нсли этот путь не существует в файдовой системе, pathlib будет работать?

[Алексей Черемисин]

verdex, А весь вопрос, что нужно. Есть метод exists(), который говорит существует или нет данный путь.

[Twelfth Doctor]

Алексей Черемисин, про метод exists я знаю. Сама программа предназначена для вывода списка файлов и папок в бэкапе. И в файловой системе сервера может не быть тех файлов и папок, которые есть в резервной копии, но эти файлы и папки всё равно должны отображаться.

[Алексей Черемисин]

verdex, Ну дак в свой дикт добавьте поле is_exist!