为什么我在这里一直未定义？

-2

total_1不断带回未定义。我究竟做错了什么？如果总分数大于它，我想增加10，如果不是，我想只增加5分。其他人来的很好，总数只有1分。为什么我在这里一直未定义？

function findMark(){ 
    var x1 = document.getElementById("fn").value; 
    var x2 = document.getElementById("ln").value; 
    var grade1 = parseFloat(document.getElementById("midterm").value*0.35); 
    var grade2 = parseFloat(document.getElementById("assign").value*0.1); 
    var grade3 = parseFloat(document.getElementById("quiz").value*0.15); 
    var grade4 = parseFloat(document.getElementById("final").value*0.4); 
    var total = Math.round(grade1 + grade2 + grade3 + grade4); 
    var total_1 = findFinalMark(total); 
    document.getElementById("outputDiv").innerHTML = "Name = " + x1 + " " + x2 + "<br>" + "Your mark is = " + total + "%" + "<br>" + "Your Final Mark = " + total_1 + "%"; 

    function findFinalMark (x) { 
    sum = 0; 
    i = 1; 
    while (i < x){ 
     if (x % i == 0){ 
      sum += i; 
     } 
     i++; 
    } 
    if (sum > x) { 
     x = x + 10; 
    } 
    else { 
     x = x + 5; 
    } 
    } 
}

来源

2015-12-15 Daniel Quomsieh

如果你的'midterm'，'assign'，'quiz'和'final'输入具有'type = text'试图对它们的值执行数学运算将不会给你一个整数。将它们设置为'type =“number”'或使用'parseInt'。 –

我对他们使用type =“number”。唯一给我定义的是total_1。我知道问题来自我的js而不是html。 –

你的'findFinalMark'函数没有返回值 –

这是很简单的：

所以它是不确定你不从函数返回任何东西。

所以你需要从函数中返回x或sum。声明变量时也使用var，以免泛滥全局名称空间。

function findFinalMark (x) { 
    var sum = 0, 
     i = 1; 
    /* your logic */ 
    return x; //or sum or whatever you want 
}

来源

2015-12-15 20:12:48 epascarello

你的意思是让var sum和var i？ –

是的，这就是你如何声明变量。 – epascarello

感谢它现在的作品。 –

你的函数findFinalMark返回和？

来源

2015-12-15 20:13:04

不，它应该放在我的if和else语句中。我应该这样做吗？（sum> x）{ \t \t \t return x = x + 10; \t \t} \t \t否则{ \t \t \t返回X = X + 5; \t \t} –

为什么我在这里一直未定义？

回答

相关问题