Цикл на повторение действия после прохождения каждой строки (xlrd)?

Question

[DoubleTap]

Помогите, пожалуйста, разобраться. Работаю с xls файлом, в котором построчно содержатся данные, которые необходимо внести в виде xml с последующей отправкой post запроса на api. Сам не программирую - совсем чайник, поэтому пока интуитивно пытаюсь разобраться и параллельно учусь в написании кода прямо на практике.
Мне необходимо создать цикл таким образом, чтобы после прохождения строки выполнялась процедура записи в xml и содержимое xml отправлял post на сервер, а затем переходил на следующую строку и выполнял эту же процедуру.

Пользуюсь библиотекой xlrd, накидал парочку строк, но к сожалению, по окончанию моего цикла, в xml записывается данные из последней строки.
Можете подсказать, пожалуйста, какие варианты можно еще придумать?
Спасибо!

import requests
import xlrd

book = xlrd.open_workbook(r"C:\Users\Admin\Documents\Pytest\data.xls")
sh = book.sheet_by_index(0)

for i in range(1,sh.nrows):
    print(f'# Итерация номер {i} из {sh.nrows}.')
    oldname  = sh.cell_value(rowx=i,colx=0)
    citype = sh.cell_value(rowx=i,colx=1)
    newname = sh.cell_value(rowx=i,colx=2)

with open('test.xml',"w",encoding="utf-8") as f:

url = "http://..." + содержимое xml

Comments

[shurshur]

И что должен делать этот код? Я вот вижу, что в цикле ничего не делается, соответственно, по его окончании все перемененые будут иметь значения из последней строки.

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

Во-первых, не понятно, для чего записывать xml в файл. Почему сразу не отправлять данные, без промежуточной записи в файл?

Во-вторых, python учитывает тело цикла по отступу. У вас строка with open с записью в файл находится вне цикла, поэтому туда может записаться только то, что останется после цикла.

В-третьих, файл постоянно перезаписывается, в итоге в нём останутся только данные последней строки. Такое поведение ожидалось? Судя по всему, не такое. Если ожидается, что файл должен пополняться, надо обратить внимание на то, чтобы либо открывать файл на добавление (“a”), либо считывать его предыдущее состояние, конкатенировать с ним новую порцию информации и вновь записывать, тогда он будет пополняться.

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

Ну и, в-пятых, я как-то не вижу, где собственно формируется структура xml?

[DoubleTap]

Алан Гибизов,
Спасибо за отклик!
Этот код я сократил, думал может мысль будет ясна, а тут получилось наоборот, извиняюсь :)

Насчет xml. Да, я изначально хотел так и сделать, отправка запроса без записи в xml тоже сработает, но опять-таки: что так, что этак - я все равно попадаю в ловушку с циклом, потому как post должен осуществляться после обработки += 1 строки. (через каждую)
Сейчас пробую как раз-таки именно через цикл while, но пока никаких успехов - не могу разобраться как останавливать этот самый цикл на моменте определенной строки.

3. Нет-нет, он не должен пополняться, он должен после каждого цикла обновляться, то бишь ("w"), но пока не пройдет весь цикл, опять-таки.

Мои все наброски можно посмотреть здесь:
https://pastebin.com/8RqLzUk3

[DoubleTap]

Весь цикл могу описать в словах:
1. Итерация 1-й строки
2. Записываем переменные "citype", "oldname", "newname" из xls в xml.
3. xml отправляем через url
4. Если statuscode [200], начинаем итерацию 2-й строки и т.д.

Answer 1

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

Что-то примерно такое должно быть:

Такое...

import requests
import xlrd
from pathlib import Path


def my_xml(citype, oldname, newname):
    return f'''<?xml version='1.0' encoding='UTF-8'?>
    <API version='1.0' >
        <citype>
            <name>{citype}</name>
                <criterias>
                    <criteria>
                        <parameter>
                            <name compOperator="IS">CI Name</name>
                            <value>{oldname}</value>
                        </parameter>
                    </criteria>
                </criterias>
            <newvalue>
                <record>
                    <parameter>
                        <name>CI Name</name>
                        <value>{newname}</value>
                    </parameter>
                </record>
            </newvalue>
        </citype>
    </API>'''


if __name__ == '__main__':
    datafilename = Path('data.xls')
    datafilepath = Path.home() / Path('Documents/Pytest/')
    datafile = datafilepath / datafilename
    book = xlrd.open_workbook(datafile)
    sh = book.sheet_by_index(0)
    url_prefix = "http://qwerty.com/api/ci?OPERATION_NAME=update&INPUT_DATA="
    for i in range(0, sh.nrows):
        print(f'## Итерация номер {i} из {sh.nrows}.')
        oldname = str(sh.cell_value(rowx=i, colx=0))
        citype = str(sh.cell_value(rowx=i, colx=1))
        newname = str(sh.cell_value(rowx=i, colx=2))
        url = url_prefix + my_xml(oldname, citype, newname)
        requests.post(url)

Comments

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

ну там еще прикрутить проверку на 200 и всё это...

[DoubleTap]

Супер! Уже что-то начинает получаться, буду сидеть дорабатывать.
Спасибо большое!