php - Den vil ikke opdatere? - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php mysql

Bruger #15305 @ 22.07.11 12:56

Nu har jeg prøvet så mange forskellige ting, at jeg endnu engang går til jer kloge hoveder herinde.

En ting er at når jeg, som i koden her, har den til at opdatere på samme side som formularen ($_SERVER['PHP_SELF']) går den direkte til forsiden, men når jeg har skrevet det direkte link til siden i "action" feltet, bliver den, men vil ikke opdatere.. Er der nogen der kan hjælpe?

PHP kode

 <?php
 include("connect.php");
 
     if(isset($_GET['id'])) {
     $id = $_GET['id'];
 
     if(isset($_SESSION["logged_in"])) 
     {
         $get = mysql_query("SELECT * from bruger WHERE id='$_SESSION[user_id]'");
         $show = mysql_fetch_array($get);
         
             if($show["rank"] == 5)
             {
                     $hent = mysql_query("SELECT * FROM nyheder WHERE id='$id'");
                     $vis = mysql_fetch_array($hent);
                 $newsid = $vis["id"];
 ?>
 <html><body>
 <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="POST">
 <input type="hidden" name="id" value="<?php echo "$vis[id]"; ?>">
 <input type="text" name="titel" value="<?php echo "$vis[titel]"; ?>"><br>
 Emne:<br>
 <select name="emne" size="1" value="<?php echo "$vis[emne]"; ?>">
     <option value="images/esbjerg.jpg">Esbjerg</option>
     <option value="images/skate.jpg">Skate</option>
     <option value="images/andet.jpg">Andet</option>
 </select><br>
 Nyheden:<br>
 <textarea name="nyhed" rows="20" cols="50"><?php echo "$vis[nyhed]"; ?></textarea><br>
 <input type="submit" name="submit" value="Opdater">
 </body></html>
 <?php
 
                             if(isset($_POST['submit']))
                             {
                             $titel = $_POST["titel"];
                             $emne = $_POST["emne"];
                             $nyhed = $_POST["nyhed"];
     
     
                             $query = "UPDATE nyhed SET titel='$titel', emne='$emne', nyhed='$nyhed' WHERE id = '$newsid'";
                             mysql_query($query) or die("Kan ikke opdatere");
                         }
                     }
                 }
             } else {
             echo "Du har ikke adgang til denne side!";
         }
 ?>

9 svar postet i denne tråd vises herunder
5 indlæg har modtaget i alt 9 karma

Sorter efter stemmer Sorter efter dato

Bruger #16025 @ 22.07.11 16:31

442

Ændre

PHP kode

 if(isset($_POST['submit'])) {
      $titel = mysql_real_escape($_POST["titel"]);
      $emne = mysql_real_escape($_POST["emne"]);
      $nyhed = mysql_real_escape($_POST["nyhed"]);
       
      $sql = "UPDATE nyhed SET titel='{$titel}', emne='{$emne}', nyhed='{$nyhed}' WHERE id = '".intval($_GET['id']) ."' ";
      mysql_query($sql) or die("Kan ikke opdatere");
  }

Til

PHP kode

 if(isset($_POST['submit'])) {
      $titel = mysql_real_escape_string($_POST["titel"]);
      $emne = mysql_real_escape_string($_POST["emne"]);
      $nyhed = mysql_real_escape_string($_POST["nyhed"]);
       
      $sql = "UPDATE nyhed SET titel='". $titel ."', emne='". $emne ."', nyhed='". $nyhed ."' WHERE id = ".intval($_GET['id']);
      mysql_query($sql) or die("Kan ikke opdatere". mysql_error());
  }

Tag udgangspunkt i den postede kode:

