El código que presentas es erróneo a varios niveles:
Algoritmo: Usas en condiciones funciones de transformación (toupper
) y usas en asignaciones funciones de decisión (islower
).
En C++ los valores numéricos se convierten implícitamente a expresiones booleanas siendo cero equivalente a "falso" y cualquier otro valor equivalente a "verdadero", así que al usar la función toupper
(que transforma un carácter a mayúscula) siempre obtendrás un valor diferente a cero ("verdadero") lo que implica que esta comprobación siempre se cumple:
if(toupper(*letra)){ // Esto siempre es verdadero.
Los caracteres imprimibles empiezan en el valor 32 (espacio:
), cualquier otro valor o no se imprimirá o actuará de manera extraña. La función islower
devuelve cero si el parámetro no es una letra minúscula o un valor indeterminado diferente a cero (probablemente uno) si la letra es minúscula; por eso si guardas el valor devuelto por islower
dentro de una variable que imprimirás luego, muy probablemente no verás nada:
*letra=islower(*letra);
Lenguaje: Mezclas cabeceras del lenguaje c pero programas en c++, las cabeceras <stdio.h>
y <string.h>
no deben usarse programando en C++.
Buenas prácticas: En C++ moderno el uso de punteros está desaconsejado. El bucle while
por convención se usa en casos en que se desconozca el número de iteraciones y se desaconseja el uso de la cláusula using namespace std
en ámbitos globales.
Si corregimos todos tus errores, tu código podría parecerse a:
#include <iostream>
#include <cctype>
#include <string>
void mayus(std::string &);
int main()
{
using namespace std;
string pala;
cout << "Digite la palabra a transformar: ";
getline(cin, pala);
cout << "La palabra anteriormente era: " << pala << endl;
mayus(pala);
cout << "Ahora la palabra es: " << pala << endl;
return 0;
}
void mayus(std::string &palabra) {
for (auto &c : palabra)
c = toupper(c);
}
Puedes ver el código funcionando en Try it online!.
islower( )
debería estar como expresión en elif( )
, ytoupper( )
en la asignación.