php - Udate i mysql - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php mysql

Bruger #21349 @ 13.03.16 03:55

Hej.

Nogen der kan se hvad der går gal her:

Jeg prøver at opdatere oplysningerne på en bruger i mit system:

PHP kode

 <?php 
     include 'header.php'; 
     
     if  ($_SESSION['user']['useres_types'] == 3){
     
     $id = $_GET['id'];    
         
     ?>
 <!-- BEGIN PAGE HEADER-->
 
             <div class="page-bar">
                 <ul class="page-breadcrumb">
                     <li>
                         <i class="fa fa-home"></i>
                         <a href="index.html">Admin</a>
                         <i class="fa fa-angle-right"></i>
                     </li>
                     <li>
                         <a href="#">Rediger Bruger</a>
                     </li>
                 </ul>
                 </div>
                 <h3 class="page-title">
                 Rediger Bruger <small></small>
                 </h3>
                 
 <?php
     
             if (isset($_POST['redigerbruger'])) {
             
                 $brugernavn = $_POST['username'];
                 $fornavn = $_POST['first_name'];
                 $efternavn = $_POST['last_name'];
                 $tlf = $_POST['tlf'];
                 $email = $_POST['email'];
                     
                     mysqli_query($conn, "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$lastname', tlf='$tlf', email='$email' WHERE id='$id')") or die(mysqli_error($conn));
         
                                                         echo "Brugeren er rettet!"; 
                                                         echo "<br>";      
                 
                 }else{
                     echo "Noget gik galt";
                 }
 
 
             $data = mysqli_query($conn, "SELECT * FROM `users` where id=$id" ) or die(mysql_error());    
             while ($info = mysqli_fetch_array($data))
             {
                 ?>
                 <form name="redigerbruger" action="<? $_SERVER['PHP_SELF']?>" method="post">
                 <?php
                 echo "<table>";
                 echo "<tr>";
                 echo "<td>Id:</td>";
                 echo "<td>".$info['id']."</td>";
                 echo "</tr>";
                 echo "<tr>";
                 echo "<td>Brugernavn:</td>";
                 echo "<td><input type=\"text\" name=\"username\" value=\"" .$info['username']. "\"></td>";
                 echo "</tr>";
                 echo "<tr>";
                 echo "<td>Fornavn:</td>";
                 echo "<td><input type=\"text\" name=\"first_name\" value=\"" .$info['first_name']. "\"></td>";
                 echo "</tr>";    
                 echo "<tr>";
                 echo "<td>Efternavn:</td>";
                 echo "<td><input type=\"text\" name=\"last_name\" value=\"" .$info['last_name']. "\"></td>";
                 echo "</tr>";    
                 echo "<tr>";
                 echo "<td>Tlf:</td>";
                 echo "<td><input type=\"text\" name=\"tlf\" value=\"" .$info['tlf']. "\"></td>";
                 echo "</tr>";
                 echo "<tr>";
                 echo "<td>Email:</td>";
                 echo "<td><input type=\"text\" name=\"email\" value=\"" .$info['email']. "\"></td>";
                 echo "</tr>";
                 echo "<tr>";
                 echo "<td>Brugertype:</td>";
                 echo "<td>";
                     if ($info['useres_types'] ==1) {
                         echo "Admin";
                         }else {
                         echo "Bruger";
                         }        
                 echo "</td>";
                 echo "</tr>";
                 echo "<tr>";
                 echo "<td>";
                 echo "<td><input type=\"submit\" value=\"Rediger bruger\" name=\"redigerbruger\"></td>";
                 echo "</td>";
                 echo "</tr>";                
                 echo "</table>";
                 ?>
                 </form> 
                 <?php
             }
         
         }else{
          header('Location: logaf.php'); 
         }
 
     include 'footer.php'; 
     
     ?>

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

Sorter efter stemmer Sorter efter dato

Bruger #16075 @ 13.03.16 11:27

1.223

hvis du ligger din sql ud i en seperat variabel, og prøver at udskrive den, kan du se fejlen

Kode

$sql = "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$last_name', tlf='$tlf', email='$email' WHERE id='$id')";
echo $sql; // test udskrivning, ser dette statement korrekt ud ??  hvis ja slettes hele denne linje

mysqli_query($conn, $sql) or die(mysqli_error($conn));

læg mærke til echo $sql

i denne linje er der 2 fejl:
1 syntax fejl (hint tæl paranteser)
1 værdi der ikke bliver sat

Bruger #21150 @ 13.03.16 11:27

Du har en parentes for meget.

last_name='$lastname', tlf='$tlf', email='$email' WHERE id='$id')") or die(mysqli_error($conn));

