Calculadora

Pessoal, boa noite.
Eu estou com problema em um exercício até que simples, mas o professor solicitou a validação dos campos, ai eu me embananei todos. Eu fiz a validação, está funcionando mas não mostra o resultado. Se eu tiro a validação mostra o resultado. Alguém pode me ajudar?

<!DOCTYPE html>
<html>
	<head>
		<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
	<title>Untitled 1</title>
		<script type="text/javascript">
		function calcular(op)
		{
			var n1 = parseFloat(document.getElementById("numero1").value);
			var n2 = parseFloat(document.getElementById("numero2").value);
			var r;
			
		function valida(op){
			if(isNaN(form1.numero1.value) || isNaN(form1.numero2.value)) 
				{    
					alert("Digite apenas números!");       
       				return false;    
				} else 
					{  
     					return false;  
					}
			}						
		switch(op)
		{
			case "+" : r = n1+n2; break;
			case "-" : r = n1-n2; break;
			case "*" : r = n1*n2; break;
			case "/" : r = n1/n2; break;
		}
			document.getElementById("resultado").value = r;	
		}
		</script>
	</head>
<body>
	<h1>Calculadora</h1>
	<form id="form1" name="form1" method="post" action="">
  		<p>Primeiro valor:<input type="text" name="numero1" id="numero1" /></p>
  		<p>Segundo valor:<input type="text" name="numero2" id="numero2" /></p>
  		<p>
    		<input type="button" name="button1" id="button1" value="+" onclick="calcular('+')"/>
    		<input type="button" name="button2" id="button2" value="-" onclick="calcular('-')"/>
    		<input type="button" name="button3" id="button3" value="*" onclick="calcular('*')"/>
    		<input type="button" name="button4" id="button4" value="/" onclick="calcular('/')"/>
		</p>
  		<p>Resultado: <input type="text" name="resultado" id="resultado" /></p>
	</form>
</body>
</html>

Pessoal, eu consegui arrumar o código e deu certo:

<!DOCTYPE html>
<html>
	<head>
		<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
	<title>Untitled 1</title>
		<script type="text/javascript">
		function calcular(op)
		{
			var n1 = parseFloat(document.getElementById("numero1").value);
			if(isNaN(form1.numero1.value)) 
				{    
					alert("Digite apenas números no campo Primeiro Valor!");       
       				document.getElementById("numero1").focus();   
				} 
				
			var n2 = parseFloat(document.getElementById("numero2").value);
			if(isNaN(form1.numero2.value)) 
				{    
					alert("Digite apenas números campo Segundo Valor!");       
       				document.getElementById("numero2").focus();   
				} 

			var r;
				
		switch(op)
		{
			case "+" : r = n1+n2; break;
			case "-" : r = n1-n2; break;
			case "*" : r = n1*n2; break;
			case "/" : r = n1/n2; break;
		}
			document.getElementById("resultado").value = r;	
		}
		
		</script>
	</head>
<body>
	<h1>Calculadora</h1>
	<form id="form1" name="form1" method="post" action="">
  		<p>Primeiro valor:<input type="text" name="numero1" id="numero1" /></p>
  		<p>Segundo valor:<input type="text" name="numero2" id="numero2" /></p>
  		<p>
    		<input type="button" name="button1" id="button1" value="+" onclick="calcular('+')"/>
    		<input type="button" name="button2" id="button2" value="-" onclick="calcular('-')"/>
    		<input type="button" name="button3" id="button3" value="*" onclick="calcular('*')"/>
    		<input type="button" name="button4" id="button4" value="/" onclick="calcular('/')"/>
		</p>
  		<p>Resultado: <input type="text" name="resultado" id="resultado" /></p>
	</form>
</body>
</html>

ae cara vlew pelo post conseguir fazer minha calculadora graças a vc :smiley: tava meio em duvida tbm mas consegui de boa agr aushauhs