Variable is not a type-CodePudding

#include<iostream>
#include<string>
#include<vector>
using namespace std;
class Course
{
    public:
        string name;
        string instructorInCharge;
        int numberOfStudents;
        int totalMarks;
        Course(string u, string v, int p, int q){
            this->name=u;
            this->instructorInCharge=v;
            this->numberOfStudents=p;
            this->totalMarks=q;

        }
    
    vector<int> studentMarks (numberOfStudents);

    

};
class Labs:public Course
{
   
        vector<int>labMarks(numberOfStudents);

};

int main()
{
    Course c("Rahul","Hota",200,300);
    cout<<"hello there";
}

When I compile this code I’m getting the following error:

1.cpp:20:31: error: 'numberOfStudents' is not a type
   20 |     vector<int> studentMarks (numberOfStudents);
      |                               ^~~~~~~~~~~~~~~~
1.cpp:28:29: error: 'numberOfStudents' is not a type
   28 |         vector<int>labMarks(numberOfStudents);
      |                             ^~~~~~~~~~~~~~~~

Please tell me what my mistake is. numberostudents was supposed to be the size of the vector. But not any function.

CodePudding user response：

vector<int> studentMarks (numberOfStudents);

This is a function declaration. The return type is vector<int>, the name is studentMarks and it accepts a single parameter of type numberOfStudents. Except, as the error points out, numberOfStudents is not a type.

You cannot use parentheses to specify a default member initialiser. You have to either use curly braces:

T member {value};

or "equals" initialiser:

T member = value;

Note however, that you don't initialise numberOfStudents member, but rather assign it later in the constructor body. This assignment is after studentMarks has been initialised, and thus the default member initialiser of studentMarks won't reflect the size that was assigned. Rather, the behaviour of the program would be undefined due to the use of an indeterminate value. This can be fixed by initialising numberOfStudents in the member initialiser list.

CodePudding user response：

If you want in-class initialization, you may do this:

vector<int> studentMarks = vector<int>(numberOfStudents);

Now the default constructor will initialize studentMarks with vector<int>(numberOfStudents) when an instance of Course is created.

CodePudding user response：

You can also use resize method of std::vector.

#include<iostream>
#include<string>
#include<vector>
using namespace std;
class Course
{
public:
    string name;
    string instructorInCharge;
    int numberOfStudents;
    int totalMarks;

    
    vector<int> studentMarks;
    
    Course(string u, string v, int p, int q){
        this->name = u;
        this->instructorInCharge = v;
        this->numberOfStudents = p;
        this->totalMarks = q;

        this->studentMarks.resize(numberOfStudents);
    }
};
class Labs : public Course
{
    vector<int> labMarks;
public:
    Labs(string u, string v, int p, int q) : Course(u, v, p, q)
    {
        labMarks.resize(p);
    }
};

int main()
{
    Course c("Rahul","Hota",200,300);
    cout<<"hello there";
}