c++ o jedna liczba za dużo

#include<iostream>
using namespace std;

int dane(int tab[], int n)
{
    srand(time(NULL));
    for(int i=0; i<n; i++)
            tab[i]=rand()%1000;
}
int main()
{
    int n;
    cin>>n;
    int *tab = new int[n];
    cout<<dane(tab, n);
    for(int i=0; i<n; i++)
            cout<<endl<<tab[i];
    delete[] tab;
    system("pause");
}
 

Program ma za zadanie wpisać losowe liczby do tablicy o długości n. Problem w tym, że jak wprowadze jak długa ma być ta tablica np. 3 to wyświetli mi 3 i dodatkowo jeszcze 3 wylosowane liczby. Kod wydaje mi sie okey, ale program działa niepoprawnie.

Powinno raczej być:

#include<iostream>
using namespace std;

void dane(int tab[], int n)
{
    for(int i=0; i<n; i++)
            tab[i]=rand()%1000;
}
int main()
{
    int n;
    srand(time(NULL));
    cin>>n;
    int *tab = new int[n];
    dane(tab, n);
    for(int i=0; i<n; i++)
            cout<<tab[i]<<'\n';
    delete[] tab;
    cin.sync();
    cin.get();
    return 0;
}

int dane(****)

widzę to dzisiaj 5 raz:) Chyba gubbi wziąłeś się do nauki or something:)

Działa :) dzięki!

P.S. Kiedyś trzeba zacząć...

Jeszcze jedno pytanie, co robi cin.get()? bo wyleciało mi z głowy.

Ban na google?
http://www.cplusplus.com/reference/iostream/istream/get/

Okey, po polsku jak mam sobie to tłumaczyć?

Tak jak masz napisane. Jeśli nie umiesz angielskiego to zrezygnuj z programowania bo daleko to nie uciągniesz.

znam angielski w jakimś stopniu. Musze wiedzieć tylko, czy dobrze sobie tłumacze cin.get()

Bardzo ogólnie mówiąc, użycie cin.get + cin.sync to lepsza wersja system("pause") czy getchar(), pisanych na koncu programu, by ten nie zamykał się od razu, aby programista mógł zobaczyć wyniki pracy.

problem napisał(a)

znam angielski w jakimś stopniu. Musze wiedzieć tylko, czy dobrze sobie tłumacze cin.get()

ja pier*** jak masz takie głupoty pisać to milcz!
Znam angielski ale nie wiem czy dobrze tłumaczę..