Forum PHP.pl

Dobry.

Kombinuje coś zmienić w moim sklepie na coś bardziej wydaje mi się bezpiecznego. Teraz używam $_GET a chciałbym to zmienić na submity.

Próbowałem zrobić tak, że w name dam wyciągnięty z bazy numer unikalny id a następnie sprawdzić isset-em który submit został wciśnięty.

[PHP] pobierz, plaintext 
if(isset($sklep["id"])){
echo 'Kupiono '.$sklep["nazwa"];
}
 
echo '<input type="submit" name="'.$sklep["id"].'" value="Kup" />';
[PHP] pobierz, plaintext 

Jak już teraz wiem nie był to dobry pomysł dlatego liczę na jakieś podpowiedzi. Z góry dzięki.

Ten post edytował Croos22 18.08.2011, 10:09:12

array_keys

Nie bardzo rozumie o co chodzi. Ma to zwracać wszystkie klucze z tablicy w moim przypadku identyfikatory.

Zrobiłem tak:

[PHP] pobierz, plaintext 
$tablica = array($sklep["id"]);
print_r(array_keys($tablica));
[PHP] pobierz, plaintext 

no i lipa
Array ( [0] => 0 ) Array ( [0] => 0 )

bez sensu pomysł, chcesz wszystkie przyciski kup zamienić teraz na submity? Nie tędy droga, to wcale nie będzie bardziej bezpieczne tylko bardziej przekombinowane, mogę wysłać na Twoją stronę $_POST, zrób normalne linki tak jak było, przy odbieraniu wartości możesz dać rzutowanie na int -> $id = (int)$_GET['id'], i sprawdzić w bazie czy taki produkt istnieje i jest dostępny.

p.s. ten array keys to miałeś dać na tablicę $_POST, żeby sprawdzić co zostało przesłane

No to niech będzie. Jeszcze prosił bym o rzucenie okiem na ten kod. Jak uczynić to bardziej bezpieczniejszym?

[PHP] pobierz, plaintext 
if($_GET['rodzaj'] == $_GET['rodzaj']){ 
if((int)$_GET['kup'] == (int)$sklep['id']){
echo 'Kupiono przedmiot '.$sklep['nazwa'].' za '.(int)$sklep['cena'].'zł.';
}
}
 
echo '<a href="?rodzaj='.$_GET["rodzaj"].'&kup='.(int)$sklep["id"].'">Kup</a>';
[PHP] pobierz, plaintext 

Ten post edytował Croos22 18.08.2011, 11:18:11

Po pierwsze w form może być chyba tylko jeden submit o ile mi się dobrze kojarzy (IMG:style_emoticons/default/smile.gif)

źle ci sie kojarzy (IMG:style_emoticons/default/smile.gif)

if($_GET['rodzaj'] == $_GET['rodzaj'])
To jest dobre (IMG:style_emoticons/default/smile.gif)

A co jeśli białe jest białe?

nospor

[HTML] pobierz, plaintext 
<a href="?rodzaj=cos1&kup=1">Kup</a>
<a href="?rodzaj=cos2&kup=1">Kup</a>
[HTML] pobierz, plaintext 

Jeśli nie sprawdzam $_GET["rodzaj"] == $_GET["rodzaj"] to nie spełniało by mi $_GET["kup"] == $sklep["id"].

Ten post edytował Croos22 18.08.2011, 11:53:01

$_GET["rodzaj"] == $_GET["rodzaj"]

Równie dobrze możesz napisać:
2==2

Efekt będzie ten sam. Włącz myślenie, potem ze mną dyskutuj (IMG:style_emoticons/default/wink.gif)