[C++] Evitare ripetizioni nell'if ...

[pixie]

Salve.
Ho un if di questo tipo:

if(cognome[ind1] == 'a' || cognome[ind1] == 'e' ||cognome[ind1] == 'o' || cognome[ind1] == 'u' ... )  // per tutte le vocali

Com'è possibile evitare di continuare a ripetere 'cognome[ind1] == ... ' ??

Ho provato a fare questo ma pur non dandomi errori di sintassi non mi da il risultato voluto.

if(cognome[ind1] == 'a' || 'e' || 'o' || 'u' ... )

Può aiutarmi un costruttore di tipo enum?
Non so ditemi voi cosa è meglio.
A presto!
Grazie!

[il-pollo]

una soluzione per accorciare soltanto la lunghezza del codice può essere

char c = cognome[ind1];
if ( c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u' ){...}

un'altra soluzione può essere quella di creare una funzione booleana che, dato un carattere, determina se è una vocale

[DanieleC88]

Una funzione. O una macro:

#define isvowel(c) ((c) == 'a' || (c) == 'e' || (c) == 'i' || (c) == 'o' || (c) == 'u')

/*
 * [...]
 */

if (isvowel(cognome[ind1])) {
    faiQualcosaDiUtile();
}

[marco.r]

Originariamente inviato da DanieleC88

Una funzione. O una macro:

#define isvowel(c) ((c) == 'a' || (c) == 'e' || (c) == 'i' || (c) == 'o' || (c) == 'u')

/*
 * [...]
 */

if (isvowel(cognome[ind1])) {
    faiQualcosaDiUtile();
}

Eviterei di usare una macro quando si puo' fare lo stesso con una funzione.
Inoltre userei una variabile temporanea. Evita la valutazione dell'operator[] piu' di una volta.

[pixie]

grz mille delle risposte! ... la soluzione di il-pollo credo sia la migliore ma già avevo pensato d fare come lui... il fatto è ke cercavo qlcs di più sintetico...
ok dai vedo cosa inventarmi ... grz e a presto!

[DanieleC88]

Originariamente inviato da marco.r

Eviterei di usare una macro quando si puo' fare lo stesso con una funzione.
Inoltre userei una variabile temporanea. Evita la valutazione dell'operator[] piu' di una volta.

Infatti il primo suggerimento era “Una funzione.”, ma magari così esplora anche il mondo delle macro.

[nuovoUtente86]

Anche se in questo caso è una soluzione eccessiva, potresti pensare ad una mappa di puntatori a dei funtori.

[banryu79]

Originariamente inviato da DanieleC88

Infatti il primo suggerimento era “Una funzione.”, ma magari così esplora anche il mondo delle macro.

Le macro sono IL male

[Noixe]

char vocali[] = {'a', 'e', 'i', 'o', 'u'};
bool vocale = false;

for (int i = 0; i < 5; i++)
    if (cognome[ind1] == vocali[i])
        vocale = true;

Se vocale è rimasta false allora cognome[ind1] non è una vocale, se è diventata true vuol dire che è una vocale.

[marco.r]

Originariamente inviato da DanieleC88

Infatti il primo suggerimento era “Una funzione.”, ma magari così esplora anche il mondo delle macro.

Si', ma non ha senso esplorare le macro per quelle cose che si fanno normalmente per le funzioni. Meglio lasciarle per quelle dove servono informazioni che si hanno solo a tempo di compilazione (e.g. header guards, salvataggio numero di linea, embedding di un argomento in una stringa ... )