[C++] Problema con sottoprogramma

[Aqua3o]

Ciao a tutti!Sarò breve...
Perké il programma scritto qui sotto mi funziona perfettamente, mentre il secondo, la cui unica differenza è la ri-kiamata di un sottoprogramma (l'unica differenza è evidenziata in rosso) non funziona?
Aiutatemi!

PROGRAMMA FUNZIONANTE

#include <iostream>
#include <stdio.h>

using namespace std;
int NumeroLettere(int Max, int Conto);
int main(){
bool Controllo=false;//Variabile di controllo (per evitare spazi doppi)
int NumeroSpazi, Conto, Max, Indice;
char Ch;
char Frase[1024];
Max = NumeroSpazi = Conto = Indice = 0;
cout<<"Scrivi una frase e premi invio: ";
while((Ch = getchar()) != '\n' && Indice<1024){
Frase[Indice++] = Ch;
if(Ch == ' ') {
if(!Controllo)//Imposto il controllo per vedere se non ci sono spazi doppi
NumeroSpazi++;
Controllo=true;
NumeroLettere(Max,Conto);
Conto = -1;
}
else Controllo=false;//Resetto il controllo
Conto++;
if (Max < Conto) Max = Conto; }
cout<<"La frase che hai inserito e':"<<Frase<<endl;
cout<<"In questa frase ci sono " <<NumeroSpazi+1<<" parole"<<endl;
cout<<"La parola piu' lunga che hai immesso e' di lunghezza pari a: "<<Max<<endl;
system ("PAUSE");
return 0;
}
int NumeroLettere(int Max, int Conto){
if(Max < Conto) Max = Conto;}


PROGRAMMA NON FUNZIONANTE

using namespace std;
int NumeroLettere(int Max, int Conto);
int main(){
bool Controllo=false;//Variabile di controllo (per evitare spazi doppi)
int NumeroSpazi, Conto, Max, Indice;
char Ch;
char Frase[1024];
Max = NumeroSpazi = Conto = Indice = 0;
cout<<"Scrivi una frase e premi invio: ";
while((Ch = getchar()) != '\n' && Indice<1024){
Frase[Indice++] = Ch;
if(Ch == ' ') {
if(!Controllo)//Imposto il controllo per vedere se non ci sono spazi doppi
NumeroSpazi++;
Controllo=true;
NumeroLettere(Max,Conto);
Conto = -1;
}
else Controllo=false;//Resetto il controllo
Conto++;
NumeroLettere(Max,Conto) }
cout<<"La frase che hai inserito e':"<<Frase<<endl;
cout<<"In questa frase ci sono " <<NumeroSpazi+1<<" parole"<<endl;
cout<<"La parola piu' lunga che hai immesso e' di lunghezza pari a: "<<Max<<endl;
system ("PAUSE");
return 0;
}
int NumeroLettere(int Max, int Conto){
if(Max < Conto) Max = Conto;}

[MItaly]

Se non indenti il codice e non lo inserisci tra i tag [CODE] ... [/CODE] non si capisce niente.

[oregon]

A parte il fatto che sono d'accordo con MItaly (come non si potrebbe ... ) sulla leggibilità del codice ...

... il problema mi sembra che sia sul fatto che richiamando quella funzione, NON modifichi il valore della variabile Max del main ... (e poi la funzione restituisce un int e non mi pare che ci sia il return ...) ...

[Aqua3o]

PROGRAMMA FUNZIONANTE

#include <iostream>
#include <stdio.h>

using namespace std;
int NumeroLettere(int Max, int Conto);
int main(){
bool Controllo=false;//Variabile di controllo (per evitare spazi doppi)
int NumeroSpazi, Conto, Max, Indice;
char Ch;
char Frase[1024];
Max = NumeroSpazi = Conto = Indice = 0;
cout<<"Scrivi una frase e premi invio: ";
while((Ch = getchar()) != '\n' && Indice<1024){
Frase[Indice++] = Ch;
if(Ch == ' ') {
if(!Controllo)
NumeroSpazi++;
Controllo=true;
NumeroLettere(Max,Conto);
Conto = -1;
if (Max < Conto) Max = Conto; 
}
else Controllo=false;
Conto++;
cout<<"La frase che hai inserito e':"<<Frase<<endl;
cout<<"In questa frase ci sono " <<NumeroSpazi+1<<" parole"<<endl;
cout<<"La parola piu' lunga che hai immesso e' di lunghezza pari a: "<<Max<<endl;
system ("PAUSE");
return 0;
}


int NumeroLettere(int Max, int Conto){
if(Max < Conto) Max = Conto;}

PROGRAMMA NON FUNZIONANTE

using namespace std;
int NumeroLettere(int Max, int Conto);
int main(){
bool Controllo=false;//Variabile di controllo (per evitare spazi doppi)
int NumeroSpazi, Conto, Max, Indice;
char Ch;
char Frase[1024];
Max = NumeroSpazi = Conto = Indice = 0;
cout<<"Scrivi una frase e premi invio: ";
while((Ch = getchar()) != '\n' && Indice<1024){
Frase[Indice++] = Ch;
if(Ch == ' ') {
if(!Controllo)
NumeroSpazi++;
Controllo=true;
NumeroLettere(Max,Conto);
Conto = -1;
}
else Controllo=false;
Conto++;
NumeroLettere(Max,Conto)  }
cout<<"La frase che hai inserito e':"<<Frase<<endl;
cout<<"In questa frase ci sono " <<NumeroSpazi+1<<" parole"<<endl;
cout<<"La parola piu' lunga che hai immesso e' di lunghezza pari a: "<<Max<<endl;
system ("PAUSE");
return 0;
}


int NumeroLettere(int Max, int Conto){
if(Max < Conto) Max = Conto;}

E' meglio così?
Ps.Come posso cambiare il valore nel main?

[oregon]

No ... non hai indentato il codice e quindi è ancora confuso ...

Comunque, la funzione deve essere

int NumeroLettere(int Max, int Conto)
{
   if(Max < Conto) 
     return Conto;
   else
     return Max;
}

e nel main

Max = NumeroLettere(Max,Conto)

[XWolverineX]

Altrimenti devi passare Max per riferimento (con un puntatore o il & dietro a Max nel prototipo di funzione)

[oregon]

Originariamente inviato da XWolverineX
Altrimenti devi passare Max per riferimento (con un puntatore o il & dietro a Max nel prototipo di funzione)

Non complicargli la situazione ...

[XWolverineX]

gne gne gne