[SOLUCIONADO] Implementar función c++

[b3salvat]

Muy buenas a todos, necesitaría un empujón para implementar esta funcion en c++:

bool valid(string, n)

-Nos devuelve "cierto" sólo si todas las letras son minusculas y su longitud está entre 3 y 8.

Os agradezco mucho vuestra ayuda, gracias un saludo!

[b3salvat]

if(valida("dsjk", 6))cout << "SI";
else cout<< "No";
cout<<endl;

if(valida("dsjkUP", 6))cout << "NO";
else cout<< "Si";
cout<<endl;

if(valida("dsjksa", 6))cout << "SI";
else cout<< "No";
cout<<endl;

En teoría el programa haría algo así no? lo que necesitaría sería la función.. gracias :P

[amchacon]

Pasa el texto a minusculas y comprueba que las dos cadenas sean iguales. Si no son iguales esporque había alguna mayúscula.

[b3salvat]

Como paso el texto a minusculas? podrías decirme como seria esa funcion?

[b3salvat]

La función es esta (igual me expliqué mal desde el principio)

bool valida(string s, int n)
Devuelve cierto si el string s tiene sólo letras minusculas y tiene la longitud de n. n tiene que ser un valor entre 3 y 8.

Gracias

[dehm]

Otra opción puede ser comparar cada caracter de la cadena con su versión en minúscula.

Si todos los caracteres son minúsculos, entonces todavía has de comprobar si estás en el rango del tamaño.
Algo así:

Código C++:

bool valida(string s, int n);
int main()
{
    string cadena="aeddddddiou";
    if (valida(cadena,0))
    {
        cout<<"La cadena ES valida"<<endl;
    }
    else
    {
        cout<<"La cadena NO es valida"<<endl;
    }
    return 0;
}
 
bool valida(string s, int n)
{
    for (int i=0; i<s.size(); i++)
    {
        cout<<"Original: "<<s[i]<<" - Copia: "<<(char)tolower(s[i])<<endl;
        if (s[i]!=(char)tolower(s[i]))
        {
            return false;
        }
    }
    return (s.size()>=3 && s.size()<=8);
}
 

Te he puesto un cout en el que se va mostrando cada caracter de la cadena s y el mismo en minúscula. También puedes ver cómo se sale de la función en el momento en el que se encuentra una mayúscula.
Si cumple la primera condición, queda evaluar el tamaño.
Como verás, la función retorna el resultado de la evaluación.
Eso es equivalente a:

Código C++:

if (s.size()>=3 && s.size()<=8)
{
    return true;
}
else
{
    return false;
}
 

Por otro lado, tal como tienes la firma de la función:
bool valida(string s, int n)

Parece que lo que quieres es que la cadena use "n" para alguna comparación de tamaño, pero tal como dices en el enunciado los límites de tamaño son un intervalo.
Si lo que quieres es que los límites de la cadena sean parametrizables, entonces deberías incluir otro parámetro en la firma de la funcion:

Código C++:

bool valida(string s, int min, int max)
{
    for (int i=0; i<s.size(); i++)
    {
        cout<<"Original: "<<s[i]<<" - Copia: "<<(char)tolower(s[i])<<endl;
        if (s[i]!=(char)tolower(s[i]))
        {
            return false;
        }
    }
    return (s.size()>=min && s.size()<=max);
}
 

O sencillamente eliminar ese parámetro int n, que ahora mismo no hace nada. (Fíjate que para usarlo en el ejemplo le he tenido que dar un valor, el cual no ha sido usado)

Código C++:

bool valida(string s)
{
    for (int i=0; i<s.size(); i++)
    {
        cout<<"Original: "<<s[i]<<" - Copia: "<<(char)tolower(s[i])<<endl;
        if (s[i]!=(char)tolower(s[i]))
        {
            return false;
        }
    }
    return (s.size()>=3 && s.size()<=8);
}
 

Saludos!

[b3salvat]

Muchas gracias dehm de verdad, me estás ayudando mucho. El int n tendría que ser un valor entre 3 y 8, por ejemplo el 4 o el 6, es decir no debería ser un intervalo, simplemente un valor concreto.

Por lo demás todo genial ^^

[dehm]

Ahhh...entonces lo había entendido mal.

Por cierto, se podría hacer una pequeña mejora para optimizar el rendimiento.
Tal como está planteada la función, se puede dar el caso de que una cadena cumpla la condición de minúsculas, pero no la del tamaño.
Eso supone que obligo a la función a realizar todas las comparaciones entre caracteres, y luego comparar el tamaño.
Se puede optimizar si primero comparo el tamaño, que es algo inmediato, y sólo entonces paso a recorrer el bucle.

Código C++:

bool valida(string s, int n)
{
    if (s.size()!=n) return false;//salida inmediata de la función
    for (int i=0; i<s.size(); i++)
    {
        cout<<"Original: "<<s[i]<<" - Copia: "<<(char)tolower(s[i])<<endl;
        if (s[i]!=(char)tolower(s[i]))
        {
            return false;
        }
    }
    return true;
}
 

Saludos

[b3salvat]

Tengo una duda: que valor le has dado a n ?

[dehm]

Bueno, el valor de n lo defines tú cuando haces la llamada a la función.

Código C++:

int main()
{
    string cadena="aeddddddiou";
    if (valida(cadena,5))//si quieres que la cadena válida tenga un tamaño de 5
    {
        cout<<"La cadena ES valida"<<endl;
    }
    else
    {
        cout<<"La cadena NO es valida"<<endl;
    }
    return 0;
}
 

[b3salvat]

Solucionado, muchas gracias y perdona por mis escasos conocimientos :(