ajax no envia datos con el metodo GET

[summerblack]

Hola chicos, si alguien me puede ayudar con mi problema, no se porque no me envia datos por medio del metodo get:
tengo el seguiente select que recibe la informacion:

Código HTML:

<label for="lblproducto">Seleccione tipo producto: </label>
<select  class="productos" id="productox" name="productox" >
            <option value="I1-x">I1-x</option>          
            <option value="I5-y">I5-y</option>
            <option value="I6-z">I6-z</option>
        </select>
<input  name="cantidad" type="text" id="cantidad" placeholder="Ingrese la cantidad" />  <br>
 

y los envio por medio de un metodo ajax

Código Javascript:

function agregar(){ 
 //select
 divcarrito = document.getElementById('divcarrito');
 cadena = document.getElementById('productox'); 
 cadenaProducto1=document.frmconsulta.productox.value;
 cantidad=document.frmconsulta.cantidad.value;
 producto = cadenaProducto1.split("-");
 item1 = producto[0];
 nombreItem1 = producto[1];
 ajax=Ajax();
 url="carrito.php?item1="+item1+"&nombreItem1="+nombreItem1+"&cantidad="+cantidad;
  alert(url);
 if(ajax.open("GET", url, true)) alert("Si entro"); else alert("No entro");
 ajax.onreadystatechange=function() {
  if (ajax.readyState==4 && ajax.status !== 404 ) {
    
   divcarrito.innerHTML = ajax.responseText;  
   alert (ajax.responseText);
  }
 } 
 ajax.send(null); 
}
 

y tengo la funcion de agregar en php

Código PHP:

Productos agregados:
<?php
 
 
class Carrito{
function carrito(){
}
 
function agregar($item1,$nombreItem1,$cantidad){
    echo 'Nombre Producto '.$nombreItem1;
    echo 'Cantidad '.$cantidad;
    echo 'agregado al carrito :) ';
}
    
}
/*$item1=$_GET['item1'];
$nombreItem1=$_GET['nombreItem1'];
$cantidad=$_GET['cantidad'];  
*/
 
if(isset($_GET["item1"])&&isset($_GET["nombreItem1"])&&isset($_GET["cantidad"])){
$p=new Carrito;
$item1=$_GET["item1"];
$nombreItem1=$_GET["nombreItem1"]; 
$cantidad=$_GET["cantidad"];
$p->agregar($item1,$nombreItem1,$cantidad);
}
?>
 

el problema es que en chrome y en FF alert (ajax.readyState) bota 0, y en IE bota 200 y todos los exploradores me botan alert("No entro"); y por ende no me muestra el contenido en el <divcarrito>, gracias a quien me pueda ayudar

[eamanesims]

es un poco lioso como llamas a ajax ¿por que no haces?:

Código:

cantidad=document.frmconsulta.cantidad.value;
 item1 = producto[0];
 nombreItem1 = producto[1];
divcarrito=$("#divcarrito").val();

var parametros={
			"item1" : item1,
			"cantidad" : cantidad,
                        "nombreItem1" :nombreItem1
		}
	
	$.ajax({
			url : 'carrito.php',
			type : 'get',
			data : parametros,
			success : function(response)
			{
				$('#' + divcarrito).html(response);
			},
                        error:function (response)
			{
			  alert("error");
			}
error
		});

no entiendo porqué compones la URL, si creas un array de parámetros con decirle que quieres el método get ya te los envía como tal y otra cosa ¿dónde llamas a la función agregar() de Javascript ?

[jall.vigo]

1.- se supone que el nombre del archivo php es carrito.php
2.- la primera línea del archivo php sobra, tal como está nunca se va a ejecutar correctamente.

espero que te haya servido

[eamanesims]

de todas formas esta mal:

divcarrito = document.getElementById('divcarrito')

le falta el .value-->divcarrito = document.getElementById('divcarrito').value asi nunca te va a coger el valor sino un objeto

[summerblack]

Ya lo solucione lo que hice fue cambiar

Código HTML:

<input  type="submit" name="input" class="boton" value="agregar" onClick="agregar()"/> <br/>
 

por

Código HTML:

<a   href="javascript:agregar()"><img src="img/imagenpedido.jpg"></img></a>
 

