Mensajes

Supongo que aun no aprendes sobre structs o clases? hacer funciones?

Voy a suponer que no y no comentare al respecto.

La variable multProductString parece superflua, la asignas, pero no la usas mas. Sugiero eliminarla.

La variable totalProduct guarda nombres, no totales. Cambiale el nombre para que sea mas facil de leer/entender el codigo.

El ciclo que tienes en la linea 143 lo haces haya o no haya dinero. Esta bien eso?
Supongo que solo quieres aceptar la venta si hay dinero suficiente, no en todos los casos.

Si lees un string que tiene X caracteres visibles tienes que reservar 1 espacio mas para poder guardar la marca de fin de string, o sea el tamano tiene que ser X+1, y si al momento de leer, lees X caracteres, asegurate de poner el 0 en la posicion final del arreglo.

Si no tienes intenciones de imprimir y solo guardas y no usas ninguna funcion de strings, no es necesario este espacio adicional, pero dado lo que te ocurre, tengo la impresion que tu si necesitas ese espacio adicional.

Tienes los datos en un archivo, los pasas a un arreglo, lo ordenas en memoria, no en disco.
No tiene sentido desplazarte en el disco con fseek si los datos ya los tienes en memoria ordenados.

Los datos en disco no se mueven de ninguna manera, tengo la impresion que crees que cuando intercambias en el arreglo, estas intercambiando en el archivo y por eso te mueves al final o al inicio para recuperar el menor o mayor, cuando en realidad deberias preguntarle directamente al arreglo por esos valores (si es que esta bien la ordenacion que hiciste).

En C++ el tipo string tiene un constructor que acepta un size_t y un char, y, al crear el string, copia el char tantas veces como diga el size_t

Un truco sucio es multiplicar el valor inicial por 10, tomar la parte entera y hacer toda la logica con numeros enteros entre 0 y 100.

Una alternativa es usar la aproximacion de Newton:

https://math.stackexchange.com/questions/575281/how-to-calculate-square-root-or-cube-root

Al menos inicializa las variables locales suma y sigma a 0.

El codigo parece tener varias cosas que se pueden remover.

La funcion iniciar_arreglo_alumnos ademas de no servir el proposito, no parece ser necesaria. Eliminala, reemplaza su invocacion por la peticion explicita de memoria.

Las variables prom y matr en el main no tienen sentido definirlas alli, pues cambian su valor, pero no se usan al interior de la funcion. Definelas al interior de la funcion imprimir.

Pareciera mejor tener otra funcion que asigne esos valores y que la funcion imprimir solo muestre estos valores. No se ve bien que una funcion encargada de imprimir asigne los valores tambien.

Las funciones aleatorio20, matricula y promedio10 dicen retornar int, pero no retornan nada. Podrias en vez de modificar el parametro que reciben, borrar el parametro que reciben, que no reciban nada, y hacer return del valor que calculan. Con esto incluso no vas a necesitar las variables locales prom y matr, puedes asignar lo retornado directamente a los campos (lineas 47 y siguientes).

JUnit se da cuenta de las excepciones que uno no captura y hace fallar los tests unitarios.
De modo que si el test espera que no hayan excepciones, sino que se devuelva un valor, es eso lo que deberia ponerse en el test unitario.

Y si por alguna razon el test falla, (porque se arrojo una excepcion para ese caso), hay que corregir el codigo, no el test.

Agregarle un fail() en el caso de excepcion no agrega nada, pues JUnit ya te hubiera avisado de la falla si hubieses usado el assert que en teoria tienes que usar.

Define una struct/clase
Devuelve un array de objetos de ese tipo.