PDA

View Full Version : [java] stampare posizioni vocali "i"


alberto t.
05-07-2009, 17:23
ciao a tutti!
ho realizzato questo programma che mi consente di cercare le vocali "i" all'interno di una stringa:


System.out.print("Le posizioni della vocale i:");
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
System.out.print((i));
}}


Ora vorrei fare in modo che le posizioni dei caratteri siano separati da una virgola...

Es, se digito:
Mario Rossi
Il risultato deve essere questo: 3,10

Come posso fare?
Ciao e grazie

luxorl
05-07-2009, 18:30
ciao a tutti!
ho realizzato questo programma che mi consente di cercare le vocali "i" all'interno di una stringa:


System.out.print("Le posizioni della vocale i:");
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
System.out.print((i));
}}


Ora vorrei fare in modo che le posizioni dei caratteri siano separati da una virgola...

Es, se digito:
Mario Rossi
Il risultato deve essere questo: 3,10

Come posso fare?
Ciao e grazie


System.out.println(i+",");

IN generale in Java puoi concatenare stringhe usando l'operatore: +
Se concateni un intero ad una stringa (come nel tuo caso) l'intero viene convertito direttamente ad una stringa :)
;)

alberto t.
05-07-2009, 19:35
Il problema è che in questo modo viene stampata anche una seconda virgola:
Le posizioni della vocale i:3,10,

luxorl
05-07-2009, 19:54
Il problema è che in questo modo viene stampata anche una seconda virgola:
Le posizioni della vocale i:3,10,

In pratica vuoi la virgola sempre tranne alla fine? Prova così allora :)


for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
System.out.print((i));
if(i == (line.length()-1)) break;
System.out.print(",");
}
}

alberto t.
05-07-2009, 20:17
Ciao
Grazie, funziona!
Però non ho capito che cosa fa questa porzione di codice:

if(i == (line.length()-1)) break;
System.out.print(",");

luxorl
05-07-2009, 20:20
Ciao
Grazie, funziona!
Però non ho capito che cosa fa questa porzione di codice:

if(i == (line.length()-1)) break;
System.out.print(",");


Scritta in italiano:

SE i è arrivato all'ultimo carattere della stringa ESCI DAL CICLO FOR (break)
Stampa ,

In pratica ad ogni ciclo controlla se è alla fine della stringa, se è così esce immediatamente senza stampare la virgola se invece non è alla fine stampa la virgola :)

break serve per uscire forzatamente dai cicli in qualsiasi momento.

Tadde
06-07-2009, 09:38
Scritta in italiano:

SE i è arrivato all'ultimo carattere della stringa ESCI DAL CICLO FOR (break)
Stampa ,

In pratica ad ogni ciclo controlla se è alla fine della stringa, se è così esce immediatamente senza stampare la virgola se invece non è alla fine stampa la virgola :)

break serve per uscire forzatamente dai cicli in qualsiasi momento.

Il tuo codice però funziona solo se la stringa di input ha 'i' come ultima lettera, altrimenti stamperà comunque una virgola alla fine.
Bisognerebbe, per ogni lettera 'i' trovata, innescare un altro controllo per percorrere il resto della stringa fino alla fine e vedere se effettivamente era l'ultima.

luxorl
06-07-2009, 09:52
Il tuo codice però funziona solo se la stringa di input ha 'i' come ultima lettera, altrimenti stamperà comunque una virgola alla fine.
Bisognerebbe, per ogni lettera 'i' trovata, innescare un altro controllo per percorrere il resto della stringa fino alla fine e vedere se effettivamente era l'ultima.

:doh: Giusto.. che erroraccio :fagiano:

banryu79
06-07-2009, 10:09
Il tuo codice però funziona solo se la stringa di input ha 'i' come ultima lettera, altrimenti stamperà comunque una virgola alla fine.
Bisognerebbe, per ogni lettera 'i' trovata, innescare un altro controllo per percorrere il resto della stringa fino alla fine e vedere se effettivamente era l'ultima.
Potresti separare la raccolta degli indici dalla stampa, tipo:

// char 'i' @ indexes: 1, 8, 16, 21
String input = "Siamo qui con voi a giocare";

// raccolgo indici come String in una lista
List<String> indexes = new ArrayList<String>();
for (int i = 0; i < input.length(); i++)
if (line.charAt(i) == 'i')
indexes.add(String.valueOf(i));

// stampo la stringa degli indici
if (indexes.size() > 0)
{
// stampo il primo e lo rimuovo dalla lista:
System.out.println(indexes.remove(0))

// stampo tutti gli altri separati dal carattere ','
for (String s : indexes)
System.out.println(", "+ s);
}

luxorl
06-07-2009, 10:10
String line = "Ilaro RIssi";
boolean ultimaI = true;

System.out.print("Le posizioni della vocale i:");
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i' || line.charAt(i) =='I') {
System.out.print((i));
for (int j = i+1; j < line.length(); j++) {
if (line.charAt(j) == 'i' || line.charAt(j) =='I') ultimaI = false;
}
if(ultimaI == true) break;

System.out.print(",");
ultimaI = true;


}
}

Così dovrebbe andare.
Ho inserito un ciclo interno che controlla se la "i" trovata è l'ultima in caso positivo interrompe tutto, se no stampa una virgola.

Inoltre ho corretto la condizione dentro l'if di verifica se il char controllato è una "i"... perché non controllava le "I" maiuscole ;)

^TiGeRShArK^
06-07-2009, 11:13
ma non sarebbe + leggibile con qualcosa del genere?

System.out.print("Le posizioni della vocale i:");
string result = "";
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
result += line.charAt(i) + ", ";
}}
if (result.length > 0)
System.out.print(result.substring(0, result.length - 2);

Tadde
06-07-2009, 11:58
ma non sarebbe + leggibile con qualcosa del genere?

System.out.print("Le posizioni della vocale i:");
string result = "";
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
result += line.charAt(i) + ", ";
}}
if (result.length > 0)
System.out.print(result.substring(0, result.length - 2);


E' leggibile ma sono richieste le posizioni delle 'i', non le 'i' stesse. Questione di una minima modifica comunque.

banryu79
06-07-2009, 12:06
E' leggibile ma sono richieste le posizioni delle 'i', non le 'i' stesse. Questione di una minima modifica comunque.
Basta che sostituisci il:

result += line.charAt(i) + ", ";

con un:

result += String.valueOf(i) + ", ";


@Tiger: naturalmente hai ragione! :D

^TiGeRShArK^
06-07-2009, 12:07
E' leggibile ma sono richieste le posizioni delle 'i', non le 'i' stesse. Questione di una minima modifica comunque.

vero basta scrivere i al posto di line.charAt(i) :p

^TiGeRShArK^
06-07-2009, 12:08
Basta che sostituisci il:

result += line.charAt(i) + ", ";

con un:

result += String.valueOf(i) + ", ";


@Tiger: naturalmente hai ragione! :D

il valueof è inutile dato che la variabile result è già inizializzata come una stringa vuota e dunque quando si va a concatenare un intero ci pensa la JVM a richiamare direttamente il toString. ;)

luxorl
06-07-2009, 13:05
ma non sarebbe + leggibile con qualcosa del genere?

System.out.print("Le posizioni della vocale i:");
string result = "";
for (int i = 0; i < line.length(); i++) {
if (line.charAt(i) == 'i') {
result += line.charAt(i) + ", ";
}}
if (result.length > 0)
System.out.print(result.substring(0, result.length - 2);


E penso non si tratti solo di leggibilità.
Con la mia soluzione nel peggiore dei casi si ha complessità quadrata :D
Qui il for resta uno solo :D