aunque no entiendo porque el submit hace que la pagina se me refresque :(

[summerblack]

Bueno chicos muchas gracias , aun sigo aprendiendo voy a ver como me va plicando los aportes

[maycolalvarez]

Aclaraciones:

Para comprobar si la petición fue exitosa, se evalúan ajax.readyState==4 && ajax.status == 200, 404 es un error, pero no el único, en cambio el éxito siempre se da si se cumplen ambas condiciones

Recomiendo usar POST para añadir / eliminar del carrito, recuerden que GET cachea la respuesta, y si el usuario quiere volver a agregar otro producto al carro, al estar chacheada la última petición no se enviará, generando defases en el carrito, además es válido porque a fin de cuentas es una operación de modificación sobre el carrito.

[eamanesims]

La verdad que no entiendo muy bien lo que quieres hacer... no entiendo por qué haces submit si lo que quieres es agregarlo a través de ajax :S

Siempre que hagas submit de un formulario te va a mandar al .php que tengas en el 'action' a no se que hagas un target a un iframe oculto por ej.

y tampoco entiendo esto:

Código HTML:

<input  type="submit" name="input" class="boton" value="agregar" onClick="agregar()"/> 

si lo que quieres es llamar a la funcion javascript 'agregar' deberias ponerlo tipo 'button' no submit, si le pones submit obvio te envía el formulario...

[maycolalvarez]

Iniciado por eamanesims

La verdad que no entiendo muy bien lo que quieres hacer... no entiendo por qué haces submit si lo que quieres es agregarlo a través de ajax :S

Siempre que hagas submit de un formulario te va a mandar al .php que tengas en el 'action' a no se que hagas un target a un iframe oculto por ej.

y tampoco entiendo esto:

Código HTML:

<input  type="submit" name="input" class="boton" value="agregar" onClick="agregar()"/> 

si lo que quieres es llamar a la funcion javascript 'agregar' deberias ponerlo tipo 'button' no submit, si le pones submit obvio te envía el formulario...

cierto, pero también sirve devolver false en el evento onsubmit del form

[summerblack]

aah claro ya entiendo porque con submit me enviaba todo lo del formulario, la vdd no lo tenia claro, muchas gracias!!

[emprear]

Iniciado por summerblack

aah claro ya entiendo porque con submit me enviaba todo lo del formulario, la vdd no lo tenia claro, muchas gracias!!

Asi aún debes usar el submit, esa es la forma correcta, ya que si el usuario deshabilita javascript el form sigue siendo funcional.
Aqui te dejo una forma de como hacerlo

Código HTML:

<!DOCTYPE html>
<html lang="es-ar">
<head>
<meta charset="utf-8" />
<title>Loguin</title>
 
<script type="text/javascript">
//<![CDATA[
//var Datos;
function valida(){
var us =document.getElementById('usuario');
var cl = document.getElementById('clave');
if((limpiar(us.value).length === 0)||(limpiar(cl.value).length === 0)){
alert('complete ambos campos');
return false; // hasta aqui si no completó los campos no pasa nada ya que devuelve false al evento onsubmit que llama la función con return
}else{
// ejecutamos ajax
var xmlHttp=null;
if (window.ActiveXObject){ 
xmlHttp = new ActiveXObject("Microsoft.XMLHTTP");
}else{ 
if (window.XMLHttpRequest){
xmlHttp = new XMLHttpRequest();
}
} // creaste el objeto
 
xmlHttp.onreadystatechange=function(){ // ejecutás el código
  if (xmlHttp.readyState==4 && xmlHttp.status==200){
 
if(xmlHttp.responseText != 'ok'){
alert('nombre o clave incorrectos');
}else{
document.getElementById('loguin').style.display = "none";
document.getElementById('mensaje').innerHTML = "Estás logueado como " + us.value;
  }
}
  }
xmlHttp.open("POST",'entrar.php',true);
xmlHttp.setRequestHeader("Content-Type", "application/x-www-form-urlencoded");
xmlHttp.send("usuario="+us.value+"&clave="+cl.value);
return false; // siempre devuelve flase para prevenir el submit
 
}
}
// función adicional para prevenir que te pasen caracteres en blanco
function limpiar(valor){
var valor_campo = valor.replace(/^\s+/g,'').replace(/\s+$/g,'');
return valor_campo;
}
//]]>
</script>
</head>
<body>
<p>Combinación válida = usuario:emprear, clave:fdw </p>
<form action="entrar.php" onsubmit="return valida();" id="loguin" method="post">
usuario: <input type="text" name="usuario" id="usuario" /><br />
passwd: <input type="password" name="clave" id="clave" /><br />
<input type="submit" value="ingresar" />
</form>
<p id="mensaje">
    <!-- mensaje loguin -->
</p>
</body>
</html>
 

un php sencillo para probarlo

entrar.php

Código:

<?php
$u = $_POST['usuario'];
$p = $_POST['clave'];
 
if(($u == "emprear") && ($p == "fdw")){
echo "ok";
exit;
}else{
echo "usuario y/o clave no validas";
}
 
?>

Demo(ejecutar con y sin javascript activado)
http://foros.emprear.com/ajax/loguin/loguin.html

Es un poco lo que te señala @maycolAlvarez

Saludos

[summerblack]

muchas gracias, precisamente tambien necesitaba un login, asi que voy a probar este muchas gracias de nuevo

[eamanesims]

Iniciado por maycolalvarez

cierto, pero también sirve devolver false en el evento onsubmit del form

Si cierto, así también se puede ^^