[AYUDA] Ejercicio concatenar

[joraloma]

Buenas gracias por .el aviso en el otro tema, puse una pregunta muy general jeje, bueno la cuestion es la siguiente en el ejercicio 3 tengo que concatenar, y hacer una especie de lista, el resultado debería corresponderse con :

salas@318CDCr12: ̃$ ./nombreCompleto

Introduzca nombre : SALIR DEL PROGRAMA

salas@318CDCr12: ̃$ ./nombreCompleto

Introduzca nombre : Fulanito

Introduzca primer apellido : de Tal

Introduzca su segundo apellido: y Cual

Nombre completo: de Tal y Cual, Fulanito

Introduzca nombre : SALIR

salas@318CDCr12: ̃$ ./nombreCompleto

Pero yo obtengo :

salas@318CDCr12: ̃$ ./nombreCompleto

Introduzca nombre : SALIR DEL PROGRAMA

salas@318CDCr12: ̃$ ./nombreCompleto

Introduzca nombre : John

Introduzca primer apellido : Smith

Introduzca su segundo apellido:

Nombre completo: Smith

John

Introduzca nombre : SALIR

salas@318CDCr12: ̃$ ./nombreCompleto

Y mi código es:

Código [Seleccionar] Expandir

/* Includes del sistema */
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
/* Includes de la aplicacion */
#include "nombreCompleto.h"
/* Definición de constantes */

/* Tipos definidos por el usuario */

/************************************************************/

int main()
{
  char * nombre = NULL;
  char * ap1 = NULL;
  char * ap2 = NULL;
  int tam;
  char * todo=NULL;

 
  nombre = (char *) calloc(TAM, sizeof(char));
  ap1 = (char *) calloc(TAM, sizeof(char));
  ap2 = (char *) calloc(TAM, sizeof(char));
  if(nombre == NULL || ap1 == NULL || ap2 == NULL)
    fprintf(stderr, MENSAJE_MEMORIA);
  else
    {
      do
{
  printf(PIDE_NOMBRE);
  fgets(nombre, TAM, stdin);
  nombre=strchr(nombre,'\n');
  *nombre='\0';
  if(strncmp(nombre, NOMBRE_SALIR, NCOMPARAR) != 0)
    {
      printf(PIDE_AP1);
      fgets(ap1, TAM, stdin);
      ap2=strchr(ap1,'\n');
      *ap2='\0';
      printf(PIDE_AP2);
      fgets(ap2, TAM, stdin);
      ap2=strchr(ap2,'\n');
      *ap2='\0';
      tam = strlen(nombre) + strlen(ap1) + strlen(ap2);
      todo = (char *) calloc(tam, sizeof(char));
      todo = strcat(todo, ap1);
      todo = strcat(todo, ap2);
      todo = strcat(todo, nombre);

      printf(TEXTO_NOMBRE_COMPLETO, ap1);
      printf("%d", tam);
    }
}
      while(strncmp(nombre, NOMBRE_SALIR, NCOMPARAR) != 0);
      free(nombre);
      nombre = NULL;
      free(ap1);
      ap1 = NULL;
      free(ap2);
      ap2 = NULL;
      free(todo);
      todo = NULL;

    }
  return 0;
}

/* Definiciones de funciones  */


Ojala sepan la solución, gracias!!!

[Yoel Alejandro]

Hola joraloma, el problema es que la función de entrada fgets() coge el carácter de nueva línea o ENTER y lo agrega al final de las cadenas ap1, ap2, por eso al concatenar e imprimir, se imprime dicho salto de línea pareciendo como si fueran dos cadenas distintas

Una solución es depurar la cadena recibida para eliminar el salto de línea hallado al final de la misma, reemplazándolo por el carácter nulo de terminación:

Código (cpp) [Seleccionar] Expandir


size_t n;

n = strlen( ap1 );
if ( n > 0 && ap1[n-1] == '\n' ) ap1[n-1] = '\0';

