首先，不要发截图，我想修改你代码都不可以。
其次吧，onclick是回调函数，当class=‘jisuan’触发onclick事件时，并不会传给你参数n，而且你这种写法永远不会到

    answer.innerHTML = factorial(n)

这条语句。
可以改成这样写 ：

    jisuan.onclick = function() {
        var n = document.getElementById('jieceng').value
         
         function factorial(n) {
             if( n > 1){
                 //你的代码
             }else {
                 //你的代码
             }
         } 
         answer.innerHTML = factorial(n)   
    }

再也不给发截图的写代码了（逃）

//更新：@克利威尔的答案是正解，我的答案考虑不周全。

1.return 会终止函数的执行，所以给 span 添加内容的语句就永远不会被执行。

2.阶乘函数的参数是 n，不过你又把 n 的值重新获取了一次，相当于不管你传的参数是多少，最终 n 永远都是 input 的 value 。所以给 n 重新赋值的那一行（25行）可以去掉，然后每次传的参数比上一次少1 。