Error 1: Punteros no inicializados
char *cadA;
scanf("%s", &cadA);
cadA
es un puntero que no está inicializado, luego apunta a una dirección aleatoria y, si intentas modificar la memoria apuntada podrán pasar dos cosas:
- Que corrompas memoria de otra aplicación. Esto sucede en sistemas operativos antiguos que no controlan los accesos a memoria.
- Que el sistema operativo mate tu aplicación... de no hacerlo sucedería lo comentado en el primer punto.
Dado que el mensaje indica que la cadena puede tener hasta 50 caracteres, lo que más sentido tiene es declarar un array de tamaño fijo con tamaño 51 (50 caracteres de longitud más el finalizador \0
).
char cadA[51], cadB[51];
Y ahora le llega el turno a cadC
... ¿Qué tamaño le ponemos? Como estás concatenando cadenas (de tamaño máximo 50) y solo va a ser necesario que tenga un finalizador, el tamaño mínimo es 101 (o 102 si quieres usar un separador entre ambas cadenas):
char cadC[101];
Error 2: Las lecturas son incorrectas
scanf("%s", &cadA);
scanf
necesita que los argumentos se le faciliten en forma de punteros porque es la única forma que hay en C para modificar variables que no pertenecen a la propia función (aparte de las variables estáticas). Pues bien, char[]
es, de por sí, un puntero, luego no hay que usar aquí el operador de referencia &
.
Al declarar char cadA[51]
, en cadA
se almacena únicamente la dirección en la que comienza el array. El uso de scanf
con punteros es así:
scanf("%s",cadA);
Y lo mismo le sucede a printf
:
printf("%s", &cadC);
// ^ Esto sobra
Cuidado con los punteros dobles
Ya hemos visto que cadA
es un puntero, luego añadir un asterisco a la declaración sirve para crear punteros dobles:
void concat(char *cadA[50], char *cadB[50], char *cadC[100]){
// ^ ^ ^
El único motivo que justificaría que concat
reciba un puntero doble es porque, bajo ciertas condiciones, concat
tiene que modificar la dirección a la que apunta uno de esos punteros... y no es el caso. concat
solo manipula la información que está apuntada por los punteros, luego lo esperable es que concat
reciba punteros simples:
void concat(char cadA[50], char cadB[50], char cadC[100]){
Consejo 1: Declaración confusa
En C, al declarar arrays de tamaño fijo, el mismo se ubica en posiciones consecutivas de la memoria. Al encontrarse los elementos en posiciones consecutivas, (y teniendo en cuenta que cadA
tiene consideración de puntero), el compilador no necesita conocer el tamaño de cada array nada más que en la declaración. La excepción la encontrarás con los arrays multidimensionales... pero no es el caso que nos ocupa en esta pregunta.
Lo que quiero decir es que al llamar a concat
no es recomendable que indices el tamaño de cada arreglo:
- No es necesario
- Puede dar lugar a confusiones si los valores indicados ahí no coinciden con los reales.
En su lugar declara la función con punteros:
void concat(char* cadA, char* cadB, char* cadC)
E incluso puedes mejorar un más la legibilidad del programa marcando cadA
y cadB
como constantes ya que su valor no debe ser modificado dentro de la función. Al declarar algo como const
permites que el compilador te avise si tu código intenta modificar la variable en cuestión (te ayuda a encontrar errores):
void concat(char const* cadA, char const* cadB, char* cadC)
Consejo 2: Aritmética de punteros
El algoritmo de concat
queda más limpio si se usa aritmética de punteros. De hecho podemos aprovechar que los punteros se pasan por valor para modificar su posición sin miedo:
void concat(char const* cadA, char const* cadB, char * cadC)
{
// Copiamos la primera cadena
for( ; *cadA != '\0'; ++cadA, ++cadC )
*cadC = *cadA;
// Copiamos la segunda cadena
for( ; *cadB != '\0'; ++cadB, ++cadC )
*cadC = *cadB;
*cadC = '\0'; // Finalizamos la cadena
}