[SOLUCIONADO] Devolver vectores en C

[pablomr960]

Hola tengo un problema con un problema en el que debo hacer una funcion que devuelva un vector que sea suma de dos vectores que yo le paso.El problema viene cuando imprimo el valor de la suma que se le va la cabeza e imprible valores alazar.
El programa guarda bien los valores que hay en el archivo que le mando abrir que son {1,2,3} y {4,5,6} asique el problema esta al devolver el vector e imprimirlo.

Código PHP:

  int Suma(int vec[10],int vec_2[10],int longi,int longi_2){ 
    int *vec_tot; 
    int longi_tot; 
    int i; 
    i=0; 
    if(longi!=longi_2){ 
        printf("Tamaño dos vectores distintos."); 
        getchar(); 
    } 
    else{ 
        longi_tot=longi+longi_2; 
        vec_tot=(int*)malloc(longi*sizeof(int)); 
        for(i=0;i<longi;i++){ 
            vec_tot[i]=vec[i]+vec_2[i]; 
        } 
        return *vec_tot; 
    } 
} 
int Suma(int vec[10],int vec_2[10],int longi,int longi_2); 
 
int main(int argc, char** argv) { 
    FILE*fp; 
    int longi; 
    int longi_2; 
    int longi_tot; 
    int *vec; 
    int *vec_2; 
    int *vec_tot; 
    int k=0; 
    int n=0; 
    int i=0; 
    fp=fopen("vectores.txt","r"); 
 
    if(fp==NULL){ 
        printf("aaaa"); 
    } 
    else{ 
             
            fscanf(fp,"%d",&longi); 
            for(k=0;k<longi;k++){ 
                fscanf(fp,"%d",&vec[k]); 
            } 
            longi_tot=longi+longi_2; 
            fscanf(fp,"%d",&longi_2); 
            for(n=0;n<longi_2;n++){ 
                fscanf(fp,"%d",&vec_2[n]); 
            } 
             
            *vec_tot=Suma(vec,vec_2,longi,longi_2); 
            
            printf("A suma e: %d",vec_tot); 
             
    } 
 
    return (EXIT_SUCCESS); 
}  
   

[vangodp]

no estoy muy seguro...Pero cuando haces return *vec_tot; estas devolviendo el valor.. O no se si me equivoco XD

¿No seria return vec_tot;? Lo digo por que vec_tot ya es un puntero, lo que necesitas retornar el la posicion donde se encuentra el primer elemento del vector. O eso creo. XDD

De todas las formas ya llegara la caballeria para salvarnos ;)

*vec_tot=Suma(vec,vec_2,longi,longi_2);
es como hacer:

*vec_tot= *vec_tot;
No tiene mucho sentido lo anterior.
Creo que seria: vec_tot = Suma(vec,vec_2,longi,longi_2); // return vec_tot;

Aun que de C estoy pillao con memoria dinamica.

[kutcher]

Primero debes cambiar el tipo de retorno de la función de int al tipo correcto int* es decir:

Código C++:

int* suma(int *vec, int *vec_2, int longi, int longi_2);
 

Esto no es técnicamente devolver un vector C no permite devolver un vector entero pero un puntero al primer elemento si; y como ya te ha indicado vangodp debes retornar vec_tot no *vec_tot

Saludos

[leosansan]

Iniciado por pablomr960

Hola tengo un problema con un problema en el que debo hacer una funcion que devuelva un vector que sea suma de dos vectores que yo le paso.El problema viene cuando imprimo el valor de la suma que se le va la cabeza e imprible valores alazar.
El programa guarda bien los valores que hay en el archivo que le mando abrir que son {1,2,3} y {4,5,6} asique el problema esta al devolver el vector e imprimirlo.

Tienes un par de errores más a lo que ya te han comentado.

* Después de obtener las longitudes no dimensionas los vectores:

Código C++:

scanf ( "%d" , &longi ) ; 
vec =  malloc ( longi * sizeof ( int ) ) ; 
................................................  
scanf( "%d" , &longi_2 ) ; 
vec_2 =  malloc ( longi_2 * sizeof ( int ) ) ; 
.........................................
 

* Intentas imprimir vec_tot como si fuera un int en :

Código C++:

printf("A suma e: %d",vec_tot);
 

Cuando lo que procede es imprimir sus componentes:

Código C++:

printf ( "LA suma es: "  ) ; 
for ( i = 0 ; i < longi_2 ; i++ ) 
  printf ( "%d  ", vec_tot [i] );
 

