Bir JavaScript nesnesi oluşturmanın en iyi yolu bu şekilde çalışır ve JSLint'i geçirir?

Question

JavaScript nesnelerini olabildiğince sezgisel olarak yapıp yapılamayacağımı ve mümkün olduğunca doğru olduğundan emin olup olmadığımı görmeye çalışıyorum. Crockford'un JSLint.com'undan bir sürü farklı senaryo çalıştırıyorum ve çok şansım olmadı. Bir hatayı düzeltmek istiyorum, o zaman değişim nedeniyle başka bir şey ortaya çıkıyor. Aşağıda alabildiğim kadar iyi. Bunun üzerinde başka kimse var mı?

function gizmo(id) { 

    /* private variables */ 

    var myId = id; 

    /* private methods */ 

    var init = function() { 
    if (myId < 1) { 
     setId(1); 
    } 
    }; 

    var setId = function (newId) { 
    myId = newId; 
    }; 

    // run 'constructor' 
    init(); 

    /* public methods */ 

    return { 
    getId: function() { 
     return myId; 
    }, 
    setId: function (newId) { 
     setId(newId); 
    }, 
    incrementId: function (inc) { 
     setId(myId + inc); 
    } 
    }; 
} 

// creating an instance of gizmo 

var myGizmo = gizmo(-2); 
console.log(myGizmo.getId()); // outputs 1 

myGizmo.setId(5); 
console.log(myGizmo.getId()); // outputs 5 

myGizmo.incrementId(2); 
console.log(myGizmo.getId()); /// outputs 7 

Bu iyi iş gibi görünüyor:

Bu bir nesne yapısı tipik bir yoldur. Bununla birlikte, bunu JSLint aracılığıyla çalıştırdığımda, bana iki özel işlevimin 'Implied Globals' olduğunu belirten bir hata veriyor.

Böyle değişkenlerle üstündeki benim fonksiyonları beyan etmektir ile gelebilir iyi:

function gizmo(id) { 

    /* private variables */ 

    var myId = id, 
     init, 
     setId; 

    /* private methods */ 

    init = function() { 
    if (myId < 1) { 
     setId(1); 
    } 
    }; 

    setId = function (newId) { 
    myId = newId; 
    }; 

    // run 'constructor' 
    init(); 

    /* public methods */ 

    return { 
    getId: function() { 
     return myId; 
    }, 
    setId: function (newId) { 
     setId(newId); 
    }, 
    incrementId: function (inc) { 
     setId(myId + inc); 
    } 
    }; 
} 

Answer 1

JSLint'te bir hata olduğundan eminim. Henüz setId görmemiş, bu yüzden küresel olduğunu varsayar. Ama gerçekte, hiçbir farketmez, çünkü tüm var s, ECMAScript 5 10.5 uyarınca kaldırılır. Bu, ilk örneğiniz ve ikincinin aynı semantik olduğu anlamına gelir. İşlevdekinumaralı yerel bir değişken bildirimi hemen işlenir ve ilk olarak undefined değerine sahip olarak bağlanır. Ancak, işlev (örneğin, init) aslında çalıştığında, kapalı değer artık undefined'dur.

bu testi yapmak, setId başlangıçta tanımlanmamış olduğunu görüyoruz, ama asla bir küresel atıfta için:

function setId() 
{ 
    alert("Global setId"); 
} 
function f() 
{ 
    var init = function() 
    { 
    setId(); 
    } 
    alert(typeof(setId)); 
    init(); 
    var setId = function() 
    { 

    } 
} 

Bu tanımsız uyaracaktır, daha sonra TypeError hatası atmak.

Answer 2

Ben jslint bilmem ama "javascript iyi parçalar" okuduktan sonra, Nesneleri her zaman açık olarak beyan ederim.

örn: Aslında yukarıdaki herhangi bir nesneyi ilan etmiyoruz

Mogwai={ 
    has_crazy_thoughts:True, 
    reacts_to_water:True, 
    name: 'Gizmo', 
    eat:function(food){ 
    // code 
    }, 
    become_gremlin:function(){ 
    // code 
    }, 
    cause_havoc:function(){ 
    // code 
    } 
} 

. Sadece bir işlev. İç işlevler Javascript'te gerçekten mevcut değil - Java gibi değil.

DÜZENLEME: Yukarıda belirtilen kitabı (bağlı olmadan) tavsiye ederim: http://oreilly.com/catalog/9780596517748 ... bize JSlint'i getiren Douglas Crockford tarafından yazılmıştır.

Answer 3

JSLint, setId'nin init dosyasında başvurulan önce tanımlanmasını bekler.

Bu, JSLint'ten geçer.

function gizmo(id) { 

    /* private variables */ 

    var myId = id; 

    /* private methods */ 

    var setId = function (newId) { 
    myId = newId; 
    }; 

    var init = function() { 
    if (myId < 1) { 
     setId(1); 
    } 
    }; 

    // run 'constructor' 
    init(); 

    /* public methods */ 

    return { 
    getId: function() { 
     return myId; 
    }, 
    setId: function (newId) { 
     setId(newId); 
    }, 
    incrementId: function (inc) { 
     setId(myId + inc); 
    } 
    }; 
} 

Answer 4

Ben ciddiye JavaScript programlanmış ettik, bu yüzden en iyi uygulama ayrıntılarına hafızam oldukça gitmiş beri uzun bir süre (birkaç yıl) olmuştur. Yaptığım şey geri döndü ve bilinçli bir karar vermene yardımcı olabilecek birkaç kaynak kazdı.

İlk olarak, nesnelerinizi oluşturma şekliniz, Module Pattern'u hatırlatıyor gibi görünüyor. Hatırladığım kadarıyla, bağlandığım makale bunun hakkında oldukça iyi bir okuma. İkincisi, belki de instantiate your objects'a farklı bir yol tercih edersiniz. Bu makale size biraz farklı bir şey kazandırıyor.