среднее значение из пяти значений в javascript

Question

Я работаю в чартерной школе, и я просто изучаю свой путь вокруг javascript. У меня есть код, написанный человеком, который раньше заполнял мою позицию, и мне кажется, что он должен работать, но это не так.

Это то, что я имею в пользовательских HTML страницы в моем SIS:

GED Status: <script language="Javascript">gedCheck('~(ELC_tspp_GED_read_score)','~ (ELC_tspp_GED_wri_score)','~(ELC_tspp_math_GED_score)','~(ELC_science_state_exam_score)','~(soc_sci_state_exam_score)')</script> 

Это, кажется, извлечения значений из различных полей БД правильно, как яваскрипта рутина оценивает каждое значение для обеспечения его по крайней мере, 410. Но это так далеко, как она идет ...

Вот Javascript рутинный код:

function gedCheck(read,wri,math,sci,soc) { 

if(read < 0 && read > 1000) 
    read = 0; 
if(wri < 0 && wri > 1000) 
    wri = 0; 
if(math < 0 && math > 1000) 
    math = 0; 
if(sci < 0 && read > 1000) 
    read = 0; 
if(soc < 0 && soc > 1000) 
    soc = 0;   

if ((read >= 410) && (wri >= 410) && (math >= 410) && (sci >= 410) && (soc >= 410)) { 
    if(read+wri+math+sci+soc >= 2250) 
     document.write("PASSED") 
} 
else 
    document.write("NOT PASSED") 
} 

предполагается проверять, что каждый счет в го e GED-тесты составляют не менее 410 и что сумма всех баллов должна быть не менее 2250. Однако она не доходит до последней части. Он возвращается «PASSED», если все баллы превышают 410.

Я пробовал это, но он также не работает.

function gedCheck(read,wri,math,sci,soc) { 

if(read < 0 && read > 1000) 
    read = 0; 
if(wri < 0 && wri > 1000) 
    wri = 0; 
if(math < 0 && math > 1000) 
    math = 0; 
if(sci < 0 && read > 1000) 
    read = 0; 
if(soc < 0 && soc > 1000) 
    soc = 0;   

if ((read >= 410) && (wri >= 410) && (math >= 410) && (sci >= 410) && (soc >= 410)) { 
    if(read+wri+math+sci+soc/5 >= 450) 
     document.write("PASSED") 
} 
else 
    document.write("NOT PASSED") 
} 

ли кто-нибудь, пожалуйста, помогите мне решить эту проблему таким образом это либо средние значения все 5 чисел и возвращает «Пройдено» только тогда, когда среднее значение равно 450, или просто добавляет все 5 чисел и возвращает «Пройдено» только если общая сумма 2250 или больше?

Answer 1

Чтобы получить среднее, вы хотите сделать это:

(((read + wri + math + sci + soc)/5) > 450) 

Скобки вокруг того гарантирует, что вы разделите сумму всех баллов на 5. Так, как вы это делаете сейчас, вы делите только soc на 5.

Редактировать (Переписывать весь метод):

function gedCheck(read, wri, math, sci, soc) { 
// As was said before, these should all be ORs 
// If the score is less than 0, OR greater than 1000 
if(read < 0 || read > 1000) { 
    read = 0; 
} 
if(wri < 0 || wri > 1000) { // I prefer to put the braces around all if/else statements just for absolute clarity 
    wri = 0; 
} 
if(math < 0 || math > 1000) { 
    math = 0; 
} 
if(sci < 0 || read > 1000) { 
    read = 0; 
} 
if(soc < 0 || soc > 1000) { 
    soc = 0;   
} 

if (read >= 410 && // Doing this will only pass the student 
    wri >= 410 && // if ALL of the conditions are met. 
    math >= 410 && 
    sci >= 410 && 
    soc >= 410 && 
    ( (read + wri + math + sci + soc) >= 2250 || // Separated more for clarity 
      ((read + wri + math + sci + soc)/5) > 450)) { 
    // Either all scores total over 2250 
    // Or the average of all 5 are over 450 to pass     
     document.write("PASSED") 
} 
else 
    document.write("NOT PASSED") 
} 

Answer 2

насчет

if ((read >= 410) && 
    (wri >= 410) && 
    (math >= 410) && 
    (sci >= 410) && 
    (soc >= 410) && 
    (read+wri+math+sci+soc >= 2250)) {  

    document.write("PASSED") 
} else { 

    document.write("NOT PASSED") 
} 

Answer 3

function gedCheck(read, wri, math, sci, soc) { 

    if(read < 0 || read > 1000) 
     read = 0; 

    if(wri < 0 || wri > 1000) 
     wri = 0; 

    if(math < 0 && math > 1000) 
     math = 0; 

    if(sci < 0 && read > 1000) 
     read = 0; 

    if(soc < 0 && soc > 1000) 
     soc = 0; 

    var total = read + wri + math + sci + soc; 

    if (read >= 410 && wri >= 410 && math >= 410 && sci >= 410 && soc >= 410 && total >= 2250) { 
     document.write("PASSED"); 
    } else { 
     document.write("NOT PASSED"); 
    } 
} 

Все это первый раздел был невозможно код. Он проверял, было ли число БОЛЬШЕ меньше нуля и больше 1000. Очевидно, это невозможно, поэтому я изменил его на использование OR.

Я также создал полную переменную, которую вы можете проверить так же, как и все остальное.

Answer 4

Использование массива здесь поможет вам уменьшить количество дублированного кода

function gedCheck(read, wri, math, sci, soc) { 
    var subjects, totalScore, averageScore; 

    subjects = [read, wri, math, sci, soc]; 
    totalScore = 0; 
    averageScore = 0; 

    for (var i = 0; i < subjects.length; i++) { 
     if (subjects[i] < 0 || subjects[i] > 1000) { 
      subjects[i] = 0; 
     } 
     totalScore += subjects[i]; 
    }; 

    averageScore = totalScore/subjects.length; 

    if (averageScore >= 450 || totalScore >= 2250) { 
     document.write("PASSED"); 
    } else { 
     document.write("NOT PASSED"); 
    } 
} 

Первый цикл повторяется через каждый предмет и устанавливают его в ноль, если это необходимо, а затем добавляют к общим переменным баллам.

Затем общий балл усредняется по количеству предметов.

Тогда, если средний балл равен или больше 450, или равен или больше, чем 2250, он проходит.