Veo dos errores en tu código en la variable $ip y al momento de insertar los datos
La variable ip debe ser
$ip = $_SERVER['REMOTE_ADDR'];
Y el mysql_query así
mysql_query("INSERT INTO usuarios (usuario,contra,correo,creditos,mensajes,publicidad,VIP_dia,VIP_mes,VIP_a,DJ_COM,MOD,REP,ADMIN,ip) VALUES ('".$usuario."','".$contra."','".$correo."','30','0','SI','".$vipdia."','".$vipmes."','".$vipa."','NO','NO','NO','NO','".$ip."')");
Esos errores lo he podido ver a simple vista, espero que se solucione el problema

---

Al respecto del color del código que haz puesto, eso tiene que ver con el sistema de coloreo del foro

Saludos.

Cambia la línea 11 por esto
if(mysql_num_rows($query) != 0){
Le quite el signo de exclamación (!), y le cambia el "=="

Saludos.

Holas:

Me he fijado en otra cosa:

No tienes bien construida la query para ver si el usuario ya existe en la bdd: no está bien concatenado.
Deberia ser algo como esto:


$query = mysql_query('SELECT * FROM usuarios WHERE usuario="' . $_POST['usuario'] . '" or correo="' . $_POST['correo'] . '"');

No has añadido las comillas que mysql quiere cuando envias cadenas...

De todos modos, para asegurarte de que la consulta esta bien construida haria lo siguiente:

$consulta = 'SELECT * FROM usuarios WHERE usuario="' . $_POST['usuario'] . '" or correo="' . $_POST['correo'] . '"';
echo $consulta;

$query = mysql_query($consulta);


El "echo" te devolverá la consulta que estas haciendo, y te podras asegurar que funciona copiando y pegandola directamente a una sesion abierta de mysql. Así vas modificando aqui y allá hasta que tengas lo que buscas. Cuando funcione le quitas el "echo" y listo!

También puedes usar un debugger, pero esta manera es mas barata

Otra cosa que tambien quitaría seria el "*" (el que hay delante del "SELECT"), pues te devolverá todos los campos, ralentizando así la consulta (No pasa nada cuando es solo 1, pero cuando haces bucles y tal... es pa tenerlo en cuenta). Yo pongo unicamente los campos que necesito: en este caso no necesitas ninguno en especial (pero al menos hay que poner 1), bastaría con hacer un "SELECT usuario FROM usuarios WHERE blah, blah......" y la respuesta seria mas corta (y por tanto, mas rápida). Pero bueno, solo un consejo que en este caso tiene poca importancia.

A ver si puedo probar el codigo cuando llegue a casa y te confirmo si esta ok...

Saludos,
M

Ah, se me olvidaban 3 cosillas:

- 1. Se te ha olvidado una "_" en la variable POST de la SELECT: $_POST[....]

- 2. La 3a condicion deberia empezar asi:  if(mysql_num_rows($query)==0){
......
"Si el numero de filas devueltas es igual a 0, insertar, de lo contrario, informa que ya existe el usuario en la bdd"

- 3. Es recomendable que en las variables $_POST, $_GET, $_SESSION y en todas sus amigas, entrecomilles (no se si existe esa palabra  , pero pal caso...) el valor del array, por ejemplo: $_POST['usuario']. En principio no pasa nada, pero los debuggers suelen dar warnings si pones una cadena sin que se pongan las respectivas comillas...

Saludos, M

Vale, gracias ahora estoy ocupado pero en cuanto pueda lo provare

Muchas gracias... KARMA +

Ahí va

<?php
$conectar = mysql_connect( &#39;localhost&#39;, &#39;usuario&#39;, &#39;************&#39;) or die(&#39;Could not connect to server.&#39; );
mysql_select_db(&#39;*********&#39;, $conectar) or die(&#39;Could not select database.&#39;);
if($_POST[&#39;registrar&#39;]){

if($_POST[&#39;contra&#39;]==$_POST[&#39;contra2&#39;]){
$consulta = &#39;SELECT usuario FROM usuarios WHERE usuario="&#39; . $_POST[&#39;usuario&#39;] . &#39;" or correo="&#39; . $_POST[&#39;correo&#39;] . &#39;"&#39;;

echo $consulta; // Quitar esta linea cuando veamos la sql bien construida

$query = mysql_query($consulta);

if(mysql_num_rows($query) == 0){ // Si el numero de filas devueltas es igual a 0, insertar, de lo contrario, muestra error

if(!$_POST[usuario]){
$error = "Usted no nos ha proporcionado un nombre de usuario</br>"; // Añado un salto de linea: Si hay mas errores, saldrán uno debajo del otro
}

if(!$_POST[contra]){
$error = "Usted no nos ha proporcionado una contrase&#38;#241;a</br>"; // Anado un salto de linea
}

if(!$_POST[correo]){
$error = "Usted se no nos ha proporcionado un mail</br>"; // Anado un salto de linea
}

$usuario = $_POST[&#39;usuario&#39;];
$contra = md5($_POST[&#39;contra&#39;]);
$correo = $_POST[&#39;correo&#39;];
$ip = $_SERVER[&#39;REMOTE_ADDR&#39;];
$vipdia = date("j");
$vipmes = date("n");
$vipa = date("Y");

} else {
$error = "El nombre de usuario y/o el correo electronico ya ha sido registrado</br>"; // Anado un salto de linea
}

} else {
$error = "Las contrase&#38;#241;as que nos ha proporcionado no coinciden</br>"; // Añado un salto de linea
}
}

if($error){
echo "Ha ocurrido un error: $error";
} else {
mysql_query("INSERT INTO usuarios (usuario,contra,correo,creditos,mensajes,publicidad,VIP_dia,VIP_mes,VIP_a,DJ_COM,MOD,REP,ADMIN,ip) VALUES (&#39;".$usuario."&#39;,&#39;".$contra."&#39;,&#39;".$correo."&#39;,&#39;30&#39;,&#39;0&#39;,&#39;SI&#39;,&#39;".$vipdia."&#39;,&#39;".$vipmes."&#39;,&#39;".$vipa."&#39;,&#39;NO&#39;,&#39;NO&#39;,&#39;NO&#39;,&#39;NO&#39;,&#39;".$ip."&#39;)");
}
?>

Una pregunta:
Por qué tienes la condicion "if($_POST['registrar']){". Yo la quitaria (junto su correspondiente "}", el que hay encima de "if($error){", logicamente). Me explico: se supone que el cargar esta página es para registrar el usuario.

Creo que con esto ya debería funcionar, no veo mas errores... De todos modos, no puedo comprobarte si las sqls, tanto la select como el insert, estan bien: no conozco la estructura de tu bdd... Si sigues teniendo problemas, pasame, al menos, la estructura de la tabla usuarios.

Otra cosa que ahora recuerdo: mysql en linux es case-sensitive, MUY importante (a mi siempre se me olvida )

Cuentanos como te ha ido!!!

Saludos,
M

P.D. Gracias por los karmas de antes

Bueno, pues ahora iremos un poco mas alla:
Qué sql se crea para el INSERT?
Haz un "echo" de la query como te pongo a continuación (sustituye todo el bloque)

if($error){
echo "Ha ocurrido un error: $error";
} else {
        $sql = "INSERT INTO usuarios (usuario,contra,correo,creditos,mensajes,publicidad,VIP_dia,VIP_mes,VIP_a,DJ_COM,MOD,REP,ADMIN,ip) VALUES ('".$usuario."','".$contra."','".$correo."','30','0','SI','".$vipdia."','".$vipmes."','".$vipa."','NO','NO','NO','NO','".$ip."')";
        echo $sql;
mysql_query($sql);
}
La sql que te salga, ejecutala en una ventana de mysql (sesion interactiva) y mira que error da. Veremos que solucion podemos darle...

Otra cosa que se me ha pasado (pero no es critica) es que las 3 condiciones que hay despues de "if(mysql_num_rows($query) == 0){", no estan "comilladas" (no se como se pone ), deben estar de esta manera: $_POST['usuario'], $_POST['contra'] y $_POST['correo']

Ah, y pasame la estructura de la tabla "usuarios", para ver si estamos haciendo el insert con los tipos de datos como tocan: las cadenas como cadenas, los enteros como enteros, etc...

Saludos,
M

G2K la ayuda se realiza mediante el foro... Parece mentira que seas Moderador de PHP...

Creo que todos queremos saber cual es el problema y donde está el fallo para futuras preguntas, ¿no?...

Bueno, pues ahora iremos un poco mas alla:
Qué sql se crea para el INSERT?
Haz un "echo" de la query como te pongo a continuación (sustituye todo el bloque)

Código: [Seleccionar]

if($error){
echo "Ha ocurrido un error: $error";
} else {
        $sql = "INSERT INTO usuarios (usuario,contra,correo,creditos,mensajes,publicidad,VIP_dia,VIP_mes,VIP_a,DJ_COM,MOD,REP,ADMIN,ip) VALUES ('".$usuario."','".$contra."','".$correo."','30','0','SI','".$vipdia."','".$vipmes."','".$vipa."','NO','NO','NO','NO','".$ip."')";
        echo $sql;
mysql_query($sql);
}
La sql que te salga, ejecutala en una ventana de mysql (sesion interactiva) y mira que error da. Veremos que solucion podemos darle...

Otra cosa que se me ha pasado (pero no es critica) es que las 3 condiciones que hay despues de "if(mysql_num_rows($query) == 0){", no estan "comilladas" (no se como se pone ), deben estar de esta manera: $_POST['usuario'], $_POST['contra'] y $_POST['correo']

Ah, y pasame la estructura de la tabla "usuarios", para ver si estamos haciendo el insert con los tipos de datos como tocan: las cadenas como cadenas, los enteros como enteros, etc...

Saludos,
M

Como te paso la estructura? :S

Como ejecuto la sql en una sesion intereactiva? :S