Skip to main content
se añadieron 543 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Aparte de que puedes hacer el INSERT basándote en el mismo SELECT con algo como:

INSERT INTO ... VALUES(SELECT ... FROM ...)

En tu código se aprecian errores, redundancias y prácticas no recomendadas:

  1. El Manual de PHP es claro con respecto a rowCount(), te dice que no deberías confiar en él para consultas del tipo SELECT. Si quieres saber si este SELECT trae datos simplemente verifica el estado de $results.

Si la última sentencia SQL ejecutada por el objeto PDOStatement asociado fue una sentencia SELECT, algunas bases de datos podrían devolver el número de filas devuelto por dicha sentencia. Sin embargo, este comportamiento no está garantizado para todas las bases de datos y no debería confiarse en él para aplicaciones portables.

  1. Este bucle tiene un error: foreach($results as $) {, no indicas nombre para la variable del as. Además, ¿por qué usas un bucle cuando dices que esperas una sola fila? Si esperas una sola fila tampoco deberías usar fetchAll. Simplemente usa fetch y lee los resultados directamente, sin bucle, dado que esperas una sola fila.

El código debería funcionar así:

$stmt#Variable =de $bdd->preparecontrol
$outPut=array("SELECT);
if($bdd) {
    $sql="SELECT * FROM pl WHERE usuario = :usuario AND id = :id LIMIT 1"1";
    if( $stmt = $bdd->prepare($sql); ) {
        $stmt->bindParam(':usuario', $usuario);
        $stmt->bindParam(':id',$id);    
        $stmt -> execute(); 
        $results = $stmt -> fetch(PDO::FETCH_OBJ);    
        if($results)   {  
            $tipo = $results->tipo;
            $info= $results->info;
            $date = $results->date;
            #Hacer el INSERT o lo que quieras
    echo "INSERT";       $outPut['msg']="Insertaremos los datos";
        } else {
    echo "ERROR";       $outPut['error']="Cero filas encontradas";
        }
    } else {
        #Cambiarlo por error personalizado
        $outPut['error']=sprintf("Error SQL: %s",$bdd->errorInfo()[2]);
    }
} else {
    $outPut['error']='No hay conexión';
}
#Test, hacer lo que necesites con $outPut
var_dump($outPut);

Aparte de que puedes hacer el INSERT basándote en el mismo SELECT con algo como:

INSERT INTO ... VALUES(SELECT ... FROM ...)

En tu código se aprecian errores, redundancias y prácticas no recomendadas:

  1. El Manual de PHP es claro con respecto a rowCount(), te dice que no deberías confiar en él para consultas del tipo SELECT. Si quieres saber si este SELECT trae datos simplemente verifica el estado de $results.

Si la última sentencia SQL ejecutada por el objeto PDOStatement asociado fue una sentencia SELECT, algunas bases de datos podrían devolver el número de filas devuelto por dicha sentencia. Sin embargo, este comportamiento no está garantizado para todas las bases de datos y no debería confiarse en él para aplicaciones portables.

  1. Este bucle tiene un error: foreach($results as $) {, no indicas nombre para la variable del as. Además, ¿por qué usas un bucle cuando dices que esperas una sola fila? Si esperas una sola fila tampoco deberías usar fetchAll. Simplemente usa fetch y lee los resultados directamente, sin bucle, dado que esperas una sola fila.

El código debería funcionar así:

$stmt = $bdd->prepare("SELECT * FROM pl WHERE usuario = :usuario AND id = :id LIMIT 1");
$stmt->bindParam(':usuario', $usuario);
$stmt->bindParam(':id',$id);    
$stmt -> execute(); 
$results = $stmt -> fetch(PDO::FETCH_OBJ);    
if($results)   {  
    $tipo = $results->tipo;
    $info= $results->info;
    $date = $results->date;
    #Hacer el INSERT o lo que quieras
    echo "INSERT";
} else {
    echo "ERROR";
}

Aparte de que puedes hacer el INSERT basándote en el mismo SELECT con algo como:

INSERT INTO ... VALUES(SELECT ... FROM ...)

En tu código se aprecian errores, redundancias y prácticas no recomendadas:

  1. El Manual de PHP es claro con respecto a rowCount(), te dice que no deberías confiar en él para consultas del tipo SELECT. Si quieres saber si este SELECT trae datos simplemente verifica el estado de $results.

Si la última sentencia SQL ejecutada por el objeto PDOStatement asociado fue una sentencia SELECT, algunas bases de datos podrían devolver el número de filas devuelto por dicha sentencia. Sin embargo, este comportamiento no está garantizado para todas las bases de datos y no debería confiarse en él para aplicaciones portables.

  1. Este bucle tiene un error: foreach($results as $) {, no indicas nombre para la variable del as. Además, ¿por qué usas un bucle cuando dices que esperas una sola fila? Si esperas una sola fila tampoco deberías usar fetchAll. Simplemente usa fetch y lee los resultados directamente, sin bucle, dado que esperas una sola fila.

El código debería funcionar así:

#Variable de control
$outPut=array();
if($bdd) {
    $sql="SELECT * FROM pl WHERE usuario = :usuario AND id = :id LIMIT 1";
    if( $stmt = $bdd->prepare($sql) ) {
        $stmt->bindParam(':usuario', $usuario);
        $stmt->bindParam(':id',$id);    
        $stmt -> execute(); 
        $results = $stmt -> fetch(PDO::FETCH_OBJ);    
        if($results)   {  
            $tipo = $results->tipo;
            $info= $results->info;
            $date = $results->date;
            #Hacer el INSERT o lo que quieras
            $outPut['msg']="Insertaremos los datos";
        } else {
            $outPut['error']="Cero filas encontradas";
        }
    } else {
        #Cambiarlo por error personalizado
        $outPut['error']=sprintf("Error SQL: %s",$bdd->errorInfo()[2]);
    }
} else {
    $outPut['error']='No hay conexión';
}
#Test, hacer lo que necesites con $outPut
var_dump($outPut);
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Aparte de que puedes hacer el INSERT basándote en el mismo SELECT con algo como:

INSERT INTO ... VALUES(SELECT ... FROM ...)

En tu código se aprecian errores, redundancias y prácticas no recomendadas:

  1. El Manual de PHP es claro con respecto a rowCount(), te dice que no deberías confiar en él para consultas del tipo SELECT. Si quieres saber si este SELECT trae datos simplemente verifica el estado de $results.

Si la última sentencia SQL ejecutada por el objeto PDOStatement asociado fue una sentencia SELECT, algunas bases de datos podrían devolver el número de filas devuelto por dicha sentencia. Sin embargo, este comportamiento no está garantizado para todas las bases de datos y no debería confiarse en él para aplicaciones portables.

  1. Este bucle tiene un error: foreach($results as $) {, no indicas nombre para la variable del as. Además, ¿por qué usas un bucle cuando dices que esperas una sola fila? Si esperas una sola fila tampoco deberías usar fetchAll. Simplemente usa fetch y lee los resultados directamente, sin bucle, dado que esperas una sola fila.

El código debería funcionar así:

$stmt = $bdd->prepare("SELECT * FROM pl WHERE usuario = :usuario AND id = :id LIMIT 1");
$stmt->bindParam(':usuario', $usuario);
$stmt->bindParam(':id',$id);    
$stmt -> execute(); 
$results = $stmt -> fetch(PDO::FETCH_OBJ);    
if($results)   {  
    $tipo = $results->tipo;
    $info= $results->info;
    $date = $results->date;
    #Hacer el INSERT o lo que quieras
    echo "INSERT";
} else {
    echo "ERROR";
}