Problema con expresion regular

[keko123]

Hola, resulta que intento validar un campo con javascript y expresiones regulares y siempre me indica que esta mal. La misma expresion regular la uso en PHP y si me funciona bien.

tiene que validar una cantidad como xxxxx.xx o xxx.xx o xx.xx o xxxx,xx o xxx. o xx.x o xxxx.x

vamos, hasta 5 numeros seguidos opcionalmente de una coma o punto y hasta otros dos numeros mas

Código:

		
if (document.frmalta.importe.value!=""){
       var re = new RegExp('/^([0-9]{1,5})[\.]?[,]?([0-9]{0,2})$/');
	if (!re.test(document.frmalta.importe.value)){
		alert ("El importe no tiene un formato válido");
		document.frmalta.importe.focus();
		return false;
	}
}

Levo tres horas intentando ver que falla y ya me doy por vencido.

Un saludo

[keko123]

Hola, me autorespondo por si a alguien le sirve de ayuda.

Lo he arreglado quitando la llamada al constructor del objeto RegExp y sustitullendolo por un string, tal que

Código:

		
if (document.frmalta.importe.value!=""){
	var re = /^([0-9]{1,5})[\.]?[,]?([0-9]{0,2})$/;
	if (!re.test(document.frmalta.importe.value)){
		alert ("El importe  no tiene un formato válido");
		document.frmalta.importe.focus();
		return false;
	}
}

[zerokilled]

buenas.
el problema era como estabas creando la expresión al pasarlo a RegExp. en ese caso lo único que tenias que indicar era el patrón como string sin los delimitadores (la barra diagonal). aparte, según tu enunciado el patrón está mal: una cifra de hasta 5 dígitos ( [0-9]{1,5} lo tienes bien), opcionalmente seguido de un punto o coma y una cifra de hasta dos dígitos ( [\.]?[,]?([0-9]{0,2}) está mal). está mal porque puedo introducir hasta 7 dígitos sin incluir el punto o coma. además de que se puede introducir ambos símbolos y considerarse como válido. lo correcto sería poner toda la parte opcional como una subexpresión con el cuantificador opcional (?). [0-9]{1,5}([\.,][0-9]{1,2})?

[keko123]

HOla, muchas gracias por tu respuesta. El problema que me ha surgido es que al sustituir mi expresion por la tuya ahora tambien puedo poner por ejemplo:

12345678

Y no me avisa de que el importe tiene formato invalido y lo introduce en la db.

Un saludo

[zerokilled]

¿en serio?

Código:

var reg = /^[0-9]{1,5}([\.,][0-9]{1,2})?$/;
console.log(reg.test('1234567')); // false
console.log(reg.test('12345')); // true
console.log(reg.test('12345.67')); // true

claro, que habia omitido los anclas pero fue solo para simplificar.

[keko123]

Ahora si, XD. Es que cuando me pusiste la expresion como no ponias el ^ ni el $ pense que a lo mejor no hacia falta y no lo puse. Al añadirlo ya fuinciona bien.

Muchas gracias!!