Функция удаления SQL не работает в таблице - PullRequest
Новая
       18

Функция удаления SQL не работает в таблице

0 голосов
/ 27 мая 2019

У меня есть три файла, соединяющихся друг с другом, моя проблема в том, что моя функция удаления не работает.Я хотел бы знать, чего не хватает в моей «публичной функции Delete_Lease ($ db) {», коды показаны ниже.По сути, я использую для каждого из них отображение таблицы, редактирование работает без проблем, но удаление просто остается на той же странице и не имеет никакого эффекта.

Имя таблицы: for_lease

Значения: аренды, тип аренды, имя аренды, адрес аренды, адрес аренды, цена аренды, условие аренды, описание аренды, Featured_photo, дата создания

table for_lease.php 

<?php
  session_start();
  require_once('for_lease.vc.php');
?>

 <?php foreach($lstProperty as $rowProperty) { ?>
       <tr align="center">
         <td>
          <a href="for_lease_edit.php?i=<?php echo($rowProperty['leaseid']); ?>"><input type="submit" class="btn bg-color-blue color-white form-control" name="edit" value="EDIT"></a>
         </td>

         <td>
           <?php
             echo($rowProperty['lease_name']);
           ?>
         </td>

         <td>
           <?php
             echo($rowProperty['lease_address']);
           ?>
         </td>

         <td>
           <?php
             echo($rowProperty['lease_type']);
           ?>
         </td>

         <td>
           <?php
             echo 'PHP'.' '.number_format(($rowProperty['lease_price']));
           ?>
         </td>

         <td>
           <?php
             echo($rowProperty['lease_condition']);
           ?>
         </td>

         <td>
           <?php
             echo( date("Y-m-d", strtotime($rowProperty['createddate']) ));
           ?>
         </td>

         <td>
           <a href="for_lease.php?delete=<?php echo($rowProperty['leaseid']); ?>" onclick="return confirm('Are you sure?');"><input type="submit" class="btn bg-color-red color-white form-control" name="delete" value="DELETE"></a>
         </td>
       </tr>
 <?php } ?>

for_lease.vc.php 

<?php
  $routePath = "../";

  require_once($routePath . "_config/db.php");
    $dbConfig = new config_db();
    $db = $dbConfig->init();

  require_once($routePath . "_mc/Property.mc.php");

  $mcProperty = new Property_MC();

  $lstProperty = $mcProperty->SelectObj_ByLeaseId($db);

  if (isset($_GET['delete'])){
    $rowProperty = $mcProperty->Delete_Lease($db);
  }
?>

Property.mc.php 

<?php
Class Property_MC {

  public function SelectObj_ByLeaseId($db) {
  $stmt = $db->prepare(
    " SELECT leaseid, lease_type, lease_name, lease_address, lease_price, lease_condition, lease_description, createddate
      FROM for_lease"
  );

  $stmt->execute();
  $row = $stmt->fetchAll(PDO::FETCH_ASSOC);

  return $row;
  }

  public function Delete_Lease($db) {
  $stmt = $db->prepare(
    " DELETE * FROM
        for_lease
      WHERE
        leaseid = :leaseid "
  );
  }

} ?>

Ответы [ 2 ]

0 голосов
/ 27 мая 2019

В Delete_Lease отсутствует привязка и выполнение, а также идентификатор аренды, который нужно удалить. SQL для удаления не нуждается в "*".

Property.mc.php 

public function Delete_Lease($db, $leaseid) {
    $stmt = $db->prepare(
        "DELETE FROM
        for_lease
        WHERE
        leaseid = :leaseid "
        );  
    $stmt->execute(['leaseid' => $leaseid]);
}

for_lease.vc.php 

if (isset($_GET['delete'])){
    $rowProperty = $mcProperty->Delete_Lease($db, $_GET['leaseid']);
}

исправлена ​​кнопка удаления, которая перезаписывает идентификатор удаления в for_lease.php 

<td>
    <a href="for_lease.php?leaseid=<?php echo($rowProperty['leaseid']); ?>" onclick="return confirm('Are you sure?');"><input type="submit" class="btn bg-color-red color-white form-control" name="delete" value="DELETE"></a>
</td>
0 голосов
/ 27 мая 2019

Вы не выполняете SQL-оператор, вы только его готовите.Кроме того, при настройке вашего кода вам нужно будет передать leaseid в вашу функцию удаления.

Попробуйте это.

for_lease.vc.php 

<?php
  $routePath = "../";

  require_once($routePath . "_config/db.php");
    $dbConfig = new config_db();
    $db = $dbConfig->init();

  require_once($routePath . "_mc/Property.mc.php");

  $mcProperty = new Property_MC();

  $lstProperty = $mcProperty->SelectObj_ByLeaseId($db);

  if (isset($_GET['delete'])){
    $rowProperty = $mcProperty->Delete_Lease($db, $_GET['delete']);
  }
?>

Property.mc.php 

 public function Delete_Lease($db, $leaseid) {
  $stmt = $db->prepare(
    " DELETE FROM
        for_lease
      WHERE
        leaseid = :leaseid "
  );

  $stmt->execute([':leaseid' => $leaseid]);  

  }
...