Ver Mensaje Individual
  #2 (permalink)  
Antiguo 30/11/2014, 09:54
Avatar de marlanga
marlanga
 
Fecha de Ingreso: enero-2011
Ubicación: Murcia
Mensajes: 1.024
Antigüedad: 13 años, 10 meses
Puntos: 206
Respuesta: Programa maquina expendedora

Tienes un montón de funciones iguales. ¿Por qué no les pones dos argumentos, moneda y cantidad, en vez de sólo cantidad?

La función que hace el cálculo es un do-while. ¿Es realmente necesario un bucle para calcular cuántas monedas de un tipo dado puedes devolver? Párate a pensar.

$precio y $dinero te los puedes ahorrar. Pregunta directamente $cambio; hacer la resta es innecesario para el problema.

En cuanto al uso de tus funciones, el trozo final de tu código donde usas $cambio, ahí sí es recomendable usar un bucle, mientras quede cambio que devolver.