y luego haces lo mismo para ap2, con lo cual debería solucionarse el problema.

[joraloma]

Hola joraloma, el problema es que la función de entrada fgets() coge el carácter de nueva línea o ENTER y lo agrega al final de las cadenas ap1, ap2, por eso al concatenar e imprimir, se imprime dicho salto de línea pareciendo como si fueran dos cadenas distintas

Una solución es depurar la cadena recibida para eliminar el salto de línea hallado al final de la misma, reemplazándolo por el carácter nulo de terminación:

Código (cpp) [Seleccionar] Expandir


size_t n;

n = strlen( ap1 );
if ( n > 0 && ap1[n-1] == '\n' ) ap1[n-1] = '\0';

y luego haces lo mismo para ap2, con lo cual debería solucionarse el problema.

SI ! Mil gracias ahora va perfecto, una pregunta la liberación de memoria esta bien situada? Esque al pasar el comprueba de mi profesor, da error, "Error in ________ free(): invalid next size(fast): 0x09425008 ***

[Yoel Alejandro]

Pues, la liberación de memoria la veo bien, habría que ver qué opinan lo otros foristas ......

Excepto la cadena "todo" cuya asignación de memoria ocurre dentro de la ejecución del primer do-while, por lo que no está garantizada en todos los casos.

Una sentencia más segura preguntaría primero si el malloc() se ejecutó primero, esto es:

Código (cpp) [Seleccionar] Expandir


if ( todo != NULL) {
    free( todo );
    todo = NULL;
}

prueba de esta manera y me comentas.

[joraloma]

Gracias lo he probado pero nada tio, no funciona... mismo fallo.. y estoy perdido

[rir3760]

Mil gracias ahora va perfecto, una pregunta la liberación de memoria esta bien situada? Esque al pasar el comprueba de mi profesor, da error, "Error in ________ free(): invalid next size(fast): 0x09425008 ***

+

Gracias lo he probado pero nada tio, no funciona... mismo fallo.. y estoy perdido

Cuando hagas cambios a un programa publica el código fuente actualizado, si no lo haces no hay forma de ayudarte.

----

En cuanto al código fuente de tu primer mensaje no es necesario inicializar las variables "nombre", "ap1" y "ap2" si lo primero que haces con ellas es asignarles el resultado de calloc y tampoco es necesaria la conversión explicita de (el valor de retorno de) esa función:

Código (c) [Seleccionar] Expandir

nombre = (char *) calloc(TAM, sizeof(char));
Basta con:

Código (c) [Seleccionar] Expandir

nombre = calloc(TAM, 1);

Un error importante se repite cuando pides el nombre, apellido paterno y materno:

Código (c) [Seleccionar] Expandir

printf(PIDE_NOMBRE);
fgets(nombre, TAM, stdin);
nombre = strchr(nombre,'\n'); /* <== */
*nombre = '\0';
Las tres mentadas variables no las debes modificar porque ellas almacenan la dirección base de los tres bloques, al almacenar en estas el resultado de strchr pierdes esas direcciones y las llamadas a free no serán validas (comportamiento no definido). En su lugar debes utilizar otra variable, por ejemplo:

Código (c) [Seleccionar] Expandir

char *p;

/* ... */

printf(PIDE_NOMBRE);
fgets(nombre, TAM, stdin);
if ((p = strchr(nombre, '\n')) != NULL)
   *p = '\0';

Otro error se genera cuando tratas de imprimir el nombre completo al utilizar "ap1" cuando debería ser "todo":

Código (c) [Seleccionar] Expandir

printf(TEXTO_NOMBRE_COMPLETO, ap1);

Por ultimo si la intención es imprimir el nombre completo en la forma "paterno materno, nombre" no es necesario reservar memoria, basta con una sola llamada a printf:

Código (c) [Seleccionar] Expandir

printf("%s %s, %s\n", ap1, ap2, nombre);

Un saludo