INSERT INTO

Dobrý deň,
mam pre niekoho asi primitivny problem:



riadok 23 je riadok kde zacina VALUES


Ďakujem za pomoc.

v prvom rade to takto určite nerob, pretože vôbec neošetruješ nebezpečný vstup.. (vkladáš do DB priamo čo ti naservíruje užívateľ - teda v prípade, ak si ich nikde neošetril, čo sa z tohto skriptu nedá vidieť, ale radšej to spomeniem). A po druhé, to isté sa dá napísať jednoduchším spôsobom:



podľa mňa je to prehľadnejšie, a výsledok je rovnaký...


inak to, čo ti spôsobuje problémy pri tom query je zápis premenných. Buď ošetri tie premenné a pri tom ich vlož do klasických, čiže nie POST a tak vkladaj, alebo skús potom tie premenné napísať v štýle $_POST[nazov], čiže bez vnútorných úvodzoviek.. to by tiež mohlo fungovať.

V každom prípade daj vedieť či to už ide

walther, nevyzerá to, že sa vyznáš - Tvoj spôsob na vkladanie funguje pri úprave, pri vkladaní zrejme nie(neštudoval som to ale pochybujem, že to ide.) A nikdy si nepočul o takom, že string musí byť vždy v úvodzovkách? Myslím, že tvoje rady mu veľmi nepomôžu.

NickVK, skús si to rozdeliť na viacero riadkov a podľa toho zistíš kde je chyba a keď nebudeš vedieť tak hoď sem po rozdelení, pretože teraz nevidím nikde chybu.

flety, prosím ťa, najskôr to vyskúšaj, potom posudzuj či to ide

to prvé s tým insertom ide určite, istý si nie som jedine pri tom spôsobe bez úvodzoviek

Odporucam si naprv ulozit Tvoje informacie dopremennych a nie ich rovno hadzat do databazy neoserene. Su to zle zvyky urcitych "programaorov" odktorych si to asi obkukal.
Takze najprv si to hod do premennych. Rada citlive info by som cez globalnu premennu POST nedaval a oseri si to na HTML tagy napr. regularnym vyrazom a samozrejme podla chuti.

Potom ukladaj ako spominal kolega waltherold

Snacker: riadny hlod ... ošetrovať je správne, ale kľudne to môžeš ošetrovať už v POST aspoň budeš mať v premenných väčší poriadok.
NickVK: tvoje SQL by malo vyzerať takto (ak som sa nikde nepomýlil).

tominator, vezmime si prípad viacerých programátorov v jednom teame a nejaký zložitejší program... keď sa pozrieš na ten skript spätne, vieš si byť 100% istý, že je tá premenná $_POST ošetrená? tým, že uložíš ošetrenú hodnotu do lokálnej premennej skript síce o dačo spomalíš (naozaj to neucítiš), ale budeš mať v každom prípade istotu, že sa jedná o lokálnu premennú, ktorá ak obsahuje hodnotu, je na 100% ošetrená... poznám x ľudí, čo tiež takto hovorili "veď to netreba, to je zbytočné" atď.. a potom sa zabudlo na maličkosť a ... ale však koľko ľudí, toľko chutí
inak by ma seriózne zaujímalo, ako vyzerá taký "poriadok v premenných", keď si nemôžeš byť hneď istý, či to už ošetrené je alebo nie
samozrejme netreba zabúdať na to, že zápis $premenna je o poznanie kratší ako $_POST['premenna']

walther: nechcem ťa uraziť, neber to v zlom, ale to nesvedčí o mojom neporiadku ale o tvojej zaostalosti.

Kontrola prebieha automaticky. Pozri si napr. Často používané scripty a stenleyho užitočnú funkciu Safety. Jednoducho do sckriptu sa ti ani reálne nedostane hodnota, ktorá by nebola ošetrená. Preto si nepotrebujem pamätať ďalších xy premenných ale pracujem s originálom. Túto automatickú bezpečnosť môžeš kedykoľvek zvýšiť alebo upraviť. Môžeš založiť na Blackliste i Whiteliste, jednoducho vyrobíš si filter.

dobre, ja som zaostalý, ty si king
ale v poriadku, keď berieš za jednoduchšie písať 100x $_POST['premenna'] miesto $premenna, tak budiž. Česť vidieť tvoje výtvory som už mal...

Myslím, že nemal (Teda nie moderné) Je tu pár ľudí na fóre, ktorý tú česť majú a ver, že mi dosvečia, že práca je efektívna

drzte sa temy! ak nemate k veci co povedat, neprispievajte!

Ja rad pouzivam sprintf()

p.s. máš tu sql injection ako krava
http://en.wikipedia.org/wiki/SQL_injection

Ďakujem, kod ktory mi dal Tominator uz nechadze chybu ale ked overujem ci vsetko prebehlo ok tak mi if hodi false
tzn: po odoslani dat z formulara vypise*:

aby ste nepovedali ze neviem pocitat alebo ze som slepy tak:


Ďakujem

*aj keď vypišem stlpce kam to ma dat

za '".$_POST['banka']."' ti (mi) chýba čiarka

Ďakujem, už všetko funguje .