[java] stampare posizioni vocali "i"

[alberto t.]

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]

Originariamente inviato da alberto t.

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.]

Il problema è che in questo modo viene stampata anche una seconda virgola:
Le posizioni della vocale i:3,10,

[luxorl]

Originariamente inviato da alberto t.

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.]

Ciao
Grazie, funziona!
Però non ho capito che cosa fa questa porzione di codice:

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

[luxorl]

Originariamente inviato da alberto t.

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]

Originariamente inviato da luxorl

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]

Originariamente inviato da Tadde

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.

Giusto.. che erroraccio

[banryu79]

Originariamente inviato da Tadde

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]

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^]

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]

Originariamente inviato da ^TiGeRShArK^

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]

Originariamente inviato da Tadde

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!

[^TiGeRShArK^]

Originariamente inviato da Tadde

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)

[^TiGeRShArK^]

Originariamente inviato da banryu79

Basta che sostituisci il:

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

con un:

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

@Tiger: naturalmente hai ragione!

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]

Originariamente inviato da ^TiGeRShArK^

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
Qui il for resta uno solo