Error con $_POST

KATHYU · #1 (**permalink**) 27/07/2013, 13:54

Señores alguien sabe porq no funciona el insert que deseo hacer ?? creo que como no he descansado desde muy temprano el coco no me funca

Código PHP:

Ver original<?php
    include('c.php');
    //$mysqli = new mysqli("localhost","root","","kosys"); no funciona a si
    if($mysqli->connect_errno){echo "Fallo la conexion a la Base de datos !!! (Error: " .$mysqli->connect_errno. ")" ;}
        else{
            
            $n = '';
            $d = '';
            $r = '';
            $p = '';
            $c = '';
            $m = '';
            $dt = '';
            
            
            if ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            
            $ins = $mysqli->query("insert into productos (id,nombre,descripcion,rubro,proveedor,costo,
            pre_may,pre_det,activo,userid,fecha) values('','$n','$d','$r','$p','$c','$m','$dt','1','1')");
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar<br>";
            }
            }
        }
?>
<html>
    <head>
        <title>Mantenimiento</title>
    </head>
    
    <body>      
        <table width='900px' height='225px' align='left'>
      <tr>
        <td>
          <fieldset>
            <legend>
              <font color='#000000' style='font-family: Comic Sans MS;'>
                ..:: Agregar Nuevo Producto ::..
              </font>
            </legend>
            <br/>
            <form method='post' action='' name='admin' id='admin' >
              <table width='100%'>
                <tr>
                  <td>
                    <b>Nombre:
                    </b><input type='text' name='nombre' size='50' maxlenght='100' class='caja' style='padding: 0.3em;'/> <br>
                  </td>
                  <td>
                    <b>Descripcion:
                    </b><input type='text' name='desc' size='50' maxlenght='200' class='caja' style='padding: 0.3em;'/> <br>
                  </td>
                </tr>
                <tr>
                  <td>
                    <b>&nbsp;&nbsp;&nbsp;&nbsp;Rubro:
                        <input type='text' name='rubro' size='5' maxlenght='1' class='caja' style='padding: 0.3em;'/>
                    <b>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;Proveedor:
                        <input type='text' name='provee' size='5' maxlenght='1' class='caja' style='padding: 0.3em;'/>
                  </td>
                  <td>
                    <br/>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<b>Costo:</b>
                    <input type='text' name='costo' id='costo' size='5' onchange='precios()' maxlenght='10' class='caja' style='padding: 0.3em;'/>
                    <b>Mayoreo:</b>
                    <input type='text' name='mayoreo'   id='mayoreo' size='5' maxlenght='10' class='caja' style='padding: 0.3em;'/>
                    <b>Detalle:</b>
                    <input type='text' name='detalle' id='detalle' size='5' maxlenght='10' class='caja' style='padding: 0.3em;'/><br><br>
                  </td>
                </tr>
                <tr>
                  <td align='center'><br/>
                    <input type='submit' value='Agregar' name='registrar' style='padding:0.3em;'/>
                    <input type='reset' value='Limpiar' name='limpiar' style='padding:0.3em;'/>
                    <a href="index.php"><input type='button' value='Cancelar'  name='cancelar' style='padding:0.3em;'/></a><br><br>
                  </td>
                </tr>
              </table>
            </form>
          </fieldset>
        </td>
      </tr>
        </table>
    
        
    </body>
</html>

Algo por ahi no a de estar en su sitio, y si tengo mal ejecutado algo me pueden decir no hay problema !!!

Basicamente es un formulario con "post" y cuando se envian los datos debe generar el codigo php que esta al inicio, ya probe solo imprimiendo los datos del _POST y si salen todos

!!!!

Nemutagk · #2 (**permalink**) 27/07/2013, 14:14

Tienes que imprimir el error que envía mysqli para saber que esta mal...

Código PHP:

Ver original//tu código....
 
 $ins = $mysqli->query("insert into productos (id,nombre,descripcion,rubro,proveedor,costo,
            pre_may,pre_det,activo,userid,fecha) values('','$n','$d','$r','$p','$c','$m','$dt','1','1')");
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar".$mysqli->error."<br>";
            }
 
//mas código....

