¿Algún alma caritativa que me eche una mano?

[MrPizza]

Buenas, estoy intentando realizar este ejercicio en el CFree:
"Crear un programa que contenga una función llamada copiarVector que reciba dos
vectores enteros y el tamaño de los mismos (deben de ser del mismo tamaño) y que copie en el segundo vector el contenido del primero"

Y esto es lo que estoy haciendo:
#include <stdio.h>
void copiarvector(int tam1, int tam2, int vector1[tam1], int vector2[tam2]);
main()
{
copiarvector(tam1, tam2, vector1[tam1], vector2[tam2]);
for(i=0;i<tam1;i++)
{
printf("El valor del segundo vector en la posicion %d es %d\n",i,vector2[i]);
}
}
void copiarvector(int tam1, int tam2, int vector1[tam1], int vector2[tam2])
{
int i, tam1, int tam2, int vector1[tam1], int vector2[tam2];
printf("Introduzca el tamano del primer vector: \n");
scanf("%d",&tam1);
tam2=tam1;
for(i=0;i<tam1;i++)
{
printf("Introduzca el valor de la posicion %d del primer vector: \n",i);
scanf("%d",&vector1[i]);
vector2[i]=vector1[i];
}
}

Me da bastantes errores, entre ellos que tam1 y compañía no están declarados, cosa que no entiendo ya que dentro de la implementación de la función los he declarado. Help pls.
Gracias de antemano

[amchacon]

Se puede postear el código así o se puede postear el código de esta forma:

Código C++:

#include <stdio.h>
void copiarvector(int tam1, int tam2, int vector1[tam1], int vector2[tam2]);
 
main()
{
    copiarvector(tam1, tam2, vector1[tam1], vector2[tam2]);
    for(i=0; i<tam1; i++)
    {
        printf("El valor del segundo vector en la posicion %d es %d\n",i,vector2[i]);
    }
}
 
void copiarvector(int tam1, int tam2, int vector1[tam1], int vector2[tam2])
{
    int i, tam1, int tam2, int vector1[tam1], int vector2[tam2];
    printf("Introduzca el tamano del primer vector: \n");
    scanf("%d",&tam1);
    tam2=tam1;
    for(i=0; i<tam1; i++)
    {
        printf("Introduzca el valor de la posicion %d del primer vector: \n",i);
        scanf("%d",&vector1[i]);
        vector2[i]=vector1[i];
    }
}
 

Mucho mejor y más fácil que te contentesmos así.

Y el problema esque no has declarado ningún vector en el main (de hecho no has declarado ninguna de las variables *_*).

[MrPizza]

He probado a declararlos en el main añadiendo esto al principio:
int i, tam1, tam2, vector1[tam1], vector2[tam2];
Pero nada, me sigue dando los mismos errores.
Por cierto como has posteado así el código?

[cursillosonline]

Iniciado por MrPizza

He probado a declararlos en el main añadiendo esto al principio:
int i, tam1, tam2, vector1[tam1], vector2[tam2];
Pero nada, me sigue dando los mismos errores.
Por cierto como has posteado así el código?

Pero no entiendo tu codigo, para que usas tam1 y tam2 si al final haces que tam2 sea igual a tam1?, quieres que tus vectores sean de igual tamaño o diferentes?, ademas tam1 y tam2 no tienen valores asignados todavía, por lo que al llamar a la función.

Código C:

copiarvector(tam1, tam2, vector1[tam1], vector2[tam2]);
 

Este tendrá un comportamiento indefinido.

Es decir, debes definir tus variables tam1 y tam2, y además asignarles un valor.

PD: Por otro lado, si tam2 llega a ser mas chico que tam1 se producira un buffer overflow en la linea 23.

[vangodp]

int i, tam1, tam2, vector1[tam1], vector2[tam2];

Esto no me gusta en absoluto para nada. al declarar a tam1 en la misma linea de vector1[tam1], tam1 no sabes que tamaño tiene. puede ser uno, puede ser cero o puede ser tropecientos trillones XDDDD

Prueba dando un valor a tam1 y tam2: int i=0, tam1=10, tam2=10, vector1[tam1], vector2[tam2];

Entonces ahora sabes que vector1 tiene 10 y vector2 tiene 10. ;)

Recuerda que dar un valor inicial a una variable aun que sea cero es buena practica.

[eferion]

Iniciado por vangodp

Esto no me gusta en absoluto para nada

No es sólo que no te guste... es que eso no pertenece al estándar, luego no está garantizada la forma de actuar de ese código. Puede que, lo haga como esperas o puede que empieces a pisar memoria sin saber el por qué.

En vez de hacer eso hay dos opciones:

* Si no quieres usar memoria dinámica, declara un buffer más grande de lo necesario (imprescindible definir un tope) y luego usa otra variable para saber cual es la capacidad real del buffer.

* La opción dos es reservar la memoria necesaria con malloc y, bueno, en este caso también necesitas una segunda variable para que te indique el tamaño del buffer.

[cursillosonline]

Bueno, una solucion al problema podria ser esta.

Código C:

#include <stdio.h>
 
int copiarvector(int tam1, int tam2, int vector1[], int vector2[]) {
    int i;
    
    if (tam2 < tam1) {
        puts("El tamano del segundo arreglo es mas chico");
        
        return 0;
    }
    else {
        for(i = 0; i < tam1; i++)
            vector2[i] = vector1[i];
            
        return 1;
    }
}
 
int main() {
    int i, tam1, tam2;
    
    printf("Introduzca el tamano del primer vector: ");
    scanf("%d", &tam1);
    
    printf("Introduzca el tamano del segundo vector: ");
    scanf("%d", &tam2);
    
    int vector1[tam1], vector2[tam2];
    
    for(i = 0; i < tam1; i++) {
        printf("Introduzca el valor de la posicion %d del primer vector: ", i + 1);
        scanf("%d", &vector1[i]);
    }
    
    if (copiarvector(tam1, tam2, vector1, vector2))
        for(i = 0; i < tam2; i++)
            printf("El valor del segundo vector en la posicion %d es %d\n", i + 1, vector2[i]);
}
 

[eferion]

Iniciado por cursillosonline

Bueno, una solucion al problema podria ser esta.

Código C:

int vector1[tam1], vector2[tam2];
 

Ahí tienes el mismo problema que se ha comentado anteriormente. Declarar un array de esta manera es algo a evitar encarecidamente

[vangodp]

¿Que problema tiene declarar int vector1[tam1], vector2[tam2]; según el estándar? ¿De alguna forma hay que declararlo no?
Lo de pisar memoria solo si fuera:
int tam1, tam2;
int vector1[tam1], vector2[tam2];

...Porque:
int tam1= 10, tam2=10;
int vector1[tam1], vector2[tam2];

...No veo en que va pisar memoria ninguna.

[eferion]

Iniciado por vangodp

¿Que problema tiene declarar int vector1[tam1], vector2[tam2]; según el estándar? ¿De alguna forma hay que declararlo no?

El problema es que el estándar dice que, en estos casos, el tamaño del arreglo tiene que quedar establecido en tiempo de compilación... tu ahí estás forzando a que el tamaño se establezca en tiempo de ejecución.

Un ejemplo en el que se ve más claro:

Código C:

int funcion( int filas, int columnas, int array[filas][columnas] )
{
  int i, j;
  for ( i=0; i < filas; i++ )
  {
    for ( j=0; j < columnas; j++ )
    {
      printf( "%d ", array[i][j] );
    }
  }
}
 

¿Me podrías decir cómo puede el compilador saber en este código cuánto vale "columnas" para calcular los saltos correctamente? No puede. Es algo que queda para tiempo de ejecución y solo en aquellos casos en los que el compilador esté preparado para ello... que como digo, al no formar parte del estándar no es algo seguro.

La forma de afrontar estos casos pasa por reservar memoria de forma dinámica con malloc.

[dehm]

No se ha hablado de que se pise memoria, sino de que no es la forma.
O se usan constantes:

Código C++:

const int tam=5;
int array[tam];
 

