corned / blabber Goto Github PK
View Code? Open in Web Editor NEWTwitter Clone for Tsoha2017-4
Twitter Clone for Tsoha2017-4
Latasin koodin ja aloitin katselmoinnin perjantaina 21.4 klo 9.
Kokonaisuudessaan ohjelma vaikuttaa toimivalta ja hyvin tehdyltä ja se täyttää dokumentaatiossa esitetyt vaatimukset.
Dokumentaatio oli hyvin tehty ja selkeytti ohjelman toimintaa. Kappaleessa järjestelmän yleisrakenne olisi voinut kirjoittaa vähän siitä miten järjestelmä käyttää istuntoa ja miten se vaikuttaa ohjelman toimintaan.
Lähdekoodi oli selkeää ja hyvin järjesteltyä. Funktioihin liitetyt kommentit olivat selkeitä, vaikka suurimmassa osassa funktion toiminta oli jo ilmankin kommentteja hyvin ymmärrettävissä.
Sisään ja uloskirjautuminen näytti toimivan hyvin. Järjestelmä estää pääsyn niille sivuille joille kirjautumattoman ei pitäisikään päästä.
Testatessani ohjelmaa huomasin pieniä virheitä, jotka olisi syytä korjata ennen lopullista palautusta:
Jos editoin viestiä ja jätän viestikentän tyhjäksi, ohjelma kaatuu sen sijaan että se palautuisi lomakkeeseen. Tämä johtuu ilmeisesti siitä että luokan blabcontroller.php funktiossa update() on käytetty sekä $errs- että $errors-muuttujaa (riveillä 114 ja 120) jolloin ohjelma ei löydä $errors-muuttujaa, koska sitä ei ole aiemmin määritelty.
Jos menen käyttäjän suosikkiblabien sivuille ja yritän painaa jonkin blabin kohdalla EXPAND-painiketta, se johtaa ohjelman kaatumiseen. Tämä johtuu ilmeisesti siitä että favourites.html-näkymässä EXPAND-painikkeen linkki on muotoa /blab/show/:id vaikka sen pitäisi olla muotoa blab/:id.
Jos haen osoitetta, jossa blabin id:ksi on määritelty jokin merkkijono, esimerkiksi http://koliver.users.cs.helsinki.fi/blabber/blab/qwe, johtaa tämä ohjelman kaatumiseen. Katsomalla koodia huomaan että haettavasta id:stä tarkistetaan fiksusti että se ei ole pienempi kuin 0 tai isompi kuin 2147483647, mutta siinä ei tarkisteta, onko haettava id oikeasti numeromuodossa.
Ohjelmaa voisi myöhemmin laajentaa esimerkiksi käyttäjän mahdollisuudella lisätä itselleen profiilikuva tai lisätä julkaistaviin viesteihin kuvia. Lisäksi käyttäjä voisi seurata muita käyttäjiä, jolloin heidän viestinsä ilmestyisivät käyttäjän henkilökohtaiseen feediin. Nämä ominaisuudet ovat kuitenkin todennäköisesti jo tämän projektin ulkopuolella. Jos aikaa jää, voisi käyttäjän hakeminen käyttäjänimen perusteella olla melko helppo toteuttaa materiaalin vinkkien avulla.
Koodia katseltu 1.4.2017 klo 22.34 lähtien.
Varhaista sanoa vielä mitään hirveän edistyksellistä, vaikka oletkin muokkaus-toiminnoilla päässyt jo aikataulua edellekin. Koodi on siistiä ja toimii, joka on todettu käytännössä.
Blab-tietokantataulussa näytti olevan eräs attribuutti deleted. Onkohan poistettaessa ajateltu muuttaa vain sen arvoa, rivin kokonaan poistamisen sijaan?
Paljon hyviä ominaisuuksia suunniteltuna. Vaikka arvosteluperusteet vaativat, niin viestien muokkaus myöhemmin voisi käytännössä olla ikävää, jos sitä on esimerkiksi uudelleenjaettu. Jos kaiken lisäksi kuitenkin sattuu ylimääräistä aikaa, niin global feediin haku- tai suodatustoiminto voisi olla hyvä plussa.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.