No funciona como deberia.

[alonsoandres]

Hola, pues ahora tengo un problema con un codigo que hice, pero no funciona como yo quisiera que funcionara. bueno, este es el codigo:

Código PHP:

  <?php
require('../include/database_connection.php'); 
 
if($_POST['yes']=='1'){ 
 
 
 
$username=$_POST[username]; 
 
$password=$_POST[password]; 
 
$email=$_POST[email]; 
 
$firstname=$_POST[firstname]; 
 
$lastname=$_POST[lastname];
 
$day=$_POST[day];
$month=$_POST[month];
$year=$_POST[year];
 
$country=$_POST[country];
$query="SELECT * FROM users WHERE username='$username'";
$result=mysql_db_query($base,$query,$conn) or die (mysql_error());
if(mysql_num_rows($result)==0)
{
//delimitation
mysql_query('INSERT INTO users (username, password, email, name, lastname, day, month, year, country)  
 
VALUES ("' . $username . '", "' . $password . '", "' . $email . '", "' . $firstname . '",  
"' . $lastname . '", "' .$day. '", "' .$month. '", "' .$year. '", "' .$country. '");')or $error = true;  
 
 
 
if(isset($error))
 
      echo 'We are sorry, but you were not able to register, please contact the administrator.<br><small>' . mysql_error() . '</small>';
 
 
 
    else
 
      echo 'Registration successful! Now you can <a href="../login.php">Log In</a>.';
 
 
 
    exit;
 
    }
 
}//End of IF
else
{
?>
<SCRIPT LANGUAGE="javascript">
    alert("The Username already exists, please try again...");
    history.go(-1);
    </SCRIPT>
<?
} //End of Else
?>   

Ahora, lo que yo quiero que haga es que no registre un usuario si ya se encuentra en la base de datos (esa parte si funciona), ahora, si encuentra un usuario en la base de datos con el mismo nombre, deberia de habrir una ventana de error diciendo que el nombre de usuario ya existe (esa es la parte que no funciona), no encuentro ningun error asi que porfavor ayudenme. Gracias de antemano.

[Marvin]

Por lo que veo estas mal cerrando las llaves de los if. Asumo que esto es lo que quieres hacer:

Código PHP:

  <?php
require('../include/database_connection.php'); 

if($_POST['yes'] == '1'){
    $username    = $_POST['username'];
    $password    = $_POST['password'];
    $email        = $_POST['email'];
    $firstname    = $_POST['firstname'];
    $lastname    = $_POST['lastname'];
    
    $day    = $_POST['day'];
    $month    = $_POST['month'];
    $year    = $_POST['year'];
    
    $country    = $_POST['country'];
    $query = "SELECT * 
            FROM users 
            WHERE username = '$username'";
    $result = mysql_db_query($base,$query,$conn) or die (mysql_error());
    if(mysql_num_rows($result) == 0){
        //delimitation
        $sql = 'INSERT INTO users (username, password, email, name, lastname, day, month, year, country)  
                VALUES ("' . $username . '", "' . $password . '", "' . $email . '", "' . $firstname . '",  
                "' . $lastname . '", "' .$day. '", "' .$month. '", "' .$year. '", "' .$country. '");';
        if(mysql_query($sql)){
            echo 'Registration successful! Now you can <a href="login.php">Log In</a>.';
        }else{
            echo 'We are sorry, but you were not able to register, please contact the administrator.<br><small>' . mysql_error() . '</small>';
        }
        exit;
    }else{
?>
<SCRIPT LANGUAGE="javascript">
    alert("The Username already exists, please try again...");
    history.go(-1);
</SCRIPT>
<?
    } //End of Else
}
?>   

Y esto es lo que tu tienes:

Código PHP:

  <?php
require('../include/database_connection.php'); 

if($_POST['yes'] == '1'){
    $username    = $_POST['username'];
    $password    = $_POST['password'];
    $email        = $_POST['email'];
    $firstname    = $_POST['firstname'];
    $lastname    = $_POST['lastname'];
    
    $day    = $_POST['day'];
    $month    = $_POST['month'];
    $year    = $_POST['year'];
    
    $country    = $_POST['country'];
    $query = "SELECT * 
            FROM users 
            WHERE username = '$username'";
    $result = mysql_db_query($base,$query,$conn) or die (mysql_error());
    if(mysql_num_rows($result) == 0){
        //delimitation
        $sql = 'INSERT INTO users (username, password, email, name, lastname, day, month, year, country)  
                VALUES ("' . $username . '", "' . $password . '", "' . $email . '", "' . $firstname . '",  
                "' . $lastname . '", "' .$day. '", "' .$month. '", "' .$year. '", "' .$country. '");';
        if(mysql_query($sql)){
            echo 'Registration successful! Now you can <a href="login.php">Log In</a>.';
        }else{
            echo 'We are sorry, but you were not able to register, please contact the administrator.<br><small>' . mysql_error() . '</small>';
        }
        exit;
    }
        
}else{
?>
<SCRIPT LANGUAGE="javascript">
    alert("The Username already exists, please try again...");
    history.go(-1);
</SCRIPT>
<?
} //End of Else
?>   

Ojo que ordene el codigo y cambie minimas cosas para que se vea mas ordenado.

Suerte!

[alonsoandres]

muchas gracias. probe tu codigo y me funciono pero te suplicaria que resaltaras donde estaba el error, porque no lo encuentro hehe.

[eits]

supongo que en el include tienes una conexión a la BD porque veo que usas un $conn, así que tomando en cuenta eso te dejo el código que creo que te funcionaría

Código PHP:

  <?php
require('../include/database_connection.php'); 
if($_POST['yes']=='1'){//supongo que tu controlas este por eso no lo quito
    $username=$_POST['username']; 
    $password=$_POST['password']; 
    $email=$_POST['email']; 
    $firstname=$_POST['firstname']; 
    $lastname=$_POST['lastname'];
    $day=$_POST['day'];
    $month=$_POST['month'];
    $year=$_POST['year'];
    $country=$_POST['country'];
    $query="SELECT * FROM users WHERE username LIKE '$username'";
    $result=@mysql_query($query) or die (mysql_error());
    if(mysql_num_rows($result)==0){
        //delimitation
        mysql_query('INSERT INTO users (username, password, email, name, lastname, day, month, year, country) VALUES ("' . $username . '", "' . $password . '", "' . $email . '", "' . $firstname . '","' . $lastname . '", "' .$day. '", "' .$month. '", "' .$year. '", "' .$country. '");') or $error = true;  
        if(isset($error)){
            echo 'We are sorry, but you were not able to register, please contact the administrator.<br><small>' . mysql_error() . '</small>';
        }
        else{
            echo 'Registration successful! Now you can <a href="login.php">Log In</a>.';
        }
    }//End of IF
    else
    {
?>
<SCRIPT LANGUAGE="javascript">
    alert("The Username already exists, please try again...");
    history.go(-1);
    </SCRIPT>
<?php
       } //End of Else
}//End of IF?>   

saludos.

EDITO: se me olvidaba, fijate en el uso del LIKE se recomienda ampliamente usar esa sentencia cuando se trata de comparar textos.

[Marvin]

Ok... te explico.

El problema era que el primer IF...

Código PHP:

  if($_POST['yes']=='1'){ 
   

...estaba cerrando con el ultimo ELSE...

Código PHP:

  }else{
?>
<SCRIPT LANGUAGE="javascript"> 
  

Por lo tanto nunca iba a entrar aca si la variable post "yes" iba con 1.

Te sugiero utilizar la indentacion de codigo en el futuro para que no tengas nuevamente estos problemas.

Suerte!