[npsr]

#webdev #fontend #js

Nawet jeśli znacie (i co dziwniejsze: stosujecie się) to warto sobie czasem przypomnieć ;)

Coding Style Guide na dzieńdobry :)

https://github.com/airbnb/javascript

[Marmite]

@npsr: (#) Z niektórymi się nie zgadzam, lub są bez sensu.

Use single quotes for strings

Rób tak bo tak? Zmieniłbym raczej na "używaj w danym projekcie jednego rodzaju cudzysłowów"

Cała sekcja "Properties" https://github.com/airbnb/javascript#properties to raczej podstawy JavaScriptu, a nie dobre praktyki.

Use one var declaration for multiple variables and declare each variable on a newline.

Co za różnica? Jak do zmiennych przypisujemy funkcje to ten var się rozrasta niepotrzebnie. Wystarczy

[mrwrotek]

@Marmite: Chodzi o to, żeby była jedna konwencja. Jeżeli się przyzwyczaisz do tej jednej wspólnej, to później dowolny kod jest przejrzysty. Dlatego właśnie to się nazywa Code Style Guide :>

[mrwrotek]

@npsr: @Marmite: ok fakt faktem, nie zgodzę się z single quotami dla stringów ponieważ jest niespójne z innymi językami programowania, w których single quote == char.

[Marmite]

@mrwrotek: (#) Wątpię, żeby rodzaj uzywanych cudzysłowów czy to czy liczba spacji przy formatowaniu wynosi 2 czy 4 kodu czyniły kod nieprzejrzystym.

[Marmite]

Tak samo - co za różnica? Dla mnie lepiej wygląda porównanie, bo wiem od razu jaki warunek testuję.

@Marmite: (#) Appendix: a już najlepiej wygląda coś takiego:

if (isCollectionEmpty())

czytelne i jasne. Jest dużo lepsze od porównywania czy

length

wynosi 0 (no chyba że funkcja operuje na niskim poziomie abstrakcji) a na pewno dużo lepsze od sprawdzenia

if (collection.length)

[npsr]

@Marmite: akurat 2 spacje, 4 spacje czy jeden tab to są 3 różne rzeczy i są chyba największą zmorą. Szczególnie, że każdy używa innego ide.

W JS powinno się zedycowanie używać single quote, podobnie jak w php.

Pamiętaj np. o takich sytuacjach:

$(this).html('dsdsdds');

Właściwie to przykład z dupy, ale piszę na szybko: wiadomo o co chodzi :)

[Marmite]

akurat 2 spacje, 4 spacje czy jeden tab to są 3 różne rzeczy i są chyba największą zmorą. Szczególnie, że każdy używa innego ide.

@npsr: (#) Jeśli ktoś używa IDE które nie wspiera możliwości dostosowania konfiguracji wcięć dla projektu no to jego strata.

Pamiętaj np. o takich sytuacjach:

$(this).html('<span class="dupa">dsdsdds</span>');

Właściwie to przykład z dupy, ale piszę na szybko: wiadomo o co chodzi :)

No właśnie w związku z tym że

[mrwrotek]

@npsr: (#) Jeśli ktoś używa IDE które nie wspiera możliwości dostosowania konfiguracji wcięć dla projektu no to jego strata.

@Marmite: a potem robisz commita i na diffie jest cały plik :)

[Marmite]

@mrwrotek: (#) No przecież ustalacie sobie chyba z resztą zespołu jaka jest konwencja formatowania i wsio? Nie wiem w czym problem.

[m4kb0l]

na diffie jest cały plik :)

@mrwrotek: W porządnym sofcie możesz sobie zrobić "ignore whitespace"