Z funkčného hľadiska na tom kóde nie je nič zlé. Funguje a robí to, čo je tam zapísané. Ináč z technického pohľadu je ten kód lamerina.
Číslo 1 a 2 spočítaš ako 1+2. Ale nemusíš použiť ani výpočet, pretože výsledkom bude 3. To je nejaká domáca úloha?
<input type="button" value="Vypočítať" onclick="alert((+this.form.n1.value)+(+this.form.n2.value))">