VNC stopt er mee :(
username availability check
ik ben ent begonnen met een beetje PHP mySQL voor mijn profielwerkstuk maar loop nu tegen een probleempje aan.
ik krijg de volgende error
Parse error: syntax error, unexpected T_STRING, expecting '(' in /home/a4665314/public_html/registration.php on line 12
dit is de code:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
<?php
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
$con =mysql_connect("$hostname","$username","$password");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("$username", $con);
if EXISTS (SELECT username FROM users WHERE username=$_POST['username'])
{
echo "username already in use";
}
if EXISTS (SELECT email FROM users WHERE email=$_POST['email'])
{
echo "email already in use";
}
else
{
$sql="INSERT INTO users (username, password, email)
VALUES('$_POST[username]','$_POST[password]','$_POST[email]')";
if (!mysql_query($sql,$con))
{
die('Error: ' . mysql_error());
}
echo "registration completed";
}
}
echo "the passwords do not match";
}
else echo "invalid data";
?>
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
$con =mysql_connect("$hostname","$username","$password");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("$username", $con);
if EXISTS (SELECT username FROM users WHERE username=$_POST['username'])
{
echo "username already in use";
}
if EXISTS (SELECT email FROM users WHERE email=$_POST['email'])
{
echo "email already in use";
}
else
{
$sql="INSERT INTO users (username, password, email)
VALUES('$_POST[username]','$_POST[password]','$_POST[email]')";
if (!mysql_query($sql,$con))
{
die('Error: ' . mysql_error());
}
echo "registration completed";
}
}
echo "the passwords do not match";
}
else echo "invalid data";
?>
ik weet niet of het op deze manier kan werken. maar het lijkt mij van wel
alvast bedankt
Gewijzigd op 21/04/2011 20:04:24 door Marijn Struijlaart
wat probeer je hier in vredesnaam te doen:
if EXISTS (
php variabelen dien je niet te quoten!!
En kijk op php.net van hoe het wel moet
http://www.php.net/mysql_query
http://www.php.net/mysql_error
http://www.php.net/mysql_fetch_assoc
Gewijzigd op 21/04/2011 20:17:50 door Noppes Homeland
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
<?php
if ($_POST['username'] && $_POST['password'] && $_POST['password2'] && $_POST['email'] )
{
if ($_POST['password' ]== $_POST['password2'])
{
if(($con =mysql_connect($hostname,$username,$password)) === false)
{
echo 'MAAK EEN NETTE FOUTAFHANDELING!';
}
else if ((mysql_select_db($username, $con)) === false)
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
$sql_get_username = "SELECT username
FROM users WHERE username= '".$_POST['username']."'
";
if(($result_get_username = mysql_query($sql_get_username)) === false)
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
else if(mysql_num_rows($result_get_username) == 1)
{
echo 'email already in use';
}
else
{
$sql_get_mail = "INSERT INTO users (username, password, email)
VALUES('".mysql_real_escape_string($_POST['username'])."','".mysql_real_escape_string($_POST['password'])."','".mysql_real_escape_string($_POST['email'])."')
";
if (!mysql_query($sql_get_mail))
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
else
{
echo "registration completed";
}
}
}
else
{
echo 'the passwords do not match';
}
}
else
{
echo 'invalid data';
}
?>
if ($_POST['username'] && $_POST['password'] && $_POST['password2'] && $_POST['email'] )
{
if ($_POST['password' ]== $_POST['password2'])
{
if(($con =mysql_connect($hostname,$username,$password)) === false)
{
echo 'MAAK EEN NETTE FOUTAFHANDELING!';
}
else if ((mysql_select_db($username, $con)) === false)
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
$sql_get_username = "SELECT username
FROM users WHERE username= '".$_POST['username']."'
";
if(($result_get_username = mysql_query($sql_get_username)) === false)
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
else if(mysql_num_rows($result_get_username) == 1)
{
echo 'email already in use';
}
else
{
$sql_get_mail = "INSERT INTO users (username, password, email)
VALUES('".mysql_real_escape_string($_POST['username'])."','".mysql_real_escape_string($_POST['password'])."','".mysql_real_escape_string($_POST['email'])."')
";
if (!mysql_query($sql_get_mail))
{
echo 'er ging iets mis -> MAAK EEN NETTE FOUTAFHANDELING!';
}
else
{
echo "registration completed";
}
}
}
else
{
echo 'the passwords do not match';
}
}
else
{
echo 'invalid data';
}
?>
- Juiste quotes gebruiken
- nette foutafhandelingen
- codeer netjes
- spring mooi in
- let op sql injectie!
(note: script is snel verbeterd, fouten voorbehouden. Het is het idee wat telt)
Gewijzigd op 21/04/2011 20:21:45 door Jasper DS
dit is nu het script. zit nog geen emailadres check bij, wel of het uberhaupt geldig is
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
<?php
function isValidEmail($email)
{
$pattern = "^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$";
if (eregi($pattern, $email))
{
return true;
}
else
{
return false;
}
}
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
if (!isValidEmail($_POST['email']))
{
echo "The email: ".$_POST['email']." is invalid";
}
else
{
$con =mysql_connect("$hostname","$username","$password");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("a4665314_cricity", $con);
$sql_get_username = "SELECT username FROM users WHERE username = '".$_POST['username']"' ";
if(($result_get_username = mysql_query($sql_get_username)) === false)
{
echo 'er ging iets mis';
}
if(mysql_num_rows($result_get_username) == 1)
{
echo "username already in use";
}
else
{
$sql_get_username="INSERT INTO users (username, password, email) VALUES ('".mysql_real_escape_string($_POST['username'])."', '".mysql_real_escape_string($_POST['password'])."', '".mysql_real_escape_string($_POST['email'])."')";
if (!mysql_query($sql,$con))
{
die('could not insert data into database: ' . mysql_error());
}
echo "registration completed";
}
}
}
else echo "the passwords do not match";
}
else echo "invalid data";
?>
function isValidEmail($email)
{
$pattern = "^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$";
if (eregi($pattern, $email))
{
return true;
}
else
{
return false;
}
}
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
if (!isValidEmail($_POST['email']))
{
echo "The email: ".$_POST['email']." is invalid";
}
else
{
$con =mysql_connect("$hostname","$username","$password");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("a4665314_cricity", $con);
$sql_get_username = "SELECT username FROM users WHERE username = '".$_POST['username']"' ";
if(($result_get_username = mysql_query($sql_get_username)) === false)
{
echo 'er ging iets mis';
}
if(mysql_num_rows($result_get_username) == 1)
{
echo "username already in use";
}
else
{
$sql_get_username="INSERT INTO users (username, password, email) VALUES ('".mysql_real_escape_string($_POST['username'])."', '".mysql_real_escape_string($_POST['password'])."', '".mysql_real_escape_string($_POST['email'])."')";
if (!mysql_query($sql,$con))
{
die('could not insert data into database: ' . mysql_error());
}
echo "registration completed";
}
}
}
else echo "the passwords do not match";
}
else echo "invalid data";
?>
als ik nu probeer om een bestaande gebruikersnaam in te vullen krijg ik deze error in line 33
Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in line 33
Gewijzigd op 22/04/2011 11:32:10 door Marijn Struijlaart
Begin eerst eens simpel met php.
Hint: operators.
Bas Kreleger op 22/04/2011 11:50:57:
Of even kijken naar het voorzetje van PHP Jasper.
Nee, eerst leren.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
<?php
function isValidEmail($email)
{
$pattern = "^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$";
if (eregi($pattern, $email))
{
return true;
}
else
{
return false;
}
}
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
$password= md5($_POST['password']);
$username = $_POST['username'];
$email = $_POST['email'];
if (!isValidEmail($_POST['email']))
{
echo "The email: ".$_POST['email']." is invalid <br />";
}
else
{
$con =mysql_connect("","","");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("", $con);
$checkuser = mysql_query("SELECT username FROM users WHERE username='$username'");
$username_exist = mysql_num_rows($checkuser);
$checkemail = mysql_query("SELECT email FROM users where email='$email'");
$emailname_exist = mysql_num_rows($checkemail);
if ($username_exist >0)
{
echo "username already in use <br />";
unset($username);
include 'index.htm';
}
if($email_exist > 0)
{
echo "email already in use <br />";
unset($email);
include 'index.htm';
}
else
{
$query = "INSERT INTO users (username, password, email) VALUES('$username', '$password', '$email')";
mysql_query($query) or die(mysql_error());
mysql_close();
echo "registration completed";
}
}
}
else echo "the passwords do not match";
}
else echo "invalid data";
?>
function isValidEmail($email)
{
$pattern = "^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$";
if (eregi($pattern, $email))
{
return true;
}
else
{
return false;
}
}
if ($_POST["username"] && $_POST["password"] && $_POST["password2"] && $_POST["email"] )
{
if ($_POST["password"]==$_POST["password2"])
{
$password= md5($_POST['password']);
$username = $_POST['username'];
$email = $_POST['email'];
if (!isValidEmail($_POST['email']))
{
echo "The email: ".$_POST['email']." is invalid <br />";
}
else
{
$con =mysql_connect("","","");
if (!$con)
{
die('could not connect: ' . mysql_error());
}
mysql_select_db("", $con);
$checkuser = mysql_query("SELECT username FROM users WHERE username='$username'");
$username_exist = mysql_num_rows($checkuser);
$checkemail = mysql_query("SELECT email FROM users where email='$email'");
$emailname_exist = mysql_num_rows($checkemail);
if ($username_exist >0)
{
echo "username already in use <br />";
unset($username);
include 'index.htm';
}
if($email_exist > 0)
{
echo "email already in use <br />";
unset($email);
include 'index.htm';
}
else
{
$query = "INSERT INTO users (username, password, email) VALUES('$username', '$password', '$email')";
mysql_query($query) or die(mysql_error());
mysql_close();
echo "registration completed";
}
}
}
else echo "the passwords do not match";
}
else echo "invalid data";
?>
Je kopieert onnodig variabelen.
Zorg voor foutafhandeling in je query's
is er nog iets van een andere simpele manier om hoofdletter te negeren?
Karl Karl op 22/04/2011 12:16:58:
Nee, want dat klopt ook al niet...
Waarom klopt dit niet?
Ocirina Ocirina op 22/04/2011 23:47:05:
Waarom klopt dit niet?
Karl Karl op 22/04/2011 12:16:58:
Nee, want dat klopt ook al niet...
Waarom klopt dit niet?
Hmm, geen idee.. Ik zal het nog eens checken of Karl moet het ons zeggen.
Ocirina Ocirina op 22/04/2011 23:47:05:
Waarom klopt dit niet?
Karl Karl op 22/04/2011 12:16:58:
Nee, want dat klopt ook al niet...
Waarom klopt dit niet?
Geen nette if() en SQL-Injection lek.
- SanThe - op 23/04/2011 13:32:50:
Geen nette if() en SQL-Injection lek.
Ocirina Ocirina op 22/04/2011 23:47:05:
Waarom klopt dit niet?
Karl Karl op 22/04/2011 12:16:58:
Nee, want dat klopt ook al niet...
Waarom klopt dit niet?
Geen nette if() en SQL-Injection lek.
Ja, er is inderdaad een sql lek.. Verder zou ik de volledige if-else constructie anders doen maar dat is de keuze ven de TS
Obelix en Idefix op 22/04/2011 15:20:17:
Eregi is vervallen; http://nl3.php.net/eregi
Je kopieert onnodig variabelen.
Zorg voor foutafhandeling in je query's
Je kopieert onnodig variabelen.
Zorg voor foutafhandeling in je query's
+
- zorg voor nette foutafhandeling -> geen or die!
- codeer netjes
Gewijzigd op 23/04/2011 15:15:15 door Jasper DS