[MySql][PHP] Formularz rejstracji, Do sprawdzenia Opcje

[artdem]

Witam,

W pocie czoła pracuję nad uruchomieniem formularza rejestracyjnego na mojej stronie. Niestety gotowy skrypt, którym dysponowałem posiada wiele wad i niestety muszę go poprzerabiać. Niestety moja znajomość PHP jest znikoma w związku z czym potrzebuję, aby ktoś fachowym okiem rzucił okiem na efekty mojej pracy.

Poniżej przesyłam kod do formularza:

[PHP] pobierz, plaintext 
<?  session_start();
include('connect.php');
$result = mysql_query("SELECT * FROM sitesettings");
while($row = mysql_fetch_array($result))
$title = "".$row["domain"]." - Zarejestruj się";
include "header.php";
$usrv = 'Wprowadź nazwę użytkownika';
$usrt = 'Nazwa użytkownika zajęta';
$usri = 'Niedozwolone znaki';
$passv = 'Wprowadź hasło';
$passn = 'Wprowadzone hasła się nie zgadzają';
$emailv = 'Wprowadź prawidłowy adres e-mail';
$emailt = 'Podany adres został już użyty';
$pattern = "/^([a-zA-Z0-9_])+$/";
$patternm = "/^[-0-9a-zA-Z_\.]+@([-0-9a-zA-Z_\.]+\.)+([0-9a-zA-Z]){2,4}$/i";
$usrn = $_POST['login'];
$usrm = $_POST['email'];
$login = mysql_real_escape_string($_POST['login']);
$pass = mysql_real_escape_string(md5(sha1($_POST['pass']))); 
$pass2 = mysql_real_escape_string(md5(sha1($_POST['pass2'])));
$email = mysql_real_escape_string($_POST['email']);
$logincheck = "SELECT COUNT(*) FROM username WHERE username='$login'";
$idlogincheck = mysql_query($logincheck);
$loginz = mysql_fetch_row($idlogincheck);
$mailcheck = "SELECT COUNT(*) FROM username WHERE email='$email'";
$idmailcheck = mysql_query($mailcheck);
$mailz = mysql_fetch_row($idmailcheck);
?>
[PHP] pobierz, plaintext 

Trochę przekombinowane chyba, no ale tak mi wyszło. Całą resztę, czyli wpisywanie danych do bazy i wysyłanie maila z kluczem jest załatwione przez plik verify.php. W pierwotnej wersji działało ok, ale po moich poprawkach nie działa. W skrypcie jest jeden kod PHP, który, jak mi się wydaje, odsyła do tego pliku:

[PHP] pobierz, plaintext 
<?
                        ob_start();
                        $pattern = "/^([a-zA-Z0-9_])+$/";
                        $usrn = $_POST['login'];
                        if(isset($_POST['register'])){
                            if(empty($_POST['login']) || $loginz >0 || $mailz >0 || empty($_POST['pass']) || $_POST['pass2'] != $_POST['pass'] || $_POST['email'] == "" || !preg_match($pattern,$usrn)){
                            }else{
                            $usname = strtolower($_POST['login']);
                            $uspass = $_POST['pass'];
                            $uspassv = $_POST['pass2'];
                            $emai = strtolower($_POST['email']);
                            $sr = "1";
                            $_SESSION['login'] = $usname;
                            $_SESSION['pass'] = $uspass;
                            $_SESSION['pass2'] = $uspassv;
                            $_SESSION['email'] = $emai;
                            $_SESSION['status'] = $sr;
                            header( 'Location: verify.php' ) ;
                            }
                        };
                        ob_flush();
?>
[PHP] pobierz, plaintext 

Nigdzie indziej w kodzie nie pojawia "verify.php", dlatego wnioskuję, że za odsyłanie do tej strony odpowiada właśnie ten fragment kodu. Tylko, że teraz to odsyłanie nie działa i nie wiem dlaczego. Proszę o pomoc.

[piszczu]

Jeżeli dobrze zrozumiałem to ten cały kod to 1 plik. Zmienne loginz i mailz to tablice, a tablice zawsze są większe od zera.
Błędy masz przy przypisywaniu wartości tym zmiennym

[PHP] pobierz, plaintext 
$logincheck = "SELECT COUNT(*) FROM username WHERE username='$login'";
$idlogincheck = mysql_query($logincheck);
$loginz = [color="#FFFF00"]intval(end[/color](mysql_fetch_row($idlogincheck)));
[PHP] pobierz, plaintext 

I analogiczna sytuacja przy $mailz. intval() może być zbędny, nie pamiętam czy jest automatyczne rzutowanie. Tablice to zbiory, a jak wiadomo nie istnieje coś takiego jak pusty zbiór, są zbiory z pustym elementem.

A co do include to zobacz jak są zbudowane.

Ten post edytował piszczu 10.05.2011, 17:48:47