Skip to main content
se eliminó 1 carácter en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75
  1. Sobreescribes los datos del usuario

    Como he comentado, n es un vector de 15 elementos de tipo char:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx
    

    Entonces, en la primera iteración almacenas un string a partir de la posición dada por n[0]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  a  v  i  d \0 xx xx xx xx xx xx xx xx xx
    

    En la segunda iteración, almacenas otro estringstring a partir de la posición de n[1]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  L  a  u  r  a \0 xx xx xx xx xx xx xx xx
    

    Y, como ves, eso provoca que se pierda la primera cadena.

  2. Desbordamiento del buffer

    Siguiendo con lo comentado en el punto anterior, cuando e vaya teniendo valores altos, el algoritmo empezará a escribir en las últimas posiciones del array, luego habrá menos espacio útil para almacenar las cadenas, lo que aumenta el riesgo de que el código escriba fuera de los límites del array. Por ejemplo, si estamos en la 12º iteración, únicamente podremos introducir una cadena de 2 caracteres:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  A  B \0
    

    Si introducimos una cadena más grande acabermos escribiendo fuera de los límites del array:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 ~~~~~~~~~ <-- desbordamiento
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  R  u  b  e  n  \0
    

    Cuando esto sucede pueden pasar dos cosas:

    • El Sistema Operativo detecta que accedes a memoria que no te pertenece y mata tu proceso
    • Sobreescribes el valor de otras variables, lo que vuelve tu programa (u otros procesos) inestables
  1. Sobreescribes los datos del usuario

    Como he comentado, n es un vector de 15 elementos de tipo char:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx
    

    Entonces, en la primera iteración almacenas un string a partir de la posición dada por n[0]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  a  v  i  d \0 xx xx xx xx xx xx xx xx xx
    

    En la segunda iteración, almacenas otro estring a partir de la posición de n[1]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  L  a  u  r  a \0 xx xx xx xx xx xx xx xx
    

    Y, como ves, eso provoca que se pierda la primera cadena.

  2. Desbordamiento del buffer

    Siguiendo con lo comentado en el punto anterior, cuando e vaya teniendo valores altos, el algoritmo empezará a escribir en las últimas posiciones del array, luego habrá menos espacio útil para almacenar las cadenas, lo que aumenta el riesgo de que el código escriba fuera de los límites del array. Por ejemplo, si estamos en la 12º iteración, únicamente podremos introducir una cadena de 2 caracteres:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  A  B \0
    

    Si introducimos una cadena más grande acabermos escribiendo fuera de los límites del array:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 ~~~~~~~~~ <-- desbordamiento
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  R  u  b  e  n  \0
    

    Cuando esto sucede pueden pasar dos cosas:

    • El Sistema Operativo detecta que accedes a memoria que no te pertenece y mata tu proceso
    • Sobreescribes el valor de otras variables, lo que vuelve tu programa (u otros procesos) inestables
  1. Sobreescribes los datos del usuario

    Como he comentado, n es un vector de 15 elementos de tipo char:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx
    

    Entonces, en la primera iteración almacenas un string a partir de la posición dada por n[0]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  a  v  i  d \0 xx xx xx xx xx xx xx xx xx
    

    En la segunda iteración, almacenas otro string a partir de la posición de n[1]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  L  a  u  r  a \0 xx xx xx xx xx xx xx xx
    

    Y, como ves, eso provoca que se pierda la primera cadena.

  2. Desbordamiento del buffer

    Siguiendo con lo comentado en el punto anterior, cuando e vaya teniendo valores altos, el algoritmo empezará a escribir en las últimas posiciones del array, luego habrá menos espacio útil para almacenar las cadenas, lo que aumenta el riesgo de que el código escriba fuera de los límites del array. Por ejemplo, si estamos en la 12º iteración, únicamente podremos introducir una cadena de 2 caracteres:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  A  B \0
    

    Si introducimos una cadena más grande acabermos escribiendo fuera de los límites del array:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 ~~~~~~~~~ <-- desbordamiento
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  R  u  b  e  n  \0
    

    Cuando esto sucede pueden pasar dos cosas:

    • El Sistema Operativo detecta que accedes a memoria que no te pertenece y mata tu proceso
    • Sobreescribes el valor de otras variables, lo que vuelve tu programa (u otros procesos) inestables
