@ozzi91: chcesz, żeby twój program był używany przez kogoś. Jest to program działający tylko na windowsie. Ludzie używają innych systemów. Ale to tam już jak wolisz.

Osobiście podzieliłbym tak:

- klasy typu DocxFile, XlsxFile, Application (w sensie *Wniosek*) w osobnych modułach
- potem dziedziczące (dziedzieczenie zazwyczaj jest złe, ale nie zawsze – tu pasuje)Pokaż całość

@ozzi91: do generowania ścieżek używaj os.dir.join()

@ozzi91: jeszcze sprawdzaj czy ścieżka istnieje: os.path.isdir() a zamiast helperow fizycznych to możesz próbować mieć coś w pamięci jeśli danych nie jest tona

@janek_: Pseudo kod: https://pastebin.com/RxZzypNc

@janek_: Hasła: agregacja oraz kompozycja - potem z dobrego zrozumienia agregacji można zrozumieć skąd bierze się agregat w DDD

W dużym skrócie wszystko zależy od tego czy obiekt może żyć / mieć własną tożsamość bez obiektu rodzica.

Przykładowo:
- książka może spokojnie istniećPokaż całość

@AnonimoweMirkoWyznania: tak, osoba relrutująca ( ͡° ͜ʖ ͡°)

@AnonimoweMirkoWyznania
1 przerób na api
2 odseparuj strzały do bazy od controllers
3 dodaj testy
4 asnotracking
5 dto lub viewmodel wysyłaj na front

@BRmedia:

1. Trzymaj się jednego języka - angielskiego. Co to ma być za ustawianie "polski"? Powinno być "polish"
2. Rzeczy typu magical number koniecznie do osobnych stałych a najlepiej do pliku gdzie masz wszystkie globalne ustawienia. Takie + 15 w kodzie nic nie mówi
3. W BooksList ten return z Bookiem wygląda źle i różni się tylko jednym propsem - ja to bym ogarnął jednym returnem z dynamicznym propsem albo wyrzucił te propsy do osobnej stałej i zrobił destrukturyzację
4. Za każdym odpaleniem "Book" kasujesz z LS'a "books".Pokaż całość

@Rst00: co do pierwszego punktu się nie zgadzam. Bardzo często ustawienia dotyczące języka są napisane właśnie w danym języku, mamy więc listę w stylu: English, Deutsch, Francois, polski

chiński, japoński, rosyjski, ukraiński, itp też są zapisane własnym alfabetem

@ProGraMajster: Poczytaj sobie o async/await vs Task.Run()

@ProGraMajster: try z pustymi catch, a inne z jakimś dzikim wypisywaniem do konsoli, inne nazwy klasy a inne pliku (form1.cs), część nazw metod po polsku a część po angielsku, troche zduplikowanego kodu też masz.

@letsgetreadytorumble: dobre

@letsgetreadytorumble: Daj programiście PR zmieniający 4 linie kodu, znajdzie ci 4 rzeczy do poprawienia. Daj PR na 400 linii, dowiesz się, że jest ok.

Popatrz ba dojrzałe projekty w django i zerznij od nich strukturę plików w projekcie bo jest troche nielogiczna u ciebie

@login_zajety_sic:
1. Linijki 17-22 - masz zduplikowaną linijkę zwracającą ścieżkę, można to zrobić mniej więcej tak
2. Jeśli chcesz połączyć stringi ze zmiennymi to używaj Pythonowego formatowania stringów - a najlepiej f-stringów.
3. Jeśli robisz operacje na ścieżkach do plików to używaj os.path.join() lub Path z modułu pathlib - one zajmą się wstawieniem slashy lub backslashy (zależnie od systemu), więc nie będziesz musiał ich ręcznie wstawiać i escape'ować.
4. Komentarze, które mówią oczywiste rzeczy jak np. "# check if the drive is of type removable" nie są potrzebne -ogólnie dobry kod powinien mieć jak najmniej komentarzy (a idealnie zero). Jednakże, jesli tworzysz jakiś komentarz (ofc jesli masz ku temu dobry powód), to rób tak, aby opisywał on dlaczego dana linijka coś robi, a nie co robi - bo co robi to widać.
5. Najpierw otwierasz file_to_save, potem otwierasz dodatkowo input_file, wpisujesz dane z niego do file_to_save po czym zamykasz oba. Więc tutaj dwie rzeczy: ZAWSZE używaj context managera (czyli konstrukcji with) gdy czytasz pliki. Drugą rzeczą byłaby lekka zmiana sterowania w twoim programie - najlepiej jakbyś najpierw otworzył input_file, przeczytał z niego dane i zapisał je do zmiennej oraz zamknął ten plik (przypominam, że konstrukcja with zamknie plik za ciebie). Następnie jak masz dane w zmiennej, to dopiero otwórz file_to_save iPokaż całość

@login_zajety_sic: osobiście jeszcze trochę popracowałbym tu na modularyzacją. Więcej mniejszych funkcji, dzięki czemu kod staje się samoopisowy i bardziej czytelny, a przy tym możesz unikać zbędnych komentarzy.

Dobrymi (choć nie jedynymi) kandydatami na osobne funkcje są np. bloki z linii 17-22 oraz 58-99.

Lepiej będzie też wyglądało jeśli początek wykonywania programu będzie jasno określony (oczywiście przed doSomething powinno być wcięcie, tylko wykop jePokaż całość

@PC27: Oba linki działają. Zmień w pierwszym linku na początku https na http i poleci.

@PC27: widocznie wygasł cert i nie łapie ssla. Gdybyś jednak kiedyś chciał zobaczyć jak strona wyglądała w przeszłości, to możesz użyć https://web.archive.org/. Tam wklejasz podstawowy link który Ciebie interesuje, czyli np. w tym przypadku http://www.urpl.gov.pl/pl/informacja-prezesa-urz%C4%99du-z-dnia-9-pa%C5%BAdziernika-2018-r-w-sprawie-dopuszczenia-do-obrotu-0 i wybierasz któryś z zapisanych snapshotów

@crasti
znajomym któtym ufam, czasem płacę komuś kto wydaje mi się lepszy ode mnie w jakimś temacie

@crasti napisz na grupie na fejsie danego języka. Tam co chwilę ktoś prosi o przegląd.

btw. ta walidacja powinna byc w $form->isValid(), czyli formularz powinien ci to zrobic.

@Klopsztanga: tak właściwie to powinna być ogarnięta przez api platform albo fos rest bundle na podstawie przyjmowanego dto i jego constraintów, nie ma tutaj potrzeby żeby bezpośrednio dotykać requesta albo się foremkami bawić ¯\_(ツ)_/¯

@h00k:
##############################
# XD
##############################

Poczytaj o https://docs.python.org/3.8/library/argparse.html

@Surykat: nice English.

Jak był tutaj przykład z tym orderem i i discountami, to gdzie oni chcieli je wyrzucić, aby ta informacja nadal gdzieś była dostępna?