Va bene ... a parte delle imperfezioni e qualche miglioramento ... pero', dato che indica di riempire e visualizzare il vettore, non farlo sarebbe un errore ...
E poi non dice di visualizzare i 1000 numeri ...
Farei queste piccole modifiche ...
Se vuoi apportare un miglioramento, rendi tutte le variabili e vettori locali ...codice:/*Programma che riempito un vettore di n elementi compresi tra 1 e 12, calcoli tramite l'utilizzo di una funzione, che ha per argomenti il numero del quale bisogna trovare la frequenza e la dimensione del vettore, la frequenza di ogni elemento del vettore*/ #include <iostream> #include <stdlib.h> #include <time.h> using namespace std; //Inizializzo le variabili const int LUN=1000; //grandezza del vettore int vett[LUN]; //vettore che contiene elementi int vfre[12]; int freq(int numero) { int n, cont; for(n=cont=0; cont<LUN; cont++) if(vett[cont]==numero) n++; return n; } main() { int cont, numero; srand(time(NULL)); cout<<"Calcolo la frequenza di uscita dei numeri in 1000 lanci di 2 dadi:"<<endl; for(cont=0; cont<LUN; cont++) vett[cont]=rand()%12+1; for(numero=1; numero<13; numero++) { vfre[numero-1]=freq(numero); cout << "\nIl numero " << numero << " e' presente " << vfre[numero-1] << " volte" << endl; } getchar(); }

Rispondi quotando