c stringa alpha o aplhanumerica

[shivi_php]

qualcuno può aiutarmi a risolvere...ho creato questa funzione per controllare se la stringa è solo aplha o aplhanumerica non riesco capire quando parte il programma mi controlla solo il primo carattere non riesco fare il controllo sulla intera stringa perchè se metto un numero alla fine della stringa output mi da sempre è una stringa

int getStringLength(const char* str){
    int length = strlen(str);
    return length;
}

int checkIfStringisAlfa(const char str[]){
    int lenStr = getStringLength(str);
    int i = 0;
    bool trueFalse = false;
    for(i = 0; i < lenStr; i++){
        if((str[i] >= 'a' && str[i] <= 'z') || (str[i] >= 'A' && str[i] <= 'Z')){
            trueFalse = true;
            break;
        }else if((str[i] >= 'a' && str[i] <= 'z') && (str[i] >= '0' && str[i] <= '9') || (str[i] >= 48 && str[i] <= 57)){
            trueFalse = false;
            break;
        }
    }


    return trueFalse;
}

int main()
{
    printf("\t\tC programming charecters\n");


    char* stringC = "test23";


    if(checkIfStringisAlfa(stringC)){
        printf("\t\tYes string\n");
    }else if(checkIfStringisAlfa(stringC) == false){
        printf("\t\tAlphanumeric string\n");
    }


    return 0;
}

[oregon]

Rivedi il codice della funzione.

Se il primo carattere è alfabetico, metti la variabile a true e termini il ciclo con la break. E gli altri caratteri?

Inoltre, anche se ininfluente, nel main non ha senso eseguire due if ... basta l'else ...

[minomic]

Ciao,
è errata la logica, cioè proprio l'algoritmo. In particolare questa parte:

    bool trueFalse = false;
    for(i = 0; i < lenStr; i++){
        if((str[i] >= 'a' && str[i] <= 'z') || (str[i] >= 'A' && str[i] <= 'Z')){
            trueFalse = true;
            break;
        }else if((str[i] >= '0' && str[i] <= '9') || (str[i] >= 48 && str[i] <= 57)){
            trueFalse = false;
            break;
        }
    }

La variabile trueFalse la devi settare a true prima del ciclo for, quindi fai semplicemente questo:

for (...)
{
    if((str[i] >= '0' && str[i] <= '9') || (str[i] >= 48 && str[i] <= 57))
    {
        trueFalse = false;
        break;
    }
}

La logica che sta dietro a tutto ciò è la seguente: consideri una stringa formata da soli caratteri fino a quando non riesci a dimostrare il contrario, cioè fino a quando non trovi un numero. Appena trovi un carattere che non sia una lettera, setti trueFalse a false ed esci dal ciclo.



EDIT: in contemporanea con oregon. Comunque lascio anche questo messaggio.

[shivi_php]

siiiii siiiii grazie mille ragazzi è da due giorni che facevo prove prove e non andava oggi con una piccola modifica va benissimo grazie mille ora funziona alla grande

[minomic]

L'importante è che tu abbia capito il perché​!

[shivi_php]

quando controllo lettere lascio scorrere tutto la strina invece nei numeri con break esco fuori dal for

[minomic]

Esatto: sono solo i numeri che attirano l'attenzione. Altrimenti, se non ci sono numeri, la stringa viene esaminata e il valore di trueFalse non viene modificato, rimanendo a true.

[shivi_php]

si ho capito la logica ero troppo confuso invece il problema era piccolo grazie mille

[oregon]

Scusa minomic, perché questa if è fatta da due parti ?

if((str[i]>='0'&& str[i]<='9')||(str[i]>=48&& str[i]<=57))

[minomic]

Scusa minomic, perché questa if è fatta da due parti ?

if((str[i]>='0'&& str[i]<='9')||(str[i]>=48&& str[i]<=57))

Effettivamente me lo sono chiesto anche io, dato che i caratteri ASCII compresi tra 48 e 57 sono proprio i numeri! Comunque, dato che non è un errore, ho evitato di sollevare la questione con shivi_php e ho lasciato come aveva scritto lui... Però in effetti la cosa è un po' strana e sicuramente il codice è poco leggibile.