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 that you have read and understand our privacy policy and code of conduct.

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