porque no me funciona bien esta funcion js

azaz · #1 (**permalink**) 26/09/2011, 04:57

Hola, he estado mirando por el foro y por google para montar esta codigo pero no consigo hacerlo funcionar bien, trata de mostrar en pantalla los nombres que tengo guardado en un archivo xml, estos nombes me los tendria que mostrar uno a uno hasta acabar con todos los nombres y volver a empezar.

Muestro codigo por si alguien me quiere corregir, saludos.

Código Javascript:

Ver originalfunction MostrarNombres(){
    if (Ajax.readyState == 4) {
        if(Ajax.status == 200) {
     var xml  = Ajax.responseXML;
     for (i = 0; i < xml.getElementsByTagName('nombre').length; i++){
        var nombres= xml.getElementsByTagName('nombre')[i].firstChild.nodeValue;
     
document.getElementById("nombres").innerHTML ='<p>'+nombres+'</p>';
                  
                  if(i>=(nombres_img.length)){i = 0}
            }
        }
    }
}
 
        window.onload = MostrarRegistos;
        setInterval(MostrarRegistos,4000)
        
var  Ajax = objetoAjax();
Ajax.open('get', 'archivo.xml');
Ajax.onreadystatechange = MostrarNombres;
Ajax.send('');
 
function objetoAjax()   {
        if (window.XMLHttpRequest)
            _ajax = new XMLHttpRequest();
        else
            if (window.ActiveXObject)
                _ajax = new ActiveXObject("Microsoft.XMLHTTP");
            else
                _ajax = false;
        return _ajax;
    }

el xml es algo asin:

Código XML:

Ver original<datos><nombre>.jose</nombre><nombre>ana</nombre><nombre>pedro</nombre><nombre>sogia</nombre></datos>

Código HTML:

Ver original<body>
<div id="registro"></div>
</body>

IsaBelM · #2 (**permalink**) 26/09/2011, 06:24

prueba a cambiar estas cosas

Cita:

azaz · #3 (**permalink**) 26/09/2011, 06:34

IsaBelM, cambie lo que me dijistes pero sigue sin funcionarme.

Saludos.

IsaBelM · #4 (**permalink**) 26/09/2011, 08:18

normal que no funcionara y yo estoy

Cita:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="es" lang="es">
<head>
<meta http-equiv="Content-Type" content="application/xhtml; charset=utf-8" />
<title></title>
<script type="text/javascript">
function MostrarNombres(){
var Ajax = objetoAjax();
Ajax.onreadystatechange = function() {

* * if (Ajax.readyState == 4) {
* * * * if(Ajax.status == 200) {
* * *var xml *= Ajax.responseXML;
* * *for (var i = 0; i < xml.getElementsByTagName('nombre').length; i++){

document.getElementById("registro").innerHTML +='<p>'+ xml.getElementsByTagName('nombre')[i].firstChild.nodeValue+'</p>';
* * * * * * * * *
* * * * * * * * *
* * * * * * }
* * * * }
* * }
}
Ajax.open('get', 'archivo.xml', false);
Ajax.send(null);
}
*
* * *

* * * *

*
function objetoAjax(){
* * * * if (window.XMLHttpRequest)
* * * * * * _ajax = new XMLHttpRequest();
* * * * else
* * * * * * if (window.ActiveXObject)
* * * * * * * * _ajax = new ActiveXObject("Microsoft.XMLHTTP");
* * * * * * else
* * * * * * * * _ajax = false;
* * * * return _ajax;
* * }

window.onload = MostrarNombres;
</script>
</head>
<body>
<div id="registro"></div>
</body>
</html>

azaz · #5 (**permalink**) 26/09/2011, 09:58

Hola IsaBelM, el codigo que posteastes funciona muy bien, pero realmente no hace lo que yo quiero, en el codigo que posteastes arriba muestra todos los resultados juntos del xml y yo lo que pretendia es mostrar los resultados uno a uno.

El codigo que quiero es este, funciona bien trata de mostrar en dos en dos los resultados en pantalla, cuyo resultados recojo de una basa de datos y los paso a un array js, pero claro no consigo adaptarlo a lo que yo quiero, yo lo que quiero es que los datos los recoja de un archivo xml, no de la base de datos, intente montar este script con lo que encontre en el foro y en google pero no consigo hacerlo funcionar bien, muestro el codigo que si funciona, pero claro recuerdo que lo que quiero cambiar es cojer los datos de un archivo xml en vez de la base de datos.

el codigo es este:

Código PHP:

Ver original//consulta
//creo el array php
$nombres_empresa = array();
while ($row = mysql_fetch_array($consulta)){
$nombres_empresa[]= $row['nombre'];
}
?>
<script language='javascript'>
//paso el array php a js
var nombres_empresa[]=new Array();
<?php
for ($i = 0, $total = count($nombres_empresa); $i < $total; $i ++)
echo "\nnombres_empresa[$i] = '$nombres_empresa[$i]';";
?>
//creo la funcion js
var pos = 0;
 function MostrarNombres(){
    
    for (i=1;i<=2;i++){
        
             nombre_empresa = nombres_empresa[pos];
           
        
document.getElementById("registro"+i).innerHTML ='<p>'+nombre_empresa+'</p>';
 
                     
                     pos++;
                if(pos>=(nombres_empresa.length)){pos = 0}
            }
        }
            
        window.onload = MostrarNombres;
        
        setInterval(MostrarNombres,2000)
        
</script>
    </head>
    <body>
            
            <div id="registro1"></div>
            <div id="registro2"></div>
            
        
        </body>
