pequeña duda del tipo: Porque no lo hace?

[lukas4]

que tal compañeros, tengo una pequeña duda de tipo: "porque no lo hace?" aqui en este codigo (tratare de resumirlo para llegar el punto importante):

Código javascript:

function showUser(){
   codigo.Javascript;
   horas()//esta es mi funcion
   mas.Codigo.Javascript;
}
 

Código javascript:

function horas(){
  validaciones.Y.EsasCosas;
}
 

el problema es que si lo pongo de esta manera no me hace lo que sigue despues de la funcion horas, que en este ejemplo seria "mas.Codigo.Javascript"

pero si lo pongo asi:

Código javascript:

function showUser(){
 codigo.Javascript;
   validadciones.Y.EsasCosas //que seria mi funcion horas
   mas.Codigo.Javascript;
}
 

asi, si funciona, alguna idea del porque?

[David]

Sin ver el código implicado es difícil saber el problema.

Muestra el código de ambas funciones.

[lukas4]

solo porque lo pediste :P

Código javascript:

function showUser(){
    xmlhttp=GetXmlHttpObject();
    if (xmlhttp==null)  {
        alert ("Browser does not support HTTP Request");
        return;
    }
    var url="getuser2.php"
    url=url+"?q="+document.getElementById("usuario").value;
    if(document.getElementById("titulo").value==""){
        alert("Seleccione la acción que se realizo");
        return;
    }
    else{
        url=url+"&t="+document.getElementById("titulo").value;
    }
    
    if(document.getElementById("hora").value=="00"||document.getElementById("hora").value==""){
        alert("Introduzca la hora a la que se realizo el servicio");
        return;
    }
        
    horas(); //despues de realizar esta funcion lo demas no lo hace
 
    if(opc==1){
        document.getElementById("lol").style.display="";
        if(document.getElementById("dato").value==""){
            alert('El campo de Información extra no debe estar vacio');
            return;
        }
    }
    if(document.getElementById("dato").value!=""){
        url=url+"&d="+document.getElementById("dato").value;
    }
    url=url+"&sid="+Math.random();
    xmlhttp.onreadystatechange=stateChanged;
    xmlhttp.open("GET",url,true);
    xmlhttp.send(null);
}
 

Código javascript:

function horas(){
    if(document.getElementById("hora").value=="00"||document.getElementById("hora").value==""){
        alert("Introduzca la hora a la que se realizo el servicio");
        return;
    }
    if(document.getElementById("minuto").value==""){
        document.getElementById("minuto").value=="00";
        alert(document.getElementById("minuto").value);
    }
    if(document.getElementById("meridiano").value=="pm" && document.getElementById("hora").value <8){
        y = parseInt(document.getElementById("hora").value) + 12;
        document.getElementById("hora").value = y;
    }
 
    var fecha = document.getElementById("mes").value+" "+document.getElementById("dia").value+" "+document.getElementById("year").value+" "+document.getElementById("hora").value+":"+document.getElementById("minuto").value+":00";
    url=url+"&f="+fecha;
 

[David]

Debes tener en cuenta el alcance de las variables, no es lo mismo asignar un valor a url en la función showUser (precedida de var) que hacerlo en la función horas (precidada o no de var)

Con este ejemplo creo que quedará claro:

Código Javascript:

function funcion1() {
    var url = 'http://test/';
    
    funcion2();
    
    alert(url);
}
 
function funcion2() {
    url = 'http://other/';
}
 
function funcion3() {
    var url = 'http://test/';
    
    url = 'http://other/';
    
    alert(url);
}
 
function funcion4() {
    url = 'http://test/';
    
    funcion2();
    
    alert(url);
}
 
funcion1(); /* http://test/ */
funcion3(); /* http://other/ */
funcion4();  /* http://other/ */
 

Debes tener en cuenta que no siempre es lo mismo usar var o no al asignar el valor de una variable.

[America|UNK]

Estás repitiendo estas 3 líneas:

Código javascript:

if(document.getElementById("hora").value=="00"||document.getElementById("hora").value==""){
 
        alert("Introduzca la hora a la que se realizo el servicio");
 
        return;
 
    }
 

Y si esta condición es cierta corta la función por el return;

Edit: No había leído el post de David

[America|UNK]

Estás repitiendo estas 3 líneas:

Código javascript:

if(document.getElementById("hora").value=="00"||document.getElementById("hora").value==""){
 
        alert("Introduzca la hora a la que se realizo el servicio");
 
        return;
 
    }
 

Y si esta condición es cierta corta la función por el return;

[lukas4]

ok, creo que entendi tu punto (o talvez no), tons lo recomendable seria poner url como variable global o en todo caso enviarsela como parametro a la funcion, tu funcion me saco muchas dudas, segun yo deberia alertar: other, other y test al final :S

[lukas4]

Iniciado por America|UNK

Estás repitiendo estas 3 líneas:

Código javascript:

if(document.getElementById("hora").value=="00"||document.getElementById("hora").value==""){
 
        alert("Introduzca la hora a la que se realizo el servicio");
 
        return;
 
    }
 

Y si esta condición es cierta corta la función por el return;

cierto, solo que se me paso borrar esto, porke ya tenia el codigo de la funcion horas en la funcion showUser, cuando lo puse aki, copie todo tal y como lo tengo,pero se paso borrar estas lineas ;)