Witam!

Raczkuję dopierow PHP, ale na tyle żeby się uczyć.

Otóż mam problem gdy chcę edytować rekord w bazie używając do tego ID wyświetla mi " udano ", ale tak nie jest.... Oto mój kod

[PHP] pobierz, plaintext 
 
<?php
 
$host="localhost"; // Host name
$username="user"; // Mysql username
$password="pass"; // Mysql password
$db_name="nazwa"; // Database name
$tbl_name="news"; // Table name
 
// Connect to server and select databse.
mysql_connect("$host", "$username", "$password")or die("cannot connect");
mysql_select_db("$db_name")or die("cannot select DB");
 
// get value of id that sent from address bar
$id=$_GET['id'];
 
// Delete data in mysql from row that has this id
$sql="UPDATE $tbl_name
SET title='$_POST[title]', body='$_POST[body]'
WHERE id='$_GET[id]'";
$result=mysql_query($sql);
 
// if successfully deleted
if($result){
echo "Edited Successfully";
echo "<BR>";
echo "<a href='http://36i6.co.uk'>Back to main page</a>";
}
 
else {
die(mysql_error());
}
 
// close connection
mysql_close();
 
?>
 
[PHP] pobierz, plaintext 

a to forma

[PHP] pobierz, plaintext 
 
<form action="edit.php" method="post">
                <div class="box">             
                <h1>Add News :</h1>             
                <label><span>Title</span>                
                <input type="text" class="input_text" name="title" id="name" value="<? echo $row['title']?>"/>             
                </label>              
                <!-- <label><span>Date</span>   -->              
                <input type="hidden" class="input_text"  name="date" id="email"/>                        
                <label><span>Body</span>                 
                <textarea class="message" name="body" id="feedback"></textarea>                 
                <input type="submit" class="button" value="Add" />             
                </label>                                  
                </div>     
                </form>
[PHP] pobierz, plaintext 

problem jak sie nie myle jest w dodawaniu tego newsa czy co tam masz

[PHP] pobierz, plaintext 
$sql="UPDATE $tbl_name SET title='$_POST[title]', body='$_POST[body]' WHERE id='$_GET[id]'";
$result=mysql_query($sql);
[PHP] pobierz, plaintext 

Zrób takie coś i powinno dzialać

[PHP] pobierz, plaintext 
$title = $_POST[title];
$body = $_POST[body];
$id = $_GET[id];
$sql="UPDATE $tbl_name SET title='$title', body='$body' WHERE id='$id'";
$result=mysql_query($sql);
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 
 
$result=mysql_query($sql) or die(mysql_error());
 
[PHP] pobierz, plaintext 

zobaczymy czy coś krzyknie

niestety nic z tego nie działa
Nadal pokazuje ze dodane poprawnie ale news sie nie zmienia


http://36i6.co.uk

mam takze w polu value dla title w ormie dla edycji zeby pokazywalo czyli echo zawartosc tabeli title ale nie pokazuje tak samo nie wiem dlaczego

a zrób
echo $_GET['id'];
zobaczymy czy coś w nim jest
a i skąd wysyłasz te $_GET['id'] do tego pliku

Nie wiem czy o to chodziło, ale normalnie edytując ID 24 forma łapie edycję Id 24... Pisze że zmieniło, ale neistety nic nie zmieniło

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 ''title'='ada', 'body'='ada' WHERE 'id'=''' at line 1 wiem juz ze nie łapie ID.

Skad wiem ?


Kliknolem edit wpisalem do title ada do body ada dalem zeby dodalo i pokazalo mi ten blad i title jest ada body ada ale nie ma id

Pytanie czy w zmiennej $id z zapytania $sql jest jakaś wartość, a z tego co widzę to raczej nie .

Tak jak sugerował Ci już kadlub, gdzieś w okolicach $id=$_GET['id']; zrób echo $_GET['id']; Zwykle taką zmienną przekazuje się w adresie.

