[PHP]Błąd nagłówka, usuwanie rekordów, Help Opcje

[MateuszS]

Witam. Mam taki problem że pod czas dodawania newsa miałem tak że po odświeżaniu dodawał mi się kolejny. Nie chcę aby tak było więc zastosowałem header wszystko widać poniżej

[PHP] pobierz, plaintext 
<?php
if($_GET['opcja']=='sklep') {
 
echo '<b><center>Aktualnie w naszym sklepiku znajdują się:</center><br /></b>';
 
$zapytanie = mysql_query("SELECT id,nazwa,cena FROM `cennik`");
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40"><b>ID</b></td><td width="200"><b>Nazwa produktu</b></td><td width="80"><b>Cena</b></td><td width="80"><b>Opcja</b></td></tr></table>';
while($wynik = mysql_fetch_assoc($zapytanie)) { 
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40">' . $wynik['id'] . '</td>
<td width="200">' . $wynik['nazwa'] . '</td>
<td width="80">' . $wynik['cena'] . '</td>
<td width="80"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usuń</td></tr></table>';
 
if($_GET['usun']==TRUE) {
$sql = mysql_query("DELETE * FROM `cennik` WHERE `id`='".$_GET['usun'].""); 
}
}
 
echo '<br />';
echo '<b><center>Dodaj nowy produkt:</center></b><br />';
echo '<form method="POST" action="page.php?id=14&opcja=sklep">Nazwa: <input type="text" name="nazwaproduktu" size="25"> 
Cena: <input type="text" size="10" name="cenaproduktu" /> zł. <input type="submit" value="dodaj" /></form>';
 
if(empty($_POST['nazwaproduktu']) or empty($_POST['cenaproduktu'])) {
echo '<font color="red">Wypełnij wszystkie pola</font>'; } else {
$zapyt = mysql_query("INSERT INTO `cennik` (nazwa,cena) VALUES ('".$_POST['nazwaproduktu']."','".$_POST['cenaproduktu']."')");
header("Location: page.php?id=14&opcja=sklep"); 
}
 
}
?>
[PHP] pobierz, plaintext 

Wywala

Kod

[b]Warning[/b]:  Cannot modify header information - headers already sent by (output started at G:\xampp\xampp\htdocs\n\gora.php:128) in [b]G:\xampp\xampp\htdocs\n\page\admin.php[/b] on line [b]103

[/b]zaznaczam że ten kod to fragment dość dużej strony, którą musiałem przerobić. Wiem że jest setki tematów na temat headera ale w żadnym nie znalazłem rozwiązania, ten błąd jest chyba unikalny jakiś. Ob_start nie działa.

I drugi problem to chcę usunąć rekordy z bazy ale mój skrypt nie działa. Proszę o pomoc.

[Foxx]

I przeczytałeś te setki tematów? Chyba nie bo byś nie dawał echo i header jedno pod drugim...

[PHP] pobierz, plaintext 
<?php
echo '<font color="red">Wypełnij wszystkie pola</font>'; 
header("Location: page.php?id=14&opcja=sklep");
?>
[PHP] pobierz, plaintext 

Co jest w tej sytuacji unikalnego, nie możesz nic wysłać do przeglądarki żeby użyć header.

A co do usuwania rekordów:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['usun'])) {
  $sql = mysql_query("DELETE * FROM `cennik` WHERE `id`='".$_GET['usun'].""); 
}
?>
[PHP] pobierz, plaintext 

Raczej to zadziała ale dobrze radzę przefiltrować jeszcze zmienną $_GET['usun'] zanim ktoś wstawi Ci do url'a jakieś OR 1=1

Ten post edytował Foxx 16.03.2009, 16:30:28

[MateuszS]

Przyznam sie o tym nie wiedzialem, ale gdzie w takim razie mam dac ten header, nie dam go na poczatku bo wtedy nawet gdy doda do bazy rekord to pojawi sie po odsiezeniu i co odswiezenie bedzie dodawal nowe

