Estoy aprendiendo PHP! y decidi hacer una pequeña calculadora... me gustaria que me den su apreciacion critica

Código:
<html>
<head>
<title>Aprendiendo PHP</title>
</head>
<body>
<h1 align="center">Caculadora</h1>
<hr>
<form action="#introduccion" method="GET">
<input type="text" value="" name="valor1">
<input type="submit" value="+" name="+">
<br>
<input type="text" value="" name="valor2">
</form>
<p>
<p>
<p>
<p>
<a name="introduccion"></a>
<?php
$valor1=$_GET['valor1'];
$valor2=$_GET['valor2'];
$suma=$_GET['+'];
if($valor1 or $valor2 = "")
echo $valor1 + $valor2;
else{
echo "Ingrese digitos a sumar";
}
?>
</body>
</html>
Esa calculadora es de un solo operador


Saludos!