Conta dei caratteri

[light95]

Buonasera a tutti, chi mi saprebbe aiutare in questa programmino?!

//Programma che legge N caratteri da tastiera e comunica quante a,b,c,d,e sono state inserite. //

Purtroppo il problema è che, quando mi stampa il numero di caratteri, risultano numeri tutti sballati... eppure penso di aver fatto tutto bene :/

Ecco il codice:

#include<iostream>
#include <conio.h>


using namespace std;


int main()
{
    int i, n, somma=0, v1[4];
 char    v[n];
    
cout<<"Quanti sono i caratteri che vuoi inserire? " ;
cin>> n;
for(i=0; i<n;i++) 
 {
     v1[i]=0;


 }
 
 
for(i=0; i<n;i++)
{
    cout<<"Inserisci il "<<i+1<<" carattere ";
    cin>> v[i];
    
}


for(i=0; i<n; i++)
{
        switch(v[i])
    {
        case 'a':
            v1[0]= v1[0] +1;
        break;
        
        case 'b':
            v1[1]= v1[1] +1;
        break;
        
        case 'c':
            v1[2]= v1[2] +1;
        break;
        
        case 'd':
            v1[3]= v1[3] +1;
        break;
        
        case 'e':
            v1[4]= v1[4] +1;
        break;
    }
}




cout<<"Sono state inserite " << v1[0] << " a \n";
cout<<"Sono state inserite " << v1[1] << " b \n";
cout<<"Sono state inserite " << v1[2] << " c \n";
cout<<"Sono state inserite " << v1[3] << " d \n";
cout<<"Sono state inserite " << v1[4] << " e \n";


system("pause");
return 0; 
}

[Scara95]

char v[n]; non è possibile farlo: la lunghezza dovrebbe essere un'espressione valutabile a tempo di compilazione. Per lo più n non è inizializzata e contiene perciò un valore casuale e impredicibile.
v1[i] = 0; 0 <= i < n ma v1 ha solo 4 elementi quindi dovrebbe essere 0 <= i < 4
v1[4] non vi puoi accedere per lo stesso motivo di cui sopra

[light95]

char v[n]; non è possibile farlo: la lunghezza dovrebbe essere un'espressione valutabile a tempo di compilazione. Per lo più n non è inizializzata e contiene perciò un valore casuale e impredicibile.
v1[i] = 0; 0 <= i < n ma v1 ha solo 4 elementi quindi dovrebbe essere 0 <= i < 4
v1[4] non vi puoi accedere per lo stesso motivo di cui sopra

EDIT: Risolto... ti ringrazio

[oregon]

Controlla questo

for(i<=0; i<n;i++)

[light95]

Controlla questo

for(i<=0; i<n;i++)

Ho risolto... in pratica non riusciva a inizializzare a 0 il vettore v1...

ho risolto facendo :

for(i=0; i<=4;i++)

Ma mi spieghi il perchè si mette i<=4 e non i<4?!
Cioè, gli elementi del vettore sono 5... da 0 a 4 sono 5 numeri... perchè bisogna mettere anche il " = " ?!

Non so se mi son spiegato bene :P

[oregon]

No, questo

for(i=0; i<4;i++)

è sbagliato (e già ti era stato detto). Se gli elementi sono 5 deve essere

for(i=0; i<5;i++)

Quello che ti avevo detto io è che è sbagliata quest'altra for

for(i<=0; i<n;i++)

prima che tu modificassi il post (cosa che, tra l'altro, non si fa, altrimenti non si capisce nulla del discorso ...)

[light95]

Si scusami,hai ragione... non lo farò più, promesso :P

Ritornando a noi... mi rispondi alla mia domanda del post 5? Vorrei capire a livello logico...

[oregon]

Se gli elementi sono 5 devi scrivere <5 oppure <=4 perché in questo modo l'ultimo elemento trattato sarà il numero 4

Ma se gli elementi sono 4 allora <=4 è sbagliato

[light95]

Capito... ti ringrazio

[Scara95]

@light attento che funziona è diverso da corretto. Funziona in ogni circostanza va già meglio.