Как упростить код для оценки тестов - новичок в JavaScript

Оффтоп javascript
1

Привет всем! Недавно начал изучать программирование и написал простую программу для оценки результатов тестов.

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

Вот мой код:

function calculatePercent(earnedPoints, totalPoints) {
    return 100 / totalPoints * earnedPoints;
};

function getResult(earnedPoints, totalPoints){
  if (calculatePercent(earnedPoints, totalPoints) >= 75) {
    return 'Отлично, тест сдан!';
  }
  else {
    return 'Тест пройден, но результат слабый';
  }
};

console.log(getResult(40, 40));
console.log('Ваш результат: ' + calculatePercent(40, 40));

Как можно оптимизировать этот код, чтобы не дублировать одни и те же значения? Буду благодарен за любые советы!

2

А можно еще через объект - функция возвращает процент и текст результата сразу:

function testResult(earned, total) {
    const percent = 100 / total * earned;
    return {
        percent: percent,
        message: percent >= 75 ? 'Отлично, тест сдан!' : 'Тест пройден, но результат слабый'
    };
}

const result = testResult(40, 40);
console.log(result.message);
console.log('Ваш результат: ' + result.percent);

Один вызов функции - все данные получаешь. Как тебе такой вариант?

3

Просто сохрани результат calculatePercent в переменную, чтобы не вызывать функцию дважды:

function getResult(earnedPoints, totalPoints){
  const percent = calculatePercent(earnedPoints, totalPoints);
  
  if (percent >= 75) {
    return 'Отлично, тест сдан!';
  }
  else {
    return 'Тест пройден, но результат слабый';
  }
};

const earned = 40;
const total = 40;
console.log(getResult(earned, total));
console.log('Ваш результат: ' + calculatePercent(earned, total));

Так изменяешь только две переменные earned и total вместо четырех одинаковых значений. Удобнее и меньше ошибок!

4

Можно еще проще - одна функция, которая сразу возвращает результат и процент:

function evaluateTest(earnedPoints, totalPoints) {
    const percent = 100 / totalPoints * earnedPoints;
    const result = percent >= 75 ? 'Отлично, тест сдан!' : 'Тест пройден, но результат слабый';
    
    console.log(result);
    console.log('Ваш результат: ' + percent);
}

evaluateTest(40, 40);

Так параметры указываешь только в одном месте. Процент считается один раз и используется везде.