ps moze cos zle robie ale usuwanie nie dziala

[PHP] pobierz, plaintext 
<?php
<td width="80"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usu&#324;</td></tr></table>';
 
if(isset($_GET['usun'])) {
 $sql = mysql_query("DELETE * FROM `cennik` WHERE `id`='".$_GET['usun']."");
}
?>
[PHP] pobierz, plaintext 

Ten post edytował MateuszScirka 16.03.2009, 16:41:15

[Foxx]

Masz bałagan w kodzie, żeby to zrobić poprawnie powinieneś:

1. zapytania sql zapisać w takiej formie:

[PHP] pobierz, plaintext 
<?php
echo $query = "DELETE * FROM `cennik` WHERE id='".$_GET['usun']."'"; // tu brakowało cudzysłowia ' 
$sql = mysql_query($query) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Dzięki czemu zobaczysz jakie zapytania ostatecznie są wykonywane po jakiej operacji na stronie i zobaczysz też ew. błędy mysql dzięki mysql_error().

2. co do przekierowania - chcesz przekierować do innego pliku czy to wszystko jest plikiem page.php? Bo jeżeli to jeden plik to header jest zbędny, po prostu wykonaj wszystkie operacje na początku pliku a potem wyświetl formularz. Jeżeli to jest przekierowanie do innego pliku, page.php to i tak wykonuj operacje na początku, razem z przekierowaniem, zanim cokolwiek wyślesz do przeglądarki (czyli formularz tak jak teraz).

[MateuszS]

tak to wszystko wyonuje sie w jednym pliku page.php. Sa tam rozne panele, do dodawania newsow, kontaktow itp a ten panel to panel dodawania do cennika roznych rzeczy.

[PHP] pobierz, plaintext 
<?php
if($_GET['opcja']=='sklep') {
 
echo '<b><center>Aktualnie w naszym sklepiku znajdują się:</center><br /></b>';
 
$zapytanie = mysql_query("SELECT id,nazwa,cena FROM `cennik`");
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40"><b>ID</b></td><td width="200"><b>Nazwa produktu</b></td><td width="80"><b>Cena</b></td><td width="80"><b>Opcja</b></td></tr></table>';
while($wynik = mysql_fetch_assoc($zapytanie)) { 
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40">' . $wynik['id'] . '</td>
<td width="200">' . $wynik['nazwa'] . '</td>
<td width="80">' . $wynik['cena'] . '</td>
<td width="80"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usuń</td></tr></table>';
}
 
echo $query = "DELETE * FROM `cennik` WHERE id='".$_GET['usun']."'"; // tu brakowało cudzysłowia '
$sql = mysql_query($query) or die(mysql_error());
 
echo '<br />';
echo '<b><center>Dodaj nowy produkt:</center></b><br />';
echo '<form method="POST" action="page.php?id=14&opcja=sklep">Nazwa: <input type="text" name="nazwaproduktu" size="25"> 
Cena: <input type="text" size="10" name="cenaproduktu" /> zł. <input type="submit" value="dodaj" /></form>';
 
if(!empty($_POST['nazwaproduktu']) or !empty($_POST['cenaproduktu'])) {
$zapyt = mysql_query("INSERT INTO `cennik` (nazwa,cena) VALUES ('".$_POST['nazwaproduktu']."','".$_POST['cenaproduktu']."')");
 
}
 
}
?>
[PHP] pobierz, plaintext 

Usuwanie nie dziala (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) Noż kurw....

[Foxx]

Ok, usuwanie nie działa ale jaki jest rezultat echo $query; ? Jeżeli potem dzieją się w tym pliku jakieś przekierowania i nie możesz nic zobaczyć do daj po echo $query; instrukcję exit; A jeżeli nadal nic nie widać to znaczy że nie wchodzi do tego ifa więc URL ma może błędną postać.

[MateuszS]

Błąd wyskakuje gdy chcę usunąć rekord.

A co do tego odświeżania to poszedłem na latwizne i prymitywizm i zastosowalem

