Ricerca delle vocali in una stringa

**JanQuarius** · 17-02-2011, 20:51

Ciao a tutti!
Vi presento l'esercizio che dovevo fare:

codice:

/*
<<<Descrizione del problema>>>
Bisogna creare un programma che
ne legge una stringa in input 
e ne conti quanti sono le vocali
nella stringa.

<<<Analisi dei dati>>>
	input:
	chiedo, leggo e memorizzo la stringa
	
	output:
	stampo quante vocali ci sono nella stringa
	
	variabili:
		parola(stringa);
		numeroVocali(int);
		numeroLettere(int);
                contatore(int);
	
	implementazioni:
		classe di utilità "Scanner";
		
<<<Pseudocodifica>>>
inizio classe Esercizio1

	dichiaro le variabili;
		contatore(int) = -1;
		numeroLettere(int) = 0;
		numeroVocali(int) = 0;
		parola(String) = "";
		
	chiedo, leggo e memorizzo la parola in input
	parola <- stringa
	vedo quante lettere ha la stringa e memorizzo il loro numero
	numeroLettere <- (int) parola
	
	ripeto
	incremento di uno il contatore
	se la lettera della stringa è una vocale
	allora
	incremento di uno numeroVocali
	
	finchè numeroLettere >= contatore
 
        stampo il risultato

fine classe Esercizio1
*/

//importo la classe Scanner
import java.util.Scanner;

class ricercaVocali
{
	public static void main (String[]args)
	{
		//dichiaro le variabili
		String parola = "";
		int numeroVocali = 0;
		int numeroLettere;
		int contatore = 0;
	
	//chiedo, leggo e memorizzo la sparola in input
	System.out.print("Inserire la parola:");
	Scanner sc = new Scanner(System.in);
	parola = sc.nextLine();

	numeroLettere = parola.length();
	
	//inizio il ciclo
	do
	{
	contatore++;
		if (parola.charAt(contatore)=='a')
		if (parola.charAt(contatore)=='e')
		if (parola.charAt(contatore)=='i')
		if (parola.charAt(contatore)=='o')
		if (parola.charAt(contatore)=='u')
		if (parola.charAt(contatore)=='A')
		if (parola.charAt(contatore)=='E')
		if (parola.charAt(contatore)=='I')
		if (parola.charAt(contatore)=='O')
		if (parola.charAt(contatore)=='U')
	numeroVocali++;
	}
	while (contatore>numeroLettere);
	
	System.out.println("Nella parola ci sono presenti "+numeroVocali+" vocali");
	
	}
}

Il problema è che qualsiasi stringa io metta in input anche di 50 caratteri di lunghezza, in output mi da "0"...

per esempio:
prendo in input il la stringa "Massimiliano"
ed ho in output questo:
"Nella parola ci sono presenti 0 vocali"

COME RISOLVERE QUESTO PROBLEMA???
Grazie a tutti in anticipo!

**VincenzoTheBest** · 17-02-2011, 21:14

Originariamente inviato da JanQuarius

codice:

import java.util.Scanner;

class ricercaVocali
{
	public static void main (String[]args)
	{
		//dichiaro le variabili
		String parola = "";
		int numeroVocali = 0;
		int numeroLettere;
		int contatore = 0;
	
	//chiedo, leggo e memorizzo la sparola in input
	System.out.print("Inserire la parola:");
	Scanner sc = new Scanner(System.in);
	parola = sc.nextLine();

	numeroLettere = parola.length();
	
	//inizio il ciclo
	do
	{
	contatore++;
		if (parola.charAt(contatore)=='a')
		if (parola.charAt(contatore)=='e')
		if (parola.charAt(contatore)=='i')
		if (parola.charAt(contatore)=='o')
		if (parola.charAt(contatore)=='u')
		if (parola.charAt(contatore)=='A')
		if (parola.charAt(contatore)=='E')
		if (parola.charAt(contatore)=='I')
		if (parola.charAt(contatore)=='O')
		if (parola.charAt(contatore)=='U')
	numeroVocali++;
	}
	while (contatore>numeroLettere);
	
	System.out.println("Nella parola ci sono presenti "+numeroVocali+" vocali");
	
	}
}

Intanto la condizione del ciclo è sbagliata. Poi ti basta un solo if nel quale devi collegare le varie condizioni mediante l'operatore logico OR, che in java si indica col simbolo ||.

