2

Me sale un error al querer generar una acción como modificar o guardar.

Error:

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'WHERE codigo = '1002'' at line 6

Código:

<?php
require_once 'producto.php';
require_once 'productoalta.php';

// Logica
$alm = new productos();
$model = new productosalta();

if(isset($_REQUEST['action']))
{
    switch($_REQUEST['action'])
    {
        case 'actualizar':
            $alm->__SET('codigo',              $_REQUEST['codigo']);
            $alm->__SET('producto',          $_REQUEST['producto']);
            $alm->__SET('descripcion',        $_REQUEST['descripcion']);
            $alm->__SET('precio',            $_REQUEST['precio']);


            $model->Actualizar($alm);
            header('Location: index.php');
            break;

        case 'registrar':
            $alm->__SET('producto',          $_REQUEST['producto']);
            $alm->__SET('descripcion',        $_REQUEST['descripcion']);
            $alm->__SET('precio',            $_REQUEST['precio']);


            $model->Registrar($alm);
            header('Location: index.php');
            break;

        case 'eliminar':
            $model->Eliminar($_REQUEST['codigo']);
            header('Location: index.php');
            break;

        case 'editar':
            $alm = $model->Obtener($_REQUEST['codigo']);
            break;
    }
}

?>

<!DOCTYPE html>
<html lang="es">
    <head>
        <title>si</title>
        <link rel="stylesheet" href="http://yui.yahooapis.com/pure/0.5.0/pure-min.css">
    </head>
    <body style="padding:15px;">

        <div class="pure-g">
            <div class="pure-u-1-12">

                <form action="?action=<?php echo $alm->codigo > 0 ? 'actualizar' : 'registrar'; ?>" method="post" class="pure-form pure-form-stacked" style="margin-bottom:30px;">
                    <input type="hidden" name="codigo" value="<?php echo $alm->__GET('codigo'); ?>" />

                    <table style="width:500px;">
                        <tr>
                            <th style="text-align:left;">producto</th>
                            <td><input type="text" name="producto" value="<?php echo $alm->__GET('producto'); ?>" style="width:100%;" /></td>
                        </tr>
                        <tr>
                            <th style="text-align:left;">descripcion</th>
                            <td><input type="text" name="descripcion" value="<?php echo $alm->__GET('descripcion'); ?>" style="width:100%;" /></td>
                        </tr>
                        <tr>
                            <th style="text-align:left;">precio</th>


                                    <td><input type="text" name="precio" value="<?php echo $alm->__GET('precio'); ?>" style="width:100%;" /></td>
                                </select>
                            </td>
                        </tr>

                        <tr>
                            <td colspan="2">
                                <button type="submit" class="pure-button pure-button-primary">Guardar</button>
                            </td>
                        </tr>
                    </table>
                </form>

                <table class="pure-table pure-table-horizontal">
                    <thead>
                        <tr>
                            <th style="text-align:left;">producto</th>
                            <th style="text-align:left;">descripcion</th>
                            <th style="text-align:left;">precio</th>
                            <th style="text-align:left;">Nacimiento</th>
                            <th></th>
                            <th></th>
                        </tr>
                    </thead>
                    <?php foreach($model->Listar() as $r): ?>
                        <tr>
                            <td><?php echo $r->__GET('producto'); ?></td>
                            <td><?php echo $r->__GET('descripcion'); ?></td>
                            <td><?php echo $r->__GET('precio') == 1 ? 'H' : 'F'; ?></td>

                            <td>
                                <a href="?action=editar&codigo=<?php echo $r->codigo; ?>">Editar</a>
                            </td>
                            <td>
                                <a href="?action=eliminar&codigo=<?php echo $r->codigo; ?>">Eliminar</a>
                            </td>
                        </tr>
                    <?php endforeach; ?>
                </table>     

            </div>
        </div>

    </body>
</html>

Código 2

<?php
class productosalta
{
    private $pdo;

