El constructor de este objeto no me funciona! (ver código)

[Albuss]

El constructor no me funciona, o al menos no consiguo que funcione correctamente, en este código no consigo que el array que le paso como texto a imprimir acabe apareciendo en pantalla!

Código:

Código Javascript:
/*Definiendo la clase */
var ClaseDePrueva = function (array1,array2){ 
               /*contructor*/ 
        this.texto1 = array1;   
            this.NombreID = document.getElementById(array2);
 
              /* Metodo */
              this.ImprimirTexto = function(){
                this.NombreID.innerHTML = this.texto1;
        }
 
 
}
 
/*Inicializo el objeto y le paso las variables para el constructor! */
var ObjetoDePrueva = new ClaseDePrueva('ArrayDeprueva1' , 'Mydiv1');
 
 
 
Código HTML:
 
<div id="Mydiv1" onload="ObjetoDePrueva.ImprimirTexto()"></div> 

[gjx2]

El div no tiene evento de onLoad

Intenta con esto

Código Javascript:

/*Definiendo la clase */
var ClaseDePrueva = function (array1,array2){ 
/*contructor*/ 
 this.texto1 = array1; 
 this.NombreID = document.getElementById(array2);
 
 /* Metodo */
this.ImprimirTexto = function(){
 this.NombreID.innerHTML = this.texto1;
}
 
 
}
 
 
/*Inicializo el objeto y le paso las variables para el constructor! */
 
window.onload = function(){
var ObjetoDePrueva = new ClaseDePrueva('ArrayDeprueva1' , 'Mydiv1');
ObjetoDePrueva.ImprimirTexto();
}
 

[Albuss]

En realidad el onload era en la etiqueta body, fallo mio, el problema es unicamente del constructor (estoy totalmente seguro) y quería saber que he hecho mal.

[gjx2]

No le vi ningun error excepto lo del onLoad al div , de la forma que lo puse deberia funcionar.

[Albuss]

Ami el constructor no me funciona y sigo sin saber por que, de todos modos es la primera vez que me falla un costructor.

[zerokilled]

¡buenas!
¿en qué momento estás creando la instancia? fijate el ejemplo de @gix2, la instancia la crea en el evento onload. en tu caso tienes que crearla luego que el documento termine de cargar en el navegador para asegurarse que existe el elemento que tomas por ID.

[Albuss]

En realidad instanciaba el objeto dentro de la hoja .js , peor he probado a hacerlo mediante un onload en el body y me da el mismo resultado

[zerokilled]

he copiado tal cual el ejemplo que muestra @gix2, y no veo donde falla. http://jsfiddle.net/xgvRu/

[Albuss]

Vale,a vosotros os funciona por que estais declarando este escript en el head del archivo .html

Yo en cambio llamo mediante scr a un archivo externo .Js , y es cuando no me funciona! , he probado declarando todo dentro de la cabecera como vosotros y si me funciona, el problema viene cuando el codigo esta en un archivo .js externo y es llamado mediante src en el archivo .html


¿por que?

[gjx2]

No importa de igual forma debe de funcionar!
verifica si esta apuntando bien al archivo externo.

[Albuss]

He reescrito todo el código unas 5 o 6 veces, no se que puede estar pasando, seguiré intentándolo.

[zerokilled]

revisa la consola del navegador para determinar si algo mal está ocurriendo. si no muestra ningún error, probablemente sea algún error logístico de tu parte. pero hasta el momento, todo parece estar bien. y el código aislado en un archivo *.js debería funcionar igual.

[Albuss]

Me santa este error en la consola :

"TypeError: Cannot set property 'innerHTML' of null"

[zerokilled]

pues lo que sucede es que el script está intentado obtener un elemento que aún no existe en el documento. por tanto, cuando getElementById no encuentra el elemento devuelve null. entonces, como null es un valor primitivo que no tiene propiedades ni métodos, devuelve error al tratarlo como un objeto. fíjate en los ejemplos corregidos que la instancia se crea cuando el navegador ha terminado de cargar el documento, evento onload.

[Albuss]

Pero la id que le paso EXISTE! , y lo he comprabado infinidad de veces! , este tipo de situaciones me dejan K.O

Pero aun se pone la cosa mas negra!, he rehecho todo el codigo otra vez, y ahora cuando en la consola de Chrome le meto la llamada al metodo de clase me sale:

"Undefined" ¿? :X


Bueno , agradezco la atencion que me habeis prestado, lo que voy a ahcer es pasar a otra cosa y retomar esto dentro de un par de dias.

[zerokilled]

no importa si la ID existe en tu código html. lo que sucede es que el navegador analiza el código html de forma líneal y va construyendo el DOM. cuando el navegador se encuentra con un elemento SCRIPT este le pasa el mando de procesamiento al motor de scripting. si el código javascript intenta acceder al DOM, este sólo podrá trabajar con lo que el navegador haya construido hasta el momento. fíjate el siguiente código y revisa lo que escribe en la consola.

Código:

<script>console.log('antes del elemento: ', document.getElementById('grabme'));</script>
<p id="grabme">elemento con ID "grabme"</p>
<script>console.log('despues del elemento: ', document.getElementById('grabme'));</script>

introducir el código directamente en la consola te funciona porque ya el navegador ha creado el DOM.

[Albuss]

Pero ese elemento script es un src al que llamo en el head, es un archivo .js . No es un elemento script en el body.

[zerokilled]

esto deberia funcionarte.

Código:

<!-- documento html -->
<html><head><script type="text/javascript" src="script.js"></script></head>
<body><p id="grabme">Grab me</p></body></html>

Código:

// documento script.js
self.onload = function(){
alert(document.getElementById('grabme').innerHTML);
};

[elarrieux]

Hola!

Concuerdo con los colegas, prueba utilizando jquery:

Código:

   <script src="jquery-1.7.1.min.js"></script>
    <script type="text/javascript">
        $(document).ready(function () {
//pon aqui tu codigo...
        });
    </script>

De esta forma tu codigo se ejecutara una vez que se haya cargado el DOM.

Sds.

[Albuss]

Lo siento, intente aprender y comprender Jquery, pero no es algo que le vea pracmatismo, no es para mi, creo que es como utilizar java cuando tienes que utilizar C++ simplemente por que es mas "facil"

[Albuss]

Iniciado por zerokilled

esto deberia funcionarte.

Código:

<!-- documento html -->
<html><head><script type="text/javascript" src="script.js"></script></head>
<body><p id="grabme">Grab me</p></body></html>

Código:

// documento script.js
self.onload = function(){
alert(document.getElementById('grabme').innerHTML);
};

Si, me sale la ventana de alert

[elarrieux]

Iniciado por Albuss

Lo siento, intente aprender y comprender Jquery, pero no es algo que le vea pracmatismo, no es para mi, creo que es como utilizar java cuando tienes que utilizar C++ simplemente por que es mas "facil"

Mire que vale la pena aprenderlo.

Si ud. tuviera que escribir su propia funcion ready() le tomaria muchisimo mas de 100 lineas de codigo.

Puede utilizar tambien el onLoad que le sugirio zerokilled pero tenga en cuenta que no funciona en todos los navegadores de la misma forma.

Sds.