php - Sletning af row i table, vha. delete-button. FEJL. - Udvikleren.dk

Tags: php mysql

Bruger #17431 @ 06.11.12 20:35

Hejsa.

Jeg har lavet en side hvorpå jeg vil kunne slette eksisterende posts (en blog). Nedenstående kode skal lave et gitter, hvori den skal lave en linje for hver row den finder i tabellen 'posts'. Det virkede fint nok, indtil jeg kom til pointen med det. I mit loop, lavede jeg også en button, som når man trykker på den, skal slette den tilhørende row i tabellen.

Jeg har fået lidt hjælp, så der nu ikke meldes om nogen fejl - men det virker stadig ikke! Kan i se hvad der er galt?

PHP kode

 <?php
 session_start();
 if(!isset($_SESSION['user_id'])){
     header('location: index.php');
     exit();
 }
 
 ?>
 
 <!DOCTYPE html>
 <html lang="en">
 <head>
 <meta charset="utf-8"/>
 <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
 <style>
     #delete-post {
         width: 100%;
         margin: auto;
         background-color: #999;
     }
 </style>
 <script src="http://code.jquery.com/jquery-1.5.min.js"></script>
 </head>
 <body>
     <div>
 <?php
              
 // Connects to your Database
 include 'db_connect.php';
 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
 mysql_select_db("pentakill_dk_db") or die(mysql_error());
 $data = mysql_query("SELECT * FROM posts")
 or die(mysql_error());
 
 print "<table border cellpadding=3>";
 while($info = mysql_fetch_array($data)){ ?>
 
     <th>
     <form method="post" action="">
     <input type="hidden" name="post_id" value="<?$info['post_id']?>" />
     <input type="submit" name="delete_id" value="Delete" />
     </form>
     </th>
     <th>Post-Title:</th> <td><?$info['title']?></td>
     <th>Body:</th> <td><?$info['post_id']?></td></tr>
 
 <?php
 }
 
 print "</table>";
 
 if (isset($_POST['post_id'])) {
 
 $post_id = $_POST['post_id'];
 
 if (isset($_POST['delete_id']) === true) {
     $delete_id = mysql_query("DELETE FROM posts WHERE post_id = '$post_id'");
 }
 }
 ?>
     </div>
 </body>
 </html>

I Netbeans, mener den at $info ikke bliver brugt i denne linje: " <th>Body:</th> <td><?$info['post_id']?></td></tr>" og at $delete_id ikke bliver brugt i denne linje: " $delete_id = mysql_query("DELETE FROM posts WHERE post_id = '$post_id'");".

På forhånd - tak!

8 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 06.11.12 23:28

4.283

Jeg ville nok gøre så'n her:

PHP kode

 <?php
 // PHP LOGIK SKAL ALTID FØRST!
 session_start();
 if(!isset($_SESSION['user_id'])){
     header('location: index.php');
     exit();
 }
  
 // Connects to your Database
 include 'db_connect.php';
  
 // mysql driver udfaces og erstattes af mysqli = needs to be fixed!
 // 2 x db connect?
 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
 mysql_select_db("pentakill_dk_db") or die(mysql_error());
  
  
 if (isset($_POST['post_id'])) {
     $post_id = $_POST['post_id'];
      
     if (isset($_POST['delete_id']) === true) {
         $delete_id = mysql_query("DELETE FROM posts WHERE post_id = '$post_id'");
     }
 }
  
 $query = mysql_query("SELECT * FROM posts");
 while($row = mysql_fetch_assoc($query)) {
     $posts_data[] = $row;
 }
  
 ?>
 <!--  HTML SKAL ALTID SIDST! -->
 <!DOCTYPE html>
 <html lang="en">
     <head>
         <meta charset="utf-8"/>
         <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
         <style>
          #delete-post {
              width: 100%;
              margin: auto;
              background-color: #999;
          }
         </style>
         <script src="http://code.jquery.com/jquery-1.5.min.js"></script>
     </head>
     <body>
         <div>
             <table>
                 <?php foreach($posts_data as $info): // php kan også bruges som template sprog ?>
                     <tr>
                         <th>
                             <form method="post" action="">
                                 <input type="hidden" name="post_id" value="<?$info['post_id']?>" />
                                 <input type="submit" name="delete_id" value="Delete" />
                             </form>
                         </th>
                         <th>Post-Title:</th> <td><?$info['title']?></td>
                         <th>Body:</th> <td><?$info['post_id']?></td>
                     </tr>
                 <?php endforeach; ?>
             </table>
         </div>
     </body>
 </html>

