Bạn cần thay đổi thông tin giống như thông tin trong MySQL của bạn mới chạy đc. :)]]> /hvaonline/posts/list/398.html#1396 /hvaonline/posts/list/398.html#1396 GMT Code:

// Nếu username này không tồn tại thì.... 
if ( @mysql_num_rows( $sql_query ) <= 0 )

Mình thì chưa thấy @mysql_num_rows( $sql_query ) <0 bao giờ nên cũng hơi buồn cười khi thấy code này . . . thường thì mình làm là Code:

if ( @mysql_num_rows( $sql_query ) == 0 )

Thứ 2 : Để đỡ tốn thời gian của user cũng như "bớt việc" cho server, mình kiểm tra toàn bộ thông tin người dùng ngay trên trang đăng kí, chứ không để lên trên server mới làm. Như vậy theo mình sẽ bỏ: Code:

// Kiểm tra 4 thông tin, nếu có bất kỳ thông tin chưa điền thì sẽ báo lỗi 
if ( ! $username || ! $password || ! $verify_password || ! $email ) 
{ 
print "Xin vui lòng nhập đầy đủ các thông tin. <a href='javascript:history.go(-1)'>Nhấp vào đây để quay trở lại</a>"; 
exit; 
} 

// Kiểm tra mật khẩu, bắt buộc mật khẩu nhập lúc đầu và mật khẩu lúc sau phải trùng nhau 
if ( $password != $verify_password ) 
{ 
print "Mật khẩu không giống nhau, bạn hãy nhập lại mật khẩu. <a href='javascript:history.go(-1)'>Nhấp vào đây để quay trở lại</a>"; 
exit; 
}

Cũng vừa để cho chắc ăn, vừa chống flood, vừa kiểm tra được user đã đăng kí hay chưa (cái kiểm tra user exists trên code chưa có à nha), em dùng một type=hidden, name="anti_flood", value=một dẫy số do em tạo ra(theo một quy luật nào đó mà mình có thể kt được nó đúng hay sai). Nếu như dẫy số này trên server kt là đúng thì sẽ : Code:

$kt_user = @mysql_query("SELECT id FROM members WHERE username='$username'"); 
if(@mysql_num_rows($kt_user)!=0)
{
echo "UserName này đã tồn tại. Mời bạn chọn User Nam khác !";
exit;
}
else 
{
// Tiến hành tạo tài khoản 
@mysql_query("INSERT INTO members (username, password, email) VALUES ('{$username}', '{$password}', '{$email}')"); 

// Thông báo hoàn tất việc tạo tài khoản 
print "Tài khoản {$username} đã được tạo. <a href='login.php'>Nhấp vào đây để đăng nhập</a>"; 
}

Còn việc Code:

if ( !$_SESSION['user_id'] ) 
{ 

print <<<EOF 
Bạn chưa đăng nhập! <a href='login.php'>Nhấp vào đây để đăng nhập</a> 
EOF; 

}

Thì chuối một tí (sorry, nếu em sai !). Như mọi người đã biết giá trị của session sẽ xuyên suốt trong quá trình user làm việc với 1 cửa sổ, cho dù user chuyển từ trang này sang trang kia, trừ khi giá trị đó bị thay đổi hoặc phá huỷ. Như vậy chỗ này, em chỉ cần tạo một cái Code:

$_SESSION['user_id']=1;//thường của admin :D

trên máy (hay trên host) rồi cũng từ cửa sổ đó chuyển sang trang đăng nhập này . . . hê hê . . . bạn đoán xem chuyện gì sẽ xảy ra ????? Theo mình là dùng setcokie (mặc dù không được tiện khi người dùng tắt cookie trên duyệt trình). Sau khi kiểm tra các bước đăng nhập xong ta sẽ : Code:

@setcookie("user","$member[1]",time()+30*24*60*60,"tendomain") //sét trong 1 tháng :D
@setcookie("pw","$member[2]",time()+30*24*60*60,"tendomain") //sét trong 1 tháng :D

