riinaalisah / sanastosovellus Goto Github PK

Projekti ladattu 24.4 klo 20:22

Koodi oli helposti luettavaa ja luokat oli sijoitettu järkeviin pakkauksiin. Metodien nimet kuvasivat hyvin niiden toimintoja ja olivat toteutukseltaan hyviä. Jotta tästä palautteesta olisi jotain hyötyäkin, niin nappasin koodista pari kohtaa ylös:

FileWordPairDao

public class FileWordPairDao implements WordPairDao {
    public List<WordPair> wordpairs;
    private String file;
    /* loput */
}

Luokassa tuo tiedosto tallennetaan parametrina saatuna merkkijonona. Tämän jälkeen aina kun tiedostoa halutaan lukea, niin luodaa uusi instanssi File -luokasta. Pystyisikö tuon tiedoston tallentamaan suoraan File -tyyppisenä oliomuuttujana?

SanastosovellusUI

public void setPracticeScene(Stage primaryStage, int index, boolean defaultDirection) {
    /* muuttujat */

    answerButton.setOnAction(e->{
            /**
             * Direction word -> translation
             */
            if (defaultDirection) {
                if (answerInput.getText().equals(usersWordpairs.get(index).getTranslation())) {
                    practiceMessage.setText("Oikein! " + usersWordpairs.get(index).getWord() + " = " + usersWordpairs.get(index).getTranslation());
                    practiceMessage.setTextFill(Color.GREEN);

                    /**
                     * Next word if there are words left
                     */
                    if (index < usersWordpairs.size()-1) {
                        setPracticeScene(primaryStage, index + 1, true);
                        primaryStage.setScene(practiceScene);
                    }

                    /* loput */
}

Pystyisikö käyttöliittymästä siirtää tuota sovelluslogiikkaa esimerkiksi luokan AppService vastuulle? Esimerkiksi tuo harjoittelusuunta (defaultDirection) vaikuttaa käytännössä vain vastauksen tarkistukseen, joka voitaisiin hoitaa AppServicen puolella ja näin eliminoitaisiin tuon defaultDirectionin else {} -haara => myös toisteisuutta saadaan pienennettyä.

Sovellus

Testasin myös sovellusta ja kaikki toiminnot toimivat juuri niinkuin pitää. Hyvää työtä! Jos aikaa ja intoa riittää, niin IndexOutOfBoundsExceptionin voisi hoidella, mikäli yritetään harjoitella ilman sanoja.