Bruger #16298 @ 06.11.12 21:02

Så vidt jeg kan se mangler du at bruge "echo" og ";", altså:

PHP kode

 38.     <th>
 39.     <form method="post" action="">
 40.     <input type="hidden" name="post_id" value="<? echo " . $info['post_id'] . "; ?>" />
 41.     <input type="submit" name="delete_id" value="Delete" />
 42.     </form>
 43.     </th>
 44.     <th>Post-Title:</th> <td><?$ echo " . $info['title'] . "; ?></td>
 45.     <th>Body:</th> <td><? echo " . $info['post_id'] . "; ?></td></tr>

Du har også et "="-tegn for meget i linje 56

PHP kode

 if  (isset($_POST['delete_id']) === true)

Indlæg senest redigeret d. 06.11.2012 21:03 af Bruger #16298

Bruger #17431 @ 06.11.12 21:17

Så vidt jeg kan se mangler du at bruge "echo" og ";", altså:

PHP kode

 38.     <th>
 39.     <form method="post" action="">
 40.     <input type="hidden" name="post_id" value="<? echo " . $info['post_id'] . "; ?>" />
 41.     <input type="submit" name="delete_id" value="Delete" />
 42.     </form>
 43.     </th>
 44.     <th>Post-Title:</th> <td><?$ echo " . $info['title'] . "; ?></td>
 45.     <th>Body:</th> <td><? echo " . $info['post_id'] . "; ?></td></tr>

Du har også et "="-tegn for meget i linje 56

PHP kode

 if  (isset($_POST['delete_id']) === true)

Det hjælper desværre ikke. Der kom røde streger under 40, 44 og 45 (i ovenstående eksempel).
Desuden kommer der til venstre for deleteknappen til at stå " /> i alle de rows den spytter ud. :-/

Bruger #10878 @ 06.11.12 23:17

Der manglede en '<tr>', og 2 '"' i <table border cellpadding=3>

PHP kode

 <?php
     session_start();
     if(!isset($_SESSION['user_id'])){
         header('location: index.php');
         exit();
     }
     if (isset($_POST['post_id'])) {
         $post_id = $_POST['post_id'];
         if (isset($_POST['delete_id'])) {
             mysql_query("DELETE FROM posts WHERE post_id = '$post_id'");
         }
     }
 ?>
 <!DOCTYPE html>
 <html lang="en">
     <head>
         <meta charset="utf-8"/>
         <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
         <style>
             #delete-post {
                 width: 100%;
                 margin: auto;
                 background-color: #999;
             }
         </style>
         <script src="http://code.jquery.com/jquery-1.5.min.js"></script>
     </head>
     <body>
         <div>
             <table border cellpadding="3">
                 <?php
                 // Connects to your Database
                 include('db_connect.php');
                 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
                 mysql_select_db("pentakill_dk_db") or die(mysql_error());
                 $data = mysql_query("SELECT * FROM posts") or die(mysql_error());
                 while($info = mysql_fetch_array($data)){ ?>
                     <tr>
                         <th>
                             <form method="post" action="">
                                 <input type="hidden" name="post_id" value="<?php echo $info['post_id']; ?>" />
                                 <input type="submit" name="delete_id" value="Delete" />
                             </form>
                         </th>
                         <th>
                             Post-Title:
                         </th>
                         <td>
                             <?php echo $info['title']; ?>
                         </td>
                         <th>
                             Body:
                         </th>
                         <td>
                             <?php echo $info['post_id']; ?>
                         </td>
                    </tr>
                 <?php }    ?>
             </table>
         </div>
     </body>
 </html>

