Foro de elhacker.net

Buenas amigos, aquí les escribe este novato.

Les explico estoy practicando web y tengo el siguiente ejercicio que estoy haciendo:

Ingreso en un Form HTML las cantidades deseadas de X productos, al ingresar dicha cantidad inmediatamente me genera un input hidden en el cual se crea un identificador por cada elemento elegido.

Envío esto a un PHP que me va a recoger todos los datos del Form de los articulos y me va a hacer el update en mi BD cambiando las cantidades de los productos escogidos.

El problema es que obtengo un error de Sintaxis que no logro entender ni porque pasa ni como repararlo.

Aquí les traigo el código del lado del PHP.

 

 <?php 
    class extraerCantidad{
    	private $identificadorCantidad;
    	private $valorCantidad;
    	private $conex;
    	private $registros;
    	public function __construct(){
    		$this->conex=mysqli_connect('localhost', 'root', '', 'base1') or die("Error en la conexion");
    	}
    	public function ingresarCantidad($cant){
    		$this->valorCantidad=$cant;
    	}
    	public function ingresarDescripcion($desc){
    		$this->identificadorCantidad=$desc;
    	}
    	public function cambioCantidad() {
			$this->registros=mysqli_query($this->conex, "update articulos set cantidad=cantidad-$this->valorCantidad where descripcion='$this->identificadorCantidad'") or die(mysqli_error($this->conex));
    		echo "Cantidad Actualizada".'<br>';
    			mysqli_close($this->conex);
    		}
    	}
    	/* Recorro todos los envios POST que me llegan desde el Form HTML*/
  foreach ($_POST as $valorCampo) {
  	/*Ignoro cualquier campo en blanco para prevenir errores*/
if ($valorCampo!=="") {
	/*Separo de todos los _POST por medio de filtrado aquellos que sean los identificadores del articulo (o descripcion) y las cantidades*/
	$cantidad=filter_var($valorCampo,FILTER_SANITIZE_NUMBER_INT);
	$descripcion=filter_var($valorCampo,FILTER_SANITIZE_STRING);
	/*Procedo a Crear las Nuevas Ordenes*/
	$nuevaOrden=new extraerCantidad();
	/*Asigno cantidad Extraida*/
	if (is_numeric($cantidad)) {
		$nuevaOrden->ingresarCantidad($cantidad);
		/*Asigno Descripcion o Identificador*/
	}elseif (is_string($descripcion)) {
$nuevaOrden->ingresarDescripcion($descripcion);
/*Llamo a la funcion que hace el Query Update en mi BD*/
	}
	$nuevaOrden->cambioCantidad();
}
  }
	?>

Recibo este error (Cabe destacar que CocaCola puede ser cualquier otro identificador que reciba): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'where descripcion='CocaCola'' at line 1

Curiosamente, cuando ingreso el identificador de forma manual en la linea del query, me sigue arrojando el error, pero me hace el cambio de cantidad en el elemento deseado.

Saludos.

Reir para no llorar,

Cuando hago el Query para consultar X articulo definido por $this->identificadorArticulo

TODO salen bien, dios mio que error puedo tener entre un simple update y un select como para que el select si se haga bien y el update no.

Este fue el cambio que hice para probar y salio bello, 0 Errores (Cabe destacar que le deje el mismo nombre a la funcion, por si se confunden pues)    

public function cambioCantidad() {
			$this->registros=mysqli_query($this->conex, "select precio,descripcion,cantidad from articulos where descripcion='$this->identificadorCantidad'") or die(mysqli_error($this->conex));
    		if ($reg2=mysqli_fetch_array($this->registros)) {
    			echo $reg2['descripcion'].'<br>';
    			echo $reg2['precio'].'<br>';
    			echo $reg2['cantidad'].'<br>';
    			echo '<hr>';
    		}

Intenta así

"update articulos set cantidad=cantidad-{$this->valorCantidad} where descripcion='{$this->identificadorCantidad}'"

Saludos Diego!

Ante todo muchas gracias por la participación.

Te cuento, he ingresado el query como me lo has indicado y el error sigue, me he fijado que la hacer el FILTER_STRING, me sigue envia las cantidades como string.

Osea que no me separa las variables _POST que tienen cantidad de las variables _POST que tienen la parte de la descripcion.

¿ Sabrás el porque ?.

Osea aveces hago una funcion de prueba que imprime las variables $valorCampo "filtradas" que son numericas por un lado y string por otro y me arroja algo así:

11 CocaCola
22 Pepsi
33 BigCola

Lo ideal debería ser

1(Numero) CocaCola (String)
2(Numero) Pepsi (String)
3(Numero) BigCola (String)

Saludos Amigo.

El error proviene en tu if.. siempre que mandas a llamar al método cambioCantidad() se llama con algunos de los dos parámetros que necesitas en el query, ya sea cantidad o la descripción y nunca ambos (y necesitas ambos para tener un sentencia SQL correcta). Deberás corregir eso, no tengo la menor idea de cómo es tu formulario pero es mejor que uses los nombres de los indices tal cuál como vienen en POST.

PD: Quién demonios es Diego?

Jajajajaj perdón Jorge! Estaba pensando en un novio (?)

Muchas gracias de nuevo!, pues te cuento todo lo que has dicho con mucho gusto lo hice, no me daba resultado.

Comprobé cuantos POST estaba recibiendo en el array por medio de un print_r($_POST)

y vi que estaba enviando 2 veces las descripciones (Esto es por error en el lado HTML + Javasciript).

He quitado ese error que me clonaba la descripción, luego user un for each con una llave y a $this->descripcionblabla le asigné el valor (osea el name) del $_POST y en el $this->valorCantidad le envié el value de cada llave.

Y santo remedio, al momento de hacer la consulta por seguridad hice un if para verificar que ninguno de los 2 tanto descripción como cantidad llegue en blanco y allí procedí a hacer el Query.

Muchísimas gracias George! de verdad, a pesar de que las soluciones dadas por tí no fueron mi definitiva aprecio mucho tu capacidad de ayuda.

Saludos desde Venezuela!

Por si acaso, aquí está el codigo

<?php 
    class extraerCantidad{
    	private $identificadorCantidad;
    	private $valorCantidad;
    	private $conex;
    	private $registros;
    	public function __construct($desc,$cant){
    		$this->identificadorCantidad=$desc;
    		$this->valorCantidad=$cant;
    		$this->conex=mysqli_connect('localhost', 'root', '', 'base1') or die("Error en la conexion");
    		$this->cambioCantidad();
    	}
 
    	public function cambioCantidad() {
 
    		if ($this->valorCantidad!=="" and $this->identificadorCantidad!=="") {
    			$this->registros=mysqli_query($this->conex, "update articulos set cantidad=cantidad-$this->valorCantidad where descripcion='$this->identificadorCantidad'") 
    			or die("Error en la consulta");
    			mysqli_close($this->conex);
    		}
    	}
    }
    	foreach ($_POST as $descripcion => $cantidad) {
$nuevaOrden=new extraerCantidad($descripcion, $cantidad);
    	}
	?>