Δημοσιεύτηκε: 16 Φεβ 2013, 23:29
Ωραία δουλεία!
Παρατηρήσεις:
1. Πρέπει σίγουρα να υλοποιηθεί μια επιλογή αναίρεσης κίνησης!
2. Ίσως θα ήταν χρήσιμο να γίνεται αυτόματη αποθήκευση για κάθε παίκτη όταν αυτός περνάει μια πίστα. Το να πρέπει να επαναλάβει κάποιες πίστες επειδή ξέχασε να πατήσει save θα είναι σίγουρα αρκετά εκνευριστικό.
3. Τα prof και prof~ δεν χρησιμοποιούνται ούτε διαγράφονται. Υπάρχει ιδιαίτερος λόγος που υπάρχουν;
Επίσης δεν υπάρχει στοίχιση στον κώδικα σου.
Πχ. ένα sample:
Μαντεύω ότι αυτό έγινε κατά λάθος. Μήπως κάτι παίζει με το IDE σου και τα έβλεπες αλλιώς όταν έγραφες;
Δες αν μπορείς να το διορθώσεις γιατί ο κώδικας είναι εξαιρετικά δυσανάγνωστος έτσι.
Α, ακόμα ο sikla folder που έχεις πάνω πάνω στο source tree είναι περιττός.
Δηλαδή το README και τα υπόλοιπα αρχεία πρέπει να βρίσκονται στο top level του αποθετηρίου.
Ή μπορούν να μπουν τα *.java αρχεία σε έναν φάκελο με όνομα sikla ή src ακριβώς κάτω απ' το top level αλλά αυτό είναι δική σου επιλογή.
Παρατηρήσεις:
1. Πρέπει σίγουρα να υλοποιηθεί μια επιλογή αναίρεσης κίνησης!
2. Ίσως θα ήταν χρήσιμο να γίνεται αυτόματη αποθήκευση για κάθε παίκτη όταν αυτός περνάει μια πίστα. Το να πρέπει να επαναλάβει κάποιες πίστες επειδή ξέχασε να πατήσει save θα είναι σίγουρα αρκετά εκνευριστικό.
3. Τα prof και prof~ δεν χρησιμοποιούνται ούτε διαγράφονται. Υπάρχει ιδιαίτερος λόγος που υπάρχουν;
Επίσης δεν υπάρχει στοίχιση στον κώδικα σου.
Πχ. ένα sample:
Spoiler: show
Μαντεύω ότι αυτό έγινε κατά λάθος. Μήπως κάτι παίζει με το IDE σου και τα έβλεπες αλλιώς όταν έγραφες;
Δες αν μπορείς να το διορθώσεις γιατί ο κώδικας είναι εξαιρετικά δυσανάγνωστος έτσι.
Α, ακόμα ο sikla folder που έχεις πάνω πάνω στο source tree είναι περιττός.
Δηλαδή το README και τα υπόλοιπα αρχεία πρέπει να βρίσκονται στο top level του αποθετηρίου.
Ή μπορούν να μπουν τα *.java αρχεία σε έναν φάκελο με όνομα sikla ή src ακριβώς κάτω απ' το top level αλλά αυτό είναι δική σου επιλογή.