Problema con cadena de caracteres en C.

[GilbertoBrewer]

Buenas noches o lo que sea cuando entren al tema.

Estoy llevando un curso de programación estructurada y hemos estado trabajando en lenguaje C utilizando Code::Blocks.

Me encargaron un programa en el cual debería guardar 10 cadenas de caracteres en una variable "nombre" y otras 10 en una variable "apellido", para después generar diez combinaciones aleatorias de nombre y apellido, y me mostrara la lista en pantalla.

Yo lo hice así:

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

int main ()
{
    int p,i,g;
    char nombre[10][10]={"Pedro","Hugo","Oliver","Luis","Jesus","Natalia","Monserrat","Vanessa","Antonio","Jorge"};
    char apellido[10][10]={"Lopez","Perez","Rojas","Garcia","Martinez","Montes","Fox","Valdez","Diaz","Hidalgo"};
    char tipo[25];
    srand(time(NULL));

    for (p=0; p<10; p++)
    {
        i=rand()%10;
        g=rand()%10;
        sprintf(tipo,"%s %s",nombre,apellido[g]);
        puts(tipo);
    }

    return 0;
}

El problema está en que en algunos casos me imprime dos apellidos juntos o no me muestra el apellido, algo así:

Jorge FoxValdez
Pedro Valdez
Jorge
Luis PerezRojas

y no tengo idea de por qué pueda ser.

También intenté escribiendo:

strcpy(tipo,nombre);
        strcat(tipo," ");
        strcat(tipo,apellido[g]);

en lugar de:

sprintf(tipo,"%s %s",nombre,apellido[g]);

pero el resultado es algo similar.

¿Podrían ayudarme a encontrar el error de mi programa y/o recomendarme alguna forma de corregirlo o alguna manera alternativa de hacerlo?

De antemano, gracias.

[leosansan]

Buenas noches o lo que sea cuando entren al tema.


El problema está en que en algunos casos me imprime dos apellidos juntos o no me muestra el apellido, ...............................

Supongo que has puesto;:

Código (cpp) [Seleccionar] Expandir

sprintf(tipo,"%s %s",nombre[i],apellido[g]);

Es que al usar Cita en lugar de las etiquetas GeSHi el código que cuelgas no sale bien,

¡¡¡¡ Saluditos! ..... !!!!

[GilbertoBrewer]

Ah, ok. Sí, ya vi el problema con la cita.

El código es así:

Código [Seleccionar] Expandir


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

int main ()
{
    int p,i,g;
    char nombre[10][10]={"Pedro","Hugo","Oliver","Luis","Jesus","Natalia","Monserrat","Vanessa","Antonio","Jorge"};
    char apellido[10][10]={"Lopez","Perez","Rojas","Garcia","Martinez","Montes","Fox","Valdez","Diaz","Hidalgo"};
    char tipo[25];
    srand(time(NULL));

    for (p=0; p<10; p++)
    {
        i=rand()%10;
        g=rand()%10;
        sprintf(tipo,"%s %s",nombre[i],apellido[g]);
        puts(tipo);
    }

    return 0;
}

También intenté escribiendo:

Código [Seleccionar] Expandir


        strcpy(tipo,nombre)[i];
        strcat(tipo," ");
        strcat(tipo,apellido[g]);
en lugar de:

Código [Seleccionar] Expandir


        sprintf(tipo,"%s %s",nombre[i],apellido[g]);

En ambos casos me muestra en la pantalla algo así:

Código [Seleccionar] Expandir

Jorge FoxValdez
Pedro Valdez
Jorge
Luis PerezRojas

[rir3760]

El programa debe funcionar correctamente si agregas al principio de este las directivas de inclusión:

Código (c) [Seleccionar] Expandir

#include <stdlib.h> /* srand y rand */
#include <time.h>   /* time */

Y no es necesario que incluyas <string.h>, en cuanto a conio la deberías evitar por las razone que se indican en el tema |Lo que no hay que hacer en C/C++. Nivel basico|.

Un saludo

[leosansan]

También intenté escribiendo:

strcpy(tipo,nombre)[ i ];
       strcat(tipo," ");
       strcat(tipo,apellido[g]);
en lugar de:

Código [Seleccionar] Expandir


       sprintf(tipo,"%s %s",nombre[i],apellido[g]);

Antes que nada elige en las etiquetas GeSHi la de C++ para que el código salga coloreado.

El [ i ] se te fue fuera del paréntesis:

Código (cpp) [Seleccionar] Expandir

strcpy ( tipo , nombre [ i ] ) ;

Y si usas strcpy y strcat si debes incluir la librería <string.h>.

¡¡¡¡ Saluditos! ..... !!!!

[GilbertoBrewer]

Ya corregí los errores y ya funcionó el programa.

Muchas gracias por las observaciones.

[ShadowA7X]

Recomendación personal amigo. Siempre cuando estés leyendo muchas cadenas de caracteres ocupa

Código (c) [Seleccionar] Expandir

fflush(stdin);

antes y/o despues de los scanf o los getch. Lo que hace esa función es limpiarte el buffer para que la próxima lectura no se vea afectada en ninguna medida por la lectura anterior. Siempre que quieras prevenir que algo malo pase con las lecturas de cadenas, ponlo (funciona como un parche por decirlo así  ). Si eso no lo soluciona entonces es un problema del código en si.


Eso, espero te sirva para afrontar futuros casos  .

[rir3760]

Recomendación personal amigo. Siempre cuando estés leyendo muchas cadenas de caracteres ocupa

Código (c) [Seleccionar] Expandir

fflush(stdin);

El uso de fflush(stdin) no se recomienda, las razones de ello en el tema |Lo que no hay que hacer en C/C++. Nivel basico|.

Un saludo

[GilbertoBrewer]

Sí había visto en ese tema que no es recomendable utilizar el fflush(stdin), pero no me quedó del todo claro el porqué.

[avesudra]

Sí había visto en ese tema que no es recomendable utilizar el fflush(stdin), pero no me quedó del todo claro el porqué.

Porque la función fflush está definida para limpiar el estandar de salida (stdout), en cualquier otro caso el comportamiento de la función está indefinido. En Windows funciona, por el contrario en Linux no lo hace y Mac OS lo ignora.