Koodikatselmointi

[ktojala]

Vertaisarviointi SkrolliEditor-ohjelmasta (sivu ladattu maanantaina 13.8.2018 klo 1037)

Aihemäärittely ja testi selaimessa:

• Aihemäärittely hyvin yksityiskohtainen, erilaisia rooleja ja osatehtäviä runsaasti. Ehkä vain osa niistä aiotaan/voidaan toteuttaa ’tsohan’ puitteissa. Käytäntöä ajatellen, onko picture-editor tarpeen erottaa editorista?
• Havainto: järjestelmä ei näytä mahdollistavan maksettuja ilmoituksia lehteen.
• Etusivulla ei linkkiä Luokka/Käsitekaavioon. Kaavio varmaan olisi hyödyksi projektille.
• Klikkaan User Stories-linkkiä. User stories on toteutettu hauskalla ja ilmeisen toimivalla tavalla. Ei ihan siten kuin OTM -kurssilla, että se esim. kuvaisi jonkun kuvitellun istunnon tai tehtävän vaiheittain alusta loppuun. Tässä vaiheessa huomasin, että github-sivun ”Issues” näyttää isoa lukemaa. User Storyt on siis tuotu issueina, ovelaa! Ilmeisesti vain owner voi ruksata niitä tehdyksi. Tulee mieleen, löytyykö tämä tuleva issue sieltä helposti.
• Alussa hämmensi se, mikä oikein oli kurssimateriaalissa annettu käyttäjätunnus. Oliko se ”testi” vai mikä? No, kun tuota aikansa kahlasi läpi, ei siellä ollut kuin yksi ja pääsin sisään Herokussa.
• Kun kirjautuu sisään Herokussa, on mahdollista päivittää testitunnuksia, mutta päätin etten tee tuota testitunnuksille. Niinpä kokeilin rekisteröintiä. Tarkistukset toimivat, mutta aluksi näytti siltä että Name ja Username vaihtuvat toisikseen. Selvisi että ne vain ovat rekisteröityessä eri järjestyksessä kuin mihin on tottunut. Kokeiltu vaihtaa salasana (Update account) – onnistui. Updatessa odottaisi, että siinä lukee Username eikä Name. Tämän jälkeen ihmetyttää, mihin tarvitsee Edit-nappulaa List people -listauksessa. Lienee jäänne viikkotehtävistä? Kokeilin Edit nappulaa rapux2:lle, ja lisäsin sille nimen rapux3. Loiko se käyttäjän nimellä testi ja salasanalla testi? Ei kai. Ahaa, List Articles näyttää että siellä on nyt artikkeli nimellä testi. rapux3 ei löydy mistään. Hmm. Artikkelia ei siis vielä pääse vielä kirjoittamaan tai editoimaan.
• Kun loggaa ulos, ruudulle jää teksti ”unfinished articles”.
• Uusia käyttäjiä ajatellen nappuloissa oleva teksti voisi kertoa tarkemmin mitä nappula tekee. Esim. ei ”Edit” vaan ”Edit article”. Ja vaihtuville sivuille otsikko ja lyhyt ohje/kuvaus sivusta – varmaan tulossa myöhemmin.
• Muutoin käyttöliittymä suunnilleen sellainen kuin sen 3. viikolla oli tarkoitus ollakin.
• Klo 1600 huomaan että sivujen ulkoasut ovat jo paranemassa. :-).
  Ohjelmakoodi:
• Latasin zipin mutta omassa koneessa voin tietty vain katsoa tiedostoja, en ajaa niitä.
• Koodi on johdonmukaisesti englanniksi.
• Kansioiden nimentä ja sisältö: Ahaa, tätä se MVC käytännössä tarkoitti. :-)
• Luokat ja nimentä johdonmukaista.
• Validaattoreissa täydennettävää, esim ArticleForm. Seuraava sivu, voisi olla hyötyä: http://nullege.com/codes/search/wtforms.validators
• Tekijä on osin pidemmällä työn koodaamisessa kuin allekirjoittanut, esim. Base-luokka, formfield-makro ja navbar toteutettu. Muuta toiminnallisuutta ei vielä kovin paljon.
• application/articles/views.py: def articles_index(), ja def articles_create(), mm. näissä on sama @app.route. Oman kokemuksen nojalla kannattaa nimetä routet eri tavoilla., koska samalla reitillä ei voida renderöidä kahta eri asiaa. Itsellä kävi niin että vain ensimmäinen renderöinti oikein toteutui jos noin teki.
• Parantaisin koodia hieman siltä osin kuin liittyy sivunäkymiin joihin jo viitattu tuolla aiemmin.