Por donde empezar...
Puedo ser más bueno que tu de programador.
Okay, vamos a tomarnos un reto y de paso de digo lo que haces mal xD.
Primero antes que nada decirte que sobre la conexion sql esta correcta.Como mucho puede hacer esto en lo de la conexion sql.
Como mucho, no, es lo que hay que hacer, no tiene ningún sentido abrir una conexión sql para imprimir "correcto" y cerrarla.
En segundo paso sobre el registro de datos del formulario esta correcto . Lo puede hacer asi.
Dices que esta correcto pero pegas un código totalmente diferente al anterior...
Veamos,
Tu primera función
validateName() yo te la reduzco a 3 lineas ·_·
function validarNombre($name){
return (preg_match($name,'/^[a-z0-9\-\_]{4,}$/i')>0); }
Sobre la
validatenota().. ¿ Has oído sobre eso de que las funciones son re-utilizables ? ¿ Te das cuenta que es absolutamente la misma que la de
validateName() ?
function validateExistnota($nota){
$consulta= "select id_codigo from producto where nombre = '$nombre'";
return false;
else
return true;
}
Dígame señor Apple Developer y señor Analista programador, esa variable, esa tal
$nombre, se declara por si sola por magia ? Dígame señor, la variable
$resultado también ? La consulta mysql se ejecuta sola ?
Y de nuevo, tu función
validatevalor() te la reduzco a 3 lineas.
function validateValor($valor){
return (preg_match('/^[0-9a-z]{5,12}$/i',$valor) > 0); }
Y tu función
validateEmail() es una bazofia (ademas no es valida para los
nuevos tlds) teniendo en cuenta que PHP ya cuenta con filtros de ese tipo.. de nuevo, 3 lineas ..
function validarEmail($email){
return (filter_var($email, FILTER_VALIDATE_EMAIL
) ?
true : false); }
Ahhh y de nuevo..
function validateExisteMail($email){
$consulta2= "select id_codigo from producto where nota = '$email'";
return false;
else
return true;
}
Seguro que el
$ilink2, se declara solito xD Por no hablar de la inyección sql que tienes ahí xD
La logica del IF a partir de la linea 99 del código que has pegado es inútil. Si no llego a enviar "send" nunca llegara a comprobar los campos.. y "send" se puede quitar muy facil modificando el POST antes del envió..
Por otra parte...
Sigue sin tener sentido.. ya que no haces la consulta en ningún lado .. pero bue.. y luego 2 lineas mas abajo tienes sqli tan grande como una casa.
En la linea 180 tienes XSS... y parece que nadie te ha hablado de que los headers se envían ANTES de imprimir (linea 188)...
Después de ver tu código, después de ver que usas expresiones regulares en un sitio y puro PHP en otro.. después de ver que usas 3 tipos de IF's... deduzco que no tienes NPI idea de lo que estas haciendo... y tampoco tienes idea de como funciona todo eso.. estoy 99% seguro que eres un copy/paster de esos que van copiando y pegando códigos de donde puedan.
AHORA YA ESTA TOTALMENTE CORRECTO Y COMPROVADO
Una *****. Ese script no funcionara ni aunque el mundo se acabe xD
Posdata y si que soy Analista Programador y miembro Developer de apple y google. Asi que no hace falta que digas que voy de aires de Creido
No me importa si eres analista programador o si pagas 99$ cada año a Apple para poder desarrollar apps para iOS. Eso no te hace mejor que nadie de aquí y no te da derecho a despreciar el trabajo de nadie.
Deberías mirarte el significado de creído porque intentas decir que no lo eres pero tus palabras dicen que lo eres xD.
http://lema.rae.es/drae/?val=creidoEn fin.. ahh y respecto a tu web, intenta no copiar datos de otros sitios quieres ? Es poco profesional =)
PD: No te voy a banear, y tampoco he dicho que lo vaya hacer. Deja de montarte películas.
Saludos
Mostrar Mensajes
