Aktualizace tabulky s relací M:N v MySQL přes PHP formulář – PHP – Fórum – Programujte.com

Zdravím,

prosím o pomoc, mám následující problém:

Tvořím databázi železničních přejezdů. Databáze obsahuje relaci M:N pro přejezdy a výstražníky (jeden železniční přejezd může mít více typů výstražníků).

Zde je PHP skript, který předává data z formuláře do MySQL:

Připojen obrázek.

<?php

$id = $_GET["id"];
$cislo = $_GET["cislo"];
$sirka = $_GET["sirka"];
$delka = $_GET["delka"];
$rok_rekonstrukce = $_GET["rok_rekonstrukce"];
$foto = $_GET["foto"];
$PZZ = $_GET["PZZ"];
$obec = $_GET["obec"];
$silnice = $_GET["silnice"];
$vystrazniky_vyber = implode(', ', $_GET["vystrazniky"]);






include("pripojeni.php");

$sql = "UPDATE ZELEZNICNI_PREJEZDY, PREJEZDY_VYSTRAZNIKY SET cislo_prejezdu='$cislo', zemepisna_sirka='$sirka', zemepisna_delka='$delka', foto='$foto', ZELEZNICNI_PREJEZDY.id_PZZ='$PZZ', ZELEZNICNI_PREJEZDY.id_obce='$obec', ZELEZNICNI_PREJEZDY.id_silnice='$silnice', WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'"; 

if($rok_rekonstrukce != '' or $rok_rekonstrukce != null) {
$sql_rok = "UPDATE ZELEZNICNI_PREJEZDY SET rok_rekonstrukce='$rok_rekonstrukce' WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'";
}
else {
$sql_rok = "UPDATE ZELEZNICNI_PREJEZDY SET rok_rekonstrukce=null WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'";
}
$vystrazniky[] = explode(', ', $vystrazniky_vyber);
foreach($vystrazniky AS $id_vystrazniku) {
$sql_prejezdy_vystrazniky = "INSERT INTO PREJEZDY_VYSTRAZNIKY (id_prejezdu, id_vystrazniku) VALUES ('$id', '$id_vystrazniku'[\"id_vystrazniku]\")";
}

if (mysql_query($sql) && mysql_query($sql_rok) && mysql_query($sql_prejezdy_vystrazniky)) {
echo "<br>Aktualizace proběhla úspěšně."; }
else {
echo "<br>Aktualizace neproběhla úspěšně."; }

?>

Nedaří se mi aktualizovat pole pro typy výstražníků - skript pro aktualizaci vyhazuje tuto hlášku:

Připojen obrázek

Děkuji za případnou pomoc

David

Tady na řádku 38 nic zajímavého není. Můžná u tebe je.

PREJEZDY_VYSTRAZNIKY je vazební tabulka, ne? Proč ji máš v tom prvním update? I kdyby jsi tam chtěl dvě tabulky, tak s jinou podmínkou a před WHERE máš nadbytečnou čárku.