[PHP] pobierz, plaintext 
<?php
$zapyt = mysql_query("INSERT INTO `cennik` (nazwa,cena) VALUES ('".$_POST['nazwaproduktu']."','".$_POST['cenaproduktu']."')");
echo '<script language="JavaScript">window.location="page.php?id=14";</script>';
?>
[PHP] pobierz, plaintext 

moze dam kod mojego kolegi rowniez z page.php gdzie jest usuwanie, sam go nie rozgryzlem

[PHP] pobierz, plaintext 
<?php
if ($_GET['opcja']=="kontakty") {
echo "<form action='page.php?id=14&opcja=kontakty' method='post'><table class='a'>
<tr><Td><b>Nazwa:</b></td><Td><input type='text' name='nazwa'></td></tr>
<tr><td><b>Email:</td><td><input type='text' name='email'></td></tr>
<tr><Td></td><Td><input type='submit' value='Dodaj' name='dodaj'></td></tr></table></form><br><br>";
if ($_POST['dodaj']){
if (empty($_POST['nazwa']) || empty($_POST['email'])) {
echo "Uzupełnij wszystkie tabelki";
} else {
mysql_query("INSERT INTO `kontakt` (`nazwa`, `email`) VALUES('".$_POST['nazwa']."','".$_POST['email']."')");
echo "Kontakt <b>{$_POST['nazwa']}</b> został poprawnie dodany do bazy danych...";
}
}
if ($_GET['usun']==TRUE){
mysql_query("DELETE FROM `kontakt` WHERE `id`='".$_GET['del']."'");
header("Location: page.php?id=14&opcja=kontakty");
exit;
}
$kontakty=mysql_query("SELECT * FROM `kontakt`");
echo "<table class='a' width=100%>
<tr><td width=53%><b>Email</b></td><Td width=33%><b>Nazwa</b></td><Td><b>Opcje</b></tr>";
while ($kontakt=mysql_fetch_assoc($kontakty)) {
$ilosc=$ilosc+1;
echo "<tr><td>{$kontakt['email']}</td><Td>{$kontakt['nazwa']}</td><Td> - <a href=\"page.php?id=14&opcja=kontakty&usun={$kontakt['id']}\">usuń</a></tr>";
}
echo "</table>";
}
?>
[PHP] pobierz, plaintext 

[Foxx]

Błąd wyskakuje gdy chcę usunąć rekord.

Wklej ten błąd, musimy go przeczytać żeby wiedzieć co nie gra.

[MateuszS]

DELETE * FROM `cennik` WHERE id=''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 '* FROM `cennik` WHERE id=''' at line 1


gdy mam

[PHP] pobierz, plaintext 
<?php
if($_GET['opcja']=='sklep') {
if(!empty($_POST['nazwaproduktu']) or !empty($_POST['cenaproduktu'])) {
$zapyt = mysql_query("INSERT INTO `cennik` (nazwa,cena) VALUES ('".$_POST['nazwaproduktu']."','".$_POST['cenaproduktu']."')");
echo '<script language="JavaScript">window.location="page.php?id=14&opcja=sklep";</script>';
} elseif ($_POST['dodaj']) {
echo '<center><font color="red">Wypełnij pola</font></center>'; }
 
echo '<b><center>Aktualnie w naszym sklepiku znajdują się:</center><br /></b>';
 
$zapytanie = mysql_query("SELECT id,nazwa,cena FROM `cennik`");
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40"><b>ID</b></td><td width="240"><b>Nazwa produktu</b></td><td width="60"><b>Cena</b></td><td width="60"><b>Opcja</b></td></tr></table>';
while($wynik = mysql_fetch_assoc($zapytanie)) { 
echo '<table class="a" border="1" cellspacing="3">
<tr><td width="40">' . $wynik['id'] . '</td>
<td width="240">' . $wynik['nazwa'] . '</td>
<td width="60">' . $wynik['cena'] . ' zł</td>
<td width="60"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usuń</td></tr></table>';
}
 
echo $query = "DELETE * FROM `cennik` WHERE id='".$_GET['usun']."'"; // tu brakowało cudzysłowia '
$sql = mysql_query($query) or die(mysql_error());
 
echo '<br />';
echo '<b><center>Dodaj nowy produkt:</center></b><br />';
echo '<form method="POST" action="page.php?id=14&opcja=sklep">Nazwa: <input type="text" name="nazwaproduktu" size="25"> 
Cena: <input type="text" size="10" name="cenaproduktu" /> zł. <input type="submit" value="dodaj" name="dodaj" /></form>';
 
}
?>
[PHP] pobierz, plaintext 

