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 :
Citarsalas@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 :Citarsalas@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:/* 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!!!
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:
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.
Cita de: yoel_alejandro en 11 Enero 2015, 16:44 PM
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:
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 ***
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:
if ( todo != NULL) {
free( todo );
todo = NULL;
}
prueba de esta manera y me comentas.
Gracias lo he probado pero nada tio, no funciona... mismo fallo.. y estoy perdido
Cita de: joraloma en 11 Enero 2015, 16:48 PMMil 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 ***
+
Cita de: joraloma en 12 Enero 2015, 00:56 AMGracias 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:
nombre = (char *) calloc(TAM, sizeof(char));
Basta con:
nombre = calloc(TAM, 1);
Un error importante se repite cuando pides el nombre, apellido paterno y materno:
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:
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":
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:
printf("%s %s, %s\n", ap1, ap2, nombre);
Un saludo