Problema programa en C

hm... hay ke ir a clase EnDeESgue.

#3 lo primero de todo thx por contestar

luego.., el problema esta en ke la dimension (es decir, el valor de columnas y el valor de filas) lo pasa el usuario por teclado a la hora de crear la matriz y yo tenia entendido ke el define era para declarar constantes..

Si consigues que me salga te hago un altar xD

Edit: Aparte de todo esto, que compilador es el mejor para trabajar en C?(En Windows)

Yo uso Dev-Cpp (gcc), pero me ha dado problemas "raros" con la asignación dinámica (en concreto con la función realloc).

El mismo código que compilado con GCC daba errores de memoria, con LCC-WIN iba perfectamente.

Salu2

Vamos a ver como lo hacemos

Primero, en tu código, falta un typedef, antes del struct matriz, y no puedes poner el mismo nombre arriba y abajo, me explico

struct matriz {
int columnas;
int filas;
float contenido[columnas][filas];
}matriz;

typedef struct tag_matriz {
int columnas;
int filas;
float contenido[columnas][filas];
} matriz;

Segundo, no puedes poner asi las columnas y las filas para la variable contenido. Puesto que son numeros que no conoces de antemano y pueden ser variables, tendrás que hacerlo con memoria dinámica. Segun tu mensaje, parece que el numero de matrices es también variable, en ese caso lo mismo te digo con la estructura "conjuntomatrices". Te recomiendo un cambio radical, aunque requiere saber usar punteros

Si no os exigen mucha calidad de código, podeis usar máximos, y "moveros" por la matriz usando los dos enteros "columnas y filas". Si es el 1º programa, no creo que os digan nada, voy a ver si me explico como:

#define MAX_FILAS 50 //Aqui cualquier numero
#define MAX_COLS 50 //lo mismo
#define MAX_MATRICES 20 //idem

typedef struct tag_matriz {
int filas, columnas;
float contenido[MAX_FILAS][MAX_COLS];
} tipoMatriz;

typedef struct tag_conjunto {
int num_matrices;
tipoMatriz matriz[MAX_MATRICES];
}

Esto creo que deberia solucionaros muuucho trabajo. De todas formas, si os hacen tener la memoria dinámica, avisa, y pongo algo mas mejor. Saludos

Saludos a todos de nuevo. Nuestros objetivos principales para el proyecto son:

• Creacion de una estructura matriz de tamaño variable.
• Creacion de una estructura conjuto de matrices que contendra un array de tamaño variable que contendra todas las matrices i el numero total que tenemos.

Hemos buscado informacion sobre las estructuras dinamicas y hemos encontrado un codigo para la creacion de una matriz con su funcion y demas. Nuestra estructura conjunto matrices sigue estando mal pero aqui os adjuntamos el codigo primordial haber si os situamos mejor y recibimos mejor vuestra ayuda.

Estructuras

struct mat {
float *datos;
int columnas,filas;
};

typedef struct mat *MATRIZ;

struct conjuntomatrices {
int n_items;
MATRIZ lista[30];
}conjuntom;

conjuntom.n_items = 0;

Funciones

MATRIZ creamatriz (int x,int y) {
MATRIZ temp;

temp = (MATRIZ) malloc (sizeof(struct mat));
temp->columnas = x;
temp->filas = y;
temp->datos = (float ) malloc (sizeof(float)x*y);
return temp;
}

void agregarmatriz(MATRIZ nueva) {
conjuntomatrices.lista[conjuntomatrices.n_items] = nueva;
conjuntomatrices.n_items++;
return;

}

Programa Principal

int fil=0, colum=0;
MATRIZ a;

printf("Introduce el numero de columnas: ");
colum = scanf("%i", &colum);
printf("Introduce el numero de filas: ");
fil = scanf("%i", &fil);
a = creamatriz(colum,fil);
agregarmatriz(a);

Como podeis ver en el programa principal queremos crear una funcion que pase una matriz a un metodo con agregarmatriz y se añada al array dinamico lista[] de tipo matriz de la estructura conjuntomatrices. N_items nos dira el numero de matrices que tenemos en el array.

Gracias por vuestros aportes, esto es lo basico que nuestro programa tendria que hacer, ya que luego tenemos que rellenar las matrices y hacer operaciones con ellas.

Seguimos mirando la funcion de agregarmatriz que esta como el culo. Cualquier informacion que nos faciliteis nos sirve de mucho.

Gracias a todos por adelantado

#11 tiene ke ser C xD

Para simplificar, hemos decicido ke vamos a poner un maximo de matrices posibles para introducir, asi ke usaremos un array y punto porq si nos tenemos ke poner con listas ya si ke la liamos parda xD

Saludos

Con C++ es más guay. ¡Vivan los constructores y destructores!

Buenas soy el compi de EndeEsgue, su ultimo post le pase yo todo dado que estaba registrado desde hace poco y no podia escribir hasta superar las 12 horas jeje.

Agradezco todas las aportaciones que nos estais brindando desinteresadamente por amor al C jeje.

• Decir que ara mismo estamos probando el codigo de Loco y adaptandolo a nuestro programa.

• javithelong, gracias por intentar optimizar el codigo de Loco pero podrias escribir el codigo igual que ha hecho el?

Hemos trabajado mucho con java a nivel de clases y metodos pero no con C y nos cuesta entender aveces lo que quereis exponer y mejorar, por eso si lo haceis mediante codigo podemos probarlo y entenderos mejor a la vez.

Gracias y feliz 2006 a todos!!!

Ante todo dejar claro que no critico a nadie, cada uno es libre de hacer lo que quiera con su tiempo libre, solo estoy dando una opinión.

