@hellooo: Ich verstehe die Frage nicht. Wenn man auf eine Zelle in einem Arbeitsblatt über die Koordinaten in Excelschreibweise zugreifen will dann sind das Buchstaben für die Spaltenangabe gefolgt von Ziffern für die Zeilenangabe, also aus Python-Sicht eine Zeichenkette/Text. Wenn man die Zeilennummer als Zahl vorliegen hat, dann muss man eine Zeichenkette erstellen in dem man die Spalten und Zeileninformation entsprechend zusammenfügt. Also den (oder die) Buchstaben mit der Zahl als Zeichenkette verbinden.
Im Videotutorial wird das auf eine eher „unpythonische“ weise gemacht mit `str()` und ``+``. Das ist eher BASIC denn Python. In Python macht man so etwas mit Zeichenkettenformatierung mit f-Zeichenketten. Oder falls die Vorlage dynamisch sein soll mit der `format()`-Methode auf Zeichenketten. Also konkret statt so etwas wie ``"C" + str(row)`` schreibt man in Python ``f"C{row}"``. Und statt `row` würde ich das auch eher `row_number` nennen, denn es ist ja keine Zeile sondern eine Zeilennummer. Eine Liste mit Werten oder Zellen für eine Zeile kann man ja auch als Wert haben und da ist `row` dann ein passender Name.
Was an dem Tutorial auch schlecht ist, ist die Namenswahl. Zum einen dauernd kryptische Abkürzungen, und dann auch ”technisch” korrekte, aber total nichtssagende Namen. Beispielsweise die Variablen `c2` und `d2` für die Zellenwerte von C2 und D2 im Arbeitsblatt. Ja das sind irgendwie korrekte Namen, aber komplett Unsinnig wenn es darum geht zu verstehen was das Programm macht. Der Leser vom Quelltext will ja nicht wissen aus welchen Zellen die Werte kommen, sondern was die Werte in diesen Zellen *bedeuten*. Im Arbeitsblatt „Bestellungen“ wären das zum Beispiel `anzahl` und `preis`.
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
sheet = workbook["Bestellungen"]
count = sheet["C2"].value
price = sheet["D2"].value
total = count * price
print("Preis ist:", total)
sheet["E2"].value = total
workbook.save(filename)
if __name__ == "__main__":
main()
Um die Schleife erweitert und mit f-Zeichenketten statt `str()` und ``+``:
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
sheet = workbook["Bestellungen"]
for row_number in range(2, 23):
count = sheet[f"C{row_number}"].value
price = sheet[f"D{row_number}"].value
sheet[f"E{row_number}"].value = count * price
workbook.save(filename)
if __name__ == "__main__":
main()
Dieses manuelle Zusammenbasteln von Excel-Zellenkoordinaten ist aber gar nicht nötig. Man kann auch Zeilenweise über die Zellen eines rechteckigen Ausschnitts in einer Schleife iterieren, in dem man für Zeilen und Spalten jeweils Start und Ende als Zeilen und Spaltennummern an die `iter_rows()`-Methode übergibt:
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
sheet = workbook["Bestellungen"]
for count_cell, price_cell, total_cell in sheet.iter_rows(2, 22, 3, 5):
total_cell.value = count_cell.value * price_cell.value
workbook.save(filename)
if __name__ == "__main__":
main()
Im Tutorial werden die Begriffe Array und Liste vermischt. Das ist nicht gut. Der Grunddatentyp der verwendet wird ist eine Liste. Es gibt auch Arrays in Python, womit in der Regel Numpy-Arrays gemeint sind. Das sollte man nicht durcheinander bringen.
Die `get_price()`-Funktion greift einfach so magisch auf das `Workbook`-Objekt zu. Das sollte nicht sein. Funktionen sollten in sich geschlossen sein, und alles was sie ausser Konstanten benötigen als Argument(e) übergeben bekommen. Globale Variablen sollten eigentlich gar nicht existieren, weshalb man das Hauptprogramm auch in eine Funktion steckt. Die heisst in der Regel `main()`.
Statt so einen komischen Wert wie 9999999 zurück zu geben, falls der Produktname nicht gefunden werden konnte, würde man eine Ausnahme auslösen. Darum kann sich der Aufrufer dann explizit kümmern und selber entscheiden was genau passieren soll. Oder man kümmert sich nicht explizit darum und lässt das Programm dann durch den Fehler abbrechen. Das Programm könnte dann so aussehen:
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def get_price(sheet, name):
for product_name, price in sheet.iter_rows(2, 11, 1, 2, values_only=True):
if product_name == name:
return price
raise KeyError(f"no price for {name}")
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
order_sheet = workbook["Bestellungen"]
#
# Preise in Tabelle ausfüllen.
#
price_sheet = workbook["Preisliste"]
for product_name_cell, _, price_cell in order_sheet.iter_rows(2, 22, 2, 4):
price_cell.value = get_price(price_sheet, product_name_cell.value)
#
# Gesamtpreis für Bestellungen berechnen.
#
for count_cell, price_cell, total_cell in order_sheet.iter_rows(
2, 22, 3, 5
):
total_cell.value = count_cell.value * price_cell.value
workbook.save(filename)
if __name__ == "__main__":
main()
Die beiden Schleifen könnte man zu einer Schleife zusammenfassen:
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def get_price(sheet, name):
for product_name, price in sheet.iter_rows(2, 11, 1, 2, values_only=True):
if product_name == name:
return price
raise KeyError(f"no price for {name}")
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
order_sheet = workbook["Bestellungen"]
price_sheet = workbook["Preisliste"]
for (
product_name_cell,
count_cell,
price_cell,
total_cell,
) in order_sheet.iter_rows(2, 22, 2, 5):
price_cell.value = get_price(price_sheet, product_name_cell.value)
total_cell.value = count_cell.value * price_cell.value
workbook.save(filename)
if __name__ == "__main__":
main()
Das lineare Suchen in der Preisliste ist ineffizient. Da würde man besser die Daten aus der Preisliste in ein Wörterbuch (Datentyp `dict`) überführen:
Code: Alles auswählen
#!/usr/bin/env python3
from openpyxl import load_workbook
def main():
filename = "Bestellungen.xlsx"
workbook = load_workbook(filename)
product_name_to_price = dict(
workbook["Preisliste"].iter_rows(2, 11, 1, 2, values_only=True)
)
for (
product_name_cell,
count_cell,
price_cell,
total_cell,
) in workbook["Bestellungen"].iter_rows(2, 22, 2, 5):
price_cell.value = product_name_to_price[product_name_cell.value]
total_cell.value = count_cell.value * price_cell.value
workbook.save(filename)
if __name__ == "__main__":
main()