e4games Posted June 14, 2009 Share Posted June 14, 2009 So I found a code for login and register...could you please tell me whats wrong with it... <?session_start(); include("database.php");/** * Returns true if the username has been taken * by another user, false otherwise. */function usernameTaken($username){ global $conn; if(!get_magic_quotes_gpc()){ $username = addslashes($username); } $q = "select username from users where username = '$username'"; $result = mysql_query($q,$conn); return (mysql_numrows($result) > 0);}/** * Inserts the given (username, password) pair * into the database. Returns true on success, * false otherwise. */function addNewUser($username, $password){ global $conn; $q = "INSERT INTO users VALUES ('$username', '$password')"; return mysql_query($q,$conn);}/** * Displays the appropriate message to the user * after the registration attempt. It displays a * success or failure status depending on a * session variable set during registration. */function displayStatus(){ $uname = $_SESSION['reguname']; if($_SESSION['regresult']){?><h1>Registered!</h1><p>Thank you <b><? echo $uname; ?></b>, your information has been added to the database, you may now <a href="main.php" title="Login">log in</a>.</p><? } else{?><h1>Registration Failed</h1><p>We're sorry, but an error has occurred and your registration for the username <b><? echo $uname; ?></b>, could not be completed.<br>Please try again at a later time.</p><? } unset($_SESSION['reguname']); unset($_SESSION['registered']); unset($_SESSION['regresult']);}if(isset($_SESSION['registered'])){/** * This is the page that will be displayed after the * registration has been attempted. */?><html><title>Registration Page</title><body><? displayStatus(); ?></body></html><? return;}/** * Determines whether or not to show to sign-up form * based on whether the form has been submitted, if it * has, check the database for consistency and create * the new account. */if(isset($_POST['subjoin'])){ /* Make sure all fields were entered */ if(!$_POST['user'] || !$_POST['pass']){ die('You didn\'t fill in a required field.'); } /* Spruce up username, check length */ $_POST['user'] = trim($_POST['user']); if(strlen($_POST['user']) > 30){ die("Sorry, the username is longer than 30 characters, please shorten it."); } /* Check if username is already in use */ if(usernameTaken($_POST['user'])){ $use = $_POST['user']; die("Sorry, the username: <strong>$use</strong> is already taken, please pick another one."); } /* Add the new account to the database */ $md5pass = md5($_POST['pass']); $_SESSION['reguname'] = $_POST['user']; $_SESSION['regresult'] = addNewUser($_POST['user'], $md5pass); $_SESSION['registered'] = true; echo "<meta http-equiv=\"Refresh\" content=\"0;url=$HTTP_SERVER_VARS[PHP_SELF]\">"; return;}else{/** * This is the page with the sign-up form, the names * of the input fields are important and should not * be changed. */?><html><title>Registration Page</title><body><h1>Register</h1><form action="<? echo $HTTP_SERVER_VARS['PHP_SELF']; ?>" method="post"><table align="left" border="0" cellspacing="0" cellpadding="3"><tr><td>Username:</td><td><input type="text" name="user" maxlength="30"></td></tr><tr><td>Password:</td><td><input type="password" name="pass" maxlength="30"></td></tr><tr><td colspan="2" align="right"><input type="submit" name="subjoin" value="Join!"></td></tr></table></form></body></html><?}?> Link to comment Share on other sites More sharing options...
jlhaslip Posted June 14, 2009 Share Posted June 14, 2009 Can you explain what it is doing (or not doing)?Any error messages? What is not working? Link to comment Share on other sites More sharing options...
e4games Posted June 14, 2009 Author Share Posted June 14, 2009 well when I put it into the w3schools thing to see what it looks like all the code shows up...like about if it works say this or if it doesn't say that ...all that shows up Link to comment Share on other sites More sharing options...
thescientist Posted June 14, 2009 Share Posted June 14, 2009 well, first of all, PHP is not going to work unless it's processed through a server, so trying to view it through your web browser alone and expecting results is not going to happen. I noticed you've been asking for lots of help for HTML/CSS related issues. Personally, I would advise you take things one step at a time and really read through the tutorials here, especially starting with the CSS one. No ill will intended, but you know, baby steps. Link to comment Share on other sites More sharing options...
e4games Posted June 18, 2009 Author Share Posted June 18, 2009 I went over the css tutorial and it doesn't ever tell you how to make a template...also so whats wrong with the code....why does it say so many things that aren't supposed to show up Link to comment Share on other sites More sharing options...
justsomeguy Posted June 18, 2009 Share Posted June 18, 2009 why does it say so many things that aren't supposed to show upBecause the PHP is not getting executed. You need to put the file on a server that supports PHP and name the file with a .php extension, then load it in the browser. It's not going to work if you just copy and paste the code into the tryit editor on the w3schools site, that's not how you use PHP. Link to comment Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.