Cada vez que llamas a arrancar
se está creando una nueva instancia de medicoInternas
, por lo que, si se llama a curarPersonas
aumentará de 0 a 1 y cuando termine la ejecución de arrancar
el objeto medico
será destruído.
Debes instanciar a medicoInternas
fuera de la función arrancar
:
var medico = new MedicoInternas("Leslie", 0, "Cardiología");
function arrancar (valor) {
if (valor === 'curar') {
medico.curarPersonas();
} else {
medico.mostrarDatos();
}
}
Es mejor añadir los métodos de instancia en el prototipo de la función
Toda función en JavaScript tiene un objeto especial, llamado prototype
que hereda de Object
. En este objeto especial están todas las propiedades y funciones que tendrá toda instancia de dicha función.
Agregar las funciones en el prototipo de una función base significa ganar rendimiento y buenas prácticas. Cuando creas funciones vía this
estás haciendo que se cree una nueva función por cada instancia de la función base; esto es muy malo ya que degrada mucho el rendimiento en aplicaciones complejas. Lo correcto es agregar funciones al prototipo:
function MedicoInternas (data) {
this.nombre = data.nombre;
this.curadas = data.personasCuradas;
this.especialidad = data.especialidadDoctor;
}
MedicoInternas.prototype.curarPersona = function () {
this.curadas++;
console.info('Persona curada');
};
MedicoInternas.prototype.mostrarDatos = function () {
console.info('Nombre:', this.nombre);
console.info('Personas curadas:', this.personasCuradas);
console.info('Especialidad:', this.especialidad);
}
De esta manera cada instancia de MedicoInternas
compartirán las mismas funciones. Esto es lo correcto y lo óptimo y debe hacerse así.
Código final
function MedicoInternas (data) {
this.nombre = data.nombre;
this.curadas = data.personasCuradas;
this.especialidad = data.especialidad;
}
MedicoInternas.prototype.curarPersona = function () {
this.curadas++;
console.info('Persona curada');
};
MedicoInternas.prototype.mostrarDatos = function () {
console.info('Nombre:', this.nombre);
console.info('Personas curadas:', this.curadas);
console.info('Especialidad:', this.especialidad);
}
let medico = new MedicoInternas({
nombre: 'Leslie',
personasCuradas: 0,
especialidad: 'Cardiología'
});
arrancar('curar');
arrancar('curar');
arrancar('curar');
console.log('\n');
arrancar('datos');
function arrancar (action) {
if (action === 'curar') {
medico.curarPersona();
} else {
medico.mostrarDatos();
}
}