Por otro lado, desperdicias por completo mysqli al realizar las consultas como lo haces, ya que haces por completo inseguro tu sistema cuando mysqli implemente por default sistemas de segurida que evitar inyección SQL pero por la forma en que lo utilizas lo desperdicias, para eso existe el método prepare() y bind_param() para asignar variables externas a la sentencia SQL y evitar inyección SQL...

KATHYU · #3 (**permalink**) 27/07/2013, 17:06

Cita:

Iniciado por Nemutagk

Tienes que imprimir el error que envía mysqli para saber que esta mal...

Código PHP:

Ver original//tu código....
 
 $ins = $mysqli->query("insert into productos (id,nombre,descripcion,rubro,proveedor,costo,
            pre_may,pre_det,activo,userid,fecha) values('','$n','$d','$r','$p','$c','$m','$dt','1','1')");
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar".$mysqli->error."<br>";
            }
 
//mas código....

Por otro lado, desperdicias por completo mysqli al realizar las consultas como lo haces, ya que haces por completo inseguro tu sistema cuando mysqli implemente por default sistemas de segurida que evitar inyección SQL pero por la forma en que lo utilizas lo desperdicias, para eso existe el método prepare() y bind_param() para asignar variables externas a la sentencia SQL y evitar inyección SQL...

Gracias averigue el uso de mysqli y he cambiado la consulta asi:

Código PHP:

Ver originalif ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            
            $ins = $mysqli->prepare("insert into productos (nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det) values(?,?,?,?,?,?,?,?)");
            $ins->bind_param($n,$d,$r,$p,$c,$m,$dt);
            $ins->executed();
            $ins->close();
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar (".$mysqli->error.")<br>";
                
            }
            }

El resultado es:

Código:

Fatal error: Call to a member function bind_param() on a non-object in C:\xampp\htdocs\OCHOA\pruebas\mantto.php on line 26

Donde 26 es la linea $ins->bind_param($n,$d,$r,$p,$c,$m,$dt);

Luego intente tomar los $_POST de un solo asi:

Código PHP:

Ver originalif ($_POST){
                
                
            
            $ins = $mysqli->prepare("insert into productos (nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det) values(?,?,?,?,?,?,?,?)");
            $ins->bind_param(       $_POST['nombre'],
                                    $d = $_POST['desc'],
                                    $r = $_POST['rubro'],
                                    $p = $_POST['provee'],
                                    $c = $_POST['costo'],
                                    $m = $_POST['mayoreo'],
                                    $dt = $_POST['detalle']);
            $ins->executed();
            $ins->close();
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar (".$mysqli->error.")<br>";
                
            }
            }

El error es el mismo ahi donde dice bind_param da el error

KATHYU · #4 (**permalink**) 27/07/2013, 17:13

La cosa es que segun he leido esta ultima forma es usando clases y haciendo el metodo _construct pero eso yo no lo estoy implementando, sera que debo usar la forma que puse en el primer post o no importa ???

Nemutagk · #5 (**permalink**) 27/07/2013, 17:21

Así no se usa bind_param(), su uso correcto es así...

Código PHP:

Ver original$query = $mysqli->prepare('INSERT INTO miTabla VALUES (?,?,?)');
$query->bind_param('iss',1,'titulo','cuerpo_mensaje');
$query->execute();

Debes tener MUY presente el primer parámetro que se le pasa, el 'iss' este no es un parámetro cualquiera, le indica a mysqli de que tipo son las variables que se le pasan, la i es de int/entero, la s de strint/texto, y hay 1 por cada variable, así que si vas a insertar 7 variables del tipo string debes poner como primer parámetro 'sssss' como poner un ejemplo, por ultimo esto esta mal....

Código PHP:

Ver original$ins->bind_param(       $_POST['nombre'],
                                    $d = $_POST['desc'],
                                    $r = $_POST['rubro'],
                                    $p = $_POST['provee'],
                                    $c = $_POST['costo'],
                                    $m = $_POST['mayoreo'],
                                    $dt = $_POST['detalle']);

O usas la variable o usas el super array, no puedes asignar una valor a una variable dentro de una función, eso dará error...

Si tienes mas dudas visita el manual de PHP siempre que te atores! bind_param()

Actialización:

Cita:

Iniciado por KATHYU

La cosa es que segun he leido esta ultima forma es usando clases y haciendo el metodo _construct pero eso yo no lo estoy implementando, sera que debo usar la forma que puse en el primer post o no importa ???

Lo siento, lo lei ya que había enviado la respuesta, en efecto, es usando mysqli como objeto (clase), por lo tanto primero debes hacer un new mysql()...

Código PHP:

Ver original<?php
$mysqli = new mysqli('host','user','pass','db');
$query = $mysqli->prepare('INSERT INTO miTabla VALUES (?,?,?)');
$query->bind_param('iss',1,'titulo','cuerpo_mensaje');
$query->execute();

Aunque también puedes usar funciones por separado...

Código PHP:

Ver original$query = mysql_prepare('INSERT INTO miTabla VALUES (?,?,?)');
mysqli_stmt_bind_param($query,'iss',1,'titulo','cuerpo_mensaje');
mysqli_stmt_execute($query);

Pero en mi opinión es mejor usarlo como objeto (clase), tiene mas lógica y orden, pero solo es mi opinión...

Dalam · #6 (**permalink**) 27/07/2013, 17:38

en el primer ejemplo usas 11 campos que son
id,nombre,descripcion,rubro,proveedor,costo,pre_ma y,pre_det,activo,userid,fecha
y en los valores utilizas solo 10, me imagino que sera por el campo id que sera autoincrement. Si es asi y no lo tienes para que pueda ser nulo , en VALUES debes introducirlo como NULL o si lo tienes que pueda ser nulo quita el campo id.
Hay que tener cuidado con esas cosas, pues mysql crea un error por que no le concuerdan las columnas con los valores introducidos.

KATHYU · #7 (**permalink**) 27/07/2013, 17:52

Gracias, estudiare mas sobre bind_param()

Resolvi el insert asi aunq debere estudiar lo de mysqli_insert mejor:

Código PHP:

Ver originalif ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            
            $ins = $mysqli->prepare("insert into productos (nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det) values(?,?,?,?,?,?,?)");
            mysqli_stmt_bind_param($ins,'ssiiddd',$n,$d,$r,$p,$c,$m,$dt);
            mysqli_stmt_execute($ins);
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar (".$mysqli->error.")<br>";
                
            }
            }

Tambien resolvi de la otra forma, pero debe actualizarme con las nuevas cosas de PHP

Código PHP:

Ver originalif ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            
            $ins = $mysqli->query("insert into productos (id,nombre,descripcion,rubro,proveedor,costo,
            pre_may,pre_det) values('','$n','$d','$r','$p','$c','$m','$dt')");
            if ($ins){
                Echo "Registro Agregado con exito !!";
            }
            else{
                echo "Error al Agregar (".$mysqli->error.")<br>";
            }
            }

Muchas Gracias Nemutagk

Nemutagk · #8 (**permalink**) 27/07/2013, 17:55

Solo como dato, para tener mejor organizado el código decídete por una forma, o usas mysqli como objeto o como funciones, no combines porque pierde lógica y resulta mas difícil leer el código...

KATHYU · #9 (**permalink**) 27/07/2013, 17:55

Cita:

Iniciado por Dalam

en el primer ejemplo usas 11 campos que son
id,nombre,descripcion,rubro,proveedor,costo,pre_ma y,pre_det,activo,userid,fecha
y en los valores utilizas solo 10, me imagino que sera por el campo id que sera autoincrement. Si es asi y no lo tienes para que pueda ser nulo , en VALUES debes introducirlo como NULL o si lo tienes que pueda ser nulo quita el campo id.
Hay que tener cuidado con esas cosas, pues mysql crea un error por que no le concuerdan las columnas con los valores introducidos.

Gracias, creo que por ahi empezo todo, pero con eso aprendi mas !!!

@Nemutagk use funciones por separado para la ejecucion del insert

ya que no quiero meterme con las clases aun

KATHYU · #10 (**permalink**) 27/07/2013, 18:01

Cita:

Iniciado por Nemutagk

Solo como dato, para tener mejor organizado el código decídete por una forma, o usas mysqli como objeto o como funciones, no combines porque pierde lógica y resulta mas difícil leer el código...

Demonios eso siempre me da lata, no se cuando uso objetos y cuando funciones

Italico76 · #11 (**permalink**) 27/07/2013, 23:27

Cita:

Iniciado por KATHYU

Demonios eso siempre me da lata, no se cuando uso objetos y cuando funciones

Hola... cuando haces algo como accion() eso es una funcion y cuando haces algo como $var->accion() ó $this->accion() ó UnaClase::accion() es que estas usando un metodo de una clase

Pero un metodo es una funcion... solo que encapsulada dentro de una clase

PD: de pronto eso lo sabias... y preguntabas otra cosa, pero eso fue lo que entendi

KATHYU · #12 (**permalink**) 28/07/2013, 09:49

Naaa sabia que era dentro de una clase porq aca lo lei, lei algo del metod construct, debo actualizarme mas en esas cosas, algo que estoy haciendo todos los dias !!!!

KATHYU · #13 (**permalink**) 28/07/2013, 16:02

Buenas, sigo con otro detalle con lo del $_POST

Fijense que puse los id de cada producto que rescato al inicio como una url, para que al darle click pueda ser modificado !!!

Código PHP:

Ver originalwhile($row = $sql->fetch_array(MYSQLI_ASSOC)){
                        //echo $row['id'] ."->".$row['nombre']."->".$row['descripcion']."<br>";
                        
                        echo "<tr>
                                <td><a href='mantto.php'>".$row['id']."</a></td>
                                <td>".$row['nombre']."</td>
                                <td>".$row['descripcion']."</td>
                                <td>".$row['rubro']."</td>
                                <td>".$row['proveedor']."</td>
                                <td>".$row['costo']."</td>
                                <td>".$row['mayoreo']."</td>
                                <td>".$row['detalle']."</td>
                                <td>".$row['stock']."</td>
                            </tr>";
                    }
                    echo "</table>";

La cosa es que aunq ya lo hice de mil formas no logro hacer que se carguen los datos en el formulario de modificacion (que seria el mismo que utilise para ingresar uno nuevo) ya que no se como pasar el parametro de la url que ha sido clickeada !!

Código PHP:

Ver original<?php
    include('c.php');
    if($mysqli->connect_errno){echo "Fallo la conexion a la Base de datos !!! (Error: " .$mysqli->connect_errno. ")" ;}
        else{
            
            $n = '';
            $d = '';
            $r = '';
            $p = '';
            $c = '';
            $m = '';
            $dt = '';
 
            
            if ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            
            $ins = $mysqli->prepare("insert into productos (nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det) values(?,?,?,?,?,?,?)");
            mysqli_stmt_bind_param($ins,'ssiiddd',$n,$d,$r,$p,$c,$m,$dt);
            mysqli_stmt_execute($ins);
            if ($ins){
                Echo "Registro Agregado con exito !!<BR>";
            }
            else{
                echo "Error al Agregar (".$mysqli->error.")<br>";
                
            }
            }
            else{
            
$id = ???????? <== aca que iria para pasarlo como variable a la consulta sql ?????
 
            $sql = $mysqli->query("SELECT id,nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det FROM productos where id='$id'");
            while($row = $sql->fetch_array(MYSQLI_ASSOC)){
                echo $row['id'] ."->".$row['nombre']."->".$row['descripcion']."<br>";
                
                }
            }
}
?>

Ese codigo estaria bien si la url del id tuviese el nombre de id (para pasarla por GET o POST) pero no se como ponerle ese parametro, no se si me puedan ayudar, el POST no es necesario o si ???

PD: impimo con echo los datos solo como prueba ya que no los logre meter a los textbox.

Nemutagk · #14 (**permalink**) 28/07/2013, 16:26

Sin parecer ofensivo, pero tienes problemas básicos sobre PHP, te recomiendo que leas un buen manual/tutorial de PHP...

Para poder modificar el item seleccionado (al dar click en el enlace) es necesario pasar el ID de dicho item, por lo cual será necesario que lo pases por GET....

Código PHP:

Ver originalecho "<tr>
     <td><a href='mantto.php?id=".$row['id']."'>".$row['id']."</a></td>
     mas código.....

Para después capturar ese ID y poder hacer la modificación...

Código PHP:

