Sólo hay que cambiar `.val` por `.value`. <!-- begin snippet: js hide: false console: true babel: false --> <!-- language: lang-js --> function valor() { var n1 = document.getElementById("n1").value; var n2 = document.getElementById("n2").value; var resultado = parseInt(n1, 10) + parseInt(n2, 10); document.getElementById("resultado").value = resultado; } <!-- language: lang-html --> <input name="n1" type="text" id="n1" size="4" maxlength="5" /> <input name="n1" type="text" id="n2" size="4" maxlength="5" /> <input name="suma" type="button" onClick="valor()" value="suma" /> <input type="text" id="resultado" /> <!-- end snippet --> **Nota:** Además, en `parseInt()` se debería poner siempre la base (10). Por ejemplo, `parseInt(n1, 10)`.