PHP kode

 <?php
  // PHP logic first!
  include("connect.php");
  
  // Check basic access.
  if(!isset($_SESSION["logged_in"])) {
       die("Du har ikke adgang til denne side!");
  } else {
  
  // Find user.
  $query = mysql_query("SELECT * from bruger WHERE id='".intval($_SESSION['user_id'].' LIMIT 1')."' ");
  $result = mysql_fetch_array($query);
  
  // Check for required access level.
  if($result["rank"] !== 5) {
      die("Du har ikke adgang til denne side!");
  }
  
  // Check required parameters
  if(!isset($_GET['id'])) {
       die("Du har ikke adgang til denne side! Mangler id...");
  }
  
  // Do the update, if the page is submittet - before the read out
  if(isset($_POST['submit'])) {
      $titel = mysql_real_escape_string($_POST["titel"]);
      $emne = mysql_real_escape_string($_POST["emne"]);
      $nyhed = mysql_real_escape_string($_POST["nyhed"]);
       
      $sql = "UPDATE nyhed SET titel='{$titel}', emne='{$emne}', nyhed='{$nyhed}' WHERE id = '".intval($_GET['id']) ."' ";
      mysql_query($sql) or die("Kan ikke opdatere". mysql_error());
  }
  
  // Read the item
  $query = mysql_query("SELECT * FROM nyheder WHERE id='".intval($id) ."' ");
  $newsitem = mysql_fetch_array($query);
  ?>
  <!-- HTML presentation second! -->
  <html>
      <body>
          <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
              <input type="hidden" name="id" value="<?php echo $newsitem['id']*1; ?>">
              <input type="text" name="titel" value="<?php echo $newsitem['titel']; ?>">
              <div>
                  Emne:<br>
                  <select name="emne" size="1" value="<?php echo $newsitem['emne']; ?>">
                       <option value="images/esbjerg.jpg">Esbjerg</option>
                       <option value="images/skate.jpg">Skate</option>
                       <option value="images/andet.jpg">Andet</option>
                   </select>
               </div>
               <div>
                   Nyheden:<br>
                  <textarea name="nyhed" rows="20" cols="50"><?php echo $newsitem['nyhed']; ?></textarea>
              </div>
              <input type="submit" name="submit" value="Opdater">
          </form>
      </body>
  </html>
 <?php } ?>

Indlæg senest redigeret d. 22.07.2011 16:38 af Bruger #16025

Bruger #15853 @ 22.07.11 13:22

Det burde virke..

HTML kode

 action=""

Indlæg senest redigeret d. 22.07.2011 13:23 af Bruger #15853

Bruger #16025 @ 22.07.11 15:50

442

Efter hver mysql_query, så skriv mysql_error(). Eksempelvis sådan her:

PHP kode

 mysql_query("SELECT * from bruger WHERE id='$_SESSION[user_id]'") or die(mysql_error());

Bruger #10216 @ 22.07.11 16:16

4.283

Hvis det var mig, ville jeg nok omstrukturere min kode til følgende. Så vil jeg nok få et bedre overblik over hvor der kan være en fejl.

PHP kode

 <?php
 // PHP logic first!
 include("connect.php");
 
 // Check basic access.
 if(!isset($_SESSION["logged_in"])) {
      die("Du har ikke adgang til denne side!");
 }
 
 // Find user.
 $query = mysql_query("SELECT * from bruger WHERE id=".intval($_SESSION['user_id'].' LIMIT 1'));
 $result = mysql_fetch_array($query);
 
 // Check for required access level.
 if($result["rank"] !== 5) {
     die("Du har ikke adgang til denne side!");
 }
 
 // Check required parameters
 if(!isset($_GET['id'])) {
      die("Du har ikke adgang til denne side! Mangler id...");
 }
 
 // Do the update, if the page is submittet - before the read out
 if(isset($_POST['submit'])) {
     $titel = mysql_real_escape($_POST["titel"]);
     $emne = mysql_real_escape($_POST["emne"]);
     $nyhed = mysql_real_escape($_POST["nyhed"]);
      
     $sql = "UPDATE nyhed SET titel='{$titel}', emne='{$emne}', nyhed='{$nyhed}' WHERE id = ".intval($_GET['id']);
     mysql_query($sql) or die("Kan ikke opdatere");
 }
 
 // Read the item
 $query = mysql_query("SELECT * FROM nyheder WHERE id=".intval($id));
 $newsitem = mysql_fetch_array($query);
 ?>
 <!-- HTML presentation second! -->
 <html>
     <body>
         <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="POST">
             <input type="hidden" name="id" value="<?php echo $newsitem['id']*1; ?>">
             <input type="text" name="titel" value="<?php echo $newsitem['titel']; ?>">
             <div>
                 Emne:<br>
                 <select name="emne" size="1" value="<?php echo $newsitem['emne']; ?>">
                      <option value="images/esbjerg.jpg">Esbjerg</option>
                      <option value="images/skate.jpg">Skate</option>
                      <option value="images/andet.jpg">Andet</option>
                  </select>
              </div>
              <div>
                  Nyheden:<br>
                 <textarea name="nyhed" rows="20" cols="50"><?php echo $newsitem['nyhed']; ?></textarea>
             </div>
             <input type="submit" name="submit" value="Opdater">
         </form>
     </body>
 </html>

Men det er bare mig...

Bruger #10216 @ 22.07.11 16:54

4.283

@Daniele har lige et par spørgsmål.

