 Autor
|
Tema: [AYUDA] Ejercicio concatenar (Leído 2,314 veces)
|
joraloma
 Desconectado
Mensajes: 11
|
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:/* 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 */
|
|
|
|
|
En línea
|
|
|
|
|
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: 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.
|
|
|
|
|
En línea
|
Saludos, Yoel.
P.D..- Para mayores dudas, puedes enviarme un mensaje personal (M.P.)
|
|
|
joraloma
Desconectado
Mensajes: 11
|
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 ***
|
|
|
|
|
En línea
|
|
|
|
|
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: if ( todo != NULL) { free( todo ); todo = NULL; }
prueba de esta manera y me comentas. |
|
|
|
|
En línea
|
Saludos, Yoel.
P.D..- Para mayores dudas, puedes enviarme un mensaje personal (M.P.)
|
|
|
joraloma
Desconectado
Mensajes: 11
|
Gracias lo he probado pero nada tio, no funciona... mismo fallo.. y estoy perdido
|
|
|
|
|
En línea
|
|
|
|
rir3760
Desconectado
Mensajes: 1.639
|
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: nombre = (char *) calloc(TAM , sizeof(char));
Basta con:
Un error importante se repite cuando pides el nombre, apellido paterno y materno: 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; /* ... */ 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
|
|
|
|
|
En línea
|
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language
|
|
|
|
| Mensajes similares |
|
Asunto |
Iniciado por |
Respuestas |
Vistas |
Último mensaje |
|
|
[Ejercicio] Aca tienen un ejercicio para practicar
« 1 2 3 »
Programación Visual Basic
|
Dreamaker
|
20
|
15,852
|
5 Septiembre 2010, 02:57 am
por TGa.
|
|
|
Ayuda ejercicio de Pascal, cuando uno no lo ve claro pide ayuda
Programación General
|
manu3742
|
1
|
3,266
|
31 Marzo 2011, 15:43 pm
por d(-_-)b
|
|
|
|
[AYUDA] Cómo puedo concatenar un char a un puntero de chars?
Programación C/C++
|
papita5
|
8
|
6,338
|
19 Octubre 2011, 20:39 pm
por papita5
|
|
|
|
Ayuda Con Ejercicio
ASM
|
sebis
|
1
|
2,934
|
10 Septiembre 2012, 13:12 pm
por Eternal Idol
|
|
|
|
Ayuda! Concatenar carácteres en C
Programación C/C++
|
ARCAME
|
1
|
2,250
|
2 Diciembre 2012, 20:50 pm
por durasno
|
 |
