1

Работаю с API Vk и хочу упросить код. Так как таких условий у меня может быть 10ки, а заполнять этим весь код не хочется. Есть ли способ сокращения?

Вот код:

def safe_file(posts):
    with open('test.csv', 'w', newline='', encoding='utf-8') as file:
        title = csv.writer(file, delimiter=';')
        title. writerow(['text', 'image', 'url'])
        for post in posts:
            try:
                if post ['id']:
                    i = post ['id']
                else:
                    i = 'pass'
                if post ['text']:
                    text = post ['text']
                else:
                    text = 'pass'
                if post ['attachments'][0]['photo']['sizes'][-1]['url']:
                    img = post ['attachments'][0]['photo']['sizes'][-1]['url']
                else:
                    img = 'pass'
            except:
                pass
                img = 'pass'

            title.writerow((text, i, img))

Мне нужно упростить эту часть кода:

for post in posts:
            try:
                if post ['id']:
                    i = post ['id']
                else:
                    i = 'pass'
                if post ['text']:
                    text = post ['text']
                else:
                    text = 'pass'
                if post ['attachments'][0]['photo']['sizes'][-1]['url']:
                    img = post ['attachments'][0]['photo']['sizes'][-1]['url']
                else:
                    img = 'pass'
            except:
                pass
                img = 'pass'
1
  • 2
    i = post['id'] or 'pass'
    – Mike
    10 авг 2020 в 7:18

3 ответа 3

5

Я бы предложил вариант с дефолтом так назовем это

def try_extract_image_url(post):
    try:
        return post['attachments'][0]['photo']['sizes'][-1]['url']
    except (KeyError, IndexError) as e:
        return None

for post in posts:
    id = post['id'] or 'pass'
    text = post['text'] or 'pass'
    img = try_extract_image_url(post) or 'pass'

Можно еще над этим поработать и придумать что то получше например заменить 'pass' на что то более специфичное что бы дальше по коду это было проще фильтровать и обрабатывать. Например завести специальный объект для этого.

2

Для начала, можно условный оператор

if post ['id']:
    i = post ['id']
else:
    i = 'pass'

Записать короче с помощью условного выыражения:

i = post['id'] if post['id'] else 'pass'

После этого я бы сделал функцию, выполняющую все три присваивания:

def con_ass(post):
    i = post['id'] if post['id'] else 'pass'
    text = ...
    img = ...
    reurn i,text,img
    

А в программе бы тогда эти присвоения выглядели бы как-то так:

i,text,img = con_ass(post)

Мне кажется - достаточно компактно и обозримо.

0

Исправленный код:

for post in posts:
            try:
                if post ['id']:
                    i = post ['id']
                elif: post ['text']:
                    text = post ['text']
                elif: post ['attachments'][0]['photo']['sizes'][-1]['url']:
                    img = post ['attachments'][0]['photo']['sizes'][-1]['url']
                else:
                    img = 'pass'
            except:
                pass
                img = 'pass'
3
  • а чей код вы исправили ?
    – S. Nick
    12 авг 2020 в 13:48
  • в данном примере получился код который не эквивалентен предыдущему указанному в вопросе. то есть он будет работать не корректно. да и еще одно маленькое замечание по поводу блоков try except. нужно их использовать с минимальным скоупом и указанием конкретных исключений которые могут быть впринципе обработаны или проигнорированы. иначе может получиться так что код будете проглатывать исключения которые могли бы говорить об ошибке в нем.
    – Mike Lazko
    13 авг 2020 в 12:16
  • в вопросе...... 13 авг 2020 в 14:11

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.