V tom INSERTu máš [\"id_vystrazniku]\". Ty uvozovky jsou špatně a hlavně to tam vůbec nemá být.

Funkce mysql_error vrací popis chyby. A hlavně nestrkej data z GETu přímo do SQL.

Viz gna.
- Kdyz oznacim, udelam ctrl+c ctrl+v na radku 38 neni nic vyznamneho. Resp, je to presposledni radek, ktery je uplne prazdny. Takze bys musel rici, co na tom radku mas ty.
- Array to string conversion znamena, ze v miste, kde ocekaval string, ty mu davas array. Obvykle to byva u implode, explode nebo kdyz pracujes se stringy, jako spojovani retezcu sql prikazu.
- INSERT. Tam mas ty uvozovky uplne spatne. Nevim sice, co jsi chtel docilit, ale takhle to urcite napsat nelze. mozna to je prave ten radek 38!
VALUES ('$id', '$id_vystrazniku'[\"id_vystrazniku]\")
- S tou vazebni tabulkou to myslel tak, ze logicky musis zaznam nejdriv vlozit do hlavnich tabulek a az pak jej muzes zapisovat do vazebnich, propojit id toho zaznamu s jinym id z jine tabulky. V tom pripade ti tam i schazi ziskani last_insert_id. Ale nejspis mas duvod, proc to delas prave takhle, pouzivas id z $_GET

Děkuji za rady, vyzkouším to upravit a odepíšu, jak to fungovalo   

David

#4 David
myslím, že data do základních tabulek má vytvořená a jde pouze o aktualizaci údajů, tzn přejezdy i typy výstražníků existují a jde jen o spárování.

- chybu ve values jsem viděl i já (php začátečník)

- možná by nebyla špatná kontrola zda je $id_vystrazniku nějaká hodnota

- otázkou je co udělá opakované doplňování, případně změna, kdy jeden výstražník nahradíš jiným, ve stávajícím kodu tam původní zůstane a přidá se nový případně se přidají další duplicitně pokud není v db indexem ošetřena duplicita (přejezd-výstražník)

Je to tak, jak píše MilanL, jde mi opravdu o to spárování. Mám vytvořené tabulky:

• ZELEZNICNI_PREJEZDY
• PREJEZDY_VYSTRAZNIKY (vazební tabulka)
• TYP_VYSTRAZNIKU

Chyby v kódu jsem se pokusil opravit. Neumím si poradit s tím, jak dostat typy výstražníků z formuláře (z prvku multiple select) do vazební tabulky - aby byly vloženy jednotlivé položky, nikoliv řetězec - např. 1 přejezd, 2 typy výstražníků. Vstupní formulář vypadá následovně:

Připojen obrázek.

Ještě jednou moc děkuji za váš čas a vaši pomoc.

David

#5 MilanL
Opakovanému doplňování jsem se pokusil zabránit tak, že při aktualizaci záznamu budou smazány původní vazby - do kódu jsem doplnil SQL dotaz: DELETE FROM PREJEZDY_VYSTRAZNIKY WHERE id_prejezdu='$id'.

Zda je $id_vystrazniku nějaká hodnota, jsem se pokusil vyřešit podmínkou - IsSet.

Dáváš do value id záznamu?

<option value="1">Metelesku</option>
<option value="2">Blesku</option>

#1 David
Při vkládání nového železničního přejezdu (také při modifikaci) by se neměla používat metoda GET, ale POST.

V příkladu máš zranitelost SQL Injection. Bylo by dobré to opravit dřív, než ti někdo zničí data v databázi.

Když dáš unikátní index na oba cizí klíče ve vazební tabulce, tak ti duplicity vznikat nebudou. Můžeš si vybrat, zda při pokusu o duplicitu bude druhý záznam odmítnut nebo přepíše ten první. MySQL umí obojí.

Dáváš do value id záznamu?

Ano, dávám, ale stále to píše chybu (Array to string conversion)

Select ve formuláři vypadá takto:

echo "Typ výstražníku: <select name=\"vystrazniky[]\" method=\"get\"" . $radek['id_vystrazniku']."\" multiple />";
if($vysledek_vystrazniky = mysql_query($sql_vystrazniky)) {
while ($radek_vystrazniky = mysql_fetch_array($vysledek_vystrazniky)) {
    echo "<option value='" . $radek_vystrazniky['id_vystrazniku'] . "'>" . $radek_vystrazniky['oznaceni_vystrazniku'] . "</option>";
}
}
echo "</select><br />";

#9 Kit
Děkuji za radu, zkusím to upravit.

#10 David
Tak tam, kde to píše asi děláš to, co se tam píše.

Pokud máš option takhle, tak by ti to mělo posílat ta čísla z value. Nechápu pak, co myslíš tím, že nechceš vkládat řetězce.

V tom selectu máš zase blbě uvozovky a method tam nemá co dělat. Podívej se na zdroják té stránky.

Typ výstražníku: <select name="vystrazniky[]" method="get"cislo" multiple />  <-- co to tam je?

#9 Kit
jde také o to když se výstražník změní na jiný, tak aby tam nazůstával i ten starý

Mozna by bylo jednodussi, kdyby jsi sem dal link na stranku a link na php kod te stranky. Podle toho, co jsem ted precetl, tam mas tale spoustu chyb.

Php generuje html, html umi precist www prohlizec.
Www prohlizec ma moznost Zobrazit zdrojovy kod stranky.
Html kod stranky je mozne nechat zkontrolovat validatorem.
Kdyz mas spatne html kod, pak nema smysl zabyvat se php kodem. Nejdriv se musis naucit spravne napsat html kod. Mit tam jen to, co tam ma byt. Po te muzes resit nejake slozitosti, jako propojovani selectu a podobne.

"Tak tam, kde to píše asi děláš to, co se tam píše."
Hadam, ze gna tou poznamkou myslel to, ze se stale bavime o jakemsi fiktivnim kodu. Ktery mas nekde ty, ale my tvuj disk nemame nasdileny. Ja to vzdy prirovnavam k autu ve tve garazi a mechanik ve sve dilne. On ti po telefonu nedokaze rici, co mas s autem spatne, potrebuje se na nej podivat.

Předně děkuji za pomoc, jsem začátečník v PHP. Aktuálně pracuji na hlavní tabulce (ZELEZNICNI_PREJEZDY) a aktualizačním formuláři.

http://projekt.vsps-su.cz/ita15/ita15krnavekd/ASW/zaverecna_prace/databaze.php?id=default

Dělám to částečně jako středoškolský projekt, ale dále to chci využít v rámci svého koníčku. Zadání ze školy bylo dost osekané a to by mě moc nebavilo...

Zde je PHP kód:

<?php

$id = $_POST["id"];
$cislo = $_POST["cislo"];
$sirka = $_POST["sirka"];
$delka = $_POST["delka"];
$rok_rekonstrukce = $_POST["rok_rekonstrukce"];
$foto = $_POST["foto"];
$PZZ = $_POST["PZZ"];
$obec = $_POST["obec"];
$silnice = $_POST["silnice"];
if(IsSet($_POST["vystrazniky"])) {
    $vystrazniky_vyber = implode(', ', $_POST["vystrazniky"]);
}

include("pripojeni.php");

if($rok_rekonstrukce != '' or $rok_rekonstrukce != null) {
$sql = "UPDATE ZELEZNICNI_PREJEZDY SET cislo_prejezdu='$cislo', zemepisna_sirka='$sirka', zemepisna_delka='$delka', rok_rekonstrukce='$rok_rekonstrukce', foto='$foto', ZELEZNICNI_PREJEZDY.id_PZZ='$PZZ', ZELEZNICNI_PREJEZDY.id_obce='$obec', ZELEZNICNI_PREJEZDY.id_silnice='$silnice' WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'"; 
}
else {
$sql = "UPDATE ZELEZNICNI_PREJEZDY SET cislo_prejezdu='$cislo', zemepisna_sirka='$sirka', zemepisna_delka='$delka', rok_rekonstrukce=null, foto='$foto', ZELEZNICNI_PREJEZDY.id_PZZ='$PZZ', ZELEZNICNI_PREJEZDY.id_obce='$obec', ZELEZNICNI_PREJEZDY.id_silnice='$silnice' WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'"; 
}

$sql = "UPDATE ZELEZNICNI_PREJEZDY, PREJEZDY_VYSTRAZNIKY SET cislo_prejezdu='$cislo', zemepisna_sirka='$sirka', zemepisna_delka='$delka', foto='$foto', ZELEZNICNI_PREJEZDY.id_PZZ='$PZZ', ZELEZNICNI_PREJEZDY.id_obce='$obec', ZELEZNICNI_PREJEZDY.id_silnice='$silnice' WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'"; 

if($rok_rekonstrukce != '' or $rok_rekonstrukce != null) {
$sql_rok = "UPDATE ZELEZNICNI_PREJEZDY SET rok_rekonstrukce='$rok_rekonstrukce' WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'";
}
else {
$sql_rok = "UPDATE ZELEZNICNI_PREJEZDY SET rok_rekonstrukce=null WHERE ZELEZNICNI_PREJEZDY.id_prejezdu='$id'";
}
if(IsSet($vystrazniky_vyber)) {
    $vystrazniky[] = explode(', ', $vystrazniky_vyber);
}
$sql_prejezdy_vystrazniky_smazat = "DELETE FROM PREJEZDY_VYSTRAZNIKY WHERE id_prejezdu='$id'";

if (IsSet($vystrazniky))
 {
    foreach($vystrazniky AS $id_vystrazniku) {
        $sql_prejezdy_vystrazniky = "INSERT INTO PREJEZDY_VYSTRAZNIKY (id_prejezdu, id_vystrazniku) VALUES ('$id', '$id_vystrazniku'[\"id_vystrazniku\"])";
        mysql_query($sql_prejezdy_vystrazniky);
    }
    if (mysql_query($sql) && mysql_query($sql_rok) && mysql_query($sql_prejezdy_vystrazniky_smazat)) {
    echo "<br>Aktualizace proběhla úspěšně."; }
    else {
    echo "<br>Aktualizace neproběhla úspěšně."; }
}
else {
    if (mysql_query($sql) && mysql_query($sql_rok)) {
    echo "<br>Aktualizace proběhla úspěšně."; }
    else {
    echo "<br>Aktualizace neproběhla úspěšně.";
    echo $id . "\r\n";
    echo $cislo . "\r\n";
    echo $sirka . "\r\n";
    echo $delka . "\r\n";
    echo $rok_rekonstrukce . "\r\n";
    echo $foto . "\r\n";
    echo $PZZ . "\r\n";
    echo $obec . "\r\n";
    echo $silnice . "\r\n";
     }
}


?>

Tu konverzi pole na řetězec máš tady.

('$id', '$id_vystrazniku'[\"id_vystrazniku\"])

Kdyby jsi chtěl chtěl hodnotu z pole, tak by to šlo napsat jako

... '{$pole['klic']}' ...

Ale ty tam chceš $id_vystrazniku, tak to tak napiš.

... '$id_vystrazniku' ...

Z toho update vyhoď tu vazební tabulku. Nic tam nevážeš. Prostě uprav data přejezdu v jeho tabulce a doplň vazby v té vazební tabulce.

A ještě jsem zapomněl napsat proč je to pole.

Takovéhle přiřazení přidává položky do pole.

$vystrazniky[] = explode(', ', $vystrazniky_vyber);

Takže v poli $vystrazniky máš jednu položku, která je taky pole. Vyhoď ty závorky []. Celkově nevidím důvod to prohánět přes implode-explode.

Velice vám děkuji za veškerou vaši pomoc. Speciál dík pro gna, díky Tvým posledním radám jsem to spatlal dohromady a vše funguje, jak má    Pár věcí mě ještě potrápilo, práce do školy je hotová, ale budu na tom dělat dál   . Až budu v koncích, přijdu zas pro radu. Mějte se