Jcalculator

13:53 2.5.2018 klooanaus aika

Risut ja Ruusut

Ruusut

• Koodi on siistiä ja hyvin dokumentoitua JavaDocilla
• JavaFXML pohjainen käyttöliittymä on selkeä ja hyvä.
• Testikattavuus on korkea

Risut

• Käyttöliittymä on vielä puutteellinen, esim Programming laskinosiosta
  ei pääse palaamaan basic tilaan. Tämä puute on varmasti teidän tiedossa.

Dao

• SQL on tehokas työkalu, mutta onko pelkän konsolihistorian tallentaminen
  sen kanssa hieman turhan monimutkaista, kun tekstitiedosto luultavasti
  olisi ajanut saman asian. (Mielipide/Pohdintaa) Ratkaisu on kuitenkin hyvä
  ja toimiva, ei oikeasti ongelma.

Ui

Jcalculator.ui.FXMLBasicViewController

Kun käyttäjä tekee virheen, suosittelisin toimimaan seuraavasti
Kohta noin rivillä 196 metodi evaluate()

inputField.setText("Malformed expression");

korvataan

inputField.clear();
inputField.setPromptText("Malformed expression")

Jolloin käyttäjän syöttäessä uutta tietoa teksti kumittuu automaattisesti.
Tämä on hieman ongelmallista koska, jos käyttäjä antoi syötteen näppäimistöltä,
niin teksti kumittuu heti. Sen takia suosittelisin käyttämään JavaFX
alerttia.
malli:

Alert alert = new Alert(AlertType.ERROR);
alert.setTitle("Virheellinen syöte");
alert.setHeaderText("Syötettäsi ei voitu tulkita");
alert.setContentText("Tarkastathan sopivat syötteet ohjeista");
alert.showAndWait();

Tämä ratkaisu on siitä hyvä, että käyttäjälle ei jää epäselväksi,
että hän on tehnyt virheen ja hän voi vielä tarkastella virheen
aiheuttanutta syötettä.

Testit

• Testikattavuus on hyvä, mutta monia metodeja testataan vain yhdellä perustapauksella
  testeihin olisi hyvä lisätä erilaisia rajatapauksia.