Не могу передать строку в writerow, что не так?

Question

[DandyDev]

Консоль выкидывает ошибку TypeError: string indices must be integers, при попытке передачи построчно

for i in range(j, len(property)):
  k = j
  write_to_csv(property[k])
  k += 1
j = len(property)

Полный текст ошибки :
Traceback (most recent call last):
File "C:/Users/Danis/PycharmProjects/parser/parser_cian.py", line 116, in
get_offices(url)
File "C:/Users/Danis/PycharmProjects/parser/parser_cian.py", line 71, in get_offices
write_to_csv(property[k])
File "C:/Users/Danis/PycharmProjects/parser/parser_cian.py", line 82, in write_to_csv
a_pan.writerow((building['адрес'], building['площадь'], building['цена'], building['цена за м²'],
TypeError: string indices must be integers

def write_to_csv(property):
    with open('cian_parser.csv', 'a', encoding='utf8', newline='') as file:
        a_pan = csv.writer(file, delimiter=',')
        for building in property:
            a_pan.writerow((building['адрес'], building['площадь'], building['цена'], building['цена за м²'],
                            building['ссылка'], building['дата выгрузки']))
    read_file = pd.read_csv(r'cian_parser.csv')
    read_file.to_excel(r'cian_parser.xlsx', index=None, header=True)

Answer 1

[DandyDev]

Разобрался, цикл был не нужен.
Меняем

def write_to_csv(property):
    with open('cian_parser.csv', 'a', encoding='utf8', newline='') as file:
     #...

на

def write_to_csv(property):
    with open('cian_parser.csv', 'w', encoding='utf8', newline='') as file:
    #...

Answer 2

[Сергей П]

Вот это кошмар...
Нагромождение каких-то ненужных приседаний и реверансов. Склажывается ощущение. что код писался вообще без понимания что и как работает.
Тут проще пересчитать строки, которые не требовалось бы исправить: ноль таких строк.
Вы бы привели хоть пример что у вас там в property подается. А-то на кофейной гуще гадать приходится какая из сотни ошибок-таки окончательно стедала неработоспособным код.

Вот здесь вообще какая-то ерунда:

for i in range(j, len(property)):
    k = j
    write_to_csv(property[k])
    k += 1
  j = len(property)

Ладно, с отступами беда. Но зачем эти бессмысленные присвоения? Зачем этот бессмысленный цикл, если вы в нем обрабатываете один и тот же элемент много раз?
Прекращайте уже такое писать. Смысла нет. Не ваше.

Похоже (разводы гущи так подсказывают за неимением других данных) вы не понимаете что передаёте в эту свою функцию. Ситуацию бы прояснила публикаия содержимого переменной roperty

Comments

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

Студент 3-го курса Прикладной информатики

Бедная прикладная информатика...

[DandyDev]

Раз уж всё настолько плохо, вы бы лучше литературу посоветовали, чем в пустую критиковать.
И по вашему ответу сразу ясно, что вы не понимаете, как вообще программа может работать
Перейду к делу,
в программе имеется глобальная переменная типа list
property = []
Далее в классе get_offices она обновляется

def get_offices(page_url):
    for office in offices:
#....
        property.append({
            "адрес": address,
            "площадь": area,
            "цена": price,
            "цена за м²": price_s_m,
            "ссылка": url,
            "дата выгрузки": today.strftime("%d-%m-%Y")
        })

        for i in range(j, len(property)):
            k = j
            write_to_csv(property[k])
            k += 1
        j = len(property)

С пробелами и правда беда, но лишь из-за того что впервые использую данный ресурс
Насчёт непонимания, я запутался с работой цикла, на ресурсе https://metanit.com/python/tutorial/3.1.php
написано:

При выполнении цикла Python последовательно получает все числа из коллекции, которая создается функцией range, и сохраняет эти числа в переменной int_var. При первом проходе цикл получает первое число из коллекции, при втором - второе число и так далее, пока не переберет все числа. Когда все числа в коллекции будут перебраны, цикл завершает свою работу.

ввиду чего запутался с работой счётчика(думаю. что он будет отсчитывать элементы коллекции помечая первый единицей ), отсюда и данная попытка решения задачи.
Каюсь, под капот залезть не догадался

[Сергей П]

Сергей Тихомиров, вам нужно взять первую попавшуюся книжку (почти любую) для новичков в языке.
Дело в том, что вы пытаетесь программироват на питоне "наугад". Это до добра не доведёт.
Преставьте, что у вас есть современная мастерская, а вы очень изобретательный человек из средневековья. Вы не догадываетесь зачем нужны синие колечки (изолента), поэтому крептие предметы друг к другу привязывая их проводами, потому что они похожи на привычные вам верёвки.
Просто почитайте тогоже Марка Лутца без остановок, как художественную. Это у вас займёт пару дней, но потом вы сможете уже работать с книгой как с справочником.

В Питоне все значения - это объекты. Функции, классы, типы - это все тоже объекты. даже целые и вещественные числа - это объекты. Всё - объекты.
У объектов есть методы - это собственные функции. Но вы лучше почитайте об основах ООП сами. Я не нанимался вас здесь обучать.
Переменная в питоне - это всегда ссылка на объект, которую можно использовать напрямую, без "разыменовывания".
Можно привести простой пример: обьекты - это предметы: кружка, ложка, нож. Переменные - это ярлычки на ниточке, которые можно привязывать к предметам. Причем к одному предмету можно привязать несколько ярлычков. При присвоении происходит привязывание нового (или старого) ярлычка к предмету. Всё, что не приязано будет выбошено "уборщицей" (сборщик мусора).
Объекты бывают изменяемые и неизменяемые.

Далее в классе get_offices она обновляется

Но ниже вы написали функцию. а не класс.

по вашему ответу сразу ясно, что вы не понимаете, как вообще программа может работать

Программы на языках программирования пишутся для того, чтобы быть понятными в первую очередь не компьютеру, а человеку. Вы не привели полный код. а те отрывки, что привели изобилуют совершенно неочевидными, контринтуитивными и откровенно вредными приёмами, которые испольуюся либо в самых азах, когда нужно освоить самые базовые основы алгоритмиации, либо в поделках "программистов", которые считают, что они и так все понимают и им не нужно изучать язык, чтобы на нем программировать.

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

Вот здесь
for i in range(j, len(property)):
у вас упоминается переменная j в функции, где она еще не была определена. Это значит, что вы ее объявили где-то глобально! Зачем?(!) Почему? (!). Короч ебред какой-то.

Этот код безнадёжен, а вам нужно читать. Начните с Мака Лутца "Программирование на Питон".