[pitrek136]

#naukaprogramowania #python

Uczę się pythona od całych 2 dni i skleciłem coś takiego https://pastebin.com/VzPHT9Ek
do wyciągania danych z teoretycznego raportu

Jak napisał by to ktoś kto umie programować?

[Arogancky]

@pitrek136: zobacz sobie any()

[redorbiter]

@pitrek136: przede wszystkim po to się używa beautifulsoup, żeby nam parsowało htmla, a nie żebyśmy musieli tekst ciachać ręcznie. Po drugie za duże wcięcie - używaj czterech spacji. Zapis do pliku bym przerobił na funkcję. Pozostałe rzeczy nie wiem, ale resztę bym zostawił jak jest i uogólniał dopiero, gdy kod przybierze większe rozmiary

[lol3pdg]

@pitrek136:
W sumie to wszędzie (w każdym ifie) używasz tego samego kodu, jeżeli nie planujesz z nim nic zrobić to możesz dać or

if "Komputer" in line or blablabla

Poczytaj też o .upper() bo zakładasz, że będzie Komputer, sprawdź plik, może jest tam komputer lub KOMPUTER (?)

[venomik]

@pitrek136: Za każdym razem jak gdzieś będziesz widział w swoim kodzie, że jakiś fragment się cały czas powtarza to niech Ci się lampka otrzegawcza zapali, w większości przypadków będzie to sygnał, że kod da się poprawić.

[COVID-21]

@venomik: W sensie ze powinien z niego zrobic funkcje?

[venomik]

@COVID-21: Czasami tak, czasami nie. Na przykład w powyższym przykładzie nie ma powodu aby robic funkcję, wystarczy, zgodnie z tym co zostało napisane wyżej, odpowiednio zmodyfikować instrukcję warunkową aby łapała wszystkie trzy warunki na raz.
Nawiasem mówiąc w kodzie od @pitrek136 jest bug o którym nic nie wspomniał - otóż jeśli w tekście jest zarówno 'Komputer', 'System operacyjny' oraz 'Domena logowania' to ta linia zostanie zapisana do pliku trzy razy, ponieważ

[croppz]

@pitrek136: @COVID-21: no i plik wystarczy otworzyć raz, nie trzeba go otwierać i zamykać co linijkę tekstu