Ejercicio compila pero no se ejecuta

[leosansan]

Funciona!!! Muchísimas gracias.
Aunque no comprendo muy bien lo de punteros a punteros, pero eso es cuestión de estudiarlo mejor.

Ya que me lo dijiste, ¿es mejor hacer

Código [Seleccionar] Expandir

int notasEnteras[alumnos]; como un array dinámico? Sólo lo uso para almacenar enteros. (perdona mi ignorancia)

Un saludo

Es otra alternativa a la que puede ponerse el pero de que algún compilador no la acepte. En general la uso para matrices normalitas. Si el tamaño es importante uso asignación dinámica de memoria.

Salu2!.

[David8]

Es otra alternativa a la que puede ponerse el pero de que algún compilador no la acepte. En general la uso para matrices normalitas. Si el tamaño es importante uso asignación dinámica de memoria.

Salu2!.

Una última cosa, el compilador me da (en modo estricto) el siguiente warning:

54 9 [Warning] 'notas' is used uninitialized in this function [-Wuninitialized]

Respecto a:

Código [Seleccionar] Expandir

char **notas;
*notas = malloc(alumnos * sizeof (char));

Un saludo.

[amchacon]

Cambia:

Código (cpp) [Seleccionar] Expandir

*notas = malloc(alumnos * sizeof (char));

Por:

Código (cpp) [Seleccionar] Expandir

notas = malloc(alumnos * sizeof (char));

[David8]

Cambia:

Código (cpp) [Seleccionar] Expandir

*notas = malloc(alumnos * sizeof (char));

Por:

Código (cpp) [Seleccionar] Expandir

notas = malloc(alumnos * sizeof (char));

Una última cosa y ya doy por solucionada la duda.
Para liberar la memoria que uso cuando reservo con malloc tendré que usar free así no?

Código [Seleccionar] Expandir

free(notas[i]);
después del

Código [Seleccionar] Expandir

while(x == 0);
O uso

Código [Seleccionar] Expandir

free(notas) solamente?

Un saludo.

[rir3760]

Como ya te comentaron el problema principal se debe a que tratas de almacenar todas las lineas cuando lo único que haces con ellas es obtener (mediante atoi) un entero. Es mejor declarar un solo array de caracteres (ya lo haces, el array "nAlumnos") y reutilizar este cada vez que debas leer una linea de la entrada estándar.

Si con todo quieres utilizar memoria dinámica debes reservar primero la memoria del bloque principal y a continuación la memoria para cada linea. Antes de terminar el programa liberas la memoria de cada linea y por ultimo la del bloque principal. De esta forma:

Código (c) [Seleccionar] Expandir

/* ... */

#define TAMANIO  10

/* ... */

char nAlumnos[TAMANIO];
int alumnos;
int longitud;
int x;
int i;
char **notas;

/* ... */

/* 1) Obtienes el numero de alumnos */

/* 2) Reservas el bloque de memoria principal */
notas = malloc(alumnos * sizeof *notas);

/* 3) Reservas el bloque de memoria para cada alumno */
for (i = 0; i < alumnos; i++)
   notas[i] = malloc(TAMANIO);

/* 4) Realizas las operaciones del programa */

/* 5) Liberas la memoria utilizada por cada alumno */
for (i = 0; i < alumnos; i++)
   free(notas[i]);

/* 6) Liberas el bloque principal */
free(notas);

Un saludo

[David8]

Como ya te comentaron el problema principal se debe a que tratas de almacenar todas las lineas cuando lo único que haces con ellas es obtener (mediante atoi) un entero. Es mejor declarar un solo array de caracteres (ya lo haces, el array "nAlumnos") y reutilizar este cada vez que debas leer una linea de la entrada estándar.

Si con todo quieres utilizar memoria dinámica debes reservar primero la memoria del bloque principal y a continuación la memoria para cada linea. Antes de terminar el programa liberas la memoria de cada linea y por ultimo la del bloque principal. De esta forma:

Código (c) [Seleccionar] Expandir

/* ... */

#define TAMANIO  10

/* ... */

char nAlumnos[TAMANIO];
int alumnos;
int longitud;
int x;
int i;
char **notas;

/* ... */

/* 1) Obtienes el numero de alumnos */

/* 2) Reservas el bloque de memoria principal */
notas = malloc(alumnos * sizeof *notas);

/* 3) Reservas el bloque de memoria para cada alumno */
for (i = 0; i < alumnos; i++)
   notas[i] = malloc(TAMANIO);

/* 4) Realizas las operaciones del programa */

/* 5) Liberas la memoria utilizada por cada alumno */
for (i = 0; i < alumnos; i++)
   free(notas[i]);

/* 6) Liberas el bloque principal */
free(notas);

Un saludo

Lo voy pillando poco a poco.
Muchas gracias por curraros tanto las respuestas

Un saludo.