Den markerede parentes skal slettes. (:

Bruger #16075 @ 13.03.16 11:36

1.223

skal vi ikke vælge den anden parantes

så sql ser sådan ud

Kode

"UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$lastname', tlf='$tlf', email='$email' WHERE id='$id'"

Bruger #21150 @ 13.03.16 11:54

Selfølgelig

Var lige hurtig nok kan jeg se..

Indlæg senest redigeret d. 13.03.2016 11:57 af Bruger #21150

Bruger #21150 @ 13.03.16 09:32

Ved ikke lige om det er det.. Men du skriver

action="<? $_SERVER['PHP_SELF']?>"

Der mangler <?php..

Men ved ikke om den selv kan finde ud af det..?

Og er din "$conn" defineret i den header.php du inkluder.

Ellers har du ikke oprettet forbindelse nogen steder.

Indlæg senest redigeret d. 13.03.2016 09:36 af Bruger #21150

Bruger #16075 @ 13.03.16 10:06

1.223

hvilke fejl får du ??
er du logget på som admin ??
er session started på siden (må ligge i header.php)
prøv at udskrive din session

er error_reporting slået til i php.ini ??
alternativt kan du lave en debugheader.php som du indsætter i toppen af dit document

debugheader.php

Kode

<?php
define('IN_DEBUG_MODE', true); // Sæt til false i production mode

if(IN_DEBUG_MODE){
	ini_set("display_startup_errors", "on");
	ini_set("display_errors", "on");
	ini_set("html_errors", "false");
	error_reporting(-1); // -1 viser alle slags fejl beskeder
	ini_set("ignore_repeated_errors", 0);
}
else{
// Turn off all error reporting
error_reporting(0);
}
?>

<?php
include_once ('debugheader.php')

// test udskrivning af session
echo "<pre>";
print_r($_SESSION);
echo "</pre>";

include 'header.php';

if ($_SESSION['user']['useres_types'] == 3) {

**********************
denne linje mangler da noget
<form name="redigerbruger" action="<? $_SERVER['PHP_SELF'] ?>" method="post">

skal da se sådan ud
<form name="redigerbruger" action="<?php echo $_SERVER['PHP_SELF'] ?>" method="post">
som Jørn Guldberg påpeger

et godt begynder råd:
læg dit sql statement ud i en seperat variabel, så er der nemmere at se fejl i det (test udskriv det)

Kode

$sql = "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$last_name', tlf='$tlf', email='$email' WHERE id='$id')";
echo $sql; // test udskrivning, ser dette statement korrekt ud ??  hvis ja slettes hele denne linje

mysqli_query($conn, $sql) or die(mysqli_error($conn));

Indlæg senest redigeret d. 13.03.2016 10:08 af Bruger #16075

Bruger #21349 @ 13.03.16 11:20

Jeg for ikke fejl der oppe. den slår først fejl nede ved :

if (isset($_POST['redigerbruger'])) {

$brugernavn = $_POST['username'];
$fornavn = $_POST['first_name'];
$efternavn = $_POST['last_name'];
$tlf = $_POST['tlf'];
$email = $_POST['email'];

mysqli_query($conn, "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$lastname', tlf='$tlf', email='$email' WHERE id='$id')") or die(mysqli_error($conn));

echo "Brugeren er rettet!";
echo "<br>";

}else{
echo "Noget gik galt";
}

Det er kun her den fejler, fjerner jeg rediger bruger køre det hele som det skal. -> det begyndte at gå galt da jeg ville integrere form'en i min php, og så har jeg aldrig prøvet at kørt en udate to mysql.

fejlen ligger her et sted. piller jeg formen ud af whilen og fjerner if rediger bruger er der ingen problemer

alt hvad jeg for af fejl denne gang.

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1

Har ikke haft problemer med mysql fejl meddelsere før.

Indlæg senest redigeret d. 13.03.2016 11:22 af Bruger #21349

Bruger #21349 @ 13.03.16 12:46

Mange mange tak

. Hvor er det dog nogen kedelige fejl jeg laver haha

Bruger #16075 @ 13.03.16 13:23

1.223

Hvor er det dog nogen kedelige fejl jeg laver haha

en del af det er jo også at fejl finde på sine fejl, og finde de metoder der passer en bedst
hvis bare du lærer af dine fejl, så er det ikke spildte fejl

**************************

som du kan se er der flere måder at lave sin sql på

som du anvender

Kode

$mysqli_query($conn, "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$efternavn', tlf='$tlf', email='$email' WHERE id='$id'") or die(mysqli_error($conn));

med en seperat variabel, nemmere at fejl finde på, da man nemt kan test udskrive den

Kode

$sql = "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$efternavn', tlf='$tlf', email='$email' WHERE id='$id'";
$mysqli_query($conn, $sql) or die(mysqli_error($conn));

sidst nævnte vil ikke give mening hvis man anvender Prepare Statement, som er noget af det nye i mysqli, og klar forbedret i det API der hedder PDO (et nyere API)
når du er mere rutineret i sql kan du læse denne lyn guide http://www.eksperten.dk/guide/1480 til Prepare Statement i mysqli

**************************

lidt suplerende, i mysqli kan man skrive i 2 style, eller blande som man har lyst:

procedural style (som du anvender), er den gamle måde at skriv på, og er et lævn fra det gamle mysql API, og kun taget med over i mysqli for at gører det nemt for brugerne at skifte.

object orinterede (OOP) style, dette er den mere moderne måde at skrive på, og er det eneste der kan anvendes i det API der hedder PDO

så dette

Kode

$sql = "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$efternavn', tlf='$tlf', email='$email' WHERE id='$id'";
$mysqli_query($conn, $sql) or die(mysqli_error($conn));

kunne også se sådan ud

Kode

$sql = "UPDATE users SET username='$brugernavn', first_name='$fornavn', last_name='$efternavn', tlf='$tlf', email='$email' WHERE id='$id'";
$conn->query($sql) or die($conn->error);

Udate i mysql

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler