C++ добавление по одному до массива до полного

Question

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

Пустота, что я использую = cpp.sh/9np2

Вот код

void GetGrades(StatsList* stats) 
{ 
    ListType grade = 0; 
    while (grade != -1) 
{ 
    cout << "Enter grade (-1 quits):"; 
    cin >> grade; 
    if (!cin) 
    { 
     cin.clear(); 
     cin.ignore(2000, '\n'); 
     cout << "Invalid number" << endl; 
    } 
    else if (grade >= MINGRADE && grade <= MAXGRADE) 
    {  
     if (!stats->NewGrade(grade)) 
     { 
      cout << grade << " not entered - grade list is full" << endl; 
     } 
    } 
    else 
    { 
     if (grade != -1) 
      cout << "Grade out of range" << endl; 
    } 

} 
} 

класс функций, которые я использую = cpp.sh/2jkz

bool StatsList::NewGrade(ListType grade) 
{ 
    int i = 0; 
    bool full = (List[6] == '\0'); 

    if (full){ 
     List[i] = grade; 
     i++; 
     cout << grade; 
} 
    return false; 
} 

Я устал AF, и я не могу понять, что Я делаю неправильно, помогите девушке. Я просто хочу понять, что я делаю неправильно, прежде чем лечь спать.

Answer 1

В bool StatsList::NewGrade(ListType grade) есть пара проблем.

1. i необходимо сохранить от вызова к вызову, иначе он всегда будет установлен в 0 в начале функции. Я рекомендую дать более описательное имя, например listindex, и сделать его переменной-членом.
2. Нет return true. Логическое место, чтобы положить его, похоже, находится в конце внутри тела if (full).

Слово на full. full действительно, если List есть нет полный. Неправда в этом наименовании будет вводить в заблуждение людей и приводить к ошибкам, введенным людьми, поддерживающими код в будущем. Я рекомендую изменить имя или изменить способ его использования. Изменение имени кажется более логичным для меня.