epiphone / pucktracker Goto Github PK

URLien rakennetta voisi miettiä, ja suunnitella sen routerin ja kirjoittaa urleja vastaavat jonkinlaiset placeholder-luokat. Esim:

urls = (
"/", "Index"
"/login", "Login",
....
)

class Index:
"""Pääsivu. Toiminnallisuudet, mitä sisältöä jne."""

class Login:
"""Kirjautuminen. Miten toimii jne... """

main.py:ssä on atm implementoitu ottelun haku sekä pelaajan ja joukkueen pelattujen pelien haku, loput vielä tekemättä.

Tällä hetkellä kaikki on boldattu ja rivit saman värisiä -> sekavan näköistä. Yahoon sivuilta voisi vaikka katsoa mallia.

Useimmilla sivuilla taulukoissa on virheellisiä sarakkeita. Katso myös kirjoitusasu Yahoon sivuilta - esim GA ja GF isolla, mutta Pts vain isolla alkukirjaimella.

fetch_from_api ja fetch_from_api_signed-funktioita käyttäessä pitää varautua tyhjään paluuarvoon.

Skreippausfunktioissa on tod. näk. aika paljon bugeja, pitäisi kirjoittaa testejä ja miettiä kaikki erikoistapaukset, ehkä estää joitain kyselyitä, tai ainakin varautua niitä varten.

API:n google docs-dokumentoinnin voisi kirjoittaa puhtaaksi, vaikka htmlään. Paluuarvojen tyypit ja jotain esimerkkejä vois ainakin laittaa.

Jotkin scrapetest.py:n funktiot eivät käytä memcachea.

Pitäisikö asiakassovellus ja API-sovellus erottaa toisistaan silleen, että asiakassovellus kutsuu APIa? Nythän jos sen asiakassovelluksen kasaa tohon API:n päälle samaan projektiin, se ei varsinaisesti käytä sitä APIa (tai ainakaan verkkoliikennettä ei ilmene) - mikä ei muuten haittaa (päinvastoin), mutta kurssia varten olisi kai ihan hyvä demonstroida sitä APIa.

Voitaisiin esim. jättää API siihen App Engine-sovellukseen ja tehdä uusi web.py-sovellus asiakasta varten. Kurssin jälkeen ne voisi sitten yhdistää. Tai jotain?