Change two positions,

Before the change: void factorial(vector<T> ivec, vector<T>::iterator iter)
After the change: void factorial(vector<T> & ivec, typename vector<T>::iterator iter)

Let’s talk about the second change position first. What is typename for? Reference http://blog.csdn.net/laojiu_/...

I can’t explain the location of the first change. If it is not quoted, VS will report a type mismatch when comparing with end(). I don’t understand this either. . .

When running on Vs2008, an error will be reported.
The reasons for the error are:
(1) vector<int> ivec = {1,2,3,4,5,6,7}; vector cannot be initialized and assigned directly like this. You can replace it with this

vector<int> ivec;
    for(int i=1;i<8;i++){
        ivec.push_back(i);
    }

(2) The function is not well written. Just write it like this. There is no need to pass ivec.begin() to the function. Because you have already passed ivec to the function, the function will get all the information of ivec without adding any extraneous information.

#include "iostream"
#include <vector>
using namespace std;
void factorial(vector<int> ivec) {
    vector<int>::iterator it;
    for(it=ivec.begin();it!=ivec.end();it++)
    cout<<*it<<endl;
}

int main()
{
    vector<int> ivec;
    for(int i=1;i<8;i++){
        ivec.push_back(i);
    }
    factorial(ivec);

    return 0;
}