Skocz do zawartości

  • 👋 Witaj na MPCForum!

    Przeglądasz forum jako gość, co oznacza, że wiele świetnych funkcji jest jeszcze przed Tobą! 😎

    • ✅ Pełny dostęp do działów i ukrytych treści
    • ✅ Możliwość pisania i odpowiadania w tematach
    • ✅ System prywatnych wiadomości
    • ✅ Zbieranie reputacji i rozwijanie swojego profilu
    • ✅ Członkostwo w jednej z największych społeczności graczy

    👉 Dołączenie zajmie Ci mniej niż minutę – a zyskasz znacznie więcej!

    Zarejestruj się teraz

[Pytanie]Czy to dobra rejestracja ?

.Stunter13

Przez .Stunter13
w Niezbędnik Programisty - Archiwum

Rekomendowane odpowiedzi

.Stunter13 Budowniczy

.Stunter13

Opublikowano
Budowniczy

.Stunter13

Opublikowano

Witam jestem w trakcie twożenia gry z kolegą i kongo powiedział ze jak bedzie potrzeba to też pomoże

i teraz mam pytanie bo moj kolega napisał script rejestracji jest to on : 

<html>
<head>
 <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2" />
 <title>Rejestracja</title>

    <script type="text/javascript">
               <!-- //
                       function regulamin()
                       {
                       if(document.getElementById('reg').checked == true )
                       {
                               document.getElementById('dalej').disabled=false;
                       }
                       else
                       {
                               document.getElementById('dalej').disabled=true;
                       }
                       }
               //  -->
   </script>

</head>

<form method="post" action="<? echo $PHP_SELF;?>" METHOD=POST enctype="multipart/form-data">
<center>
   <center><b>Login:<b><br />
   <input type="text" name="login" style="font-size: 10pt; font-family: Tahoma; font-weight: bold" /><br />
       <b>Email:<b><br />
   <input type="text" name="email" style="font-size: 10pt; font-family: Tahoma; font-weight: bold" /><br />
       <b>Hasło:</b><br />
   <input type="password" name="haslo" style="font-size: 10pt; font-family: Tahoma; font-weight: bold" /><br />
   <b>Oświadczam iż zaakceptowałem regulamin</b>
       <input type="checkbox" id="reg" onclick="regulamin()" /><br />
       <input type="submit" name="submit" value="Rejestracja" id="dalej" disabled="disabled" style="font-size: 10pt; font-family: Tahoma; font-weight: bold" /></center>


</form>
<?
//Jak wiadomo dobrze użyć funkcji trim aby usunšć zbędne znaki
$login = trim($_POST['login']);
$email = trim($_POST['email']);    
$pass_md5 = md5($_POST['haslo']);
$pass = $_POST['haslo'];
if($_POST['submit']) {
$checkuser = mysql_query("SELECT login FROM users WHERE login='$login'");
$username_exist = mysql_num_rows($checkuser);

//Jezeli zarejestrowanych takich nazw jest wiecej niz 0
if($username_exist > 0){
   echo '
     <h2><center><b>Ta nazwa jest już zajęta wybierz inną</b></center></h2>
';
   unset($login);
   exit();
}

//Wysylamy zapytanie z logowaniem
$zapytanie = "INSERT INTO `users` (`id`, `login`, `email`, `haslo`) VALUES (NULL, '$login', '$email', '$pass_md5')";
$idzapytania = mysql_query($zapytanie);

if(!$idzapytania)
{
   echo 'blad!' .  mysql_error();
}
else
{
   echo '
     <h2><center><b>Poprawnie się zarejestrowałeś, <a href="logowanie.php">możesz się zalogować</a></b></center></h2>
';
}}
?>

 

 

Czy script jest dobry ? co w nim moge udoskonalić co zmienic itd ?

2627930384.png


reklama w sygnie - MrPiotrek122


Gość

Gość

Opublikowano

Gość

Opublikowano

SCRIPT jak SCRIPT, normalne rejestrowanie z wczesniejszym zaakceptowaniem ragulaminu.

Moim zdaniem tam nic nie trzeba poprawiac bo co moze byc w rejestracji - rejestracja :)

eXu Specjalista

eXu

Opublikowano
Specjalista

eXu

Opublikowano

Patrząc na sam kod to dane wejściowe bym filtrował.

Bo patrząc na ten kod, można wstrzyknąć tutaj coś głupiego.

 

najlepiej użyć jakiegoś "algorytmu" preg_match ;)

Może też pobawić się z opcją ctype_ albo is_ ;)

 

a żeby to udoskonalić dodaj captcha ;) oraz oddzial kod css od reszty ;)

eXu Specjalista

eXu

Opublikowano
Specjalista

eXu

Opublikowano

Szczerze powiedziesz to według opini publicznej html5 zaczyna wypierać flash ;(

Lecz nie dawno zostały znów znowione prace nad flash, więc jak kto woli.

A poza tym nie znam kursów flash.

Zarchiwizowany

Ten temat przebywa obecnie w archiwum. Dodawanie nowych odpowiedzi zostało zablokowane.

Tematy
×
×
  • Dodaj nową pozycję...