From 5e92b8d9987c4a32a13f2cfe045796de9fcf72e9 Mon Sep 17 00:00:00 2001 From: Aminda Suomalainen Date: Fri, 12 Jul 2024 10:58:51 +0300 Subject: [PATCH] python: low-hanging pylint fixes --- .pre-commit-config.yaml | 14 +------- python/taitotalo/.editorconfig | 15 --------- python/taitotalo/.gitattributes | 3 -- .../{arvaa-numero.py => arvaa_numero.py} | 12 ++++--- ...an-comparison.py => boolean_comparison.py} | 4 +-- .../{if-salasana.py => if_salasana.py} | 4 +-- .../{ika-tarkistus.py => ika_tarkistus.py} | 12 ++++--- ...aampi.py => ikatarkistus_kohteliaammin.py} | 12 ++++--- ...joita-uusi-txt.py => kirjoita_uusi_txt.py} | 0 ...sakset-paperi.py => kivi_sakset_paperi.py} | 9 +++-- ...ityo-parit.py => muodosta_parityoparit.py} | 6 ++-- python/taitotalo/nelilaskin.py | 15 ++++++--- python/taitotalo/plussaaja.py | 3 +- ...vetuloa_merkkijono_muuttujayhteenlasku.py} | 11 ++++--- ...{tulosta-numerot.py => tulosta_numerot.py} | 3 +- ...ikosti.py => tunnistautuminen_heikosti.py} | 0 ...e-suomenlippu.py => turtle_suomenlippy.py} | 33 +++++++++++-------- python/taitotalo/viestikolmesti.py | 3 +- 18 files changed, 80 insertions(+), 79 deletions(-) delete mode 100644 python/taitotalo/.editorconfig delete mode 100644 python/taitotalo/.gitattributes rename python/taitotalo/{arvaa-numero.py => arvaa_numero.py} (84%) rename python/taitotalo/{boolean-comparison.py => boolean_comparison.py} (80%) rename python/taitotalo/{if-salasana.py => if_salasana.py} (81%) rename python/taitotalo/{ika-tarkistus.py => ika_tarkistus.py} (83%) rename python/taitotalo/{ikatarkistus-kohteliaampi.py => ikatarkistus_kohteliaammin.py} (89%) rename python/taitotalo/{kirjoita-uusi-txt.py => kirjoita_uusi_txt.py} (100%) rename python/taitotalo/{kivi-sakset-paperi.py => kivi_sakset_paperi.py} (89%) rename python/taitotalo/{muodosta-parityo-parit.py => muodosta_parityoparit.py} (91%) rename python/taitotalo/{tervetuloa-merkkijono-muuttujayhteenlasku.py => tervetuloa_merkkijono_muuttujayhteenlasku.py} (59%) rename python/taitotalo/{tulosta-numerot.py => tulosta_numerot.py} (95%) rename python/taitotalo/{tunnistautuminen-heikosti.py => tunnistautuminen_heikosti.py} (100%) rename python/taitotalo/{turtle-suomenlippu.py => turtle_suomenlippy.py} (70%) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index c38fa81..6e24e2b 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -33,8 +33,7 @@ repos: #- id: check-xml - id: debug-statements - id: mixed-line-ending - # .gitattributes is trusted to handle this - args: [--fix=no] + args: [--fix=lf] - id: pretty-format-json args: [--autofix, --no-ensure-ascii] @@ -56,17 +55,6 @@ repos: - id: mypy additional_dependencies: [types-tabulate] - - repo: https://github.com/asottile/pyupgrade - rev: v3.16.0 - hooks: - - id: pyupgrade - args: [--py310-plus] - - - repo: https://github.com/asottile/add-trailing-comma - rev: v3.1.0 - hooks: - - id: add-trailing-comma - - repo: https://github.com/pycqa/isort rev: 5.13.2 hooks: diff --git a/python/taitotalo/.editorconfig b/python/taitotalo/.editorconfig deleted file mode 100644 index 831f06a..0000000 --- a/python/taitotalo/.editorconfig +++ /dev/null @@ -1,15 +0,0 @@ -root = true - -[*] -# https://peps.python.org/pep-0008/#indentation -indent_style = space -indent_size = 4 -max_line_length = 79 - -[*.py] -profile = black -max_line_length = 79 -#end_of_line = crlf - -[*.md] -end_of_line = lf diff --git a/python/taitotalo/.gitattributes b/python/taitotalo/.gitattributes deleted file mode 100644 index 6f891f7..0000000 --- a/python/taitotalo/.gitattributes +++ /dev/null @@ -1,3 +0,0 @@ -* text=auto -# This is for doctoc should I decide to use it in the future -*.md text eol=lf diff --git a/python/taitotalo/arvaa-numero.py b/python/taitotalo/arvaa_numero.py similarity index 84% rename from python/taitotalo/arvaa-numero.py rename to python/taitotalo/arvaa_numero.py index 0c35510..8464fb3 100644 --- a/python/taitotalo/arvaa-numero.py +++ b/python/taitotalo/arvaa_numero.py @@ -33,7 +33,8 @@ def arvaa_numero(): laskuri += 1 - # Pyydetään käyttäjältä luku varautuen virheelliseen syötteeseen. Se on tärkeää muuttaa int:iksi tai muuten se on string! + # Pyydetään käyttäjältä luku varautuen virheelliseen syötteeseen. Se + # on tärkeää muuttaa int:iksi tai muuten se on string! try: arvaus = int(input("Arvaa numero? ")) # Virhesyötteen tapahtuessa @@ -54,13 +55,16 @@ def arvaa_numero(): ) break - # Muuten avustetaan käyttäjää antamalla vihje. Printistä myös poistetaan erottavat välilyönnit, jotka tehdään itse, jotta arvauksen ja pisteen väliin ei tule välilyöntejä - elif arvaus < oikeavastaus: + # Muuten avustetaan käyttäjää antamalla vihje. Printistä myös + # poistetaan erottavat välilyönnit, jotka tehdään itse, jotta + # arvauksen ja pisteen väliin ei tule välilyöntejä + if arvaus < oikeavastaus: print("Oikea luku on suurempi kuin ", arvaus, ".", sep="") else: print("Oikea luku on pienempi kuin ", arvaus, ".", sep="") -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan +# pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/boolean-comparison.py b/python/taitotalo/boolean_comparison.py similarity index 80% rename from python/taitotalo/boolean-comparison.py rename to python/taitotalo/boolean_comparison.py index afccc81..7baf699 100644 --- a/python/taitotalo/boolean-comparison.py +++ b/python/taitotalo/boolean_comparison.py @@ -6,8 +6,8 @@ import datetime tanaan = datetime.date.today() vuosi = int(tanaan.year) -nimi = "Jussi" +NIMI = "Jussi" print(vuosi) print(vuosi == 2014) -print(vuosi is vuosi) +# print(vuosi is vuosi) diff --git a/python/taitotalo/if-salasana.py b/python/taitotalo/if_salasana.py similarity index 81% rename from python/taitotalo/if-salasana.py rename to python/taitotalo/if_salasana.py index d68848f..2e18e33 100644 --- a/python/taitotalo/if-salasana.py +++ b/python/taitotalo/if_salasana.py @@ -2,12 +2,12 @@ Sovellus tekee yksinkertaisen salasanatarkistuksen if:illä """ -koodisana = "python" +KOODISANA = "python" print("Tervetuloa") syote = input("Anna salasana: ") -if syote == koodisana: +if syote == KOODISANA: print("Matka voi alkaa") else: print("Yritä uudelleen") diff --git a/python/taitotalo/ika-tarkistus.py b/python/taitotalo/ika_tarkistus.py similarity index 83% rename from python/taitotalo/ika-tarkistus.py rename to python/taitotalo/ika_tarkistus.py index 71dd284..e864831 100644 --- a/python/taitotalo/ika-tarkistus.py +++ b/python/taitotalo/ika_tarkistus.py @@ -1,16 +1,19 @@ """ -Sovellus kysyy käyttäjän ikää, kertoo alle 18 heidän olevan alaikäinen, 18-65 aikuinen ja yli 65 seniori. +Sovellus kysyy käyttäjän ikää, kertoo alle 18 heidän olevan alaikäinen, 18-65 +aikuinen ja yli 65 seniori. """ # Tuo sys -kirjasto, jotta voimme poistua virhekoodiin. import sys -# Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin numeroita ja varaudutaan asiaan +# Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin +# numeroita ja varaudutaan asiaan try: # Tallennetaan kokonaislukuna käyttäjältä pyydetty ikä ika = int(input("Mikä on ikäsi? ")) -# Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla, jos käyttäjä ei syöttänyt numeroita +# Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla, +# jos käyttäjä ei syöttänyt numeroita except ValueError: print("Syötä ikäsi numeroina!") sys.exit(1) @@ -18,7 +21,8 @@ except ValueError: # Aloitetaan kokeilemalla onko käyttäjä alle 18. if ika < 18: print("Olet alaikäinen") -# Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän alle 65. +# Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän +# alle 65. elif (ika >= 18) and (ika < 65): print("Olet aikuinen") # Mikäli ikä on 65 tai yli, käyttäjä on seniori. diff --git a/python/taitotalo/ikatarkistus-kohteliaampi.py b/python/taitotalo/ikatarkistus_kohteliaammin.py similarity index 89% rename from python/taitotalo/ikatarkistus-kohteliaampi.py rename to python/taitotalo/ikatarkistus_kohteliaammin.py index d06b97e..fdc00d7 100644 --- a/python/taitotalo/ikatarkistus-kohteliaampi.py +++ b/python/taitotalo/ikatarkistus_kohteliaammin.py @@ -24,11 +24,13 @@ def main(): print("Olen", ikani, "-vuotias", nimeni) nimi = input("Mikä sinun nimesi on?: ") - # Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin numeroita ja varaudutaan asiaan + # Odotetaan käyttäjän mahdollisesti pyrkivän syöttämään muutakin kuin + # numeroita ja varaudutaan asiaan try: # Tallennetaan kokonaislukuna käyttäjältä pyydetty ikä ika = int(input("Entä kuinka vanha olet? ")) - # Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan virhekoodilla, jos käyttäjä ei syöttänyt numeroita + # Annetaan käyttäjäystävällinen virheilmoitus ja poistutaan + # virhekoodilla, jos käyttäjä ei syöttänyt numeroita except ValueError: print("Syötä ikäsi numeroina!") sys.exit(1) @@ -40,7 +42,8 @@ def main(): # Aloitetaan kokeilemalla onko käyttäjä alle 18. if ika < 18: print("Olet alaikäinen") - # Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän alle 65. + # Jos käyttäjä ei ollut alaikäinen, varmistetaan onko hän 18 ja onko hän + # alle 65. elif (ika >= 18) and (ika < 65): print("Olet täysi-ikäinen") # Mikäli ikä on 65 tai yli, käyttäjä on seniori. @@ -51,6 +54,7 @@ def main(): print("Jokin meni pieleen. Yritä uudelleen.") -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan +# pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/kirjoita-uusi-txt.py b/python/taitotalo/kirjoita_uusi_txt.py similarity index 100% rename from python/taitotalo/kirjoita-uusi-txt.py rename to python/taitotalo/kirjoita_uusi_txt.py diff --git a/python/taitotalo/kivi-sakset-paperi.py b/python/taitotalo/kivi_sakset_paperi.py similarity index 89% rename from python/taitotalo/kivi-sakset-paperi.py rename to python/taitotalo/kivi_sakset_paperi.py index e96d8c1..87db07d 100644 --- a/python/taitotalo/kivi-sakset-paperi.py +++ b/python/taitotalo/kivi_sakset_paperi.py @@ -34,13 +34,16 @@ def main(): elif tkvalinta == "paperi" and valinta == "sakset": print("Voitit, sakset voittavat paperin") else: - # Tästä tuli vahingossa catch-all virheellisille syötteille, enkä ole varma toteutinko kaikki skenaariot + # Tästä tuli vahingossa catch-all virheellisille syötteille, enkä ole + # varma toteutinko kaikki skenaariot print( - "Joko tilannetta ei ole vielä toteutettu tai annoit virheellisen syötteen. Ilmoittaisitko ongelman kehittäjälle?", + "Joko tilannetta ei ole vielä toteutettu tai annoit \ +virheellisen syötteen. Ilmoittaisitko ongelman kehittäjälle?" ) print("Tietokone valitsi:", tkvalinta, "syötteesi oli", valinta) -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan +# pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/muodosta-parityo-parit.py b/python/taitotalo/muodosta_parityoparit.py similarity index 91% rename from python/taitotalo/muodosta-parityo-parit.py rename to python/taitotalo/muodosta_parityoparit.py index a14733f..c89648f 100644 --- a/python/taitotalo/muodosta-parityo-parit.py +++ b/python/taitotalo/muodosta_parityoparit.py @@ -23,7 +23,8 @@ def main(): random_names1 = random.sample(nimet, 3) random_names2 = random.sample(nimet2, 3) - # Luo tuplen, jossa zip yhdistää random_names{1,2} vastaavan kohdan kanssa, jotka sitten laitetaan listaan. + # Luo tuplen, jossa zip yhdistää random_names{1,2} vastaavan kohdan + # kanssa, jotka sitten laitetaan listaan. name_pairs = list(zip(random_names1, random_names2)) print(name_pairs) @@ -38,6 +39,7 @@ def main(): print(table) -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, +# suoritetaan pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/nelilaskin.py b/python/taitotalo/nelilaskin.py index fdf179b..84040cf 100644 --- a/python/taitotalo/nelilaskin.py +++ b/python/taitotalo/nelilaskin.py @@ -13,21 +13,25 @@ def main(): Pääfunktio. """ - # Pyydetään käyttäjältä kaksi lukua varautuen siihen, ettei käyttäjä syötä lukuja oikeassa muodossa. + # Pyydetään käyttäjältä kaksi lukua varautuen siihen, ettei käyttäjä + # syötä lukuja oikeassa muodossa. try: luku1 = float(input("Anna ensimmäinen luku: ")) luku2 = float(input("Anna toinen luku: ")) - # Mikäli käyttäjä syöttää jotakin, mitä emme voi muuttaa float:iksi tai int:iksi, annetaan virheilmoitus ja poistutaan virhekoodiin. + # Mikäli käyttäjä syöttää jotakin, mitä emme voi muuttaa float:iksi tai + # int:iksi, annetaan virheilmoitus ja poistutaan virhekoodiin. except ValueError: print("Syötä luku!") sys.exit(1) - # Tulostetaan laskutoimitukset. Käytetään int siisteyden vuoksi, vaikka uhrataan tuki desimaalien laskemiselle. + # Tulostetaan laskutoimitukset. Käytetään int siisteyden vuoksi, vaikka + # uhrataan tuki desimaalien laskemiselle. print(int(luku1), "+", int(luku2), "=", int(luku1 + luku2)) print(int(luku1), "-", int(luku2), "=", int(luku1 - luku2)) print(int(luku1), "*", int(luku2), "=", int(luku1 * luku2)) - # Varaudutaan siihen, että käyttäjä syöttää luvuksi nollan ja siinä tapauksessa kieltäydytään suorittamasta tehtävää. + # Varaudutaan siihen, että käyttäjä syöttää luvuksi nollan ja siinä + # tapauksessa kieltäydytään suorittamasta tehtävää. if luku1 == 0 or luku2 == 0: print("Nollalla ei voi jakaa.") else: @@ -35,6 +39,7 @@ def main(): print(int(luku1), "/", int(luku2), "=", (luku1 / luku2)) -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan +# pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/plussaaja.py b/python/taitotalo/plussaaja.py index 9d7b4e2..2d39d5a 100644 --- a/python/taitotalo/plussaaja.py +++ b/python/taitotalo/plussaaja.py @@ -39,6 +39,7 @@ def main(): print("Syötä luku!") -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, +# suoritetaan pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/tervetuloa-merkkijono-muuttujayhteenlasku.py b/python/taitotalo/tervetuloa_merkkijono_muuttujayhteenlasku.py similarity index 59% rename from python/taitotalo/tervetuloa-merkkijono-muuttujayhteenlasku.py rename to python/taitotalo/tervetuloa_merkkijono_muuttujayhteenlasku.py index 95aa4bb..a49e5c9 100644 --- a/python/taitotalo/tervetuloa-merkkijono-muuttujayhteenlasku.py +++ b/python/taitotalo/tervetuloa_merkkijono_muuttujayhteenlasku.py @@ -1,5 +1,6 @@ """ -Tämä on ensimmäinen Taitotalossa kirjoitettu Python koodi koulutuspäivänä yksi. Se vain tulostaa asioita. +Tämä on ensimmäinen Taitotalossa kirjoitettu Python koodi koulutuspäivänä +yksi. Se vain tulostaa asioita. """ @@ -9,8 +10,8 @@ print("Tervetuloa") # toinen kommentti print("esi" + "merkki") -luku1 = 5 -luku2 = 7 +LUKU1 = 5 +LUKU2 = 7 -summa = luku1 + luku2 -print("Vastaus", summa) +SUMMA = LUKU1 + LUKU2 +print("Vastaus", SUMMA) diff --git a/python/taitotalo/tulosta-numerot.py b/python/taitotalo/tulosta_numerot.py similarity index 95% rename from python/taitotalo/tulosta-numerot.py rename to python/taitotalo/tulosta_numerot.py index b15ae7d..8b93b56 100644 --- a/python/taitotalo/tulosta-numerot.py +++ b/python/taitotalo/tulosta_numerot.py @@ -29,6 +29,7 @@ def main(): print() -# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, suoritetaan pääfunktio. +# Jos suoritamme koodia suoraan skriptistä importtauksen sijaan, +# suoritetaan pääfunktio. if __name__ == "__main__": main() diff --git a/python/taitotalo/tunnistautuminen-heikosti.py b/python/taitotalo/tunnistautuminen_heikosti.py similarity index 100% rename from python/taitotalo/tunnistautuminen-heikosti.py rename to python/taitotalo/tunnistautuminen_heikosti.py diff --git a/python/taitotalo/turtle-suomenlippu.py b/python/taitotalo/turtle_suomenlippy.py similarity index 70% rename from python/taitotalo/turtle-suomenlippu.py rename to python/taitotalo/turtle_suomenlippy.py index a0790b2..6213a06 100644 --- a/python/taitotalo/turtle-suomenlippu.py +++ b/python/taitotalo/turtle_suomenlippy.py @@ -1,6 +1,11 @@ +""" +Tämän skriptin olisi tarkoitus piirtää Suomen lippu turtle-kirjastolla, +mutta en koskaan tehnyt siitä valmista. +""" + # Tuodaan turtle -kirjasto # Satunnaislukuja -import random +# import random import turtle # ikkkuna = turtle.Screen() @@ -48,37 +53,37 @@ turtle.showturtle() # Piirretään suorakulmio -vaaka = 300 -pysty = 200 +VAAKA = 300 +PYSTY = 200 # -turtle.forward(vaaka) +turtle.forward(VAAKA) turtle.left(90) -turtle.forward(pysty) +turtle.forward(PYSTY) # turtle.left(90) -turtle.forward(vaaka) +turtle.forward(VAAKA) # turtle.left(90) -turtle.forward(pysty) +turtle.forward(PYSTY) turtle.left(90) # Piirretään risti -turtle.forward(vaaka / 2) +turtle.forward(VAAKA / 2) turtle.left(90) -turtle.forward(pysty / 3) +turtle.forward(PYSTY / 3) turtle.right(90) -turtle.forward(vaaka / 3) +turtle.forward(VAAKA / 3) turtle.left(90) -turtle.forward(vaaka / 5) +turtle.forward(VAAKA / 5) turtle.left(90) -turtle.forward(vaaka / 3) +turtle.forward(VAAKA / 3) turtle.right(90) -turtle.forward(vaaka / 3) +turtle.forward(VAAKA / 3) -# turtle.forward(pysty / 3) +# turtle.forward(PYSTY / 3) # Pidetään ikkuna auki turtle.mainloop() diff --git a/python/taitotalo/viestikolmesti.py b/python/taitotalo/viestikolmesti.py index 170cff5..56cca75 100644 --- a/python/taitotalo/viestikolmesti.py +++ b/python/taitotalo/viestikolmesti.py @@ -7,5 +7,6 @@ Sovellus pyytää käyttäjältä nimen, sekä viestin ja tulostaa viestin kolme nimi = input("Nimi: ") viesti = input("Viesti: ") -# Tulostetaan nimi ja kolmesti viesti rivinvaihdolla. Lopuksi poistetaan rivinvaihto ja erotin tulostuksesta +# Tulostetaan nimi ja kolmesti viesti rivinvaihdolla. Lopuksi poistetaan +# rivinvaihto ja erotin tulostuksesta print(nimi, " sanoo:\n", (viesti + "\n") * 3, end="", sep="")