Me parece fatal la postura de que estáis manteniendo pidiendo que os escriban el código, sobre todo en #16 donde directamente le dices a otro que por que no te escribe algo de código en lugar de solo hablar.

Una cosa es escribir algo de código para aclarar una idea, y otra pedir que os hagan vuestro trabajo... Creo que con la información que se os ha dado tenéis de sobra para trabajar vosotros mismos, y pienso que es lo que deberías de hacer, ahora bien, si preferis pedir que os lo hagan y alguien está dispuesto a haceroslo, pues como ya dije, cada uno es libre de gastar su tiempo en lo que quiera.

No no, no me ha ofendido ni nada, precisamente intente dejar claro en el post que no tengo nada en contra de lo que estáis haciendo, simplemente era una opinión nada más.

Como ya dije antes, solo con aprender a utilizar malloc() y free() tenéis el camino abierto para resolver esto vosotros mismos. No me molesta para nada que hagáis estas preguntas, pero sinceramente creo que sacariais más provecho aprendiendo a utilizar malloc() y free() vosotros mismos y preguntando las dudas que os surjan, que directamente pidiendo partes de código ya hechas.

Por cierto, veo en vuestro código que cada vez que usáis malloc() haceis un cast:

temp = b[/b] malloc (sizeof(struct mat));

temp->datos = (float ) malloc (sizeof(float)x*y);

malloc() devuelve un puntero de tipo void, y se puede asingar directamente a cualquier puntero, no es necesario hacer una conversión de tipos (en C++ sí es necesario, en C no).

Releido el ejemplo editado, parece que ya va todo bien (tampoco lo he probado, solo lo he leido por encima )

Un par de comentarios na mas, nada importante :

A los del problema, estoy con Cabron, no es bueno que os den código ya hecho, y que lo adapteis a vuestro problema, es mejor que lo vayais pariendo vosotros. La programación no se aprende leyendo programas dabuti hechos por otros, muy a mi pesar; aqui se aprende currando, y dejandose los cuernos contra el monitor para encontrar los putos fallos.

Cabron, emplear castings en el malloc es una práctica bastante útil, obviamente no es necesaria (ningún casting es necesario, pues en realidad un casting no es nada), pero, primero, te quitas warnings, y segundo, ayuda a la depuracion del código (hablo de C, no se nada de C++, igual en C++ los castings si hacen algo)

Loco, ya vi que pasabas la matriz como referencia, decia, que no hace falta que la pases como referencia, puedes pasarla directamente por valor, pues es una estructura, y podrás modificar el valor de sus campos (vamos, me parece ^^) si pones matriz.datos en lugar de matriz->datos, todo irá guay (ya te dije, lo que tu hiciste esta bien)

Conio no es ansi, y deberias aprender a no usarla (todo lo no-ansi, se puede hacer con código ansi)

No se me ocurre que mas poner, asi que me voy a dormir, que ya son horas... si es que tienen razón, los informáticos somos como las putas... trabajamos en horarios extraños, y lo hacemos gratis a parientes y amigos ^^. Saludos

La función rand() de stdlib.h os valdría para eso (Tb os hace falta incluir time.h para la semilla).

float aleatorio;

srand(time(NULL)); //Normalmente se usa como semilla el timestamp

aleatorio=((float)rand())/100; //Esto te genera un float pseudo-aleatorio

Salu2

#25

time(NULL) varía cada segundo, que normalmente es suficiente... Busca en Google otras semillas para la función rand()

En cuanto a lo que comentas es cierto. Me he puesto a debuggear y ya sé dónde falla pero no entiendo por qué. Voy a mirarlo un ratillo más hasta la cena. Ahora te cuento.

EDITADO: NO entiendo por qué falla, lo siento

Salu2

Saludos de nuevo. Llevamos viento en popa esta practica sobre el 70 % o algo mas y todo es gracias a vosotros gente!

Una nueva cuestion a surguido en nuestro programa. Estamos intentando simplemente destruir una matriz que elijamos con este codigo un poco variado de Loco pero el mismo vaya.

printf("Total de matrices: %i\n\n",conjunto.n_items);
printf("Introduce el num. de la matriz que quieres:");
scanf("%i",&elec);

for(conta_columnas=0; conta_columnas<conjunto.lista[elec-1].columnas; conta_columnas++)
free(conjunto.lista[elec-1].datos[conta_columnas]);

           free(conjunto.lista[elec-1].datos);

Nuestra duda surge en que luego hacemos el listado de las matrices y nos aparecen datos sin liberarse. Creo entender por dato liberado el que su valor sea un 0.0

Por ejemplo normalmente si creamos una matriz 3x3 con el codigo de arriba solo liberamos la primera columna. Con otra matriz se libera solo las dos primeras filas de 3 que tiene, cosas asi mu raras jeje

Seguro que se liberan todos los datos correctamente de una matriz con ese codigo?

Gracias de nuevo!!

Buenos dias a toda nuestra comunidad de C
He probado tus nuevos cambios y ya no peta si entras una matriz de:
3filas x 2 columnas, 4 filas x 3 columnas... increible gracias!!!

No obstante el tema de la liberacion nos sigue pasando que una vez destruida una matriz o como tu mismo lo as hecho con el bucle de destruirlas todas, sin liberar el array final, pues siguen quedando valores que no son 0.0 al hacer un listado de lo que hay. Nu sabemos que puede ser.

PD: Seria un honor que te pudieramos enviar por pm nuestro codigo despues de tanta ayuda que nos estas dando una vez acabado todo claro ^_