ΘΕΜΑ: Number to Greek

Έχω ανεβάσει στο PYPI, ένα πρόγραμμα που μετατρέπει αριθμούς (integer ή float), σε ελληνικές λέξεις.
Επειδή είναι ακόμη σε beta έκδοση, θα ήθελα να παρακαλέσω, όποιος μπορεί να το δοκιμάσει και να μου στείλει τις παρατηρήσεις του, ή τα σφάλματα που εντόπισε.

Μπορείτε να το κατεβάσετε από εδώ.
Στον ίδιο σύνδεσμο θα βρείτε και οδηγίες για την εγκατάστασή του. Συνιστώ virtualenv.

Αν κάποιος θέλει να το εγκαταστήσει σε κανονικό περιβάλλον με pip ίσως χρειαστεί να χρησιμοποιήσει την παράμετρο --pre.

Τρόπος Χρήσης:

Χαχα ωραίο πρόγραμμα. Παρατήρησα ότι πρέπει να κάνεις

from numtogreek import n2g και όχι from numtogreek.numtogreek
import n2g.

Επίσης είδα ότι φτάνει μέχρι εκατό επτάκις εκατομμύρια.Άμα
δώσω αριθμό με ένα περισσότερο μηδενικό με πετάει αυτό το
error:

dol = temp + place[count] + dol
KeyError: 9

Αλλά εντάξει λογικό είναι, οι αριθμοί είναι άπειροι δεν μπορείς να τους καλύψεις πλήρως.

Πάντως ωραίο, καλή δουλειά

babaliaris1 έγραψε:

Το πρώτο δεν έχει να κάνει με το ίδιο το πρόγραμμα. Προφανώς έχει να κάνει με τη δομή του καταλόγου. Θα το κοιτάξω.

Όσο για το δεύτερο, όπως γράφω και στο README στο pypi, έχω βάλει άνω όριο τα 99 τρισεκατομμύρια.
Θα το διορθώσω και αυτό.

Διορθώθηκε!

Όποιος έχει μεγαλύτερο τραπεζικό λογαριασμό από μερικά οκτάκις εκατομμύρια, να μου δανείσει.

Δεν το εγκατέστησα, αλλά όσο κοίταξα τον κώδικα φαίνεται ωραία δουλειά.

Επειδή δεν το αναφέρεις στο documentation, και δεν έχεις και test για αυτό, τι γίνεται αν δώσεις ως input κάτι με περισσότερα από 3 δεκαδικά ψηφία;

pmav99 έγραψε:
Αν βάλεις πάνω από δύο δεκαδικά, τα επιπλέον απορρίπτονται σιωπηλά. Στο κάτω - κάτω της γραφής, η λογική είναι να μετατρέπει «πραγματικούς αριθμούς» σε λέξεις. Τι θέλω να πω.

Αν γράφεις ένα τιμολόγιο ή απόδειξη, το τελικό ποσό είναι ακέραιος ή δεκαδικός με το πολύ δύο δεκαδικά, ακόμη και αν τα επιμέρους στοιχεία (τιμή μονάδος) έχουν τρία δεκαδικά (π.χ λίτρο βενζίνης).

Τώρα για το test, αν θέλεις, ίσως να μου έγραφες ένα παράδειγμα, πως θα χειριζόσουν το συγκεκριμένο θέμα. Οφείλω να ομολογήσω ότι δεν έχω ασχοληθεί ιδιαίτερα με το θέμα των test.

Για τα tests, προσωπικά, χρησιμοποιώ την βιβλιοθήκη py.test. Πατσάρει την assert και δεν χρειάζεσαι όλες τις συναρτήσεις που χρησιμοποιεί η unittest (πχ assertEqual, assertListEqual κτλ). Χρησιμοποιείς παντού σκέτο assert.

Όπως και αν έχει, εφόσον απορρίπτεις σιωπηλά τα παραπάνω δεκαδικά ψηφία, νομίζω ότι κάτι σαν και το ακόλουθο θα ήταν επαρκές (θεωρώ ότι δεν κάνεις στρογγυλοποίηση!)
Από εκεί και πέρα, ίσως να είχε νόημα να έχεις δύο modes. Ένα lenient στο οποίο θα έχεις την συμπεριφορά που έχεις τώρα, και ένα strict όπου τα παραπάνω δεκαδικά ψηφία θα πετάνε exception ή έστω ένα warning (pymotw.com/2/warnings/).

Τα παραπάνω φυσικά εξαρτώνται από το τι features χρειάζεσαι.

Α! και όσον αφορά το representation, νομίζω θα είχε νόημα και ένα 3ο mode εκτός των ("upper", "lower"). Πιο συγκεκριμένα κάτι σαν το "sentence"
Όχι ότι είναι δύσκολο να το κάνεις μόνος σου μετά, αλλά δεν είναι και τίποτα να το υλοποιεί η βιβλιοθήκη απευθείας.

Όσον αφορά την απόδοση, αυτή τη στιγμή δημιουργείς τα ίδια dictionaries κάθε φορά που καλείς την «n2g()». Θα μπορούσες να την βελτιώσεις λίγο μετατρέποντας τα dictionaries σε constants και περνώντας τα σαν arguments στις διάφορες συναρτήσεις.

Και μια παρατηρήση όσον αφορά το API. Προσωπικά νοίωθω λίγο περίεργα με τα greeklish στον προγραμματισμό. Πχ η παράμετρος «nomisma», δύσκολα θα είναι κατανοητή από κάποιον που δεν γνωρίζει ελληνικά. Ναι, η συντριπτική πλειοψηφία αυτών που θα χρησιμοποιήσουν την εφαρμογή είναι έλληνες, αλλά γιατί να αποκλείσεις κάποιον;

Τέλος, ίσως η παράμετρος «nomisma» να έχει νόημα να μετατρεπεί από boolean σε string και να έχει default τιμή το «eur». Πχ:

Ωραία! Θα κοιτάξω τις παρατηρήσεις σου.

Όσο για τα warnings ή τα exceptions, δεν θέλω να εκτυπώνεται μήνυμα στη θέση το αποτελέσματος.
Ειδικά για τα greeklish, η σκέψη μου ήταν ότι κατά βάση, η χρήση θα ήταν για ελληνόφωνους, όμως τώρα· σίγουρα θα το διορθώσω.
Επίσης, η ιδέα για το νόμισμα, είναι πολύ καλή. Ίσως προσθέσω και τη δραχμή.

kamar έγραψε:
Να την προσθέσεις γιατί έτσι όπως παν τα πράματα το βλέπω να
γυρίζουμε στην δραχμή