**lolide** · 17-02-2011, 21:16

Sono sbagliate molte cose in questo esercizio;

Intanto quegli if così messi non vanno bene.

codice:

		if (parola.charAt(contatore)=='a')
		if (parola.charAt(contatore)=='e')
		if (parola.charAt(contatore)=='i')
		if (parola.charAt(contatore)=='o')
		if (parola.charAt(contatore)=='u')
		if (parola.charAt(contatore)=='A')
		if (parola.charAt(contatore)=='E')
		if (parola.charAt(contatore)=='I')
		if (parola.charAt(contatore)=='O')
		if (parola.charAt(contatore)=='U')
	numeroVocali++;

Così il compilatore intende: se la lettera in posizione contatore è 'a' allora se la lettera in posizione contatore è 'e' allora se la lettera in posizione ecc...
Devi concatenare tutte le condizioni con l'operatore || (or), che verifica se una condizione è vera o una condizione è vera o una condizione è vera ecc...

Poi il while:

codice:

	//inizio il ciclo
	do
	{
	contatore++;
        ....
	}
	while (contatore>numeroLettere);

Intanto incrementi contatore all'inizio del ciclo, quindi siccome charAt vuole un indice che parta da 0, o all'inizio setti contatore a -1 invece di 0, oppure usi while invece del do-while.
Poi la condizione di ciclo è sbagliata. Tu cicli fin quando contatore è maggiore di numero lettere (cosa impossibile), quindi fa il controllo della 2^ lettera (perchè contatore diventa 1) ed esce subito dal ciclo.

**Celebron** · 17-02-2011, 21:17

in più mettendo gli if in quel modo l'operazione viene eseguita solo se la prima lettera è una 'a'
infatti se non lo è, gli altri if non vengono nemmeno visionati

come lo hai scritto tu, va letto
se la lettera in esame è una 'a' allora
controlla se la lettera in esame è anche una 'e' allora
eccetera

noterai che non ha nemmeno un lontano senso logico

edit: preceduto :P

**JanQuarius** · 20-02-2011, 10:53

Ok...ho corretto!

codice:

import java.util.Scanner;

class ricercaVocali
{
	public static void main (String[]args)
	{
		//dichiaro le variabili
		String parola = "";
		int numeroVocali = 0;
		int numeroLettere;
		int contatore = -1;
	
	//chiedo, leggo e memorizzo la sparola in input
	System.out.print("Inserire la parola:");
	Scanner sc = new Scanner(System.in);
	parola = sc.nextLine();

	numeroLettere = parola.length();
	
	//inizio il ciclo
	do
	{
	contatore++;
		if ((parola.charAt(contatore)=='a')||(parola.charAt(contatore)=='e')||(parola.charAt(contatore)=='i')||(parola.charAt(contatore)=='o')||(parola.charAt(contatore)=='u')||(parola.charAt(contatore)=='A')||(parola.charAt(contatore)=='E')||(parola.charAt(contatore)=='I')||(parola.charAt(contatore)=='O')||(parola.charAt(contatore)=='U'));
	numeroVocali++;
	}
	while (contatore<numeroLettere);
	
	System.out.println("Nella parola ci sono presenti "+numeroVocali+" vocali");
	
	}
}

Solo che adesso mi da un'altro errore:

...
Inserire la parola:Giardino

Exception in thread "main" java.lang.StringIndexOutOfBoundsException:
String index out of range: 8
at java.lang.String.charAt(String.java:686)
at ricercaVocali.main(ricercaVocali.java:76)
...

**VincenzoTheBest** · 20-02-2011, 11:48

Originariamente inviato da JanQuarius
...
Inserire la parola:Giardino

Exception in thread "main" java.lang.StringIndexOutOfBoundsException:
String index out of range: 8
at java.lang.String.charAt(String.java:686)
at ricercaVocali.main(ricercaVocali.java:76)
...

Inizializza contatore a 0, ed incrementalo dopo l'istruzione di incremento di numeroVocali.

**Alex'87** · 20-02-2011, 12:00

Attento, alla fine dell'if non ci va il ; (punto e virgola)!

Poi curiosità, perché hai un do-while e lo usi come se fosse un for?

Il while/do-while si usa quando non è noto a priori quanto iterare. Al contrario, se conosci il numero esatto di iterazioni il costrutto più adatto è il for.

codice:

for (int contatore = 0; contatore < numeroLettere; contatore++) {
  if ((parola.charAt(contatore) == 'a') || (parola.charAt(contatore) == 'e') ||
      (parola.charAt(contatore) == 'i') || (parola.charAt(contatore) == 'o') ||
      (parola.charAt(contatore) == 'u') || (parola.charAt(contatore) == 'A') ||
      (parola.charAt(contatore) == 'E') || (parola.charAt(contatore) == 'I') ||
      (parola.charAt(contatore) == 'O') || (parola.charAt(contatore) == 'U')) {
    numeroVocali++;
  }
}

**VincenzoTheBest** · 20-02-2011, 12:24

Originariamente inviato da Alex'87
Poi curiosità, perché hai un do-while e lo usi come se fosse un for?

Perchè nella specifica sopra gli hanno detto di usare un do-while.

**JanQuarius** · 20-02-2011, 12:31

Allora, ho coretto tutto quanto:

codice:

/*
<<<Descrizione del problema>>>
Bisogna creare un programma che
ne legge una stringa in input 
e ne conti quanti sono le vocali
nella stringa.

<<<Analisi dei dati>>>
	input:
	chiedo, leggo e memorizzo la stringa
	
	output:
	stampo quante vocali ci sono nella stringa
	
	variabili:
		parola(stringa);
		numeroVocali(int);
		numeroLettere(int);
		contatore(int);
	
	implementazioni:
		classe "Scanner";
		
<<<Pseudocodifica>>>
inizio classe Esercizio1

	dichiaro e inizializzo le variabili;
		contatore(int) = 0;
		numeroLettere(int) = 0;
		numeroVocali(int);
		parola(String) = "";
		
	chiedo, leggo e memorizzo la parola in input
	parola <- stringa
	vedo quante lettere ha la stringa e memorizzo il loro numero
	numeroLettere <- (int) parola
	
	ripeto
	
	se la lettera della stringa è una vocale
	allora
	incremento di uno numeroVocali
	e
	incremento di uno il contatore
	
	finchè numeroLettere < contatore
	
	stampo il risultato

fine classe Esercizio1
*/


import java.util.Scanner;

class ricercaVocali
{
	public static void main (String[]args)
	{
	
		String parola = "";
		int numeroVocali = 0;
		int numeroLettere;
		int contatore = 0;
	
	System.out.print("Inserire la parola:");
	Scanner sc = new Scanner(System.in);
	parola = sc.nextLine();

	numeroLettere = parola.length();
	
	do
	{
		if ((parola.charAt(contatore)=='a')||(parola.charAt(contatore)=='e')||(parola.charAt(contatore)=='i')||(parola.charAt(contatore)=='o')||(parola.charAt(contatore)=='u')||(parola.charAt(contatore)=='A')||(parola.charAt(contatore)=='E')||(parola.charAt(contatore)=='I')||(parola.charAt(contatore)=='O')||(parola.charAt(contatore)=='U'))
		numeroVocali++;
		contatore++;
	}
	while (contatore<numeroLettere);
	
	System.out.println("Nella parola ci sono presenti "+numeroVocali+" vocali");
	
	}
}

Poi curiosità, perché hai un do-while e lo usi come se fosse un for?

Il while/do-while si usa quando non è noto a priori quanto iterare. Al contrario, se conosci il numero esatto di iterazioni il costrutto più adatto è il for.

Non l'ho usato perchè il prof. non me l'ha ancora insegnato...sicuramente hai ragione dal punto di vista logico! Appena lo vedo, lo chiedo perchè non ci insegna di più funzioni...di solito ci fa programmare in maniera strana, mi sembra di aver pochi strumenti a disposizione.
Tu che ne pensi di questo?

Grazie a tutti!
Il programma mi è venuto!

**Alex'87** · 20-02-2011, 13:17

Originariamente inviato da VincenzoTheBest
Perchè nella specifica sopra gli hanno detto di usare un do-while.

Ops, non l'avevo letta quella parte

Discussione: Ricerca delle vocali in una stringa

Strumenti discussione

Ricerca discussione

Visualizza

Ricerca delle vocali in una stringa

Re: Ricerca delle vocali in una stringa

Re: Ricerca delle vocali in una stringa

Permessi di invio