0

Tengo este programa, quisiera que lo que se imprima sea Mayusculas independientemente lo que reciba (MAYUS o minus). Esta casi listo solo me falta que imprima en mayusculas.

Como ven imprimo mi void en el main, pero quiero que imprima mi void en MAYUSCULAS. Ejemplo

Hola mundo. (con desplazamiento 3)

KROD PXQGR

#include <iostream>
#include <cstring>

using namespace std;



void encriptar(char secreto[], int);



int main(){
    char secreto[1000];
    int n;
    cin.getline(secreto, 1000);
    cin >> n;
    
    encriptar(secreto, n);
    
    cout<< secreto;
    
    return 0;
}

void encriptar(char secreto[], int n){
    int i, j;
    char abecedario []= "abcdefghijklmnopqrstuvwxyzABCDFGHIJKLMNOPQRSTUVWXYZ";

    if(n>0 && n<27){
        for(i=0; secreto[i]; i++){
            j=0;
            for(j=0; j<52; j++){
                if(secreto[i]==abecedario[j] && j<78){
                    secreto[i]=abecedario[j+n];
                    j=52;
                }
            }
        }
    }   
}
11
  • 4
    Estaría bien que, antes de seguir enviando preguntas, revises todas las que ya has hecho y marques alguna respuesta como aceptada (o pongas tu solución y la marques para compartirla con los demás). La comunidad entiende por tu historial actual que no estás muy interesado en aportar.
    – Alfabravo
    Commented el 7 ago. 2020 a las 20:39
  • Disculpa, es que no usar muy bien esta plataforma. Commented el 7 ago. 2020 a las 20:40
  • 2
    por favor, hace el recorrido de bienvenida y tambien mira Cómo preguntar. La seccion respuestas es solo para respuestas a la pregunta, no es para hacer nuevas preguntas ni un seguimiento de la realizada.
    – gbianchi
    Commented el 7 ago. 2020 a las 20:41
  • Ademas,si se fijan en las otras preguntas. Praticamente yo solo termine los programas y en todo caso, MI APORTE FUE ENVIARLO RESUELTO CON CASOS NO EXPUESTOS AQUI. Commented el 7 ago. 2020 a las 20:44
  • 2
    Amigo, piensa que otros pueden tener tu misma pregunta, tu mismo problema, así como viniste tú. La idea es que otros vengan en el futuro, mañana o en un año, y puedan resolver su duda sabiendo lo que ahora tú sabes. Si consideras que eso no tiene valor, los demás no estaremos tan inclinados a responder tus preguntas.
    – Alfabravo
    Commented el 7 ago. 2020 a las 21:15

2 respuestas 2

3

Para pasarlos todos a mayúsculas puedes usar la función toupper en todos los caracteres. Esta recibe un carácter y lo devuelve hecho mayusula.

   for (int i=0; secreto[i]; i++) {
        secreto[i] = toupper(secreto[i]);
    }

Aqui esta ya puesto en el codigo:

#include <iostream>
#include <cstring>

using namespace std;



void encriptar(char secreto[], int);



int main(){
    char secreto[1000];
    int n;
    cin.getline(secreto, 1000);
    cin >> n;
    
    encriptar(secreto, n);
    
    for (int i=0; secreto[i]; i++) {
        secreto[i] = toupper(secreto[i]);
    }
    
    cout<< secreto;
    
    return 0;
}

void encriptar(char secreto[], int n){
    int i, j;
    char abecedario []= "abcdefghijklmnopqrstuvwxyzABCDFGHIJKLMNOPQRSTUVWXYZ";

    if(n>0 && n<27){
        for(i=0; secreto[i]; i++){
            j=0;
            for(j=0; j<52; j++){
                if(secreto[i]==abecedario[j] && j<78){
                    secreto[i]=abecedario[j+n];
                    j=52;
                }
            }
        }
    }   
}


Otros

Se que tu pregunta no va de esto, pero es para avisarte que tienes un pequeño bug en tu codigo. Prueba darle como entrada

ZZZ
1

Y no saca nada

Le das esto como entrada:

HHHHHHHHHHH
25

Te da como resultado:

66666666666

Tienes que revisar que los resultados se mantengan en los limites.

Otra cosita es que este pedazo de codigo:

            for(j=0; j<52; j++){
                if(secreto[i]==abecedario[j] && j<78){
                    secreto[i]=abecedario[j+n];
                    j=52;
            }

Ver si es una letra de ese modo es muy lento. Un código equivalente haciendo podría ser este:

if (secreto[i] >= 'a' && secreto[i] <= 'z' || secreto[i] >= 'A' && secreto[i] <= 'Z') {
    secreto[i] += n;
}

Sin importar cual uses, lo que tienes que hacer es mantener los resultados entre a y z & A y Z

1
  • Hola, ya pude realizarlo. Voy a editar mi pregunta; pero igual, muchas gracias. Tu aporte de simplificar el for es muy buenoo. Commented el 7 ago. 2020 a las 21:27
1

A peticion de un miembro de la comunidad puse la respuesta a mi duda en la parte de respuestas. Esperando pueda ser de utilidad.

#include <iostream>
#include <cstring>

using namespace std;



void encriptar(char secreto[], int);



int main(){
    char secreto[1000];
    int n;
    cin.getline(secreto, 1000);
    cin >> n;
    
    encriptar(secreto, n);
    
    cout<< secreto;
    
    return 0;
}

void encriptar(char secreto[], int n){
    int i, j;
    char abecedarioT []= "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ";
    char abecedariominus []="abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz ";
    char abecedarioMAYUS []="ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZ";
    if(n>0 && n<27){
        for(i=0; secreto[i]; i++){
            j=0;
        if (secreto[i] >= 'a' && secreto[i] <= 'z' || secreto[i] >= 'A' && secreto[i] <= 'Z') {
            for(j=0; j<78; j++){
                if(secreto[i]==abecedarioT[j] && j<78){
                    secreto[i]=abecedarioMAYUS[j+n];
                    j=78;
                }
                if (secreto[i]==abecedariominus[j] && j<78){ 
                    secreto[i]=abecedarioMAYUS[j+n];
                    j=78;       
                }
                if (secreto[i]==abecedarioMAYUS[j] && j<78){ 
                    secreto[i]=abecedarioMAYUS[j+n];
                    j=78;   
                }
            }
       }    
    }   
    }
    
    else
    {
        for(i=0; secreto[i]; i++){
            j=0;
        if (secreto[i] >= 'a' && secreto[i] <= 'z' || secreto[i] >= 'A' && secreto[i] <= 'Z') {
            for(j=0; j<78; j++){
                if(secreto[i]==abecedarioT[j] && j<78){
                    secreto[i]=abecedarioMAYUS[j];
                    j=78;
                }
                if (secreto[i]==abecedariominus[j] && j<78){ 
                    secreto[i]=abecedarioMAYUS[j];
                    j=78;       
                }
                if (secreto[i]==abecedarioMAYUS[j] && j<78){ 
                    secreto[i]=abecedarioMAYUS[j];
                    j=78;   
                }
                
                
           }
        }   
        
        }   
    }



}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.