@xXDraklordXxTienes una gran cantidad de fallos en ese código. Antes de nada ese código no debería compilar, no sé que compilador usas pero desde luego es muy permisivo
Voy a obviar los #includes
void makeText (char *t1, char *t2, char *S){
strcat(t2, S);
strcat(t2, t1);
printf ("%s\n", t2);
}
int main(int argc,char *argv[]){
if (argc != 3){
printf("Uso: Texto1 Texto2\n");
return; <--- return 0
}
if (!strcmp(argv[1], "-c")) makeText(argv[1], argv[2], " ");
return 0
}
Fallos (si es que pueden llamarse así) de estilo. Sí, esto es cosa de gustos, pero a la larga afectan a la calidad y la mantenibilidad del código
void makeText (char *t1, char *t2, char *S){
strcat(t2, S);
strcat(t2, t1);
printf ("%s\n", t2);
}
int main(int argc, char *argv[]){
if (argc != 3){
printf("Uso: Texto1 Texto2\n");
}else if (!strcmp(argv[1], "-c")){//Condiciones mutuamente excluyentes y uso de corchetes, aunque sea para delimitar una sola instrucción.
makeText(argv[1], argv[2], " ");
}
//Un único punto de salida de la función, ya tendrás ocasión de romper esta regla.
return 0;
}
Correcciones en el uso de los parametros
int main(int argc, char *argv[]){
//una llamada a un ejecutable consta de:
//nombreEjecutable parámetro1 parámetro2 parámetro3 ... parámetroN
//que se corresponden respectivamente con:
//argv[0] argv[1] argv[2] argv[3] ... argv[N]
//el valor de argc no es exactamente el numero de parámetros, sino el número de parámetros + 1
//por tanto como mínimo siempre vale 1, el 1 es por el nombre del ejecutable que como ya te he
//comentado es uno de los elementos de la matriz *argv[]
//Por tanto el valor válido de argc para 3 parámetros es 4, 3 parámetros (-c texto1 text2) y nombreEjecutable
if (argc != 4){
printf("Uso: Texto1 Texto2\n");
}else if (!strcmp(argv[1], "-c")){
makeText(argv[2], argv[3], " ");//Aqui supongo que el primer argumento debia ser argv[2], no 1 (que sera "-c")
}
return 0;
}
Aviso:
aviso: conversión obsoleta de una constante de cadena a ‘char*’
producido por
makeText(argv[2], argv[3], " ")
La signatura de la fúncion makeText es
void makeText (char *t1, char *t2, char *S);
Es decir, espera tres vectores (o cadenas o, más generalmente, tres punteros) de char, sin embargo estás pasando dos punteros (más tarde te diré que aquí hay otro error) y una constante. Hay muchas soluciones para arreglar esto:
Pasando el espacio como un vector puedes hacer:
char espacio[] = " ";
...
makeText(argv[2], argv[3], espacio);
Ahora lo grave:
void makeText (char *t1, char *t2, char *S){
strcat(t2, S);
strcat(t2, t1);
printf ("%s\n", t2);
}
main(){
..
makeText(argv[2], argv[3], espacio);
...
}
Estás concatenando sobre t2, es decir, todas las cadenas estan copiandose en t2 (se entiende que una a continuación de la otra) sin embargo no se ha reservado memoria para t2, t2 no tiene un espacio propio, de hecho lo que realmente estas haciendo es usar la memoria donde esta almacenado argv[2] como t2, esto pareces saberlo por el codigo que has hecho. Sin embargo esto es un gran problema, no solo porque estás machacando el contenido de argv[1] y potencialmente también el contenido de los demás parámetros, sino porque probablemente te saldrás de la zona reservada y el programa abortará.
Ejecuta este ejemplo con parametros "AAA BBB CCC DDD" para ver lo que ocurre
#include <stdio.h>
#include <string.h>
void makeText (char *t1, char *t2, char *S){
strcat(t2, S);
strcat(t2, t1);
}
int main(int argc,char *argv[]){
char espacio[] = " ";
if (argc == 5){
printf("ANTES:|%s|%s|%s|%s|\n", argv[1], argv[2], argv[3], argv[4]);
makeText(argv[2], argv[3], espacio);
printf("DESPU:|%s|%s|%s|%s|\n", argv[1], argv[2], argv[3], argv[4]);
}
return 0;
}
la salida será
ANTES:|AAA|BBB|CCC|DDD|
DESPU:|AAA|BBB|CCC BBB|BBB|
Como ves se ha machado el DDD al aumentar de longitud la cadena contenida en argv[3], esto se debe a que argv es una matriz y los datos en las matrices se encuentran almacenados consecutivamente, además, pese a que se ha sobrepasado la longitud al añadir el espacio, no cascará porque la memoria reservada para los parametros es mayor que la necesaria.
Haciendolo con strcat
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
void makeText (char *t1, char *t2, char *S){
int longitudTotal = 0;
char *cadenaFinal;
//primero se calcula el tamaño de la cadena
longitudTotal += strlen(t2);
longitudTotal += strlen(t1);
longitudTotal += strlen(S);
//se reserva el espacio necesario
cadenaFinal = (char*) malloc (sizeof(char) * (longitudTotal + 1));//+1 debido a que hay que dejar espacio para el caracter terminador
//se monta la cadena
cadenaFinal[0] = '\0'; //para indicar que la cadena esta vacia, el terminador como primer elemento
strcat(cadenaFinal, t2);
strcat(cadenaFinal, S);
strcat(cadenaFinal, t1);
printf ("%s\n", cadenaFinal);
//se libera la memoria reservada
free(cadenaFinal);
}
De mejor manera:
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
void makeText (char *t1, char *t2, char *S){
int longitudTotal = 0;
char *cadenaFinal;
longitudTotal += strlen(t2);
longitudTotal += strlen(t1);
longitudTotal += strlen(S);
cadenaFinal = (char*) malloc (sizeof(char) * (longitudTotal + 1));
sprintf(cadenaFinal, "%s%s%s", t2, S, t1);
printf ("%s\n", cadenaFinal);
//se libera la memoria reservada
free(cadenaFinal);
}
Lo ideal, en cualquier version, es que te cargues el parámetro donde pasas el espacio.
Espero que haya quedado claro.
![[oki]](/images/smilies/net_thumbsup.gif "Ok!")
![[carcajad]](/images/smilies/nuevos/risa_ani2.gif "carcajada")
![[tadoramo]](/images/smilies/adora.gif "Adorando")
)