* Si no son de igual longitud debes salir del programa. El getchar es sólo una pequeña pausa pero el código seguirá en main e imprimirá basura en ese caso:

Código C++:

if ( longi != longi_2 ) {
  printf("Tamaño dos vectores distintos.") ;
  exit (1) ; 
}
 

* Como hagas un programa un poco serio se te van a acabar las líneas si usas una para cada variable. Un método es agruparlas por tipo, algo como:

Código C++:

int  i , longi , longi_2  ; 
int *vec , *vec_2, *vec_tot ;
 

Y acostúmbrate a usar si procede una misma variable para diferentes bucles que no estén anidados, no un índice para cada bucle:

Código C++:

for ( i = 0  ; i < longi ; i++ )
    scanf ( "%d" , &vec [i] ) ; 
......................................... 
for ( i = 0 ; i < longi_2 ; i++ )
  scanf( "%d" , &vec_2 [i] ) ;
........................................... 
printf ( "LA suma es: "  ) ; 
for ( i = 0 ; i < longi_2 ; i++ ) 
  printf ( "%d  ", vec_tot [i] );
 

Como ves he usado un sola variable para los tres bucles.

* return no es una función por lo que no es necesario el uso del paréntesis en EXIT_SUCCESS.

Y para que no se diga, y ya que has propuesto un código trabajado, te pongo uno pero con scanf, para que veas que funciona. Ya tú tendrás que pasarlo a fscanf y al fichero:

Código C++:

#include <stdio.h>
#include <stdlib.h>
 
int *Suma ( int *vec  , int *vec_2  , int longi , int longi_2  ) ;
 
int main ( void ) { 
  int  i , longi , longi_2  ; 
  int *vec , *vec_2, *vec_tot ; 
  
  scanf ( "%d" , &longi ) ; 
  vec =  malloc ( longi* sizeof ( int ) ) ; 
  for ( i = 0  ; i < longi ; i++ )
    scanf ( "%d" , &vec [i] ) ; 
    
  scanf( "%d" , &longi_2 ) ; 
  vec_2 =  malloc ( longi_2* sizeof ( int ) ) ; 
  for ( i = 0 ; i < longi_2 ; i++ )
    scanf( "%d" , &vec_2 [i] ) ;
    
  vec_tot = Suma ( vec , vec_2 , longi , longi_2  ) ; 
  
  printf ( "LA suma es: "  ) ; 
  for ( i = 0 ; i < longi_2 ; i++ ) 
    printf ( "%d  ", vec_tot [i] );        
  return EXIT_SUCCESS ; 
}  
 
int *Suma( int *vec  , int *vec_2 , int longi , int longi_2  ) { 
    int i , *vec_tot; 
    if ( longi != longi_2 ) {
      printf("Tamaño dos vectores distintos.") ;
      exit (1) ; 
    }
    else { 
      vec_tot =  malloc ( longi * sizeof ( int ) ) ; 
      for ( i = 0 ; i < longi ; i++ )  
        vec_tot [i] = vec [i] + vec_2 [i] ;  
    } 
    return vec_tot;
}
 

¡¡¡Saluditos!!!

[kutcher]

Iniciado por pablomr960

El programa guarda bien los valores que hay en el archivo que le mando abrir que son {1,2,3} y {4,5,6}

Seguro que guarda bien los números? porque yo veo algunas inconsistencias en tu código si tu intención es leer datos de un fichero con el siguiente formato:

Código C++:

1 2 3
4 5 6
 

Te propongo el siguiente cambio:

Código C++:

int main(void)
{
    FILE* fp;
    int cols = 3;
    int *vec_1 = malloc(sizeof(int)*cols);
    int *vec_2 = malloc(sizeof(int)*cols);
    int *vec_tot = NULL;
    int i;
    char ch;
 
    fp = fopen("vectores.txt","r");
 
    if (fp == NULL)
        perror("Error");
    else
    {
        for(i = 0; i < cols; )
        {
            fscanf(fp, "%d%c", &vec_1[i++], &ch);
            if(ch == '\n') break;
        }
        int longi_1 = i;
        i = 0;
        while(1 == fscanf(fp, "%d", &vec_2[i]))
            if(++i == cols) break;
        int longi_2 = i;
        fclose(fp);
 
        vec_tot = Suma(vec_1, vec_2, longi_1, longi_2);
    //imprimir el resultado de la suma
    }
    free(vec_1);
    free(vec_2);
    free(vec_tot);
 
    return 0;
}
 

Saludos