</html>

Espero que me podais ayudar,saludos.

refreegrata · #6 (**permalink**) 26/09/2011, 10:15

El código que han posteado sirve como base, pero con eso tú ya deberías ser capaz de alterar el código para hacer lo que quieres. Podrías declarar algunas variables globales y jugar con eso:

Código:

var procesamiento, xml = null, contador=0;
function MostrarRegistro() {
        try {
                document.getElementById("registro").innerHTML ='<p>'+ xml.getElementsByTagName('nombre')[contador].firstChild.nodeValue+'</p>';
                contador ++;
        }
        catch(e) {
                //Y aqui se reiniciaría, o podrías detener detener el ciclo infinito u otra cosa
                document.getElementById("registro").innerHTML = '';
                contador = 0;
        }
}

function MostarNombres() {
        var ajax = objetoAjax();
        ajax.open('get','archivo.xml');
        ajax.onreadystatechange = function() {
                if(ajax.readyState == 4){
                        if(ajax.status == 200) {
                                xml=ajax.responseXML;
                                if(xml.getElementsByTagName('nombre').length > 0)
                                        procesamiento = window.setInterval(MostrarRegistro, 4000);
                        }
                }
        }
        ajax.send(null);
}
....

Otra cosa sería si piensas que los datos a devolver van a cambiar y por tanto quieres actualizar cada cierto tiempo la variable XML volviendo a consultar o algo así.

El último código que posteaste no te funcionará por que que deseas crear y ejecutar javascript directamente desde el archivo al que llamas con open. Todo tu javascript debería estar en el fichero js vinculado al archivo html principal. De otra forma tendrías que valerte de trucos poco trasparentes , que no son necesarios para resolver la problemática que te aqueja. Te recomiendo leer un poco más de Ajax.

Saludos.

Saludos

azaz · #7 (**permalink**) 26/09/2011, 10:33

refreegrata, probe tu codigo pero no me funciona, aunque tampoco lo entendi muy bien si pudieras postearlo entero con explicaciones para entenderlo, saludos.

refreegrata · #8 (**permalink**) 26/09/2011, 10:57

hay un pequeño error, le puse mal el nombre a una función:

"MostarNombres" en vez de "MostrarNombres"

Cambiando eso funciona. Obviamente está consultando a archivo.xml y no a un archivo.php, por que es lo consultaste en el primer post y deberías partir probando eso. Una vez funcionando intenta buscar los valores de la base de datos. Tampoco te he posteado todo el código javascript, sólo lo que cambie con respecto a lo posteado por IsaBelM.

La idea es simple, al cargar la página consulta el archivo XML con Ajax, guarda la respuesta en una variable y, de haber resultados, iniciar el ciclo para mostrarlos. Para acceder a estos deberás definir la variable con los datos, asociar el setInterval y mantener un contador, todos como variables globales. Una vez se acabe de recorrer la respuesta reiniciar el contador para volver a mostrar los resultados, o podrías detener el ciclo si no los quieres volver a mostar(por eso defíne "procesamiento" como global).

Mas o menos eso. Saludos.

A propósito, ¿no usas herramientas como Firebug, o chequeas regularmente la consola de errores?, lo digo por qué el pequeño detallito con el nombre de la función debiste haberlo visto fácil.

azaz · #9 (**permalink**) 26/09/2011, 11:10

ahora funciona, gracias y saludos.

refreegrata · #10 (**permalink**) 26/09/2011, 11:14

Que bueno que te haya funcionado.

azaz · #11 (**permalink**) 26/09/2011, 14:28

hola refreegrata, bueno estoy aqui de nuevo porque estoy intentando cambiar el codigo, para cuando recorro todos los datos xml, no se quede la pantalla en blanco, probe de colocar en catch(e) lo mismo que el try y funciono bien, pero mi problema biene cuando intento crear mas div registro con un bucle for, que no me los muestra seguidos los datos, es decir cuando acaba de recorrer todo el xml, lo que me hace es repetirme los nombres hasta acabar los div creados y luego vuelve a mostrar los nombres por su orden, yo no quiero eso quiero que los muestre seguidos, no se si me explique bien. esto es posible?

Saludos.

refreegrata · #12 (**permalink**) 26/09/2011, 17:30

Mira, pero eso es fácil. Si quieres que no se vaya borrando debes sumar en vez de asignar en el innerHTML. Si no quieres que los borre al final y no repita el ciclo sólo debes pararlo:

Código:

...
function MostrarRegistro() {
        try {
                document.getElementById("registro").innerHTML +='<p>'+ xml.getElementsByTagName('nombre')[contador].firstChild.nodeValue+'</p>';
                contador ++;
        }
        catch(e) {
                //Y aqui se reiniciaría, o podrías detener detener el ciclo infinito u otra cosa
                try { window.clearInterval(procesamiento); } catch(e) {}
                contador = 0;
        }
}
...

azaz · #13 (**permalink**) 27/09/2011, 03:17

Hola refreegrata, gracias de nuevo por tu colaboracion, ahora si que lo entendi perfectamente.

Saludos.

azaz · #14 (**permalink**) 04/10/2011, 15:45

Cita:

Otra cosa sería si piensas que los datos a devolver van a cambiar y por tanto quieres actualizar cada cierto tiempo la variable XML volviendo a consultar o algo así.

Estoy intentando cambiar el xml de forma dinamica cada vez que seleciono una opcion en un select y funciona correctamente el xml me lo cambia, pero los datos en pantalla no alguna idea de como conseguir que me cambien los datos.
Saludos.