请问一下这个的正确做法以及我的代码问题在哪里

用c++语言，是找两个最大值那里出错误了么，前面的输入n得到n个字符串的关系如何编写

仅供参考，望采纳~

#include <iostream>
using namespace std;
int main(void)
{
    int n,i,max1,max2;
    cin>>n;
    int a[n];
    for(i=0;i<n;i++)
    {
        cin>>a[i];
    }
    max1=a[0];
    max2=a[0];
    for(i=0;i<n;i++)
    {
        if(max1<a[i])
        {
            max1=a[i];
        }
        if(max2<a[i]&&max1!=a[i])
        {
            max2=a[i];
        }
    }
    cout<<max1<<" "<<max2<<endl;
    return 0;
}