concatenate - jstl concat string and variable

C-Cuerdas Concat

Question

Intentando concatenar cadenas (* char) usando C y teniendo muchas fallas de segmentación:

void printDateFormat( char *in ) { /* begin function printDateFormat */
char *month; // month by char
int month_int; // month by digit
char *day; // day by char
char *year; // year by char
char *dateToken; // date token in split
char *formatted; // formatted string
dateToken = strtok (in, "/");
month = &dateToken;
formatted = formatted = getMonth(month);
dateToken = strtok (NULL, "/");
day = &dateToken;
formatted = strcat (formatted, day);
formatted = strcat (formatted, ", ");
dateToken = strtok (NULL, "/");
year = &dateToken;
formatted = strcat (formatted, year);
in = *formatted;
} /* End function printDateFormat */
char *getMonth( int d) { /* begin function *getMonth */
switch (d) {
case 1:
return "January";
// break;
case 2:
return "February";
// break;
case 3:
return "March";
// break;
case 4:
return "April";
// break;
case 5:
return "May";
// break;
case 6:
return "June";
// break;
case 7:
return "July";
// break;
case 8:
return "August";
// break;
case 9:
return "September";
// break;
case 10:
return "October";
// break;
case 11:
return "November";
// break;
case 12:
return "December";
// break;
}
} /* End function *getMonth */

Se espera la entrada en printDateFormat () como otra cadena en el formato: MM / dd / aaaa ... es decir. 31/03/2013. El propósito es convertirlo en: 31 de marzo de 2013.

EDITAR:

Así es como paso a printDateFormat

void option1( void ) { /* begin function option1 */
char date[10]; /*user input date string */
printf("/n/nEnter date [Format: MM/dd/yyyy]: ");
fgets(date, 10, stdin);
scanf("%s", &date);
printDateFormat(date);
printf("/n%s", date);
} /* End function option2 */

EDICION 2:

Ok, hice algunos cambios pero aún no hay dados ...

aquí está mi advertencia de compilación:

asgn9.c: In function `printDateFormat'':
asgn9.c:224: warning: passing arg 1 of `getMonth'' makes integer from pointer without a cast
asgn9.c:237: warning: assignment makes pointer from integer without a cast

se refieren al uso de getMonth() dentro de mi printDateFormat()

Aquí está mi código actualizado, todavía tengo un error de segmentación en el mismo lugar ...

void printDateFormat( char *in ) { /* begin function printDateFormat */
char *month; // month by char
int month_int; // month by digit
char *day; // day by char
char *year; // year by char
char *dateTkn; // date token in split
char *formatted; // formatted string
dateTkn = strtok (in, "/");
month = dateTkn;
formatted = getMonth(month);
dateTkn = strtok (NULL, "/");
day = dateTkn;
formatted = strcat (formatted, day);
formatted = strcat (formatted, ", ");
dateTkn = strtok (NULL, "/");
year = dateTkn;
formatted = strcat (formatted, year);
in = *formatted;
} /* End function printDateFormat */
char *getMonth( int d) { /* begin function *getMonth */
static char *months[] = {"January", "February", "March", "April", "May",
"June", "July", "August", "September", "October", "November", "December"};
return strcpy(malloc(32), months[d]);
} /* End function *getMonth */

Answer 1

C no es un lenguaje para el procesamiento de cadenas. Si puede, use C ++ y std :: string.

Una forma de hacer esto en C es definir un char buffer[BUFFERSIZE] grande char buffer[BUFFERSIZE] para copiar las dos cadenas en el fin de concatenarlas. ¡Ten cuidado de no sobrepasar el buffer!

También puede malloc / liberar el búfer, pero ese es otro conjunto de dolores de cabeza.

Answer 2

El valor devuelto por

getMonth(month);

es un punto a una cadena constante . No puede modificar esta cadena, incluido strcat (). Puede cambiar la función para cumplir con su requerimiento como:

char formatted[MAXSIZE]; // formatted string
getMonth(month, formatted);
void getMonth( int d, char *cache) { /* begin function *getMonth */
switch (d) {
case 1:
strcpy(cache, "January");
return;
......
}

Después de eso, puede continuar modificando el contenido en la matriz de caracteres "formateada".

Answer 3

No ha asignado ninguna memoria para el formato. Cambiar la declaración de formateado a

char formatted[80];

y cambie la primera asignación a formateada para

strcpy ( formatted, getMonth ( month ), sizeof ( getMonth ( month ) ) );

y deberías estar bien.

Answer 4

Puede resolver algunos de sus problemas y simplificar el código con:

char *getMonth(int d) {
static char *months[] = { "January", "February", "March",
. . .
};
return strcpy(malloc(60), months[d]);
}

De esta manera,

• el valor de retorno es escribible
• tiene espacio adicional para tus operaciones strcat ()

^{Para la fuerza industrial, es posible que desee comprobar el valor de retorno de malloc() , ver si n está dentro del rango, y cosas por el estilo.}

Answer 5

Supongamos que su publicación es correcta, entonces los problemas son

month = &dateToken;
....
day = &dateToken;
....
year = &dateToken;

strtok return char *, así que simplemente elimine y de cada ''dateToken''. Y su prototipo de entrada ''getMonth'' es de tipo ''int'', pero usted le da a la función el tipo ''char *'', la tabla ''switch'' no reconocerá el caso de la tabla sin dígitos.

Answer 6

Your getMonth devuelve un puntero a un literal de cadena. Intentar modificarlo (por ejemplo, con strcat ) no está permitido; conduce a un comportamiento indefinido.

Sugeriría (fuertemente) usar strftime para manejar la fecha del formato y / o las cadenas de tiempo para imprimir. Esto no solo reducirá su código de formato a una línea, sino que también le permitirá respaldar resultados localizados cuando / si lo desea también.

Editar: si no puedes usar strftime , querrás construir una fecha formateada en tu propio buffer, probablemente usando sprintf :

char buffer[256];
static const char *months[] = {
"January",
"February",
/* ... */ ,
"November",
"December"
};
sprintf(buffer, "%s %d %d", months[monthnum], day, year);