Skip to main content
3 de 4
se eliminaron 34 caracteres en el cuerpo
Dev. Joel
  • 23.9k
  • 3
  • 30
  • 47

En primer Lugar debe indicar la conexión de tipo orientado a objetos (Podría ser de tipo procedimiento) pero no es bueno mezclar los tipos.

$con= new mysqli("localhost", "my_user", "my_password", "midatabase");

En segundo lugar , debería emplear sentencias preparadas y evitar la concatenación de valores en sus querys ya que es un gran problema en cuanto a seguridad se refiere . Luego de esto no tendría por qué emplear mysqli_real_escape_string() ya que no sería necesario.

La query del update sería :

$query = "UPDATE tbl_employee SET name=?, address=?, gender= ?,
         designation = ?, age = ?  WHERE id= ? " ;

Para el Insert

$query = "INSERT INTO tbl_employee(name, address, gender, designation, age)  
           VALUES(?, ?, ?, ?, ?); "; 

Para luego dependiendo de la query que desea ejecutar, emplea el bindeo de los valores que llegan por POST , para la sintaxis revisar la documentación de bind_param(), esto después de preparar la sentencia, se coloca en un if porque a veces el prepare de la sentencia retorna false por algún error en la consulta.

if ($stmt = $con->prepare($query)) {
   $stmt->bind_param("sss", $one,$two,$three);
   $stmt->execute();
}

El reemplazo de if(mysqli_query($con, $query)) sería dependiendo de lo que quiere realizar, si fuese un insert simplemente podría validar el retorno de execute()

if($stmt->execute()){
    echo "Insert Exitoso";
}

Pero si es un Update , no podría con lo anterior ya que no siempre se validaría correctamente , para esto podría obtener el número de filas afectadas affected_rows

$stmt->execute();
if($con->affected_rows > 0){
   echo "Update Exitoso";
}

Para el ultimo select , podría hacerlo directamente

$query = "SELECT * FROM tbl_employee ORDER BY id DESC"
$result = $con->query($query);

while ($row = $result->fetch_assoc() ) {... }

Posible código Final

if(!empty($_POST))  
 {  
      $output = '';  
      $message = '';  
      $name = $_POST["name"];  
      $address =  $_POST["address"];  
      $gender = $_POST["gender"];  
      $designation = $_POST["designation"];  
      $age = $_POST["age"];  
      // Variable para saber que tipo de consulta se realizará
      $tipo = 0;
      // Variable para validar la correcta ejecución de la query, Insert o Update
      $estado = 0;  
      if($_POST["employee_id"] != '')   
      {  
        $query = "UPDATE tbl_employee SET name=?, address=?, gender= ?,designation = ?, age = ?  WHERE id= ? " ; 
        $tipo = 1; // seteamos para update
      }  
      else  
      {  
        $query = "INSERT INTO tbl_employee(name, address, gender, designation, age)  
           VALUES(?, ?, ?, ?, ?); ";   
        $tipo =2; //seteamos el tipo para insert
      }  
      // Sí será update 
      if($tipo === 1){
        if ($stmt = $con->prepare($query)) {
           $stmt->bind_param("sssssi", $name,$address,$gender,$designation,$age,$id);
           //Asignamos el valor de la ejecución del update
           $estado = $stmt->execute();
        }
      }
      //Sí sera Insert
      else if($tipo === 2){
        if ($stmt = $con->prepare($query)) {
           $stmt->bind_param("sssss", $name,$address,$gender,$designation,$age);
           $stmt->execute();
           //Asignamos el valor de la ejecución del insert , 
           //si el número afectado de filas es mayor a 0
           $estado = ($con->affected_rows>0);
        }
      }

      // Para saber si se ejecutó correctamete update o insert 
      if($estado){
        $query = "SELECT * FROM tbl_employee ORDER BY id DESC";
        $result = $con->query($query);
        while ($row = $result->fetch_assoc() ) {
            $output .= '  
               <tr>  
                  <td>' . $row["name"] . '</td>  
                  <td><input type="button" name="edit" value="Edit" id="'.$row["id"] .'" class="btn btn-info btn-xs edit_data" /></td>  
                  <td><input type="button" name="view" value="view" id="' . $row["id"] . '" class="btn btn-info btn-xs view_data" /></td>  
             </tr>  
                    '; 
        }
        echo $output;  
      }


      
 } 
Dev. Joel
  • 23.9k
  • 3
  • 30
  • 47