På linje 8, er else fuldkommen overflødigt, da hvis linje 6 er sand vil linje 7 stoppe al fremtidig afvikling. Men hvis $_SESSION["logged_in"] er sat, overses logikken på linje 7. Hvad du har tilføjet er blot hvad der allerede er impliceret i den struktur kode har fået. Altså i dit eksempel er linje 8 og linje 60 overflødig, eller hvordan?

Denne kode:

Kode

$sql = "UPDATE nyhed SET titel='". $titel ."', emne='". $emne ."', nyhed='". $nyhed ."' WHERE id = ".intval($_GET['id']);

hvor du breaker strengen op, er egentlig også unødvendig. Årsagen er at PHP parser tekst inden for dobbelt citationstegn.

Eksempel:

PHP kode

 $value = 'bar';
 
 echo "Foo {$value}."; // Foo bar.
 echo 'Foo {$value}.'; // Foo {$value}.

Det du tilføjer er blot en masse ekstra parsing.

@mark du skal ændre

Kode

if($result["rank"] !== 5) {

til

Kode

if($result["rank"] != 5) {

Det er en tastefejl da jeg typisk arbejder med strict type checking.

Indlæg senest redigeret d. 22.07.2011 16:59 af Bruger #10216

Bruger #15305 @ 22.07.11 13:36

Det burde virke..

HTML kode
 action=""

Nej desværre ikke

Bruger #15305 @ 22.07.11 16:19

Tak for hjælpen, men det virker desværre stadig ikke.. Jeg tænker at det jo må være selve "mysql-update delen". Jeg har fjernet "PHP_SELF" fra 'action' til ="" og nu skriver den "kan ikke opdatere".

Hvis det var mig, ville jeg nok omstrukturere min kode til følgende. Så vil jeg nok få et bedre overblik over hvor der kan være en fejl.

PHP kode

 <?php
 // PHP logic first!
 include("connect.php");
 
 // Check basic access.
 if(!isset($_SESSION["logged_in"])) {
      die("Du har ikke adgang til denne side!");
 }
 
 // Find user.
 $query = mysql_query("SELECT * from bruger WHERE id=".intval($_SESSION['user_id'].' LIMIT 1'));
 $result = mysql_fetch_array($query);
 
 // Check for required access level.
 if($result["rank"] !== 5) {
     die("Du har ikke adgang til denne side!");
 }
 
 // Check required parameters
 if(!isset($_GET['id'])) {
      die("Du har ikke adgang til denne side! Mangler id...");
 }
 
 // Do the update, if the page is submittet - before the read out
 if(isset($_POST['submit'])) {
     $titel = mysql_real_escape($_POST["titel"]);
     $emne = mysql_real_escape($_POST["emne"]);
     $nyhed = mysql_real_escape($_POST["nyhed"]);
      
     $sql = "UPDATE nyhed SET titel='{$titel}', emne='{$emne}', nyhed='{$nyhed}' WHERE id = ".intval($_GET['id']);
     mysql_query($sql) or die("Kan ikke opdatere");
 }
 
 // Read the item
 $query = mysql_query("SELECT * FROM nyheder WHERE id=".intval($id));
 $newsitem = mysql_fetch_array($query);
 ?>
 <!-- HTML presentation second! -->
 <html>
     <body>
         <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="POST">
             <input type="hidden" name="id" value="<?php echo $newsitem['id']*1; ?>">
             <input type="text" name="titel" value="<?php echo $newsitem['titel']; ?>">
             <div>
                 Emne:<br>
                 <select name="emne" size="1" value="<?php echo $newsitem['emne']; ?>">
                      <option value="images/esbjerg.jpg">Esbjerg</option>
                      <option value="images/skate.jpg">Skate</option>
                      <option value="images/andet.jpg">Andet</option>
                  </select>
              </div>
              <div>
                  Nyheden:<br>
                 <textarea name="nyhed" rows="20" cols="50"><?php echo $newsitem['nyhed']; ?></textarea>
             </div>
             <input type="submit" name="submit" value="Opdater">
         </form>
     </body>
 </html>

Men det er bare mig...

Det ser en hel del mere overskueligt ud, det må jeg give dig

Men nu kan jeg ikke få adgang til siden, selvom jeg opfylder "kravende".

Indlæg senest redigeret d. 22.07.2011 16:32 af Bruger #15305

Bruger #15305 @ 22.07.11 16:51

Sådanne og mange mange tak til jer alle sammen! Hvad skulle man dog gøre uden udvikleren nogen gange?

Bruger #16025 @ 24.07.11 09:41

442

#Michael
Det er bare sådan jeg plejer at kode. Men det gør jo ikke nogen forskel om der er det ekstra

Jeg havde ikke lige set den tastefejl med !== ellers havde jeg også rettet den

Den vil ikke opdatere?

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler