concatenar varias cadenas

[minette1988]

Hola, tengo un subprograma que concatena 3 cadenas , y devuelve la longitud total al programa principal, pero me da el siguiente error: concatenar_cadenas.c:26: error: el objeto ‘longitud_cad’ llamado no es una función
concatenar_cadenas.c:26: error: el objeto ‘longitud_cad’ llamado no es una función
concatenar_cadenas.c:26: error: el objeto ‘longitud_cad’ llamado no es una función

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(){
         char cad1[100], cad2[100], cad3[100];
         int long_total;
         int concatenar(char[], char[], char[]);
 
         printf("Introduce una cadena: ");
         scanf("%99s", cad1); 
         printf("Introduce una cadena: ");
         scanf("%99s", cad2); 
         printf("Introduce una cadena: ");
         scanf("%99s", cad2);
 
         long_total=concatenar(cad1, cad2, cad3);
         printf("La concatenación tiene un total de %d caracteres.\n", long_total);
}
int concatenar(char cad1[], char cad2[], char cad3[]){
         int tamano;
         int i, j;
         int error = 1;
         int longitud_cad;
 
         tamano=longitud_cad(cad1) + longitud_cad(cad2) + longitud_cad(cad3);
 
         if(tamano > sizeof(cad3)){
             error=0;
             printf("No hay espacio suficiente para la concatenación.\n");
         }
         else{
              i = 0;
              while(cad1[i] != '\0'){
                  cad3[i]=cad1[i];
                  i++;
              }
              j=0;
              while(cad2[j] != '\0'){
                  cad3[i]=cad2[j];
                  j++;
                  i++;
              }
              cad3[i+1]='\0';
         }
         return(tamano);
}
 

[iwkillyou]

¿entiendes ese codigo? :/

[iwkillyou]

puedes usar, strcpy, strcat, strlen, para hacer eso

[sam90]

Tienes el problema que longitud_cad es una variable y no una funcion. Reemplaza longitud_cad por strlen que es una funcion que devuelve la longitud de una cadena.

[minette1988]

Voy a plantear bien el problema:

1º Introducir dos cadenas de teclado(programa principal).
2º Función que devuelva la longitud de la primera y segunda cadena.
3º Función que concatene ambas cadenas y devuelva como resultado la suma de ambas cadenas.

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
     
int main(){
         char cad1[100], cad2[100], cad3[200];//cad3[200] porque es la suma de cad1[100]+cad2[100]; 
         int long_total;
         int concatenar(char[], char[], char[]);
 
         printf("Introduce una cadena: ");
         scanf("%99s", cad1);
         printf("Introduce una cadena: ");
         scanf("%99s", cad2);
         
 
         long_total=concatenar(cad1, cad2, cad3);
 
         printf("La concatenación tiene un total de %d caracteres.\n", long_total);
}
int long_cad(char cad1[]){
         int i;
         i=0;
          
 
         while(cad1[i] != '\0')
              i++;
        
         return(i);
} 
 
int concatenar(char cad1[], char cad2[], char cad3[]){
               int tamano;
               int i, j;
               int error=1;
 
               tamano=long_cad(cad1) + long_cad(cad2);
 
               if(tamano > sizeof(cad3)){
                   error=0;
                   printf("No hay espacio suficiente para la concatenacion.\n");
               }
               else{
 
                   i = 0;
 
                   while(cad1[i] != '\0'){
                      cad3[i]=cad1[i];
                      i++;
                   }
                   j=0;
 
                   while(cad2[j] != '\0'){
                     cad3[i]=cad2[j];
                     j++;
                     i++;
                   }
                   cad3[i+1]='\0';
               }     
               return(error);
}
 

Resultado de la compilación:
Introduce una cadena: pepe
Introduce una cadena: ana
No hay espacio suficiente para la concatenacion.
La concatenación tiene un total de 0 caracteres.

[minette1988]

Ayudadme con este programa, por favor. Gracias.

[sam90]

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int long_cad(char cad1[]){
         int i=0;
         while(cad1[i] != '\0')
              i++;
         return i;
} 
 
char * concatener (char * cad1, chae * cad2)
{
     int longitud = long_cad(cad1) + long_cad(cad2)
     char * buffer  = malloc(longitud * sizeof(char));
     int i=0,j=0;
 
     while (cad1[i] != '\0')
                  buffer[i] = cad1[i++];
     while (cad2{[j] != '\0')
                 buffer[i++] = cad2[j++];
    buffer[i] = '\0'
    return buffer;        
}
 
int main(){
         char cad1[100], cad2[100]; 
         int long_total;
         char * cadena; 
         printf("Introduce una cadena: ");
         scanf("%99s", cad1);
         printf("Introduce una cadena: ");
         scanf("%99s", cad2);
         cadena = concatenar(cad1, cad2);
 
         printf("Resultado de la concatenacion:  %s\n",cadena)
         printf("La concatenación tiene un total de %d caracteres.\n", long_cad(cadena));
         free(cadena);
}
 

Ahi tenes tu codigo. Use memoria dinamica para realizar la concatenacion. Por eso despues la libero.
Tambien fijate que en la libreria "string.h" ya vienen esta funciones hechas. Pero nunca esta demas volver a hacerlas.

[minette1988]

¿Por qué pones el main al final en lugar de al principio?
Si quiero ponerlo al principio que tendría que cambiar?

Por qué en la función concatenar utilizas un puntero a char si lo que interesa es que sea de tipo int ya que lo que queremos es que devuelva la suma de cad1 y cad2

char * concatenar
La instrucción que sigue no entiendo lo de buffer puntero a char, ¿la función malloc?
Lo de sizeof(char) es el tamaño en bytes de la variable tipo char?

char * buffer = malloc(longitud * sizeof(char));

[sam90]

Si el main no esta a lo ultimo, el compilador no va a encontrar las demas funciones que defini. Obviamente que hay una solucion para esto que es agregar la cabeceras al principio:

int long_cad(char cad1[]);
char * concatener (char * cad1, chae * cad2);

Si agregas eso entre los #include y la primera funcion, da lo mismo el orden en que defina las funciones.

Mi funcion concatener devuelve una cadenas de caracteres, no de enteros. Vos acasos querias que devuelva solo la longitud de la concatenacion??
En mi caso devuelve el puntero a la cadena concatenada.

La variable buffer la utilizo para llevar un puntero a memoria. En donde guardare la cadena final. Con malloc lo que se hace es reservar memoria para la cadena. Por eso reservo la suma de las longitudes y lo multiplico por sizeof(char) para me uqe guarde espacio para caracteres.

Despues lo que hago en la funcion concatener, return buffer, devuelvo el valor de la variable buffer, que tiene la direccion a la reserva de memoria. Por lo tanto, al retorno tengo una cadena de caracteres concatenada.

Para el trabajo de cadenas de caracteres te recomiendo que lees sobre punteros en C++....

[CalgaryCorpus]

La reserva de memoria para un string tiene que considerar el caracter de marca de fin de string, por lo tanto, al pedir memoria, hay que sumar 1 a la cantidad de caracteres "visibles".

Para el caso particular, la variable buffer, al solicitar memoria tiene que sumar 1 a lo que ya pide, quedaria:

char * buffer = malloc(longitud +1);

En lenguaje C se define que sizeof(char) es 1, asi que es posible ahorrarse la multiplicacion por 1.

[sam90]

El sizeof(char) lo deje para que sea mas descriptivo el ejemplo y entienda mejor lo que es memoria dinamica. Para que no se cierre a que solo se usa con caracteres.

Y es verdad, me falto el +1 para el caracter nulo asi se detecta el fin de cadena.