Problema con clases en javascript

[dvd181191]

Buenas tengo una duda con la clases en javascript que me estan volviendo loco

tengo esta clase

Código:

// JavaScript Document

// JavaScript Document

function DefinirTropas (uno,dos,tres,cuatro,cinco,seis,siete,ocho,nueve) {
	
	//definimos todos los valores de las tropas
	this.nombre = uno;
	this.imagen = dos;
	this.ataque = tres;
	this.defensa = cuatro;
	this.movilidad = cinco;
	this.distancia = seis;
	this.salud = siete;
	this.bando = ocho;
	this.posicion = nueve;
	
	//definimos la posicion en el mapa
	this.CrearPosicion = DefinirPosicion	
	
	function DefinirPosicion  () {
		
		this.posicion = "NuevaPosicion";
		
	}
}

y para crearla

Código:

//creamos un array para guardar las tropas
			Tropa = new Array ();
				
			//definimos todas las tropas
			Tropa[0] = new DefinirTropas ("Zombie","",100,10,1,1,100,1,"");		
			Tropa[1] = new DefinirTropas ("Arquero","",100,10,3,3,100,0,"");
			Tropa[2] = new DefinirTropas ("Zombie","",100,10,1,1,100,1,"");
			Tropa[3] = new DefinirTropas ("Arquero","",100,10,3,3,100,0,"");
			Tropa[4] = new DefinirTropas ("Arquero","",100,10,3,3,100,0,"");
			Tropa[5] = new DefinirTropas ("Zombie","",100,10,1,1,100,1,"");

el problema es que cuando quiero redefinir un atributo de esta clase me da error

Código:

//colocamos las tropas en el mapa
			for (a = 0; a <= Tropa.length; a++)	 {
				Tropa[a].CrearPosicion;				
			}

Que estoy haciendo mal?

y mi segunda duda es como ago para que las funciones de mi clase sean privadas y solo se pueda aceder a ellas en la clase?

Gracias adelantada por la respuesta :)

saludos

[chwc]

hola!
la función esta mal creada

Código Javascript:

this.DefinirPosicion =function(){
        this.posicion = "NuevaPosicion";
    }
 

y luego no estas ejecutando la funcion aca

for (var a = 0; a <= Tropa.length; a++) {
Tropa[a].DefinirPosicion();
}

con eso en rojo ahi se ejecuta

un saludo ! y espero que os haya iluminado !


si te sirvio podraimos arreglar con unas de tus plantas ;) xD

[Reedyseth]

Nada mas para aclarar, cuando esta en el for usando CrearPosicion(), es lo mismo que usar la funcion como mencionaste chwc, por que al final CrearPosicion es una referencia de DefinirPosicion(), su error esta que intenta llamar la funcion como si estuviera llamando una propiedad, y es que le falta los parentesis, algo asi:

for (a = 0; a <= Tropa.length; a++) {
Tropa[a].CrearPosicion();
}

Saludos.

[chwc]

Iniciado por Reedyseth

Nada mas para aclarar, cuando esta en el for usando CrearPosicion(), es lo mismo que usar la funcion como mencionaste chwc, por que al final CrearPosicion es una referencia de DefinirPosicion(), su error esta que intenta llamar la funcion como si estuviera llamando una propiedad, y es que le falta los parentesis, algo asi:



Saludos.

Hola !

he leido lo que comentas, y hay que ver bien a fondo, un problema radica en esta parte

Código Javascript:

this.CrearPosicion = DefinirPosicion    
    
    function DefinirPosicion  () {
        
        this.posicion = "NuevaPosicion";
        
    }
 

primero que nada, CrearPosicion esta obteniendo como valor undefined, ya que en ese momento DefinirPosicion no existe, pero DefinirPosicion es generado en la proxima linea

ya que

Código Javascript:

this.CrearPosicion = function() {       
        this.posicion = "NuevaPosicion";        
    }
 

bastaria,

asi que seria mejor que la funcion sea asi ( le agrego DefinirPosicion por las dudas )

Código Javascript:

function DefinirTropas (uno,dos,tres,cuatro,cinco,seis,siete,ocho,nueve) {
    
    //definimos todos los valores de las tropas
    this.nombre = uno;
    this.imagen = dos;
    this.ataque = tres;
    this.defensa = cuatro;
    this.movilidad = cinco;
    this.distancia = seis;
    this.salud = siete;
    this.bando = ocho;
    this.posicion = nueve;
    
    //definimos la posicion en el mapa
        
    
    this.CrearPosicion = function() {       
        this.posicion = "NuevaPosicion";        
    }
this.DefinirPosicion = this.CrearPosicion
}
 

igual me parece que lo que quiso hacer, era tener DefinirPosicion como una funcion Global, pero lo puso adentro de esa funcion

un saludo !!

[dvd181191]

antes que nada gracias por responder :), pero sigo con el mismo problema

lo que intento hacer como bien supondreis es un juego.

la idea es que yo cuando creo un personaje defino unos datos de este como son nombre, imagen, movilidad...

y ahora lo que quiero es que al llamar por ejemplo a

Tropa[0].DefinirPosicion

me redefina la posicion de este objeto haciendo que

Tropa[0].posicion (que es donde se guarda la posicion en el mapa de ese soldado)

se redefina con un nuevo valor

todo esto lo ago para que cada x segundos traves de setInterval pueda editar la posicion en el mapa de ese soldado y este pueda moverse.

Tendre varios metodos como:

Tropa[0].Atacar, Tropa[0].Moverse, Tropa[0].Morir ...

y en cada una de estas llamadas lo que quiero conseguir es redefinir las variables que se crean para ese objeto

no se si me explico muy bien, pero se agradece muchisimo vuestra ayuda

saludos

[Reedyseth]

Que tal companero, mira la solucion es mas simple de lo que el problema parece, tienes todos errores que son criticos y por eso no funciona tu codigo:

Tu problema se presenta mas que en la clase en tu for

//colocamos las tropas en el mapa
for ( a = 0; a <= Tropa.length; a++) {
Tropa[a].CrearPosicion
}

Como te puedes percatar en el for le tienes que a = 0 y que sea menor igual a 5, como tienes un arreglo de 5 y el arreglo comienza de 0 le estas diciendo que vaya auna sexta posicion, lo que tendrias que hacer es dejar que sea menor que 5, o que iniciar tu variable a en 1, tampoco estas invocando el metodo con Tropa[a].CrearPosicion sino que con eso haces referencia a una propiedad como te habia comentado en un post pasado, entonces la funcion deberia de quedar algo asi:

Código Javascript:

//colocamos las tropas en el mapa
            for ( a = 0; a < Tropa.length; a++) {
                Tropa[a].CrearPosicion();
            }
 

o asi:

Código Javascript:

//colocamos las tropas en el mapa
            for ( a = 1; a <= Tropa.length; a++) {
                Tropa[a].CrearPosicion();
            }
 

Ya probe el codigo asi y funciona a la perfeccion, lo que si te recomiendo es que te metas mas en el tema de objectos por que estas haciendo algo muy generico y es programacion no muy controlada, te invito a que leas un poco sobre contextos en la clase(publico,privado, privilegiado) y tambien prototype, para que puedas hacer un buen uso de los objectos en javascript,

saludos. :)

[dvd181191]

Muchisimas gracias :)

funciona perfectoooooooooo!

La verdad es que estoy buscando algun videotutorial o algun libro bueno para aprender POO de javascritp pero es dificil de encontrar.

Las clases no distan mucho de php pero este me resulta mas llevadero.

Si me pudierais sugerir algun buen libro seria de mucha ayuda

y gracias a todos los que me han ayudado

saludos

[Reedyseth]

Una vez que sepas lo basico de javascript puedes leer este libro "Pro JavaScript Techniques" de John Resig el creador de jQuery.

Saludos.

[chwc]

codeacademy es de lo mejor que he visto para aprender ! un saludo