Foreach SQL

V řádku 15 chybí uvozovky a tečky:

Nepochopí náhodou php tento apostrof jako ukončení stringu?

Tomasds
Jo, to jsem přehlédl.

kromě toho.. neznám logiku té aplikace, ale nějak se mi nechce zdát, že by mělo smysl ukládat do id_objednavky prázdný string. Nula by byla trochu zvláštní, ale představit si to umím. Takto bude nejspíš chyba, už ne syntaktická na úrovni php ale na úrovni sql.

smysl ukládat do id_objednavky prázdný string
To bude sloupec s nastaveným auto increment, tam to projde, ovšem nemělo by se to takto psát.

nightrider
Když dám prázdnej, tak auto_increment přičte id objednávky,ne?

To nevím, možná to projde, jak píše Mastodont, nezkoušel jsem to. Ale je nesmysl to psát.
Jestli ten kód nějak upravuješ a nefunguje, dej sem aktuální verzi, ať se v tom dá orientovat. Ne všechno, stačí ten inkriminovaný řádek.

Mastodont „To bude sloupec s nastaveným auto increment“
Myslíš?

Mastodont
To bude sloupec s nastaveným auto increment,
Když se na to dívám, tak spíš nebude. Mám pocit, že nightrider trochu tápe.

nightrider
Ten sloupec id_objednavky z INSERTu úplně vypusť, být tam nemusí (a nemá).

nightrider
Ujasni si, co od toho chceš. V session máš košík, po jeho potvrzení chceš vytvořit objednávku. To znamená že potřebuješ tabulku objednávek, kde bude asi jen ID a ID_zakaznika. A pak potřebuješ tabulku polozky_v_objednavkach, kde bude ID_objednavky a pak jednotlivé položky objednávky. Nejjednodušší by bylo tam nastrkat jen jejich ID a aktuální cenu, případné další informace joinovat.
Teď mě napadá, že i v té tabulce objednávek by to chtělo místo ID_zakaznika jeho komplet aktuální adresu, protože se ta se může změnit.

nightrider
Tohle všechno už je
Není. Ty snad chceš mít v databázi deset objednávek, když si zákazník objedná deset věcí?

Opakuju: jedna tabulka: objednavky. Po potvrzení košíku vytvoříš záznam, kde bude ID (samozřejmě automaticky vytvořené) a jméno a adresa zákazníka.
druhá tabulka: polozky_v_objednavkach. Tam bude ID (pouze pro účely administrace), IDobjednavky, IDzbozi, cena

Je poznámku ke kódu, jak ho udělat krásnější:


Druhá věc, která mě tam bije do očí je, že se v druhém foreach nepoužívá $val, je to v pořádku?
A ještě hint - je lepší sestavit v tom druhém foreach jeden dotaz a pak ho odeslat najednou:
Místo n-krát

použít dotaz


Co se týče tabulek, podpořím mého jmenovce a jím navrhovanou strukturu.

Možná složitější, asi o 10 řádků. Ale určitě lepší než jedna tabulka pro všechno. Za tu dobu, co se o tom bavíme, už jsi to mohl mít napsaný.

Složitější možná při vkládání (a řekl bych, že nepatrně), ale má to několik podstatných výhod:
- představ si situaci, že budeš chtít měnit email zákazníka. Tvůj návrh musí změnit spousty řádků, Tomasdsův jeden řádek v tabulce zákzníci.
- tvá databáze bude několikanásobně větší - jméno zákazníka nebo produktu tam bude uloženo spoustakrát na jednom místě.
- některé dotazy budou při tvém návrhu dramaticky náročnější na databázi - třeba seznam zákazníků
- uložení zákazníka, který si nic neobjednal (nebo např. zrušil objednávku) v tvém případě nejde (nebo jen s obtížemi)

Aneb zkušenosti říkají, že stojí za to to změnit ;-)

Jak říká Tomasds
Tabulky:
zakaznici(id, jmeno, prijmeni, adresa, ...)
produkty(id, nazev_produktu, ...)
objednavky(id, zakaznik_id)
polozky(id, objednavka_id, produkt_id)

Seznam produktů bys měl naplnit sám produkty, které prodáváš. Nechť jsem pak na stránce, kde se ukládají objednávky. Pokud přijde nový zákazník, vložíš ho do tabulky zákazníci. Pokud už existuje, zjistíš jen jeho id. Vytvoříš pro zákazníka novou objednávku (=záznam v tabulce objednávky) a zapamatuješ si její id jako id_objednavky. Pro každý kus zboží, který má v košíku, vytvoříš jeden záznam v tabulce polozky, objednavka_id použiješ to, které sis zapamatoval.

Zkusím někde najít link - postup, jak vytvořit podobnou strukturu je někde určitě popsaný, já si ho nevymýšlím, jen aplikuju

Edit: http://cs.wikipedia.org/wiki/3NF - tvé řešení je na úrovni druhého pokusu, Tomasdsovo na úrovni třetího pokusu

Škatulkovat zákazníky je potřeba - ale podle IP bych to nedělal - může být víc stejných jmenovců pod jednou IP. Zkusil bych třeba kombinaci jména, příjmení a telefonu nebo emailu, to by mělo být unikátní.

Někde určitě jo, free eshopů je hodně :) Kód po ruce bohužel nemám...

Při odesílání objednávky uděláš v tomto pořadí:
- zkusíš najít v databázi zákazníka, pokud tam není, vytvoříš ho
- vytvoříš nový záznam v tabulce objednávky, který bude obsahovat id nalezeného zákazníka
- v cyklu pro každou položku si [ najdeš id produktu, vložíš záznam do tabulky polozky, ktery bude obsahovat id produktu a id objednavky ]

V tabulce produkty bys měl mít seznam produktů, které nabízíš, ten vložíš dřív než přijde první zákazník.

Je to víc akcí než předtím, ale zdá se mi, že žádná z nich není příliš těžká. Když tohle přepíšeš do kódu, tak si vyhrál :)

nightrider
Je otázka, jestli předpokládáš, že se ti zákazníci budou vracet a tudíž má smysl jim umožňovat nějakou registraci a pozdější přihlášení.
Pokud ne, tak tabulka zakaznici(id, jmeno, prijmeni, adresa, ...) , jak ji navrhuje TomášK, nemá smysl a v tabulce objednavky nebude zakaznik_id, ale komplet adresa. Píšu to z toho důvodu, že by se ti to tím všecho zjednodušilo o registraci a přihlašování. TomášK to samozřejmě myslí dobře a jeho návrh je správný.

Takže v té jednodušší variantě, o které píšu, je postup tento. Zákazník není přihlášen (protože možnost přihlášení neexistuje). Potvrdí, že má nakoupeno a chce dokončit objednávku. Tam vypíše adresu a formulář odešle.
Ty vytvoříš nový řádek v tabulce objednavky (INSERT) a tam vložíš adresu zákazníka. Zjistíš id právě vloženého záznamu (mysql_insert_id) a do tabulky polozky vložíš toto id a id objednaných produktů. A to je asi tak všechno, tádydádydá.

Propracovanější varianta předpokládá přihlašování, sessions a do objednávek nevkládá jméno, ale ID zákazníka.

TomášK
Díky za spolupráci a za tu wiki, dozvěděl jsem se něco teorie na téma OLAP vs. OLTP :-)