[d.....3]

#naukaprogramowania #programowanie #angularjs

Hej.

Zabrałem się za naukę angularjs i mam problem z poprawnym rozdzieleniem controllera, serwisu, widoku oraz ogólnie mam kulka zastrzeżeń co do kodu wywoływanie funkcji z jsp, angular.merge do łączenia widoku.

Poproszę o review code.

Controller
https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/resources/js/controllers/login/loginController.js

Service
https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/resources/js/controllers/login/services.js

JSP
https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/WEB-INF/views/home/login.jsp

[niewidzialnyktos]

@dinor913: Po co przypinać do $scope jakiś globalny error, po co on jest? Wszystkie pola które są Ci potrzebne masz w formularzu.

https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/resources/js/controllers/login/services.js#L83
https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/resources/js/controllers/login/services.js#L79
Po co te funkcje? (Nigdzie nie użyte)
Większość tekstów użytych na stronie lepiej trzymać w widoku (subiektywnie), jak tak nie robimy to później przy translacjach pojawia się mix, połowa jest w js, połowa w html - boli

https://github.com/dinor913/WorkServices/blob/master/src/main/webapp/resources/js/controllers/login/services.js#L74
Co to za magiczna funkcja? Po co?

Generalnie widać

[o.....r]

@dinor913: Źle używasz promisów, request z $http sam z siebie zwraca Promise, opakowywanie tego w defery z $q jest redundantne.