czyli nic nie masz w tym GET pewnie jak wpiszesz na sztywno to dodoa

Próbowałem różnych kombinacji

w pliki edit.php do którego jest " post " formy


żeby łapało ID mam linię

[PHP] pobierz, plaintext 
 
$id=$_GET['id'];
 
[PHP] pobierz, plaintext 

w sql mam

[PHP] pobierz, plaintext 
 
$sql="UPDATE $tbl_name SET title='$title', body='$body' WHERE id='$id'";
[PHP] pobierz, plaintext 

nie odpowiedziałeś na moje pytanie skąd wysyłasz tego geta

ska wysyłam ? Jak pisałe mw początku tematu raczkuje dopiero więc prosiłbym jasniej jak krowie na rowie jesli chodzi z jakiego pliku to z edit.php i z delete.php Jesli ktos chce sie smiac z mojego podejscia do czegos bez znajomosci tematu to prosze bardzo

Pisze jak byk, że wartość $id jest pobierana z paska adresu.

Troszke się pogubiłem, ale spróbuj czegoś takiego:

[PHP] pobierz, plaintext 
<form action="edit.php?id=( tutaj wpisujesz wartość id )" method="post">
<div class="box">
<h1>Add News :</h1>
<label><span>Title</span>
<input type="text" class="input_text" name="title" id="name" value="<? echo $row['title']?>"/>
</label>
<!-- <label><span>Date</span> -->
<input type="hidden" class="input_text" name="date" id="email"/>
<label><span>Body</span>
<textarea class="message" name="body" id="feedback"></textarea>
<input type="submit" class="button" value="Add" />
</label>
</div>
</form>
[PHP] pobierz, plaintext 

wlasnie podsunoles mi pewien pomysl Wyprubuje go najpierw

[PHP] pobierz, plaintext 
<form action="edit.php?id=24" method="post">
<div class="box">
<h1>Add News :</h1>
<label><span>Title</span>
<input type="text" class="input_text" name="title" id="name" value="<? echo $row['title']?>"/>
</label>
<!-- <label><span>Date</span> -->
<input type="hidden" class="input_text" name="date" id="email"/>
<label><span>Body</span>
<textarea class="message" name="body" id="feedback"></textarea>
<input type="submit" class="button" value="Add" />
</label>
</div>
</form>
[PHP] pobierz, plaintext 

dziala jak marzenie ale jak uzyje

zamiast id=24 dam <? echo $row['id']; ?> juz nie dziala

No to podsune Ci kolejny pomysł:

PLIK cos.php

[PHP] pobierz, plaintext 
<?php
$id=24;
 
 
echo "<a href=cos2.php?id=".$id.">Klik</a>"
 
?>
[PHP] pobierz, plaintext 

PLIK cos2.php

[PHP] pobierz, plaintext 
<?php
$id=$_GET['id'];
echo $id;
?>
[PHP] pobierz, plaintext 

nie wiem czy to ma jakieś znaczenie ale do Updare używam tego samego kodu c do usuwania tylko ze zmienionym $sql

Dodaj sobie do formularza:

[HTML] pobierz, plaintext 
<input type="hidden" name="id" value="<?php echo $row['id']; ?>" />
[HTML] pobierz, plaintext 

i odczytuj id za pomocą tablicy $_POST, bo takie przesyłanie części danych w getem, a części postem, to niezbyt dobre rozwiązanie.

Zapytanie możesz zapisać tak:

[PHP] pobierz, plaintext 
sql='UPDATE '.$tbl_name.' SET title="'.$_POST['title'].'", body="'.$_POST['body'].'" WHERE id="'.$_POST['id'].'"';
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
sql="UPDATE $tbl_name SET title='{$_POST['title']}', body='{$_POST['body']}' WHERE id='{$_POST['id']}'";
[PHP] pobierz, plaintext 

tak czy inaczej daj echo $sql, żeby zobaczyć co siedzi w zapytaniu,
jeżeli id nadal będzie puste to dodaj var_dump($_POST), żeby zobaczyć, czy dane zostały przesłane.

