Почему в while оценку последнего человека он выводит еще раз и с новой строки?

Question

[dzzaad]

Код

#include <iostream>
#include <Windows.h>
#include <fstream>
using namespace std;
struct Student
{
    char firstname[20], name[20], patronymic[20];
    int math, physics, drawing, chemistry, copromat;
};
int main() {
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    ofstream zapis;
    zapis.open("first_file.txt", ios_base::out | ios_base::trunc | ios_base::binary);
    struct Student cheliki;
    int count_students, i = 1;
    float average_mark, count_physics_mark = 0.0;
    cout << "Введите количество учеников: ";
    cin >> count_students;
    while (i <= count_students) {
        cout << "Введите фамилию " << i << "-ого ученика: ";
        cin >> cheliki.firstname;
        cout << "Введите имя " << i << "-ого ученика: ";
        cin >> cheliki.name;
        cout << "Введите отчество " << i << "-ого ученика: ";
        cin >> cheliki.patronymic;
        cout << "Введите оценку " << i << "-ого ученика по математике: ";
        cin >> cheliki.math;
        cout << "Введите оценку " << i << "-ого ученика по физике: ";
        cin >> cheliki.physics;
        count_physics_mark += cheliki.physics;
        cout << "Введите оценку " << i << "-ого ученика по черчению: ";
        cin >> cheliki.drawing;
        cout << "Введите оценку " << i << "-ого ученика по химии: ";
        cin >> cheliki.chemistry;
        cout << "Введите оценку " << i << "-ого ученика по сопромату: ";
        cin >> cheliki.copromat;
        i++;
        zapis << cheliki.firstname << " " << cheliki.name << " " << cheliki.patronymic << " " <<  cheliki.physics << endl;
    }
    ifstream rFile;
    rFile.open("first_file.txt", ios_base::in | ios_base::binary);
    while (!rFile.eof()) {
        rFile >> cheliki.firstname >> cheliki.name >> cheliki.patronymic >> cheliki.physics;
        cout << cheliki.firstname << cheliki.physics << ' ' << endl;
    }
    average_mark = count_physics_mark / count_students;
    cout << "Средний балл группы по физике = " << average_mark;
    rFile.close();
    return 0;
}

Comments

[dzzaad]

gd1xza, опечатка, по сопромату

Answer 1

[Александр Александров]

zapis << cheliki.firstname << " " << cheliki.name << " " << cheliki.patronymic << " " << cheliki.physics << endl;
при такой записи (endl в конце) файл всегда будет с пустой строкой в конце. Соответственно при выводе из файла цикл делается на 1 раз больше чем надо.

Comments

[dzzaad]

Если убираю endl , то в файле все записывается в одну строчку и уже в цикле он не выводит Петров5, Иванов4, но продолжает выводить последнее число

[Александр Александров]

Нужно просто удостоверится что при чтении с файла все поля структуры корректно заполнились. Когда в последней итерации не получится заполнить поля структуры возникнет ошибка и ее можно проигнорить..например так:
while (!rFile.eof()) {
rFile >> cheliki.firstname >> cheliki.name >> cheliki.patronymic >> cheliki.physics;
if(!rFile.fail()){
cout << cheliki.firstname << cheliki.physics << ' ' << endl;
}
}

[Александр Александров]

соответственно endl не нужно будет убирать

[dzzaad]

Александр Александров, Спасибо большое. Корректно работает. Выручили!