Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [MySQL][PHP]Sprawdzenie który submit został wciśnięty.
Croos22
post 18.08.2011, 10:06:46
Post #1





Grupa: Zarejestrowani
Postów: 316
Pomógł: 4
Dołączył: 22.03.2010

Ostrzeżenie: (0%)
-----


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.

  1. if(isset($sklep["id"])){
  2. echo 'Kupiono '.$sklep["nazwa"];
  3. }
  4.  
  5. echo '<input type="submit" name="'.$sklep["id"].'" value="Kup" />';


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
Go to the top of the page
+Quote Post
skowron-line
post 18.08.2011, 10:12:22
Post #2





Grupa: Zarejestrowani
Postów: 4 340
Pomógł: 542
Dołączył: 15.01.2006
Skąd: Olsztyn/Warszawa

Ostrzeżenie: (0%)
-----


array_keys


--------------------
I'm so fast that last night I turned off the light switch in my hotel room and was in bed before the room was dark - Muhammad Ali.
Peg jeżeli chcesz uprawiać sex to dzieci muszą wyjść, a jeżeli chcesz żeby był dobry ty też musisz wyjść - Al Bundy.

QueryBuilder, Mootools.net, bbcradio1::MistaJam
http://www.phpbench.com/
Go to the top of the page
+Quote Post
Croos22
post 18.08.2011, 10:29:51
Post #3





Grupa: Zarejestrowani
Postów: 316
Pomógł: 4
Dołączył: 22.03.2010

Ostrzeżenie: (0%)
-----


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

Zrobiłem tak:
  1. $tablica = array($sklep["id"]);
  2. print_r(array_keys($tablica));


no i lipa
Array ( [0] => 0 ) Array ( [0] => 0 )
Go to the top of the page
+Quote Post
tehaha
post 18.08.2011, 10:36:21
Post #4





Grupa: Zarejestrowani
Postów: 1 748
Pomógł: 388
Dołączył: 21.08.2009
Skąd: Gdynia

Ostrzeżenie: (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
Go to the top of the page
+Quote Post
Croos22
post 18.08.2011, 11:09:02
Post #5





Grupa: Zarejestrowani
Postów: 316
Pomógł: 4
Dołączył: 22.03.2010

Ostrzeżenie: (0%)
-----


No to niech będzie. Jeszcze prosił bym o rzucenie okiem na ten kod. Jak uczynić to bardziej bezpieczniejszym?
  1. if($_GET['rodzaj'] == $_GET['rodzaj']){
  2. if((int)$_GET['kup'] == (int)$sklep['id']){
  3. echo 'Kupiono przedmiot '.$sklep['nazwa'].' za '.(int)$sklep['cena'].'zł.';
  4. }
  5. }
  6.  
  7. echo '<a href="?rodzaj='.$_GET["rodzaj"].'&kup='.(int)$sklep["id"].'">Kup</a>';


Ten post edytował Croos22 18.08.2011, 11:18:11
Go to the top of the page
+Quote Post
Fifi209
post 18.08.2011, 11:11:58
Post #6





Grupa: Zarejestrowani
Postów: 4 655
Pomógł: 556
Dołączył: 17.03.2009
Skąd: Katowice

Ostrzeżenie: (0%)
-----


Po pierwsze w form może być chyba tylko jeden submit o ile mi się dobrze kojarzy smile.gif


--------------------
Zainteresowania: C#, PHP, JS, SQL, AJAX, XML, C dla AVR
Chętnie pomogę, lecz zanim napiszesz: Wujek Google , Manual PHP
Go to the top of the page
+Quote Post
nospor
post 18.08.2011, 11:16:26
Post #7





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004




Cytat
Po pierwsze w form może być chyba tylko jeden submit o ile mi się dobrze kojarzy
źle ci sie kojarzy smile.gif

if($_GET['rodzaj'] == $_GET['rodzaj'])
To jest dobre smile.gif

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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
Croos22
post 18.08.2011, 11:52:24
Post #8





Grupa: Zarejestrowani
Postów: 316
Pomógł: 4
Dołączył: 22.03.2010

Ostrzeżenie: (0%)
-----


nospor
  1. <a href="?rodzaj=cos1&kup=1">Kup</a>
  2. <a href="?rodzaj=cos2&kup=1">Kup</a>

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
Go to the top of the page
+Quote Post
nospor
post 18.08.2011, 11:53:57
Post #9





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004





$_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 wink.gif


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 14.08.2025 - 11:50