hacelo por separado
Código HTML:
Ver original<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml"> <meta http-equiv="content-type" content="text/html; charset=utf-8" />
<script type="text/javascript"> //<![CDATA[
function verificar(){
var p = document.getElementById('preg').value;
var v = document.getElementById('valora').value;
p= parseInt(p);
v = parseInt(v);
if((v < 1)||(v > 100)){
alert('error v');
}
if(( p < 1)||(p > 90)){
alert('error p');
}
}
//]]>
pregunta:
<input type="text" value="" id="preg" name="preg" /><br />valorar:
<input type="text" value="" id="valora" name="valora" /><br /><br /><input type="button" value="verifica" onclick="verificar();" />
En tu condición,
elcampo = "pregunta" no es nada
El problema es que en tu ejemplo en ningún momento habías mencionado que hubiese dos tipos de campos, por eso integrarlos todo en una sola función se complica si no se sabe que es pregunta y que es valoración
Saludos
PD:
LAs urgencias se atienden en los hospitales, no en los foros