và đoạn này Code:

if ( !$_SESSION['user_id'] ) 
{ 

print <<<EOF 
Bạn chưa đăng nhập! <a href='login.php'>Nhấp vào đây để đăng nhập</a> 
EOF; 

}

thay bằng Code:

if ( ! $_COOKIE["user"] || !$_COOKIE["pw"] ) 
{ 

print <<<EOF 
Bạn chưa đăng nhập! <a href='login.php'>Nhấp vào đây để đăng nhập</a> 
EOF; 

} 
else
{
...... <== kt cookie. Nếu đúng thì thông báo đã đăng nhập, nếu sai thì echo "Thằng nào thích fake hả ?"; :D
}

^_^ đó là những gì trong cái đầu bé ít chữ của em . . . các bác tham khảo nếu có sai thì các bác đừng vác đá táng đầu em :D HAVE FUN!]]> /hvaonline/posts/list/398.html#1484 /hvaonline/posts/list/398.html#1484 GMT

phamquoc_truong wrote:

Thứ 2 : Để đỡ tốn thời gian của user cũng như "bớt việc" cho server, mình kiểm tra toàn bộ thông tin người dùng ngay trên trang đăng kí, chứ không để lên trên server mới làm. Như vậy theo mình sẽ bỏ: Code:

// Kiểm tra 4 thông tin, nếu có bất kỳ thông tin chưa điền thì sẽ báo lỗi 
if ( ! $username || ! $password || ! $verify_password || ! $email ) 
{ 
print "Xin vui lòng nhập đầy đủ các thông tin. <a href='javascript:history.go(-1)'>Nhấp vào đây để quay trở lại</a>"; 
exit; 
} 

// Kiểm tra mật khẩu, bắt buộc mật khẩu nhập lúc đầu và mật khẩu lúc sau phải trùng nhau 
if ( $password != $verify_password ) 
{ 
print "Mật khẩu không giống nhau, bạn hãy nhập lại mật khẩu. <a href='javascript:history.go(-1)'>Nhấp vào đây để quay trở lại</a>"; 
exit; 
}

 

ehèm, cái thứ hai của bạn đóa , đành rằng công việc kiếm tra nên để ở phía client thì tốt hơn nhưng mà cũng không thể nào không kiểm tra ở phía Server, hãy tưởng tượng, nếu dữ liệu được gửi đến Server của bạn không thông qua trình duyệt (nơi mà có sẵn các mã JavaScript xử lý trước dữ liệu) mà được gửi lên từ một ứng dụng thứ ba thì sao ? Khi đó thì rõ ràng bạn không thể kiểm soát được cái nào được gửi lên và dĩ nhiên khả năng xảy ra lỗi là rất lớn. Thật ra việc để cho trình duyệt phía Client tự kiếm tra dữ liệu trước chỉ là một thủ thuật để nhằm tiết kiệm thời gian cho USER và phần nào giúp đỡ Server đỡ mệt mỏi thôi. Chúng ta không thể phủ định tầm quan trọng của việc kiếm tra dữ liệu phía Server. ]]> /hvaonline/posts/list/398.html#2180 /hvaonline/posts/list/398.html#2180 GMT Thì chuối một tí (sorry, nếu em sai !). Như mọi người đã biết giá trị của session sẽ xuyên suốt trong quá trình user làm việc với 1 cửa sổ, cho dù user chuyển từ trang này sang trang kia, trừ khi giá trị đó bị thay đổi hoặc phá huỷ. Như vậy chỗ này, em chỉ cần tạo một cái Code:

$_SESSION['user_id']=1;//thường của admin

trên máy (hay trên host) rồi cũng từ cửa sổ đó chuyển sang trang đăng nhập này . . . hê hê . . . bạn đoán xem chuyện gì sẽ xảy ra ?????