Где проблема в коде?

Question

[Oli]

Здравствуйте. Возникла проблема, я с сайта пытаюсь достать нужную мне информацию. Достать я смог, но погрузить в базу данных не получается. Почему-то мой тег разбивается по буквам. Можете, пожалуйста, помочь

def write_to_db(block):
    tags = []
    for tag in block['tags']:
        if tag in db_tags:
            tags.append(db_tags[tag])
        else:
            db_tags[tag] = len(db_tags) + 1 
            cur.execute('INSERT INTO tags VALUES (?, ?, ?)', (len(db_tags), tag, block['author']))
            conn.commit()
            tags.append(db_tags[tag])
    text_id = len(seen_news) + 1
    cur.execute(
        'INSERT INTO texts VALUES (?, ?, ?, ?)',
        (text_id, block['date'], block['full_text'], block['hse_id'])
    )
    tags = [(text_id, t) for t in tags]
    cur.executemany(
        'INSERT INTO text_to_tag (id_text, id_tag) VALUES (?, ?)',
        tags
    )
    conn.commit()
    seen_news.add(block['hse_id'])

Comments

[Алан Гибизов]

Тег разбивается по буквам потому, что вы его так разбиваете. А вы его так разбиваете потому, что не имеете понятия об основах языка программирования. Рекомендую проработать «Изучаем Python» М.Лутца. Еще первые 400 страниц первого тома книги приведут ваше понимание языка в норму.

[Алан Гибизов]

Теперь по коду:
Вы показали код функции, но как обращаетесь к функции, не показали.

Рекомендую прикладывать кусок рабочего кода, так чтобы мы взяли его, попробовали и быстро выяснили, в каком месте косяк. А так только косвенно догадываться приходится, где вы итерируете строку поэлементно вместо итерирования элементов списка.

[Oli]

Алан Гибизов, Можете, пожалуйста, в чем ошибка?

[Алан Гибизов]

Видимо, содержимое переменной block имеет атрибут «tags», а в нем - текстовая строка. По ней вы циклом for и пробегаетесь, посимвольно.
Наверно, надо строку подготовить - либо посредством метода split, либо посредством других функций, для того предназначенных - в зависимости от структуры строки, видимо содержащей теги.

[Oli]

Алан Гибизов, block['tags'] = soup.find('a', {'class': 'a-cats__link'}).text

Это функция вытягивает тег, например, проишествие

[Алан Гибизов]

irina222222, рекомендую посмотреть да хоть print-ом, как выглядит block ['tags'], и если там что-то вроде:
‘Тег 1 тег2 тег3’, то сделать
block['tags'].split(“ “)
В итоге строка разобьётся по разделителю (в данном примере по пробелу) и уж по результату разбиения пробегайтесь циклом.

[Oli]

Алан Гибизов, просто, я знаю, что у меня на сайте только один тег всегда. что может склеиваться?

[Алан Гибизов]

irina222222, ещё раз говорю: я не умею читать мысли.

[Алан Гибизов]

irina222222, если там один тег, то зачем по нему цикл for проходится? Что он, по вашему, делает?
Откомментируйте ваш код, как вы полагаете, что делает каждая операция, что получает на вход и что возвращает.

[o5a]

irina222222, если у вас

block['tags'] = soup.find('a', {'class': 'a-cats__link'}).text

то чего вы ожидаете от перебора этого значения (которое по факту текст)
for tag in block['tags']:
соответственно и получаете, что tag = одиночная буква.

Наверное предполагалось не soup.find, а soup.find_all чтобы вытянуть список всех тэгов.

[Oli]

o5a, Я исправила, но теперь вообще ничего не записывается

[Oli]

o5a, Он мне пишет

ResultSet object has no attribute 'text'. You're probably treating a list of elements like a single element. Did you call find_all() when you meant to call find()?

[o5a]

irina222222, потому что нельзя просто бездумно заменить find на find_all. Нужно различать разницу.

find - возвращает одиночный объект тэга, у которого есть атрибут .text

find_all - возвращает список таких объектов, у каждого из которых есть атрибут text. Но нельзя же просто приписать к списку text и ожидать, что сработает.

[tag1, tag2].text - так же не делают

Нужно из каждого возвращаемого элемента достать text, например так

block['tags'] = [tag.text for tag in soup.find_all('a', {'class': 'a-cats__link'})]