Tengo un problema con el uso de una matriz como campo de Javascript.
var Object = function () { var admins = []; this.addAdmin = function(admin){ this.admins.push(admin) } }Normalmente, esperaría que el administrador sea empujado a los administradores de la matriz, pero en su lugar obtengo un 'no se puede leer la propiedad 'empujar' de indefinido'.
Si no me equivoco cuando inicialicé el Objeto con el nuevo Objeto(), admins = []; debe inicializar la matriz. ¿Es esto una limitación de Javascript?
Gracias de antemano.
var array crea una variable local . No crea una propiedad en el objeto.
Necesitas:
this.admins = [];o
admins.push(admin) /* without this */En su función, los admins son una variable local para la función. Debe declarar los admins como una propiedad en la instancia.
function Obj(){ this.admins = []; } Obj.prototype.addAdmin = function(admin){ this.admins.push(admin); } obj = new Obj(); obj.addAdmin('tester'); Además, debido a que Object es el objeto base global, no cree funciones u objetos llamados Object .
Sospecho que te has confundido (lo cual es fácil :-)) porque has visto un código como este:
class Obj { admins = []; addAdmin(admin) { this.admins.push(admin); } } Eso usa la sintaxis moderna class y campos de clase para colocar una propiedad de admins en el objeto construido a través de new Obj . (Tenga en cuenta que no hay var antes de admins = []; .) Pero en su código, ha usado la sintaxis anterior basada en function . Dentro de su función, var admins = []; simplemente crea una variable local, no una propiedad.
Sugeriría que si desea crear funciones de constructor, usar la nueva sintaxis de class anterior es la forma más simple y poderosa de hacerlo. Sin embargo, si desea utilizar la sintaxis anterior, otras respuestas han mostrado cómo, pero para completar, haga que los admins sean una propiedad del objeto:
let Obj = function() { this.admins = []; // *** this.addAdmin = function(admin){ this.admins.push(admin) }; }; o quizás con addAdmin en el prototype :
let Obj = function() { this.admins = []; // *** }; Obj.prototype.addAdmin = function(admin){ this.admins.push(admin) }; o use el hecho de addAdmins cierra la llamada a Obj y, por lo tanto, a los admins locales:
let Obj = function() { const admins = []; this.addAdmin = function(admin){ admins.push(admin) // <=== No `this.` here, you want to close over the // `admins` local }; };