    public function __CONSTRUCT()
    {
        try
        {
            $this->pdo = new PDO('mysql:host=localhost;dbname=carro_compras', 'root', '');
            $this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);                
        }
        catch(Exception $e)
        {
            die($e->getMessage());
        }
    }

    public function Listar()
    {
        try
        {
            $result = array();

            $stm = $this->pdo->prepare("SELECT * FROM productos");
            $stm->execute();

            foreach($stm->fetchAll(PDO::FETCH_OBJ) as $r)
            {
                $alm = new productos();

                $alm->__SET('codigo', $r->codigo);
                $alm->__SET('producto', $r->producto);
                $alm->__SET('descripcion', $r->descripcion);
                $alm->__SET('precio', $r->precio);


                $result[] = $alm;
            }

            return $result;
        }
        catch(Exception $e)
        {
            die($e->getMessage());
        }
    }

    public function Obtener($codigo)
    {
        try 
        {
            $stm = $this->pdo
                      ->prepare("SELECT * FROM productos WHERE codigo = ?");


            $stm->execute(array($codigo));
            $r = $stm->fetch(PDO::FETCH_OBJ);

            $alm = new productos();

            $alm->__SET('codigo', $r->codigo);
            $alm->__SET('producto', $r->producto);
            $alm->__SET('descripcion', $r->descripcion);
            $alm->__SET('precio', $r->precio);


            return $alm;
        } catch (Exception $e) 
        {
            die($e->getMessage());
        }
    }

    public function Eliminar($codigo)
    {
        try 
        {
            $stm = $this->pdo
                      ->prepare("DELETE FROM productos WHERE codigo = ?");                    

            $stm->execute(array($codigo));
        } catch (Exception $e) 
        {
            die($e->getMessage());
        }
    }

    public function Actualizar(productos $data)
    {
        try 
        {
            $sql = "UPDATE productos SET 
                        producto          = ?, 
                        descripcion        = ?,
                        precio            = ?, 

                    WHERE codigo = ?";

            $this->pdo->prepare($sql)
                 ->execute(
                array(
                    $data->__GET('producto'), 
                    $data->__GET('descripcion'), 
                    $data->__GET('precio'),

                    $data->__GET('codigo')
                    )
                );
        } catch (Exception $e) 
        {
            die($e->getMessage());
        }
    }

    public function Registrar(productos $data)
    {
        try 
        {
        $sql = "INSERT INTO productos (producto,descripcion,precio) 
                VALUES (?, ?, ?, ?)";

        $this->pdo->prepare($sql)
             ->execute(
            array(
                $data->__GET('producto'), 
                $data->__GET('descripcion'), 
                $data->__GET('precio'),

                )
            );
        } catch (Exception $e) 
        {
            die($e->getMessage());
        }
    }
}
?>

Código 3

<?php
class productos
{


    private $codigo;
    private $producto;
    private $descripcion;
    private $precio;


    public function __GET($k){ return $this->$k; }
    public function __SET($k, $v){ return $this->$k = $v; }
}
?>
1
  • No tengo idea de que framework o herramienta usas. El problema evidentemente es la construcción del query. ¿Puedes imprimir la consulta completa y mostrarla? Para poder validar que es lo que está fallando.
    – jrodriguez
    el 15 dic. 2016 a las 10:59

3 respuestas 3

3

Cuando está en el modo de emulación PDO::ATTR_EMULATE_PREPARES (cual está activado por defecto), PDO sustituye a los marcadores de posición (Placeholder) con los datos actuales.

Y con la vinculación perezosa (usando array en execute()), PDO trata cada parámetro como una cadena (string).

Como ejemplo, el límite preparado ?,? en la consulta se convierte en LIMIT '10', '10' que es una sintaxis no válida que hace que la consulta falle.

Hay dos soluciones:

  • Desactiva la emulación (MySQL puede ordenar todos los marcadores de posición (placeholder) correctamente).

  • Vincular el número explícitamente y establecer el tipo apropiado (PDO::PARAM_INT) para esa variable.

Para desactivar la emulación, se puede ejecutar este código (o establecer en una arreglo de opciones de conexión):

$conn->setAttribute( PDO::ATTR_EMULATE_PREPARES, false );

O para enlazar estas variables explícitamente con el tipo de parámetro:

$stm = $pdo->prepare('SELECT * FROM table LIMIT ?, ?');
$stm->bindParam(1, $limit_from, PDO::PARAM_INT);
$stm->bindParam(2, $per_page, PDO::PARAM_INT);
$stm->execute();
$data = $stm->fetchAll();

Fuente Original en Inglés

Solución en tu caso añade:

$this->pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
0
0

Tienes un error de syntax en la siguiente parte... introducir la descripción de la imagen aquí

public function Actualizar(productos $data)
    {
        try 
        {
            $sql = "UPDATE productos SET 
                        producto          = ?, 
                        descripcion        = ?,
                        precio            = ?, // AQUI!!! ESA COMA NO VA DENTRO DE LA CONSULTA.

                    WHERE codigo = ?";

            $this->pdo->prepare($sql)
                 ->execute(
                array(
                    $data->__GET('producto'), 
                    $data->__GET('descripcion'), 
                    $data->__GET('precio'),

                    $data->__GET('codigo')
                    )
                );
        } catch (Exception $e) 
        {
            die($e->getMessage());
        }
    }
0

muchas gracias por la observación, estuve presentando el mismo codigo de excepción y lo solucioné quitan esa coma después del where.Saludos,

1

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.