PHP script

len pripomienka (neber to ako vrtanie) - ked pises kod, tak hore v liste mas tlacitko code - je to potom prehladnejsie.

//Odkaz pro smazání novinky
<a href="?smaz=<? echo $zanam['id']; ?>">Smazat</a>

//Skript pro smazání

<?php
if(isset($_GET['smaz']))
{
$id = $_GET['smaz'];
(mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1')) ? echo "OK" : echo "Error";
}
?>

Lze nahradit skript

(mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1')) ? echo "OK" : echo "Error"; 

pouze tímto (nevypíše to ty hlášky OK a Error)

mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1'); 

PS: <form action="zapis.php" name="post" method="post"> IMHO by bylo vhodnější nedávat tam to action="zapis.php" ale jen action="" nebo nic, protože když budeš svůj systém rozvíjet tak budeš třeba i měnit názvy souborů a hned ti to nepůjde

PS 2: Pokud bys nerozuměl tomuto řádku:

(mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1')) ? echo "OK" : echo "Error";

tak je to to samé jako:

if(mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1'))
{
echo "OK";
}
else
{
echo "Error";
}

Mazat záznamy z databáze fyzicky, není zrovna nejlepší řešení...

moc ti dekujo za script ale nemohu to nejak rozchodit. kdyz ho vlozim k scriptu:

<?
mysql_connect("localhost","--","--");
mysql_select_db("--");

$vyber = mysql_query("SELECT * FROM `Novinky` ORDER BY `Novinky`.`cas` DESC");
?>

<?
while ($zanam = mysql_fetch_array($vyber, MYSQL_NUM))
{
 echo "<div style='border: 2px solid'><b>$zanam[1] - $zanam[0]</b></div><br>";
}
?>

tak se mi zobrazi jen bila stranka. muzete mi prosim poradit kde to mam zpravne vlozit??

Udělej export struktury dat z databáze nebo vymaž všechny novinky a přidej si tam sloupec id s auto-increment a Primary key.

no to jsem udelal a co dale??

<?
mysql_connect("localhost","--","--");
mysql_select_db("--");

$vyber = mysql_query("SELECT * FROM `Novinky` ORDER BY `Novinky`.`cas` DESC");

if(isset($_GET['smaz']))
{
$id = $_GET['smaz'];
(mysql_query('DELETE FROM `Novinky` WHERE `id` = '$id' LIMIT 1')) ? echo "OK" : echo "Error";
}
?>

<?
while ($zanam = mysql_fetch_array($vyber, MYSQL_NUM))
{
 echo "<div style='border: 2px solid'><b>$zanam[1] - $zanam[0]</b>&nbsp;<a href="?smaz=<? echo $zanam['id']; ?>">Smazat</a></div><br>";
}
?>

presne tak to udelam a delal jsem to uz drive a proste to nejde hodi to bilou obrazovku a kdyz dam zobrazit zdrojovy kod nic tam neni

Napiš mi na ICQ (373475016) a podívám se ti na to, pak sem postnu konečné řešení.

Vyřešeno, celé to bylo o tom, že Vaseks tam trochu popletl závorky a zápis apostrofů, takže tam bylo pár drobných chybiček, které jsem opravil.

Vyreseno, moc dekuji marekdrtic za pomoc. Je sikovnej.

Hoci je to vyriešené, napíšem ešte pár poznámok, ktoré sú dúfam k veci:

1. Keď vyvíjaš skript, tak si nastav vypisovanie chýb (error_reporting) buď v konfigurácii alebo priamo v PHP súbore. Oveľa jednoduchšie potom nájdeš príčinu problému podľa chybovej hlášky.

2. V SQL dopytoch používaš neošetrené vstupy od používateľa, takže hrozí SQL injection. Pred použitím reťazca v SQL dopyte sa používa funkcia mysql_real_escape_string, v prípade čísla ho môže stačiť pretypovať.

O té funkci mysql_real_escape_string jsem nic nevěděl, vždycky jsem na to buď kašlal nebo obcházel složitěji, díky

v prípade čísla ho môže stačiť pretypovať.

Mozem poprosit o podrobnejsie vysvetlenie? Dakujem.

Napr. po priradení $id = (int)$_GET["id"]; bude v $id určite číslo, takže sa dá bezpečne použiť v SQL dopyte.

Vdaka, doteraz som si myslel, ze ciselne typy (smallint, int atd) su bez problemov a neriesil som na nich kontrolu.