Ver original$id = $_GET['id'];
//aunque lo mejor es
if (isset($_GET['id'])) {
     $id = $_GET['id'];
}else {
     //si no existe el ID regresar al listado de items...
     header('location: tu_listado_de_items.php');
     exit();
}
 
            $sql = $mysqli->query("SELECT id,nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det FROM productos where id='$id'");
            while($row = $sql->fetch_array(MYSQLI_ASSOC)){
                echo $row['id'] ."->".$row['nombre']."->".$row['descripcion']."<br>";
                
                }
            }
}

Cita:

Iniciado por KATHYU

...el POST no es necesario o si ???...

Si, es necesario porque si no, no sabrás que item se va a modificar (update....), así que tendrás que tener un input que aloje el ID del item que se esta modificando...

Y por ultimo, tu código de mysqli sigue siendo igual que al comienzo del post insertando directamente los valores en la sentencia SQL...

Italico76 · #15 (**permalink**) 28/07/2013, 17:46

Revisa tambien la forma en la que haces el manejo de errores..... control de flujo ... utiliza NULL cuando puedas..... etc

Código PHP:

  <?php 
    include('c.php'); 
    if($mysqli->connect_errno) 
    { 
        throw new Exception ("Fallo la conexion a la Base de datos !!! (Error: " .$mysqli->connect_errno. ")" ; 
    } 
             
                
    $n = isset($_POST['nombre']) ? $_POST['nombre'] : null; 
    $d = isset($_POST['desc']) ? $_POST['desc'] : null; 
    // .. etc

Creo esa parte quedo mucho mas compacta además ... comparada con...

Cita:

Iniciado por KATHYU

Código PHP:

Ver original<?php
    include('c.php');
    if($mysqli->connect_errno){echo "Fallo la conexion a la Base de datos !!! (Error: " .$mysqli->connect_errno. ")" ;}
        else{
            
            $n = '';
            $d = '';
            $r = '';
            $p = '';
            $c = '';
            $m = '';
            $dt = '';
 
            
            if ($_POST){
                $n = $_POST['nombre'];
                $d = $_POST['desc'];
                $r = $_POST['rubro'];
                $p = $_POST['provee'];
                $c = $_POST['costo'];
                $m = $_POST['mayoreo'];
                $dt = $_POST['detalle'];
                
            //...

KATHYU · #16 (**permalink**) 29/07/2013, 10:41

Cita:

Iniciado por Nemutagk

Sin parecer ofensivo, pero tienes problemas básicos sobre PHP, te recomiendo que leas un buen manual/tutorial de PHP...

Para poder modificar el item seleccionado (al dar click en el enlace) es necesario pasar el ID de dicho item, por lo cual será necesario que lo pases por GET....

Código PHP:

Ver originalecho "<tr>
     <td><a href='mantto.php?id=".$row['id']."'>".$row['id']."</a></td>
     mas código.....

Para después capturar ese ID y poder hacer la modificación...

Código PHP:

Ver original$id = $_GET['id'];
//aunque lo mejor es
if (isset($_GET['id'])) {
     $id = $_GET['id'];
}else {
     //si no existe el ID regresar al listado de items...
     header('location: tu_listado_de_items.php');
     exit();
}
 
            $sql = $mysqli->query("SELECT id,nombre,descripcion,rubro,proveedor,costo,pre_may,pre_det FROM productos where id='$id'");
            while($row = $sql->fetch_array(MYSQLI_ASSOC)){
                echo $row['id'] ."->".$row['nombre']."->".$row['descripcion']."<br>";
                
                }
            }
}

Si, es necesario porque si no, no sabrás que item se va a modificar (update....), así que tendrás que tener un input que aloje el ID del item que se esta modificando...

Y por ultimo, tu código de mysqli sigue siendo igual que al comienzo del post insertando directamente los valores en la sentencia SQL...

Pero eso es lo que habia hecho lo habia pasado por GET, y no funcionaba bien, es lo extraño he hecho muchas veces estas cosas sencillas y sin tener este tipo de problemas !!! en serio nose que es lo que pasa

otra vez creo lo que pasa es que mucho paso frente al codigo y mi cerebro se bloquea !!!

(¬_¬) lo que mas me da rabia es que lo dejaste igual que yo y hoy el mengico codigo ya funciona

@Italico76: Leere e implementare sobre lo que usted me aconseja !!!