Last active
August 29, 2015 14:03
-
-
Save szmeku/17ec52817a265bd3b680 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
// FORMATOWANIE | |
function something(x, y, z){ | |
return (x-20 + 2*y)%2*z; | |
} | |
// czytelniejsza wersja | |
function something(x, y, z) | |
{ | |
return (x-20 + 2*y)%2*z; | |
} | |
// ANGULAR | |
// - podstawowa zasada angulara - nigdzie poza dyrektywami nie modyfikujemy i bawimy sie DOM'em, | |
// programowanie deklaratywne, a nie imperatywne | |
// - nie podpinać funkcji do scope'a gdzie nie trzeba | |
// - korzystać z przekazywania do funkcji argumentów co zwiększa czytelność kodu | |
// - przekazywać do funkcji tylko tyle ile dana funkcja potrzebuje wiedzieć, ogolna zasada kazda rzecz ma widzieć jak najmniej | |
// przykład skrajnie złej funkcji: | |
// na pierwszy rzut oka nie wiem co robi funkcja, z czego korzysta itd, długie warunki, ogólny syf | |
// w dodatku nie jest uzywana w scopie | |
$scope.something = function() | |
{ | |
if(angular.isArray($scope.profile.foo.boo.woo) && !_.isNull($scope.profile.foo.boo.wii)){ | |
$scope.profile.foo.boo.woo.push($scope.profile.foo.boo.wii)); | |
} | |
} | |
// lepiej napisac | |
// latwo testowalna | |
// od razu widac ze dziala bez zadnych dziwnych dependencji | |
function something(woo, wii) | |
{ | |
if(angular.isArray(woo) && !_.isNull(wii)){ | |
woo.push(wii); | |
} | |
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment