Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> Prosty system rejestracji
Strarus
post 23.02.2009, 15:54:12
Post #1





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


cześć:)
Chciałbym abyście ocenili pod względem bezpieczeństwa i praktyczności mój skrypt rejestracji, który znajduje się tu:

http://strarus.cba.pl/testy/register.php
Go to the top of the page
+Quote Post
bim2
post 23.02.2009, 15:56:37
Post #2





Grupa: Zarejestrowani
Postów: 1 873
Pomógł: 152
Dołączył: 9.04.2006
Skąd: Berlin

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


Może podaj jak długie ma być hasło.... Wrrr ;/ Uzupełniłem dane, klikam rejestruj i "Hasło za krótkie" Dalej nie testuje, bo się wkruzyłem smile.gif

PS. Po co ta captcha?


--------------------
Go to the top of the page
+Quote Post
Foxx
post 23.02.2009, 16:01:11
Post #3





Grupa: Zarejestrowani
Postów: 896
Pomógł: 76
Dołączył: 15.11.2003
Skąd: Sosnowiec/Kraków

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


Komunikat mógłby się wyświetlać na tej samej stronie co formularz albo chociaż z linkiem powrotnym.
Nie sprawdzasz poprawności adresu e-mail.
Go to the top of the page
+Quote Post
.radex
post 23.02.2009, 16:01:35
Post #4





Grupa: Zarejestrowani
Postów: 1 657
Pomógł: 125
Dołączył: 29.04.2006

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


bezsensowna CAPTCHA - łatwo ją złamać (stała czcionka, stały kolor, stałe tło)


--------------------
blog | Tadam — minutnik do Pomodoro na Maka :)
Go to the top of the page
+Quote Post
Strarus
post 23.02.2009, 19:33:55
Post #5





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


OK, już poprawiłem walidację adresu email i dodałem link powrotny przy błędach. Proszę o sprawdzenie czy potwierdzanie rejestracji działa.
Go to the top of the page
+Quote Post
phpion
post 23.02.2009, 20:03:24
Post #6





Grupa: Moderatorzy
Postów: 6 072
Pomógł: 861
Dołączył: 10.12.2003
Skąd: Dąbrowa Górnicza




Walidacja danych daremna i wkurzająca - za każdym razem muszę wypełniać cały formularz.
Go to the top of the page
+Quote Post
Strarus
post 24.02.2009, 13:14:09
Post #7





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


Ale jak zła walidacja danych? Chodzi Ci o to, że po kliknięciu w link powrotny pola są puste?
Go to the top of the page
+Quote Post
phpion
post 24.02.2009, 13:42:23
Post #8





Grupa: Moderatorzy
Postów: 6 072
Pomógł: 861
Dołączył: 10.12.2003
Skąd: Dąbrowa Górnicza




Cytat(Strarus @ 24.02.2009, 15:14:09 ) *
Chodzi Ci o to, że po kliknięciu w link powrotny pola są puste?

Tak.
Go to the top of the page
+Quote Post
marian12
post 24.02.2009, 13:44:10
Post #9





Grupa: Zarejestrowani
Postów: 1
Pomógł: 0
Dołączył: 24.02.2009

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


dobrze by bylo jak bys dodal wysylanie potwierdzenia na email
Go to the top of the page
+Quote Post
Strarus
post 24.02.2009, 14:15:55
Post #10





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


Przecież wysyłanie potwierdzenia i link aktywacyjny są? A w jaki sposób zrobić te wypełnione pola? Dać wszystko do sesji i tylko wczytywać potem dane?
Go to the top of the page
+Quote Post
zegarek84
post 24.02.2009, 14:40:04
Post #11





Grupa: Zarejestrowani
Postów: 1 332
Pomógł: 294
Dołączył: 12.10.2008
Skąd: Olkusz

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


jeśli ma to być tylko i wyłącznie w php to w sesji - ale możesz też walidację danych w js i nie dopuścić do wysłania niepoprawnych danych - co nie zmienia faktu że i tak walidacja musi być też po stronie php gdyż ktoś może mieć wyłączoną obsługę js

Cytat(Strarus @ 24.02.2009, 14:15:55 ) *
Przecież wysyłanie potwierdzenia i link aktywacyjny są? A w jaki sposób zrobić te wypełnione pola? Dać wszystko do sesji i tylko wczytywać potem dane?

ale daruj subie w sumie też sesje winksmiley.jpg - tak na szybko to przecież masz wszystkie potrzebne dane po przesłaniu do skryptu sprawdzającego dane - po prostu jeśli dane nie będą poprawne to je wyświetl winksmiley.jpg - czyli trzymaj razem skrypt formularza i skrypt sprawdzający (jak nie przejdzie to wyświetlisz znowu formularz na false - możesz kasować niepoprawne dane i zaznaczać które jeszcze)


--------------------
Jeśli twoja ręka rusza do przodu powstrzymaj swój gniew; gdy wyprzedza cię twój gniew - wycofaj rękę.

Go to the top of the page
+Quote Post
bliitz
post 24.02.2009, 14:43:23
Post #12





Grupa: Zarejestrowani
Postów: 155
Pomógł: 12
Dołączył: 5.04.2008

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


jeśli uzupełnię źle wszystkie pola to i tak zawsze najpierw informuje o złym emailu, powinien chyba informować o błędach w kolejności ich występowania


--------------------
Go to the top of the page
+Quote Post
Strarus
post 24.02.2009, 14:54:20
Post #13





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


Do zegarek84: czyli po prostu tam gdzie sprawdzam poprawność dawać żeby wyświetlił formularz?
Do bliitz: ponieważ sprawdzam je po kolei i w pierwszej kolejności jest email, a jeśli chciałbym wyświetlać wszystkie błędy, to w jaki sposób, bo gdy nie korzystam z break; to skrypt dodaje niepełne, bądź fałszywe dane do bazy?
Go to the top of the page
+Quote Post
zegarek84
post 24.02.2009, 15:12:12
Post #14





Grupa: Zarejestrowani
Postów: 1 332
Pomógł: 294
Dołączył: 12.10.2008
Skąd: Olkusz

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


Cytat(Strarus @ 24.02.2009, 14:54:20 ) *
Do zegarek84: czyli po prostu tam gdzie sprawdzam poprawność dawać żeby wyświetlił formularz?
Do bliitz: ponieważ sprawdzam je po kolei i w pierwszej kolejności jest email, a jeśli chciałbym wyświetlać wszystkie błędy, to w jaki sposób, bo gdy nie korzystam z break; to skrypt dodaje niepełne, bądź fałszywe dane do bazy?

możesz użyć jakichś dodatkowych zmiennych zapamiętujących wartość true bądź false dla danego pola formularza, sprawdzić najpierw wszystko i przy koniunkcji na true (AND) wszystkich zmiennych/bądź warunków - to już Twoja koncepcja - zapisać dane z formularza (czyli zapisać jeśli wszystkie warunki jednocześnie zostaną spełnione), z kolei gdy jeden z warunków nie zostanie spełniony to zmienne z formularza po obróbce (zostawiasz np. tylko poprawne dane a niepoprawne dajesz jako pusta zmienna) przekazujesz dalej i wyświetlasz formularz z wartościami pól odpowiadającymi poprawnym zmiennym....

Ten post edytował zegarek84 24.02.2009, 15:13:17


--------------------
Jeśli twoja ręka rusza do przodu powstrzymaj swój gniew; gdy wyprzedza cię twój gniew - wycofaj rękę.

Go to the top of the page
+Quote Post
Strarus
post 24.02.2009, 15:21:27
Post #15





Grupa: Zarejestrowani
Postów: 339
Pomógł: 12
Dołączył: 22.11.2008

Ostrzeżenie: (20%)
X----


nie rozumiem... sad.gif ?
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 - 09:53