search

sonjaolkkonen / spell-checker

0 stars 0 forks source link

Vertaisarvio 2 #2

Open jayzstep opened 5 months ago

jayzstep commented 5 months ago

Vertaisarvio

Repo kloonattu 20.6 klo 10.45

Mielenkiintoinen projekti!

Ohjeista:

UI pärähtää käyntiin hienosti, vaikka Poetryltä tuleekin sama herjaus kuin edellisellä vertaisarvioijalla.

Pitäisikö poetry install tehdä kuitenkin ennen shellin käynnistämistä? Onko sillä väliä?

UI

Koko sivu latautuu uusiksi kun submittaa, se vähän häiritsee pienellä ruudulla, sillä sivu “hyppii” ja poloinen käyttäjä joutuu scrollaamaan uudestaan alas. Joo, UI on UI, eikä tämän kurssin pointti ollenkaan, kunhan sanon :) Plus tämä taitaa olla Flaskin “ominaisuus”.

Ensikosketuksellä minulle oli vähän epäselvää mitä tapahtuu, kun tuli viesti “huom kaikkia sanoja ei voitu korjata” syötteellä “Oltiin tehty sillä lailla tyhmästi että tytöille näytetty juttuja”. Se meinaa vissiin että kaikki ok?

Mukavaa, että on toimiva UI. Se esittelee oikein hyvin algoritmin toimintaa, mikä onkin tärkeintä.

Algoritmi

Algoritmi vaikuttaa ihan oikein rakennetulta. Lauseen ja sanan tsekkaus on varsin hidas välillä. Yhden kerran jollain siansaksalla sain aikaan “fatal python errorin”, mutta en saanut sitä toisinnettua, saattoi olla joku oman koneen vipstaaki (olin lisäillyt omia print-lauseita kun selvittelin miten homma toimii). Joka tapauksessa testailun jälkimauksi jää pohdinta siitä, olisiko tämä vielä optimoitavissa?

Spellchecker.py:n suggest-funtkiossa joka sanaa verrataan aina koko vocabularyyn, eli “etsintäalue” on aina maksimaalinen? Olisiko hölmöä rajoittaa vaikka samalla kirjaimella alkaviin sanoihin? No voihan se kirotusvirhe olla heti ekassa kirjaimessa. :olankohautushymiö

Ihan pienenä huomiona suggestissa:


        for letter in word:
            if letter.isnumeric():
                return "Et voi antaa numeroita"

        for letter in word:
            if letter in string.punctuation:
                return "Et voi antaa erikoismerkkejä"

voisi varmaan olla yksi tsekki: “Et voi antaa numeroita tai erikoismerkkejä”.

Testit

Testit vaikuttavat tosi kattavilta. 5/5. Mock-asia vaikuttaa kiinnostavalta. Se ei ole ennestään tuttu, pohdin jopa olisiko siitä iloa omassa prokkiksessa..

Yleistä

Koodi on helppolukuista. Laitoin heti alkuun docstringit piiloon (tiedän että ne on vaatimuksena, mutta minusta ne vaan häiritsee) ja väitän, että tajusin ihan hyvin missä mennään.

Kaikki toimii. Esimerkillistä duunia! Nyt sidukka auki ja Juhannuksen viettohon!

sonjaolkkonen commented 5 months ago

Kiitos palautteesta, hyviä pointteja :)

Olin itseasiassa aluksi käyttänytkin tuota, että haettiin ainoastaan samalla sanalla alkaneita sanoja, mutta tämä sai noottia ohjaajalta :D