[Foxx]

No ale czy nie widzisz tego, że zmienna usun nie ma wartosci? (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif)

WHERE id=''

A po drugie, zajrzyj do manuala żeby przeczytać jak wygląda instrukcja delete:
http://www.w3schools.com/php/php_mysql_delete.asp

[MateuszS]

[PHP] pobierz, plaintext 
<?php
WHERE id='".$_GET['usun']."'"
?>
[PHP] pobierz, plaintext 

Jak to nie ma? Wartosc to ID z bazy, i powinno usuwac wszystkie pola z ID ktory jest w gecie.

[Foxx]

Wiem, że w kodzie ma, ale zobacz jak wygląda jak się je wyświetli tuż przed wykonaniem:

DELETE * FROM `cennik` WHERE id=''You have an error

To cytat z Twojego posta. Czyli zmienna $_GET['usun'] nie ma wartości, może nie mam jej przypisanej w urlu.

I pamiętaj o tej składni DELETE, która u Ciebie jest błędna.

Ten post edytował Foxx 16.03.2009, 17:43:52

[MateuszS]

bleah...

[PHP] pobierz, plaintext 
<?php
<td width="60"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usu&#324;</td></tr></table>';
 
$_GET['usun'] = $ide;
$query = "DELETE FROM `cennik` WHERE id='$ide";
?>
[PHP] pobierz, plaintext 

Nie mam pojecia szczerze mowiac o co ci chodzilo z ta pusta wartoscia ale mniejsza... zrobilem tak jak powyzej i uwaga!!!!! nie dziala ;/ Doceniam Twoje poswiecenie i licze ze jakos pomozesz mi z tego wybrnąć.

Ten post edytował MateuszScirka 16.03.2009, 18:06:29

[Foxx]

Jak sam widzisz, proces się składa z 2 elementów: linka i instrukcji wykonującej zapytanie DELETE na jego podstawie.
Zrób tak:

[PHP] pobierz, plaintext 
<?php
echo '<td width="60"><a href="page.php?id=14&opcja=sklep&usun=' . $wynik['id'] . '">Usuń</td></tr></table>';
 
$ide = $_GET['usun'];
echo $query = "DELETE FROM `cennik` WHERE id='".$ide."'";
mysql_query($query) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Teraz powinieneś już nie mieć błędów składni ale nadal możesz mieć ten problem co poprzednio - w zmiennej $_GET['usun'] nic nie było. Dlatego przyjrzyj się linkowi, jeżeli ma postać page.php?id=14&opcja=sklep&usun= to źle a powinno być page.php?id=14&opcja=sklep&usun=23 - powpisuj ręcznie jakieś id rekordów i sprawdź czy usuwa Ci z bazy, sprawdź czy zmienna, którą wstawiasz w to miejsce, czyli $wynik['id'] ma na pewno jakąś wartość.

[MateuszS]

Metoda dziala czesciowo. Pod tabela mam napisane DELETE blabla zwyczajnie wyswietlone a na pasku mam niby ...&usun=6 (przykladowe id) ale usuwa dopiero gdy klinne enter (czyi wejde w link)

[Foxx]

Zobacz w źródle strony czy w linku jest jakiś numer id czy puste usun=
Nie wiem teraz czy mówisz o wpisanym id na sztywno czy to działa tak jak trzeba.
Wiadomo, że musisz kliknąć w link żeby usuń się wykonało.