Векторы с проблемой оператора Cout в C ++

0

В моей программе я сталкиваюсь с ошибкой при попытке распечатать end() вектора. Вот код:

#include <iostream>
#include <string>
#include <fstream>
#include <vector>
#include <cstdlib>
#include <time.h>
#include "inFrench.h"

using namespace std;

class student
{
    int m_studentNumber;
public:
    string nameFirst;
    string nameLast;
    string nameFull;
    int getStudentNumber() { return m_studentNumber; }
    void setStudentNumber(int studentNumber) { m_studentNumber = studentNumber; }
};

class group
{
public:
    vector<student> groupMembers;
};

ostream& operator<<(ostream& os, const student& s)
{
    return os << s.nameFirst << ' ' << s.nameLast;
}

student typeName()
{
    student bar;
    cout << "Type in a student first name: ";
    cin >> bar.nameFirst;
    cout << "Type in that student last name: ";
    cin >> bar.nameLast;
    bar.nameFull = bar.nameFirst + " " + bar.nameLast;
    return bar;
}

void displayStudents(student listOfStudents[50], int studentHeadCount)
{
    for (int i = 0; i < studentHeadCount; i++)
    {
        cout << listOfStudents[i].nameFull << endl;
        cout << listOfStudents[i].getStudentNumber() << endl;
        cout << "\n";
    }
}

void generateGroups(int numberOfGroups, int maxStudents, int studentsPerGroup, int remainder, group foo, student theStudents[], int studentsBeenAssigned)
{
    int k;
    numberOfGroups = maxStudents / studentsPerGroup;
    cout << numberOfGroups << endl;
    srand(time(NULL));
    while (studentsBeenAssigned << maxStudents)
    {
        for (int j = 0; j < maxStudents; j++)
        {
            k = rand() % maxStudents;
            foo.groupMembers.push_back(theStudents[k]);
            cout << foo.groupMembers.end() << endl;
            studentsBeenAssigned++;
        }
    }
    if (remainder < studentsPerGroup && remainder > 0) // Still coding this section
    {
        foo.groupMembers.push_back(theStudents[k]);
    }
}

void languageChoices()
{
    cout << "Select your language from the following:\n";
    cout << "a) English\n";
    cout << "b) French\n";
    cout << "\n";
}

void options()
{

    cout << "Select what you want to do:\n";
    cout << "1) Exit application\n";
    cout << "2) Enter a Student\n";
    cout << "3) Display Students\n";
    cout << "4) Display Groups\n";
    cout << "5) Output groups as text file\n";
    cout << "\n";
}

int main()
{
    char selectedLanguage;
    languageChoices();
    cin >> selectedLanguage;
    switch (selectedLanguage)
    {
        case 'a':
        {
            group finishedListOfStudents;
            student allStudents[50]; // Having 50 students alone is ridiculous!
            bool endProg = 0;
            int maxStudents;
            int studentsPerGroup;
            int optionSelect;
            int studentHeadCount = 0;
            int remainder = 0;
            int numberOfGroups;
            int studentsBeenAssigned;
            cout << "GroupPicker 1.0\n";
            cout << "Note: This version of the program is intended for purposes of education only, " 
            << "specifically for teacher use in a classroom.\n\n";
            cout << "How many students are in the class?\n" << "(Note: You cannot have more than 50 in this program)\n";
            cin >> maxStudents;
            if (maxStudents > 50)
            {
                cerr << "Too many students!\n" << "Exiting program...\n";
                system("PAUSE");
                exit(1);
            }
            if (maxStudents >= 35 && maxStudents <= 50)
            {
                cout << maxStudents << " students? You are a pro!\n";
            }
            cout << "How many students per group?\n";
            cin >> studentsPerGroup;
            if (studentsPerGroup >= maxStudents || studentsPerGroup <= 1)
            {
                cerr << "You're kidding, right?\n" << "Exiting program...\n";
                system("PAUSE");
                exit(1);
            }
            while (endProg == 0) {
                options();
                cin >> optionSelect;
                switch (optionSelect) {
                    case 1:
                        endProg = 1;
                        break;
                    case 2:
                    {
                        if (studentHeadCount == maxStudents)
                        {
                            cerr << "You can't enter more than " << maxStudents << " students\n";
                        }
                        else
                        {
                            allStudents[studentHeadCount] = typeName();
                            allStudents[studentHeadCount].setStudentNumber(studentHeadCount);
                            cout << "Student (" << allStudents[studentHeadCount].nameFull << ") entered.\n";
                            cout << "\n";
                            studentHeadCount++;
                        }
                        break;
                    }
                    case 3:
                        cout << "Current list of students:\n\n";
                        displayStudents(allStudents, studentHeadCount);
                        break;
                    case 4:
                    {
                        if (studentHeadCount < studentsPerGroup || studentHeadCount < maxStudents)
                        {
                            cerr << "Invalid group parameters.\n" << "Returning to main menu...\n\n";
                            break;
                        }
                        else
                        {
                            cout << "Here are the groups:\n";
                            generateGroups(numberOfGroups, maxStudents, studentsPerGroup, remainder, finishedListOfStudents, allStudents, studentsBeenAssigned);
                        }
                        break;
                    }
                    case 5:
                    {
                        cout << "Saving groups to file...\n";
                        ofstream studentGroups;
                        studentGroups.open("studentGroups.txt");
                        break;
                    }
                }
            }
            break;
        }
        case 'b':
        {
            mainInFrench();
        }
    }
}

Я получаю следующие результаты:

error C2679: binary '<<' : no operator found which takes a right-hand operand of type 'std::_Vector_iterator<_Myvec>' (or there is no acceptable conversion) [Line 66]
IntelliSense: no operator "<<" matches these operands. operand types are: std::ostream << std::_Vector_iterator<std::_Vector_val<std::_Simple_types<student>>> [Line 66]

Я уже рассмотрел перегрузку ostream operator<< оператора ostream operator<< снова, но есть ли другой вариант?

  • 0
    Пожалуйста, также поделитесь своим кодом "inFrench.h" .
  • 0
    Вы НЕ МОЖЕТЕ напечатать end() , так как это значение «один за другим». Вы, вероятно, хотите back() . Я не думаю, что это вызывает вашу ошибку компиляции, хотя.
Показать ещё 1 комментарий
Теги:
vector
operator-overloading

2 ответа

4
Лучший ответ

std::vector::end() возвращает итератор в конец вектора, а не фактический последний элемент.

Вместо этого вы можете напечатать последний элемент:

cout << foo.groupMembers.at(foo.groupMembers.size() - 1);
  • 6
    Или foo.groupMembers.back() .
  • 0
    Или foo.groupMembers[foo.groupMembers.size() - 1)] .
Показать ещё 2 комментария
0

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

  • 2
    Массивы злые.
  • 0
    Использование вектора обычно будет означать меньше боли в дальнейшем. Они заботятся об управлении памятью, следят за размером, могут гарантировать, что вы не будете читать / записывать вне границ и т. Д. Все это должно быть сделано вручную, если вы используете обычные массивы. std::vector довольно часто является лучшим выбором для обработки непрерывного набора однородных данных.
Показать ещё 4 комментария

Ещё вопросы

Сообщество Overcoder
Наверх
Меню