Konwerter ceny do postaci słownej - wystawiam na krytykę :)

Question

Witam,

jestem w trakcie projektowania dla jednego klienta aplikacji w node, w której stanąłem przed problemem pobrania kwoty w postaci np. "120.50" i przekonwertowania jej na postać słowną ("sto dwadzieścia złotych pięćdziesiąt groszy").

Udało mi się rozwiązać ten problem, i przy okazji postanowiłem stworzyć swój pierwszy publiczny projekt NPM, dostępny pod linkiem: price-to-polish-words. 

Do testowania kodu wykorzystuję w projekcie Mocha+chai więc i ten moduł otrzymał testy w tej technologii (testy są dostępne w module).

price-to-polish-words przyjmuje kwotę jako Number lub String i konwertuje są do odpowiednio ustawionej postaci słownej. W kodzie zaimplementowałem kilka możliwości prezentacji słownej gdyż tego wymaga moja aplikacja (muszę zapewnić możliwość konfigurowania sposobu wyświetlania kwot).

 

Może komuś kiedyś się przyda "takie coś" :) a jeśli ktoś miałby ochotę zerknąć w kod to chętnie wysłucham konstruktywnych uwag i krytyki co dodać, zmienić itp. Nie muszą to być uwagi stricte do algorytmów, chętnie nawiążę również dyskusję np. na temat formatowania kodu (i ewentualnych zmian w moich ustawieniach ESLint) itp.

Pozdrawiam,

Tomek

Comments

Coś podobnego w Qt pisałem do zamiany kwoty na słowną postać chociaż bez tak zaawansowanego formatowania.

Answer 1

https://github.com/drogimex/priceToPolishWords/blob/master/priceToPolishWords.js#L36

W pętlach for..of oraz for..in nie zmieniasz (raczej) wartości zmiennej, więc można używać const. W "zwykłych" pętlach for, czy while / do..while zmienia się iterator (i++, i += 1 itd.), więc tam const nie przejdzie.

https://github.com/drogimex/priceToPolishWords/blob/master/src/setPriceFormat.js#L23

https://github.com/drogimex/priceToPolishWords/blob/master/src/setPriceFormat.js#L57

https://github.com/drogimex/priceToPolishWords/blob/master/src/numberToWords.js#L67

https://github.com/drogimex/priceToPolishWords/blob/master/src/ConvertMethods.js#L75

https://github.com/drogimex/priceToPolishWords/blob/master/src/ConvertMethods.js#L95

Tam możesz użyć const. Mogłem też przeoczyć jakieś miejsca, ale jeśli nie przypisujesz ponownie jakiejś wartości do tej samej zmiennej, tylko ją mutujesz - czyli operujesz na obiektach, to stosuj const. Tak samo, jeśli tworzysz "stały string", np. jako klucz do tłumaczenia.

https://github.com/drogimex/priceToPolishWords/blob/master/src/priceToArray.js#L30

W takich momentach, czytelność kodu moim zdaniem spada. Podobnie, jak w innej dyskusji na forum: https://forum.pasja-informatyki.pl/271101/fukcja-z-tablicami?show=271107#a271107

Lepiej, przy arrow function jednak otoczyć parametr nawiasami i przy tworzeniu łańcuszka wywołań użyć klamer. Za bardzo kodu nie rozbudowujesz, a mimo wszystko czytelność wzrasta.

https://github.com/drogimex/priceToPolishWords/blob/master/src/numberToWords.js#L25

Tutaj też IMO troszkę za bardzo "one liner":

Comments

i o takie coś mi chodziło, wielkie dzięki kolego :)

Co do const to spuszczam głowę i przyznaję całkowitą rację. Nie wiem dlaczego, ale jakoś zafiksowałem się na tym "let" i nie mogę odpuścić, mimo iż zdaję sobie sprawę z tego jak dokładnie działa const, a jak let (nawet pisałem o tym posta na swoim blogu.. :) ale na pewno to poprawię, żeby wyrobić sobie lepsze nawyki.

Co do tej czytelności to szczerze mówiąc są to pewne nawyki po nauce JS na platformie CodeWars, gdzie zrobiłem wiele zadań i w części wymagane było użycie jak najmniejszej liczby znaków i stąd wyrobiłem sobie nawyk tych wywołań liniowych i arrow function, ale już Comandeer niedawno zwracał mi na to uwagę i powolutku pracuję nad tym :)

---

Też lubię skracać kod, ponieważ można wtedy lepiej poznać quirki języka. Lecz przesadzanie ze skrótowcami zmniejsza czytelność kodu na tyle, że albo ja sam po jakimś czasie mam problem z szybkim zrozumieniem "o co tu chodzi", albo ktoś inny spogląda i się pyta "kto to pisał?!". Więc przyjąłem metodę - dla zabawy, z ciekawości i "for sake of science" czasami dłużej pobawię się jakąś funkcją, żeby zobaczyć jak bardzo mogę ją skrócić. Jednak ostatecznie ("na produkcję") staram się zostawić ją w czytelnym stanie.