jQuery переменная scope? говорит, что не определено

Question

У меня есть следующий код:

function postToDrupal(contacts, source, owner) { 
    (function ($) { 

    var contact, name, email, entry; 
    emails = {}; 

    for (var i = 0; i < contacts.length; i++) { 
     contact = contacts[i]; 
     emails[i]['name'] = contact.fullName(); 
     emails[i]['email'] = contact.selectedEmail(); 
    } 

    $.post("/cloudsponge-post",emails,function(data) { 

    }); 
    }(jQuery)); 
} 

Я получаю следующее сообщение об ошибке, когда я пытаюсь запустить его:

WARN: Attempt to invoke callback [afterSubmitContacts] failed: TypeError: Cannot set property 'name' of undefined 

Я не уверен, что проблема IS-я вполне новый для JS и нахожу это немного сложнее. Какая причина, по которой она сломана, и как мне ее исправить?

Answer 1

Есть куча способов, которыми вы могли бы написать этот код, но лично я хотел бы сделать это:

function postToDrupal(contacts, source, owner) { 
    // TODO: source and owner are unused 

    var emails = jQuery.map(contacts, function(contact) { 
     return { 
      name: contact.fullName(), 
      email: contact.selectedEmail() 
     } 
    }); 

    jQuery.post('/cloudsponge-post', emails, function(data) { 
     // ... 
    }); 
} 

Answer 2

Этот объект emails[i] еще не определен. попробуйте таким образом:

for (var i = 0; i < contacts.length; i++) { 
    contact = contacts[i]; 
    emails[i] = {}; //Instantiate it here 
    emails[i]['name'] = contact.fullName(); 
    emails[i]['email'] = contact.selectedEmail(); 
} 

Answer 3

Я подозреваю, что вам нужен массив вместо объекта. Поэтому вы должны изменить emails = {} на emails = [].

Если вы, как @PSL предложил вам закончить с таким объектом (который не является массивом):

{ 
    0: { 
    name: 'john' 
    email: 'john@john.com' 
    }, 
    1: { 
    name: 'lennon' 
    email: 'lennon@lennon.com' 
    } 
} 

Одно из возможных решений:

var contact, name, email, entry, 
    emails = []; 

for (var i = 0; i < contacts.length; i++) { 
    contact = contacts[i]; 
    emails.push({name: contact.fullName(), email: contact.selectedEmail()}); 
} 

Вы будете в конечном итоге с этим:

[ 
    { 
    name: 'john' 
    email: 'john@john.com' 
    } 
    ,{ 
    name: 'lennon' 
    email: 'lennon@lennon.com' 
    } 
]