Indlæg senest redigeret d. 06.11.2012 23:26 af Bruger #10878

Bruger #17431 @ 07.11.12 10:13

Der manglede en '<tr>', og 2 '"' i <table border cellpadding=3>

PHP kode

 <?php
     session_start();
     if(!isset($_SESSION['user_id'])){
         header('location: index.php');
         exit();
     }
     if (isset($_POST['post_id'])) {
         $post_id = $_POST['post_id'];
         if (isset($_POST['delete_id'])) {
             mysql_query("DELETE FROM posts WHERE post_id = '$post_id'");
         }
     }
 ?>
 <!DOCTYPE html>
 <html lang="en">
     <head>
         <meta charset="utf-8"/>
         <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
         <style>
             #delete-post {
                 width: 100%;
                 margin: auto;
                 background-color: #999;
             }
         </style>
         <script src="http://code.jquery.com/jquery-1.5.min.js"></script>
     </head>
     <body>
         <div>
             <table border cellpadding="3">
                 <?php
                 // Connects to your Database
                 include('db_connect.php');
                 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
                 mysql_select_db("pentakill_dk_db") or die(mysql_error());
                 $data = mysql_query("SELECT * FROM posts") or die(mysql_error());
                 while($info = mysql_fetch_array($data)){ ?>
                     <tr>
                         <th>
                             <form method="post" action="">
                                 <input type="hidden" name="post_id" value="<?php echo $info['post_id']; ?>" />
                                 <input type="submit" name="delete_id" value="Delete" />
                             </form>
                         </th>
                         <th>
                             Post-Title:
                         </th>
                         <td>
                             <?php echo $info['title']; ?>
                         </td>
                         <th>
                             Body:
                         </th>
                         <td>
                             <?php echo $info['post_id']; ?>
                         </td>
                    </tr>
                 <?php }    ?>
             </table>
         </div>
     </body>
 </html>

Jeg havde haft problemer med at få den til at vise post_id og title, men det virker nu igen med din kode. Desværre virker delete-knappen stadig ikke. Men før, hvor der bare ikke skete noget, kan man nu se at den lige tænker, og så går ud. Fik lige en besked fra min bror, som sagde at jeg skulle kigge på hvornår jeg brugte " og ' - da den godt kan misforstå de tegn og tro at det er slutningen på en string ( eller noget i den dur)

Ps. Netbeans melder ikke om nogle ubrugte variabler, så det er sikkert bare en lille ændring det kræver.

Bruger #10878 @ 07.11.12 21:45

Prøv at flytte

PHP kode

 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
 mysql_select_db("pentakill_dk_db") or die(mysql_error());

op under

PHP kode

 session_start();

Det bedste er jo at oprette forbindelse til databasen. Før man prøver at ændre i databasen.

Til sidst kan du fjerne

PHP kode

 include('db_connect.php');

ellers opretter du forbindelse til databasen 2 gange.

Bruger #17431 @ 07.11.12 21:59

Prøv at flytte
PHP kode
 mysql_connect("localhost", "root", "hejmeddig") or die(mysql_error());
 mysql_select_db("pentakill_dk_db") or die(mysql_error());
op under
PHP kode
 session_start();
Det bedste er jo at oprette forbindelse til databasen. Før man prøver at ændre i databasen.

Til sidst kan du fjerne
PHP kode
 include('db_connect.php');
ellers opretter du forbindelse til databasen 2 gange.

TAAAAAK. Det virkede!!! :-) Årh man, endelig! Jeg takker meget.

Bruger #10878 @ 07.11.12 22:12

Også er det tid til at du kigger på Michael Larsens kode. Det sådan php bruges bedst

Sletning af row i table, vha. delete-button. FEJL.

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler