Использование вектора со структурами

Question

У меня проблемы с инициализацией базы данных, но я не могу понять, что это такое. Эта программа компилируется без ошибок, но всегда сбой при выполнении. Любая идея, что не так внутри функции initialise_database?

#include <iostream> 
#include <cstddef> 
#include <cstdlib> 
#include <fstream> 
#include <string> 
#include <vector> 
#include <algorithm> 
using namespace std; 

struct Date 
{ 
int day, month, year; 
}; 


struct Grade 
{ 
int s1, s2, s3, s4, s5, s6; 
}; 

struct Student 
{ 
bool free; 
long studentID; 
string fname; 
string sname; 
Date DOB, DOE, DOG; 
Grade semester1, semester2; 
}; 

void initialise_database(vector<Student>, int size); // initialize each free variable to free 

int main(int argc, char** argv) 
{ 
fstream fin; 
char choice_readfile; 
int rowcount; 

int size; 
cout << "Enter number of student:\n"; 
cin >> size; 

vector<Student> BENG; 


    do           //verify choice to read from file is Y,y or N,n 
    { 
     cout << "Do you wish to read from file (Y/N)? (file name must be named students.txt)" <<  endl;   //choice for user to read from external file 
     cin >> choice_readfile; 
      while(cin.fail())  
      { 
       cin.clear(); 
       cin.ignore(80,'\n'); 
       cout << "Please Re-Enter choice" << endl; 
       cin >> choice_readfile;    // choice to read from file 
      } 
    } 
    while(choice_readfile != 'Y' && choice_readfile != 'y' && choice_readfile != 'N' && choice_readfile != 'n'); 

    if(choice_readfile == 'Y' || choice_readfile == 'y') 
     { 
      fin.open("students.txt", ios::in|ios::out); //opens mygrades.txt 
      if(fin.fail()) 
      { 
       cout << "Error occured while opening students.txt" << endl; 
       exit(1); 
      } 
      fin.clear();   
      fin.seekg(0);  

      string line;  
      while(getline(fin, line))  //counts the rows in the external file 
       { 
        rowcount++; 
       }   

      cout << "Number of rows in file is " << rowcount << endl; 
      cout << size << " " << rowcount << endl; 
     } 

size += rowcount; 
int i=0; 
initialise_database(BENG, size); 
return 0; 
} 

void initialise_database(vector<Student> BENG, int size) 
{ 
    for(int i=0;i<size;i++) 
    { 
     BENG[i].free = false; 
    } 
} 

Answer 1

vector<Student> BENG; здесь вы определяете пустой вектор. Позже в функции:

void initialise_database(vector<Student> BENG, int size) 
{ 
    for(int i=0;i<size;i++) 
    { 
     BENG[i].free = false; 
    } 
} 

поля доступа Вы с индексами до size в этом векторе. Поскольку вектор пуст, любой доступ элемента будет терпеть неудачу. Вызовите изменение размера вектора перед вызовом initialise_database. Что-то вроде этого:

size += rowcount; 
BENG.resize(size); 
initialise_database(BENG, size); 

Answer 2

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

1) std :: vector не растет автоматически. Чтобы увеличить размер вектора, вам нужно использовать методы resize(), push_back(), emplace_back() или insert(). Вы не используете их. Ваш вектор остается пустым. Попытка доступа к несуществующим элементам вектора приводит к неопределенному поведению.

2) Параметр std :: vector для initialize_database() передается по значению. Это делает копию вектора, и любые изменения в векторе в initialize_database() не будут отражены в векторе, который вы фактически передали этой функции.