to co mi wyszło

UPDATE news SET title='adam', body='adam ' WHERE id=''array(4) { ["title"]=> string(4) "adam" ["date"]=> string(0) "" ["id"]=> string(0) "" ["body"]=> string(6) "adam " }

A dodaj sobie przed tym formularzem <?php var_dump($row); ?> i napisz co ci wyrzuca, bo z tego co napisałeś, to widać, że brakuje parametru id.
Pokaż też w jaki sposób pobierasz dane do $row.

co wiem do tej pory to to że forma nie dostaje żadnego ID już robię to o co poprosiłeś

daje mi NULL


o to ci chodziło ?

[PHP] pobierz, plaintext 
<?
                while($row=mysql_fetch_array($result)){
                ?>
[PHP] pobierz, plaintext 

A dodałeś to pole hidden, co napisałem ci wcześniej.
var_dump($row) zwraca null? To dlaczego w formularzu masz <?php echo $row['title']; ?> ?
a jak nie chodziło ci o $row, to napisz w jaki sposób ładujesz do tej zmiennej dane.

teraz plik


edit_form.php

[PHP] pobierz, plaintext 
 
 
 
<?php var_dump($row); ?>
<form action="edit.php?id=<? echo $row['id']; ?>" method="post">
                <div class="box">             
                <h1>Add News :<? echo $_GET['id']; ?></h1>             
                <label><span>Title</span>                
                <input type="text" class="input_text" name="title" id="name" value="<? echo $row['title']?>"/>             
                </label>              
                <!-- <label><span>Date</span>   -->              
                <input type="hidden" class="input_text"  name="date" id="email"/> 
                <input type="hidden" name="id" value="<?php echo $row['id']; ?>" />                       
                <label><span>Body</span>                 
                <textarea class="message" name="body" id="feedback"></textarea>                 
                <input type="submit" class="button" value="Add" />             
                </label>                                  
                </div>     
                </form>
 
[PHP] pobierz, plaintext 

i edit.php

[PHP] pobierz, plaintext 
 
<?php
 
$host="localhost"; // Host name
$username="web230-simplecms"; // Mysql username
$password="y3babd50"; // Mysql password
$db_name="web230-simplecms"; // Database name
$tbl_name="news"; // Table name
 
// Connect to server and select databse.
mysql_connect("$host", "$username", "$password")or die("cannot connect");
mysql_select_db("$db_name")or die("cannot select DB");
 
// get value of id that sent from address bar
$id=$_GET['id'];
 
// Delete data in mysql from row that has this id
 
 
      $sql='UPDATE '.$tbl_name.' SET title="'.$_POST['title'].'", body="'.$_POST['body'].'" WHERE id="'.$_POST['id'].'"';
 
 
$result=mysql_query($sql) or die(mysql_error());
 
 
// if successfully deleted
if($result){
echo "Edited Successfully";
echo "<BR>";
echo "<a href='http://36i6.co.uk'>Back to main page</a>";
}
 
else {
die(mysql_error());
}
 
// close connection
mysql_close();
 
?>
 
[PHP] pobierz, plaintext 

A wykonanie jakiego zapytania daje ci wynik do zmiennej $result?

$sql chyba z tego co pamietam.. czytalem dopeir opodstawy i na tutorialach sklejem wiec prosze o wyrozumialosc

Przed chwilą wkleiłeś coś takiego:

[PHP] pobierz, plaintext 
      <?
      while($row=mysql_fetch_array($result)){
      ?>
[PHP] pobierz, plaintext 

chodzi mi o to co masz wyżej, a dokładnie o treść zapytania, które wysyłasz do bazy, żeby pobrać dane.

to znaczy tego to akurat uzywam w home.php


to chyba o to ci chodzi

[PHP] pobierz, plaintext 
// select record from mysql
$sql="SELECT * FROM $tbl_name";
$result=mysql_query($sql);
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
<? include('connect.php');?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Language" content="en" />
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta name="Keywords" content="36i6, w36i6, web design, web development, templates, layouts" />
<meta name="Description" content="36i6 Interactive" />
<meta name="Author" content="Marek Prymula" />
<meta name="robots" content="index,follow,all" />
<title>36i6 Interactive</title>
<link rel="stylesheet" type="text/css" href="style.css" />
<link rel="shortcut icon" type="image/png" href="favicon.png" />
<script type="text/javascript" src="js/cufon-yui.js"></script>
<script type="text/javascript" src="js/Titillium.font.js"></script>
<script type="text/javascript">
Cufon.replace('.logo', {textShadow: '#000 2px 1px', hover: true});
</script>
</head>
<body>
 
    <div class="center"><!-- Center -->
 
        <div class="header"><!-- Header Section -->
 
            <a class="logo" href="#">36i6 Development</a><!-- Logo -->
 
            <div class="social"><!-- Social Icons -->
 
 
 
            </div><!-- Social Icons -->
 
        </div><!-- Header Section -->
 
        <div class="page"><!-- Main Content -->
 
            <div class="news">
 
                <?
                while($row=mysql_fetch_array($result)){
                ?>
 
<? $id=24;?>
               ID: <? echo $row['id'];?> <a href="/delete.php?id=<? echo $row['id']; ?>">delete</a>|<a href="/edit_form.php?id=<? echo $row['id']; ?>">edit</a>
               <br />
               Title: <? echo $row['title']?><br />
               Date: <? echo date("d/m/Y",$row['date']) ?><br />
               Body: <? echo $row['body'] ?>
 
               <hr />
 
 
               <?
 
              // close while loop
               }
 
              // close connection;
              mysql_close();
 
              ?> 
 
            </div>
 
            dsdds
            <div class="add_news">
 
                <form action="insert.php" method="post">
                <div class="box">             
                <h1>Add News :</h1>             
                <label><span>Title</span>                
                <input type="text" class="input_text" name="title" id="name"/>             
                </label>              
                <!-- <label><span>Date</span>   -->              
                <input type="hidden" class="input_text"  name="date" id="email"/>                        
                <label><span>Body</span>                 
                <textarea class="message" name="body" id="feedback"></textarea>                 
                <input type="submit" class="button" value="Add" />             
                </label>                                  
                </div>     
                </form>
 
            </div>
 
        </div><!-- Main Content -->
 
    </div><!-- Center -->
 
</body>
</html>
 
 
[HTML] pobierz, plaintext 

Zmień
<input type="hidden" name="id" value="<?php echo $row['id']; ?>" />
na
<input type="hidden" name="id" value="<?php echo $_GET['id']; ?>" />

kocham cie
dziala

temat uważam za zamknięty użytwkonik bastard13 sprostał zadaniu

Jeszcze chciałem się dowiedziec jesli mam więcej problemów mam zakładać nowy temat czy juz tu pisac ?

Miłości nigdy za wiele:)
Następnym razem staraj się zwracać większą uwagę na to skąd, co powinieneś pobierać.
A jak nie wiesz w czym jest problem, to świetne do prostego debugowania skryptów są funkcje var_dump() i die()


Nowy temat.

Właśnie DIE użyłem prawie na koncu gdy juz nie wiedzialem co robic i niby troche mi pomoglo bo najpierw patrzylem w zrodlo kodu i wiedzialem ze ID nie dostaje i kombinowalem z tym id na takie sposoby ze nie jeden programista by za brak bezpieczenstwa mnei powiesil , ale kiedys umeiszczalem tu ogloszenie o prace jako pracodawca i wiedzialem ze sa tu tacy kompetentni ludzie ze zawsze pomoga w tym przypadku bastard13 okazal sie moim guru gdzie nie obrazam innych gdyz oni tez wykazali sie nie lada wsparciem
Zakładam nowy temat