O se usa memoria dinámica.

Código C++:

int tam=5;
int *array= new int[tam];
 

Aquí dos enlaces:
http://c.conclase.net/curso/?cap=010
http://gcc.gnu.org/onlinedocs/gcc-4....ariable-Length

En el 2º enlace te dice que el estandar C99 sí que permite esta forma.
Pero luego, viendo otros artículos, se alerta del problema que puede ocurrir si el tamaño de la memoria a reservar es demasiado grande y nos quedamos sin espacio en la pila.
http://www.clarkcox.com/blog/2009/04...vlas-are-evil/
De ahí que esté desaconsejada esta forma

Saludos!

[dehm]

Ala, ya te has adelantado!

[eferion]

Iniciado por dehm

En el 2º enlace te dice que el estandar C99 sí que permite esta forma.

Siempre había oído la versión opuesta, me lo apunto.

[cursillosonline]

Iniciado por dehm

En el 2º enlace te dice que el estandar C99 sí que permite esta forma.
Pero luego, viendo otros artículos, se alerta del problema que puede ocurrir si el tamaño de la memoria a reservar es demasiado grande y nos quedamos sin espacio en la pila.
[URL="http://www.clarkcox.com/blog/2009/04/07/c99s-vlas-are-evil/"]http://www.clarkcox.com/blog/2009/04/07/c99s-vlas-are-evil/[/URL]
De ahí que esté desaconsejada esta forma

Saludos!

Leyendo un poco más (se que se podian hacer VLAs desde el C99, pero no he leído mucho del tema) el problema es que a diferencia de usar malloc (que retorna null en caso de haber fallado), un VLA puede ser más peligroso por que no retorna null en caso de que no halla memoria suficiente para el arreglo (y en el stack el espacio de memoria es pequeño), pero para arreglos de tamaño pequeños no creo que halla mucho drama, aun así la mayoría recomienda no usarlos.

[vangodp]

¿Me podrías decir cómo puede el compilador saber en este código cuánto vale "columnas" para calcular los saltos correctamente?

Código C:

#include <stdio.h>
#include <stdlib.h>
 
#define f 0
#define c 0
 
void funcion ( int filas, int columnas, int array[f+filas][c+columnas] ) {
    int i, j;
    
    for ( i = 0; i < filas; i++ ) {
        for ( j = 0; j < columnas; j++ ) {
            printf ( "%d ", array[i][j] );
        }
    }
}
 
int main () {
    int fil, col, contador = 0;
    
    fil = col = 3;
    int a[fil][col];
    
    int i, j;
    
    for ( i = 0; i < fil; i++ ) {
        for ( j = 0; j < col; j++ ) {
            a[i][j] = contador;
            contador++;
        }
    }    
    
    funcion( fil, col, a );
 
 
    getchar();
    return 0;
}
 

¿Que te parece? XDDDD

[dehm]

Hola:

Yo entiendo que si se usa esta forma en funciones, las cuales van a dimensionar un array y además no sabemos muy bien qué tamaño podría llegar a manejar, estamos jugando con fuego.

En casos como el que indicas, en los que me limito a almacenar el tamaño del array en una variable, y luego declarar el array en función de esta variable, pues no parece peligroso, siempre que estemos seguros de que esa variable no se va a usar para otros usos y cosas raras así.(que ciertamente es fácil controlar esta cuestión)

Pero aún así, siempre va a dar un warning, y en mi caso, con el -pedantic-errors activado, no me compilaría.

[vangodp]

¿Pero va decir que no mola lo de?:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
 
int main () {
    const int FILAS = 0; 
    const int COLUMNAS = 0;
    
    int f = 5;
    int c = 5;
    
    char arreglo[FILAS+f][COLUMNAS+c];
 
    getchar();
    return 0;
}
 

Es una constante no constante jajaja
XDDD

Ademas si es para grandes cantidades de memoria creo que cualquiera de aquí sabe que debe ser en el heap.

[MrPizza]

Pues muchas gracias a tod@s, me han venido bastante bien las respuestas. Saludos ;)