[SOLUCIONADO] Error al guardar arreglo bidimensional en unidimensional

[TriN]

Buenas noches comunidad de forosweb se les saluda, por aqui me encuentro aprendiendo cada dia mas pero hoy me encuentro atascado en un codigo en un programa que guarde una matriz 5 x 5 en un vector de 25 elementos el codigo es:

#include <stdio.h>
#include <conio.h>

main()
{
int vectora[5][5],v[25],i,j;

for (i=0;i<25;i++)
{
vectora[i]=0;
}



for (i=0;i<25;i++)
for (j=0;j<25;j++)
{
printf("[%d,%d] = ",i,j);
scanf("%d",&a[i][j]);
}
for (i=0;i<5;i++)
for (j=0;j<5;j++)
{
vector[i]=a[i][j];
}


for (i=0;i<25;i++)

{
printf("%d ",v[i]);
}
getch();
}

Compila el codigo pero al imprimir los valores solo me guarda los valores 1,3 2,3 y 3,3 que podria estar sucediendo gracias de antemano

[CalgaryCorpus]

Suponiendo que es cierto que compila, estas asignando una variable e imprimiendo otra.

[TriN]

Hola Colgary gracias por tu pronta repuesta ya he arreglado el cod, pero igual me sale el mismo error

#include <stdio.h>
#include <conio.h>

main()
{
int ma[5][5],v[25],i,j;

for (i=0;i<25;i++)
{
v[i]=0;
}



for (i=0;i<5;i++)
for (j=0;j<5;j++)
{
printf("[%d,%d] = ",i,j);
scanf("%d",&ma[i][j]);
}
for (i=0;i<5;i++)
for (j=0;j<5;j++)
{
v[i]=ma[i][j];
}


for (i=0;i<25;i++)

{
printf("%d ",v[i]);
}
getch();
}

solo guarda los valores de la posicion 1,5 2,5 3,5 4,5 y 5,5

[CalgaryCorpus]

Una pista:

Si el arreglo "v" tiene 25 posiciones, que indice estas usando para asignar esas 25 posiciones? ¿Que valores toma ese indice?

[TriN]

Hola calgary, los valores de la matriz 5 x 5 al mi parecer, pero esta mal ejecutado

[CalgaryCorpus]

En un arreglo o matriz hay valores guardados y llegas a ellos usando indices. Olvida los valores.

Repito :

Si el arreglo "v" tiene 25 posiciones, que indice estas usando para asignar esas 25 posiciones? ¿Que valores toma ese indice?

Sugerencia:
Al poner codigo aqui en el foro destaca usando el tipo C o C++. Con ello las lineas aparecen con nros a su lado y es mas facil hablar de ellas.

Código C:

Int main ()
{
}
 

[vosk]

@CalgaryCorpus: "...Una pista..."

Otra pista :)) Haz el bucle donde asignas los valores de 'ma' a 'v' con palel y lapiz para ver en que posiciones guardas:

Código C:

for (i=0;i<5;i++) {
    for (j=0;j<5;j++) {
        v[i]=ma[i][j];
    }
}
 

No obstante puede que te resulte mas atractivo algoritmo de los videojuegos (aplicable en muchos casos):

Código C:

posicion lineal = (ancho matriz * posicion y) + posicion x
 

Código C:

#define W 5
#define H 5
 
int x, y, offset;
for(y = 0; y < H; y++) {
    for(x = 0; x < W; x++) {
        offset = (W * y) + x;
        v[offset] = ma[y][x];
    }
}
 

Solo una observacion, en este caso estoy leyendo de derecha a izquierda y de arriba a abajo, por eso recojo los valores con las posiciones cambiadas ma[y][x]

Esto te deja el array bidimensional como lineal; de esta forma puedes acceder a cualquier posicion del array lineal para lectura i/o escritura sabiendo las coordenadas que ocuparia en la matriz:

Código C:

(3x3)
0 1 2
3 4 5
6 7 8
 
P.ej. la posicion 5 = columna 2, fila 1 -> offset = (3 * 1) + 2
 

Espero que te sea de ayuda
Saludos
vosk

[yoel_monsalve]

Como dice vosk, el problema está en la relación matemática que hace corresponder cada par de índices del arreglo bidimensional, en un índice de un arreglo unidimensional.

Si M y N son respectivamente la cantidad de filas y de columnas de una matriz a, mientras v es un vector de M*N elementos, entonces la sentencia:

Código:

for (j=0;j<5;j++)
{
vector[i]=a[i][j];
}

debe ser cambiada por

Código C++:

for (i = 0; i < M; i++) 
    for (j = 0; j < N; j++)
        v[i * M + j] = a[i][j];
 

Es como antes, usando M y N en lugar de H (height) y W (width) respectivamente.
Sin embargo, hay algunos otros detalles erróneos así como sugerencias que quisiera mencionar:

1. Mezclas nombres de variables. El nombre del vector a veces es "vector", y a veces es "v". Debería ser uno o el otro.

2. En el llenado de la matriz:

Código C++:

for (i=0;i<25;i++)
    for (j=0;j<25;j++)
    {
        printf("[%d,%d] = ",i,j);
        scanf("%d",&a[i][j]);
    }
 