se añadieron 2541 caracteres en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

Realmente no necesitas usar matrices en n. Si sigues leyendo verás por qué.

Aunque claro, quizá n no sea el nombre más apropiado para una variable... no te van a cobrar ningún suplemento, así que te sugiero que los nombres de las variables sean representativos:

Una vez que ya has pedido el nombre y el género copias los datos a otro array (dependiendo del género):

if(g[i]=='m'){  
  n[i]=h[i];
  c=c+1;
}

Este código tiene también dos problemas:

  1. Las asignaciones se hacen justamente al revés, es decir, ahora mismo estás copiando caracteres de h en n y debería ser al revés. Además, fíjate que estás copiando caracteres, no cadenas.

    Un detalle importante en C es que las cadenas se copian usando funciones, como strcpy. Si esta función te da errores probablemente se deba a que no estás intentando copiar cadenas sino, como es este caso, caracteres sueltos (el compilador puede ser tu amigo)

    Como te comenté antes, no es necesario que uses matrices en n. El motivo es que tu código se limita a pedir un nombre y a copiarlo en otra lista en función del género.

    Es decir, las listas finales que deben ser matrices, pero n puede ser una cadena de caracters al uso:

     // Cadena de caracteres
     char nombre[20];
    
     // Listas de cadenas
     char hombres[15][20];
     char mujeres[15][20];
    

    De esta forma, reutilizas nombres para pedir los datos al usuario:

     for(i=0;i<e;i++){
         printf("ingrese el nombre %d\n",i+1);
         scanf("%s", nombre);
    
         strcpy(hombres[i], nombre);
    
  2. Para el array h deberías usar el índice c en vez de i

    Efectivamente, usas c y p en lo que parece que debieran ser los índices para las dos listas de géneros... pero no aprovechas sus valores al rellenar dichas listas.

Aplicando las correcciones:

int main( ) {
  char nombre[20];
  char genero[15];
  char hombres[15][20];
  char mujeres[15][20];

  int indice_hombres = 0;
  int indice_mujeres = 0;

  int total_datos;
  printf("\nCuantos nombre desea ingresar?\n");
  scanf("%d",&total_datos);

  for(int i=0;i<total_datos;i++)
  {
    printf("ingrese el nombre %d\n",i+1);
    scanf("%s", nombre);
    printf("escoja el genero de %s\n", nombre);
    scanf("%s", genero);

    if(genero[0] == 'm')
    {  
      strcpy(hombres[indice_hombres], nombre);
      indice_hombres++;
    }
    else if( genero[0] == 'f')
    {
      strcpy(mujeres[indice_mujeres], nombre);
      indice_mujeres++;
    }
  }

Aunque claro, quizá n no sea el nombre más apropiado para una variable... no te van a cobrar ningún suplemento, así que te sugiero que los nombres de las variables sean representativos:

Realmente no necesitas usar matrices en n. Si sigues leyendo verás por qué.

Aunque claro, quizá n no sea el nombre más apropiado para una variable... no te van a cobrar ningún suplemento, así que te sugiero que los nombres de las variables sean representativos:

Una vez que ya has pedido el nombre y el género copias los datos a otro array (dependiendo del género):

if(g[i]=='m'){  
  n[i]=h[i];
  c=c+1;
}

Este código tiene también dos problemas:

  1. Las asignaciones se hacen justamente al revés, es decir, ahora mismo estás copiando caracteres de h en n y debería ser al revés. Además, fíjate que estás copiando caracteres, no cadenas.

    Un detalle importante en C es que las cadenas se copian usando funciones, como strcpy. Si esta función te da errores probablemente se deba a que no estás intentando copiar cadenas sino, como es este caso, caracteres sueltos (el compilador puede ser tu amigo)

    Como te comenté antes, no es necesario que uses matrices en n. El motivo es que tu código se limita a pedir un nombre y a copiarlo en otra lista en función del género.

    Es decir, las listas finales que deben ser matrices, pero n puede ser una cadena de caracters al uso:

     // Cadena de caracteres
     char nombre[20];
    
     // Listas de cadenas
     char hombres[15][20];
     char mujeres[15][20];
    

    De esta forma, reutilizas nombres para pedir los datos al usuario:

     for(i=0;i<e;i++){
         printf("ingrese el nombre %d\n",i+1);
         scanf("%s", nombre);
    
         strcpy(hombres[i], nombre);
    
  2. Para el array h deberías usar el índice c en vez de i

    Efectivamente, usas c y p en lo que parece que debieran ser los índices para las dos listas de géneros... pero no aprovechas sus valores al rellenar dichas listas.

Aplicando las correcciones:

int main( ) {
  char nombre[20];
  char genero[15];
  char hombres[15][20];
  char mujeres[15][20];

  int indice_hombres = 0;
  int indice_mujeres = 0;

  int total_datos;
  printf("\nCuantos nombre desea ingresar?\n");
  scanf("%d",&total_datos);

  for(int i=0;i<total_datos;i++)
  {
    printf("ingrese el nombre %d\n",i+1);
    scanf("%s", nombre);
    printf("escoja el genero de %s\n", nombre);
    scanf("%s", genero);

    if(genero[0] == 'm')
    {  
      strcpy(hombres[indice_hombres], nombre);
      indice_hombres++;
    }
    else if( genero[0] == 'f')
    {
      strcpy(mujeres[indice_mujeres], nombre);
      indice_mujeres++;
    }
  }
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

Esto es un vector de 15 elementos de tipo char:

char n[15];

Y aquí escribes en dicho vector:

for(i=0;i<e;i++){
    scanf("%s",&n[i]);

¿Problemas de estas tres líneas? Dos, básicamente:

  1. Sobreescribes los datos del usuario

    Como he comentado, n es un vector de 15 elementos de tipo char:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx
    

    Entonces, en la primera iteración almacenas un string a partir de la posición dada por n[0]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  a  v  i  d \0 xx xx xx xx xx xx xx xx xx
    

    En la segunda iteración, almacenas otro estring a partir de la posición de n[1]:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n ->  D  L  a  u  r  a \0 xx xx xx xx xx xx xx xx
    

    Y, como ves, eso provoca que se pierda la primera cadena.

  2. Desbordamiento del buffer

    Siguiendo con lo comentado en el punto anterior, cuando e vaya teniendo valores altos, el algoritmo empezará a escribir en las últimas posiciones del array, luego habrá menos espacio útil para almacenar las cadenas, lo que aumenta el riesgo de que el código escriba fuera de los límites del array. Por ejemplo, si estamos en la 12º iteración, únicamente podremos introducir una cadena de 2 caracteres:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  A  B \0
    

    Si introducimos una cadena más grande acabermos escribiendo fuera de los límites del array:

          00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 ~~~~~~~~~ <-- desbordamiento
     n -> xx xx xx xx xx xx xx xx xx xx xx xx  R  u  b  e  n  \0
    

    Cuando esto sucede pueden pasar dos cosas:

    • El Sistema Operativo detecta que accedes a memoria que no te pertenece y mata tu proceso
    • Sobreescribes el valor de otras variables, lo que vuelve tu programa (u otros procesos) inestables

Para resolver estos problemas tienes que usar un array doble o matriz

// Puede almacenar 15 nombres, cada uno con una longitud máxima de 19 caracteres
char n[15][20];

Aunque claro, quizá n no sea el nombre más apropiado para una variable... no te van a cobrar ningún suplemento, así que te sugiero que los nombres de las variables sean representativos:

char nombre[15][20];

Tu código quedará más legible y será más dificil que metas la pata al usar las variables.