[inteli]

Dwa pytania, chcę napisać jedną funkcję perfekcyjnie, o czym powinnienem pamiętać oprócz czytelności kodu, sprawdzania danych wejściowych czy są poprawne, rzucania w innym przypadku wyjątków i zasadzie pojedynczej odpowiedzialności?
Testy jednostkowe do takiej funkcji w czym najlepiej dziś pisać? Chai?

KOD: https://pastebin.com/xyb8rCzp
#programowanie #informatyka #javascript

[andrew1985]

Powinna robić tylko jedną rzecz.

[inteli]

@andrew1985 no to mówię zasada pojedynczej odpowiedzialnosci - jedna funkcja jest w drugiej żeby nie była dostępna publicznie, tylko dana funkcja z niej korzysta

[O.....i]

@inteli: Do testów polecam "Jest". Zapomniałeś przede wszystkim o dokumentacji http://usejsdoc.org i prawdopodobnie nie używasz lintera np. ESLint

[andrew1985]

Sorki, niedoczytalem do konca ;) Tak patrze na kod i funkcja findNumbers robi trzy rzeczy:
1) tworzy funkcje getElementNumber
2) sprawdza czy elements nie jest puste
3) mapuje elements

Czyta funkcja, ktora robi jedna rzeczy powinna wygladac tak:

[inteli]

@andrew1985 to jak zrobić tą funkcje nie dostępna globalnie? Chyba nie będę specjalnie klasy tworzyć

[andrew1985]

Generalnie to Twoja funkcja findNumbers, tez nie powinna byc dostepna globalnie. Czyli dodana do window, jezeli to masz na mysli. W tym celu nalezy calosc opakowac w fukcje anonimowa z jej natychmiastowym wywolanie. Czyli:

(function() {
...
}())