2

Quiero que los datos que ingreso de la estrutura "Libros" se pasen directamente a mayuscula. Se que hay que usar la funcion toupper() pero no se como aplicarlo cuando trabajo con estructuras.

#include<stdio.h>
#include<string.h>
#include<conio.h>


//Estructura del libro.
typedef struct{
    char titulo[100];
    char genero[100];
    int paginas;
    int anio_de_edicion;
    char numero_isbn[100];
    char editorial[100];
}Libros;

//Funcion para datos del libros.
void alta(Libros * libros){
    fflush(stdin);
    printf("Titulo: ");
    gets(libros->titulo);
    printf("genero: ");
    gets(libros->genero);
    printf("paginas: ");
    scanf("%d", &libros->paginas);
    printf("año de edicion: ");
    scanf("%d", &libros->anio_de_edicion);
    fflush(stdin);
    printf("numero isbn: ");
    gets(libros->numero_isbn);
    printf("editorial: ");
    gets(libros->editorial);
 }

1 respuesta 1

4

Yo optaría por crear primeramente una función que permitiese convertir una cadena cualquiera a mayúsculas, como por ejemplo:

#include <ctype.h>

void aMayusculas(char* ptr)
{
  for( ; *ptr, ++ptr )
    *ptr = toupper(*ptr);
}

El segundo paso sería adaptar el código de lectura para realizar las conversiones una a una:

void alta(Libros * libros){
  // ...

  aMayusculas(libros->titulo);
  aMayusculas(libros->genero);
  aMayusculas(libros->numero_isbn);
  aMayusculas(libros->editorial_isbn);
}

Como nota adicional, fflush está diseñado para ser usado única y exclusivamente con salidas. fflush(stdin) no está soportado por el estándar, luego es probable que con otros compiladores el resultado no sea el que tu esperas. Yo en tu lugar evitaría ese uso de fflush.

12
  • El utiliza struct con char[] de tamaño fijo, que no tienen porque terminar en un \0. Esto es un despiste por tu parte, seguro ;-) Edita la respuesta y tienes mi voto.
    – Trauma
    Commented el 14 nov. 2016 a las 12:43
  • @Trauma La respuesta está dada en base al código que ha puesto y fgets finaliza las cadenas con \0. Bueno, realmente no sería únicamente fgets sino la inmensa mayoría de las funciones de entrada/salida que trabajan con char*. De hecho si no terminasen en \0 no podría hacer posteriormente printf("%s",libros->titulo), como asumo que está haciendo. Tendría que hacer uso de artificios los cuales al menos yo pongo la mano en el fuego no está utilizando.
    – eferion
    Commented el 14 nov. 2016 a las 12:47
  • @Trauma Por supuesto que se podría hacer una versión tal que void aMayusculas(char*, size_t); pero su uso es más complejo y seguramente no es ni necesario. No veo necesario complicar el código si con la versión básica se cumplen las especificaciones.
    – eferion
    Commented el 14 nov. 2016 a las 12:48
  • Gracias eferion!
    – P. Matias
    Commented el 14 nov. 2016 a las 12:48
  • 1
    @Trauma Además, en este caso las cadenas se procesan después de escribirlas... si se produce buffer overflow no será en mi función sino en la lectura de gets
    – eferion
    Commented el 14 nov. 2016 a las 12:55

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.