[ozzi91]

Ludzki #programowanie #python

Kod tutaj: https://pastebin.com/ti7Yq3fq

Co miał robić program:

Docelowo miał pobrać dane od użytkownika, a potem pozamieniać konkretne wartości w przygotowanych wcześniej plikach docx (takich jakby szablonach). Takie szybkie generowanie powtarzalnych formularzy i wniosków.

Po kolei kodu może nie będę tłumaczył, coś tam pomógł chat gpt, ale najważniejsze:
mamy słownik wszystkich gotowych formularzy z parami kluczy i wartości gdzie klucz to rzecz podmieniana a po dwukropku już gotowa wartość (docelowo wiadomo, będę pytał użytkownika). Podmiana też nie zmienia stylu formatowania tekstu w plikach.

Później mamy pętlę, gdzie w zależności od tego, jaką cyfrę podał użytkownik, do pustego słownika kopiujemy wartość z tego głównego słownika. Później pary kluczy i wartości są używane w metodzie i generują docelowy plik.

Jako że jestem na początku swojej przygody z programowaniem i się tak wdrażam to jak widzicie jakieś rażące błędy to dawajcie znać, chętnie poczytam i na 100% się czegoś też nauczę. Na dzień dobry już mi nie pasuje, że w pętli mam jeżeli warunek == 1 to, analogicznie jeżeli warunek == 2 to itd. Na bank się da to jakoś poprawić.

Docelowo chciałbym, żeby użytkownik mógł wpisać nie tylko jedną cyfrę, np. 1, 2, 4, ale na tę chwilę nie umiem sformułować warunku, bo w obecnej chwili wyszedłby z tego niemały potwór.

Aha, wiem, że nie ma to żadnej walidacji i nie sprawdza w żaden sposób użytkownika, ale na to przyjdzie jeszcze czas.

[Calka_internowana]

@ozzi91: Jest to zdecydowanie bardziej zagmatwane niż mogło by być, a objawem tego jest na przykład to, że funkcja
replacetextindocx efektywnie pracuje na słowniku z jednym tylko kluczem. Słownik z jednym kluczem nie ma racji bytu, dane których potrzebujesz są w słowniku zagnieżdżonym w środku, tam jak najbardziej ma on sens. Zauważ że iterujesz po docreplacements.items() tylko raz, po to żeby wyciągnąć do osobnej zmiennej jedyny klucz

[Calka_internowana]

@ozzi91: Dalej - widzimy że w wywołaniu tej funkcji występuje teraz dwa razy ten sam literał "zdO.docx". Skąd ta redundancja? Bierze się ona stąd że w dictOfRecepientRequests przechowujemy akurat docxpath jako klucz i replacements jako wartość. Nie musimy więc przekazywać za każdym części dictOfRecepientRequests, wystarczy przekazać klucz i już wiemy skąd wziąć replacements - z dictOfRecepientRequests[docxpath]. Funkcja replacetextindocx ma teraz postać:

def replace_text_in_docx(docx_path):
    replacements = dictOfRecepientRequests[docx_path]
    doc = docx.Document(docx_path)
...

a

[Calka_internowana]

@ozzi91: Dobrze Ci świta że da się pozbyć tych ifów. Potrzebne jest mapowanie liczb, które wpisuje użytkownik, na docxpath. Można to zrobić na wiele sposobów, na przykład definiując dodatkowy słownik:

choices_map = {
    '1': "zdO.docx",
    '2': "umigO.docx",
}

wtedy zupełnie pozbywasz się tych ifów a wywołanie funkcji zawsze wygląda tak samo:

replace_text_in_docx(choices_map[chosenRecepients])

Jednak teraz mamy dwa słowniki a literały takie jak "zdO.docx" są kluczami jednego a wartościami drugiego, można więc pokusić

[Calka_internowana]

@Calka_internowana: Oprócz tego według konwencji zmienne w pythonie nazywa się

snake_case

a stałe

SCREAMING_SNAKE_CASE

tego już nie zmieniałem żeby nie gmatwać dodatkowo

[ozzi91]

@Calka_internowana po pierwsze, wielkie dzięki za odpowiedź.

części merytorycznej nie sprawdzę teraz, pewnie dopiero wieczorem ale ma to oczywiście sens.

bardzo ważna uwaga co do zmiennych - uczyłem się z jakiegoś kursu na udemy i tam właśnie zmienne były pisane aaaaBaaaaCaaa stąd te zmienne u mnie ale już będę się stosował do tego nazewnictwa.

i przy okazji: dużo wiecej trzeba na juniora? :Do

[Krolik]

. Potrzebne jest mapowanie liczb, które wpisuje użytkownik, na docxpath. Można to zrobić na wiele sposobów, na przykład definiując dodatkowy słownik:

@Calka_internowana: do mapowania liczb nie jest potrzebny słownik, wystarczy zwykła tablica, chyba że przewidujesz dziury w numeracji.

[mrq]

@ozzi91:
@Calka_internowana:

To jest tak dobre że powinno trafić pod tag #codereview.

[ozzi91]

@mrq nie znałem tagu bo bym pewnie dodał

[mrq]

@ozzi91: spoko, dobrze ze się w ogóle coś dzieje na tagu pythona :)

[Calka_internowana]

@Krolik: Tak też napisałem, że można to zrobić na wiele sposobów, ale jakieś powiązanie liczb z formatami plików musi istnieć.

[Calka_internowana]

@ozzi91: https://pastebin.com/NBsR7g7c

[ozzi91]

@Calka_internowana przetestuje wieczorem

[ozzi91]

@Calka_internowana: przetestowane, działa super, ale wychodzi na to, że "pythondocxreplace" nie daje rady podmieniać tekstu napisanego w polach tekstowych.

Wniosek do jednego urzędu jest udostępniany w pdf z polami które można wypełnić bezpośrednio w przeglądarce pdfów. Stwierdziłem, że może przekonwertuje sobie wniosek do docx, ale jak to bywa, wykrzaczyło bo wniosek ma dużo grafik, kolumn i innych problematycznych rzeczy. Myślałem, że obejdę to poprzez pola tekstowe.

Pola tekstowe, tak

[Calka_internowana]

@ozzi91: 2. https://realpython.com/lessons/type-hinting/ to jest po prostu oznaczenie typów, żeby wiedzieć co przekazać do funkcji i co ona zwraca. Poza specyficznymi przypadkami, nie ma wpływu na działanie programu.
3. template jest obiektem typu Path i jako taki ma atrybut name https://docs.python.org/3/library/pathlib.html#pathlib.PurePath.name