los ciclos for deben ser el primero desde 0 hasta uno menos que la cantidad de filas, mientras el segundo debe ser desde cero hasta uno menos que la cantidad de columnas. Es decir, en tu caso, el primero desde 0 hasta 4, y el segundo desde cero hasta 4. Así:

Código C++:

for (i=0; i<M; i++)
    for (j=0; j<N; j++)
    {
        printf("[%d,%d] = ", i, j);
        scanf("%d", &a[i][j]);
    }
 

3. No uses "getch()" ni "conio.h". Hará tu programa no-portable (que no se pueda pasar de una computadora a otra. Estas funciones y bibliotecas son sólo para Windows. Si lo que quieres es que el programa se quede esperando a que se pulse la tecla ENTER, usa [tt]getchar()[/tt] y la bilbioteca "stdio.h"

4. Te falta el tipo de valor de retorno de main(), que debe ser int. Por lo mismo, hay que poner una instrucción return al final de main().

5. Por último, no es un error pero sería recomendable en el caso de dimensiones de la matriz que las mismas pudieran se cambiadas a otros valores. Por eso se aconseja definir M y N (o H y W) mediante constantes #define (también llamadas macros o directivas). Así:

Código:

#define M 5
#define N 5

(sin punto y coma al final). Luego, escribe tu código en función de M y N. Si algún día quieres cambiar el tamaño de la matriz, sólo cambia los valores de estas constantes en el código, y compila de nuevo.

Te dejo un código terminado, y listo para compilar y probar. Recuerda (es un hábito que desarrollarás con el tiempo) ser más prolijo con la indentación o sangría, y colocar espacios entre los símbolos para hacer tu código maś legible. Más o menos así como el que te voy a presentar

Código C++:

#include <stdlib.h>
#include <stdio.h>
 
#define M 5
#define N 5
 
int main() {
    
    int a[M][N], v[M*N], i, j;
 
    /* inicializa vector v con elementos nulos */
    for (i = 0; i < M*N; i++)
    {
        v[i]=0;
    }
 
    /* Ahora pedimos por los elementos de la matriz */
    for (i=0; i<M; i++)
        for (j=0; j<N; j++)
        {
            printf("[%d,%d] = ", i, j);
            scanf("%d", &a[i][j]);
        }
        
    /* y asignamos el contenido al vector 
     * ¡aquí estaba el mayor error! */
    for (i = 0; i < M; i++) 
        for (j = 0; j < N; j++) 
            v[i * M + j] = a[i][j];
 
    /* imprimimos el contenido del vector para verificar */
    for (i = 0; i < M*N; i++)
    {
        printf("%d\n",v[i]);
    }
    
    getchar( );
    return 0;
}
 

[CalgaryCorpus]

Creo que hay que corregir

Código C++:

v[i * M + j] = a[i][j];
 

por

Código C++:

v[i * N + j] = a[i][j];
 

sino. cuando i valga M-1, la expresion i*M + j sera cercano a M^2, lo que está mal.

Por ejemplo si M vale 10 y N vale 2, (un total de 20 distintos enteros) la expresion i*M + j podria llegar a valer 90.

[RaKKoS]

Usa un índice para controlar los valores

[TriN]

Chicos gracias por sus respuestas ya logre guardar la matriz en un vector, realmente me da pena decirlo pero fue suerte, haciendo cambios, lo que hice fue controlar los valores con índice pero no entiendo el código al 100% que es lo que realmente quiero, que me deje esa enseñanza igual les dejo el código funcionando

#include <stdio.h>
#include <conio.h>

main()
{
int m[3][3],v[9],a,i,j;
a=1;

printf("Ingrese los valores de la matriz ");

for (i=0;i<3;i++)
for (j=0;j<3;j++)
{
printf("[%d,%d] = ",i,j);
scanf("%d",&m[i][j]);
v[a]=m[i][j];
a=a+1;

}
for (i=0;i<9;i++)
{

printf("%d ",v[i]);

}

getch();
}

Como pueden observar controle el índice del vector con la variable a, ya que si coloco la variable i para controlar los datos me da erróneo, Saludos y gracias por sus pistas.

[CalgaryCorpus]

Una excelente alternativa. Sin embargo, el índice "a" deberia partir en 0, no en 1.

[vizen]

otra opcion mas. ;-)

Código c:

#include <stdio.h>
 
#define FILAS 2
#define COLUMNAS 2
 
int main(void){
 
   int matriz[FILAS][COLUMNAS], vector[FILAS * COLUMNAS];
 
   for(int i=0, ii=0; FILAS>i; i++)
      for(int j=0; COLUMNAS>j; j++){
         printf("\nintroduce el elemento de la matriz [%i][%i] ",i,j);
         scanf("%i", &matriz[i][j]);
         vector[ii++]=matriz[i][j];
      }
 
   puts("\n\nlos elementos del vector son:");
   for(int i=0, ii=0; FILAS>i; i++){
      putchar('\n');
      for(int j=0; COLUMNAS>j; j++){
         printf("%6i ", vector[ii++]);
 
   }  }
   getchar();
   return 0;
}