Konwencje kodu JavaScript - deklaracja zmiennej

Question

Jakie jest mniej podatne na błędy podejście do deklarowania zmiennych w JavaScript?

var a; 
var b; 
var c; 

lub

var a, 
    b, 
    c; 

jQuery i Dojo użyć drugiego podejścia i osobiście to moja ulubiona pod względem czytelności kodu, problemem jest to, że trudniej debugowania.

Przykład 1:

var a, 
    b, 
    c; // oops.. semicolon instead of comma 
    d, 
    e; 

Przykład 2:

Podczas poszukiwania pewnej zmiennej w projekcie var wyróżnia się lepiej, że zmienna jest zadeklarowana.

EDIT: Oryginalne przykłady były błędne, popełniłem błąd podczas kopiowania i wklejania. Kod jest już zaktualizowany, przepraszam za incovenient.

Answer 1

Wiele deklaracji zmiennych (zamiast łączonych) jest mniej podatnych na błędy z powodów, o których wspomniałeś. Jednak połączone deklaracje mają kilka innych zalet:

• JSLint i ewentualnie inne linters będą narzekać na wiele słów kluczowych var według zakresu.
• Jedno słowo kluczowe var zminimalizuje lepiej niż wiele z oczywistych powodów.
• Łączenie deklaracji zmusza do zadeklarowania zmiennych w jednym miejscu, prawdopodobnie blisko początku funkcji, w której się znajdują, co jest uważane za dobrą praktykę.

Z drugiej strony, jak już wspomniano, błędy można wprowadzać za pomocą złożonych deklaracji zmiennych, i mogą one być niezręcznie reprezentowane w różnicach.

Dopóki zachowasz spójny styl deklaracji zmiennych w całym projekcie, styl, który wybierzesz, nie powinien mieć znaczenia.

Answer 2

Druga opcja jest lepsza w mojej książce, ponieważ jest mniej zbędna. Nowoczesne przeglądarki i IDE sprawiają, że dość łatwo jest znaleźć błąd, który wywołujesz.

Oto jsFiddle, który pokazuje, co się dzieje. Przypadkowe wstawienie średnika w serii powoduje Uncaught ReferenceError. Jest to szybkie rozwiązanie i mały kompromis dla ładniejszej składni. Jeśli jest to najgorszy błąd, z którym musisz sobie poradzić podczas pisania JavaScript, masz świetny dzień.

Jak na ironię, w drugim przykładzie miałeś literówkę - średnik po a. Whoops. :)

EDIT: Oryginalny plakat poprawiono literówka; Zostawiam wzmiankę, ponieważ jest to istotne dla pierwotnego pytania.

Answer 3

var a; 
var b; 
var c; 

jest lepszym rozwiązaniem, ponieważ nie spowoduje odrzucenia nieokreślonego błędu. Również zmienne będą mieć odpowiednio zakres & nie w zasięgu globalnym.

var a=10; 
    b=5; 

spowoduje b w zasięgu globalnym.

EDYCJA: To było w kontekście z próbką wysłaną pierwotnie https://stackoverflow.com/revisions/9746359/2. Dzięki zaktualizowanemu kontekstowi sprowadza się on do bardziej indywidualnych preferencji. Moim zaleceniem będzie, aby nie było miejsca na takie błędy, jak trudno je znaleźć.

Answer 4

Preferuję Twoje drugie podejście do używania tylko jednego słowa kluczowego var. Niedawno zrobiłem to o krok dalej i utworzyłem instancję typów dla każdej zmiennej, aby pomóc mi zapobiec późniejszemu przekształcaniu typu niezamierzonego. Przykład:

var a = 0, 
    b = 0, 
    c = {}, 
    d = [], 
    f = ""; 

Podam tutaj wartość rzeczywistą, jeśli mam wartość do dostarczenia w tym miejscu. Jeśli nie mogę dostarczyć następujące wartości nieużywane:

• 0 dla typów numerycznych
• [] na tablicy
• {} dla obiektu dosłownym i DOM węzłów
• "" na ciąg
• function() {return;} dla funkcji
• false dla boolean
• w tym celu nie martwię się o wstępne przypisywanie typów dla typów Math, Date lub RegExp.

Answer 5

Istnieje jedna premia do posiadania wielorakiej deklaracji zmiennej za pomocą 1 var.

Możesz uzyskać niższy skrócone skrypt. Oczywiście 4 bajty wymagane do otrzymania deklaracji var nie mają większego znaczenia, dopóki nie pojawi się takie miejsce jak stackoverflow.com. Miliony na miliony pobrań == gigabajty transferu w oparciu o kilka dodatkowych deklaracji var.