planner's Issues
Koodikatselmointi
**-**Tykkäsin aiheenmäärittelystä, oli selkeä ja sovelluksen yleinentoiminta tuli esiin.
**-**Osa testeitä aika yksinkertaisia, esim. DayTest testit. Gui:n testejä oli aika vähän.
**-**Kun ajoin ohjelman toimi hyvin, mutta oli kohtia joihin ohjelma tyssäsi, esim. toisen merkinnän laittaminen sanoi aina "invalid output". Joitain koodinpätkiä olit laittanut kommenttien sisään.
**-**Dokumentaation kuvassa ei näkynyt luokkien välisiä viivoja. (en varma temppuileeko oma koneeni...) mutta tarkistamisen arvoinen asia.
**-**PlannerGui-luokan luomisesta plussaa.
Lataus aika: 23:33 05.02.2016
Koodikatselmointi 2
Ladattu 17.2.2016 klo 17:45
Dokumentointi
Aihekuvausta voisi hiukan tarkentaa, vaikka aihe onkin selkeä. Itse lisäisin vaikkapa kuvan graafisestä käyttöliittymästä selkeyttämään kuvausta. Selkeyttä lisäisi myös mainin sijainnin lisäys luokkakaavioon.
Ohjelma
Ohjelma toimi tosi hyvin. Yritin löytää bugeja, taikka muuten vaan rikkoa ohjelmaa, ja en onnistunut kummassakaan. Printteri ja värivalikot oli mahtava lisäys, ja ainakin värivalikko toimi oikein. Printtaamista en päässyt harmikseni kokeilemaan. Ohjelma suostui jopa lukemaan jotain oudompia syötteitä. Esim ¸¸♬·¯·♩¸¸♪·¯·♫¸¸Happy Birthday To You¸¸♬·¯·♩¸¸♪·¯·♫¸¸ .
Pientä viilausta: Filteröi pelkkää välilyötiä olevat syötteet. Jos voit niin disabloi musta taustaväreistä, koska siitä ei pysty lukemaan mitään.
Koodi
Koodi on täynnä ihmeellisiä swing -olioita ja niiden kutsuja, jotka ilmeisesti luovat kaiken käyttöliittymä magian. Voisit hiukan kommentoida niiden toiminnallisuutta, koska jos nyt todella haluaisin ymmärtää kaiken mitä tapahtuu joutuisin googlettamaan aika paljon.
Omat luokkasi ovat hyvin jäsenneltyjä ja luettavuus on huippuluokkaa. Vaihtaisin itse luokan CreateTask nimen TaskFactoryksi tai joksikin vastaavaksi, koska se kuvaisi sen toimintaa paremmin.
Siis koodi on tosi selkeää muutamaa pientä kohtaa lukuunottamatta. Muutenkin ohjelman rakenne on selkeä ja järkevä.
Testit
Testikattavuus on tosi hyvä. Voisit jakaa jotain testejä osiin, koska tarkoitus on testata yhtä asiaa per testi.
Eli siis tosi hyvältä näyttää. Ohjelmahan on käytännössä valmis, vaikka aikaa on vielä paljolti. Siis tässä muutama kehitysidea:
- Tapahtumille vapaaehtoinen kuvaus, joka esim pop-uppaisi esille, kun tapahtumaa klikkaa käyttöliittymästä.
- Tapahtuman alkamisajankohdan voisi valita hiirellä taulukosta.
2.1 Tapahtuman ajankohdan voisi maalata taulukosta (alkamisaika --- loppumisaika) - Lisää päivämäärät
3.1 Mahdollista viikkojen välillä liikkuminen
3.1.1 Viikottain/Päivittäin jne toistuvat tapahtumat
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.