Sistema bancario (cajero automático) [C++] [CMD]

[razpeitia]

Uuuuufff no se por donde empezar, aunque las observaciones que haga coinciden con lo que dicen arriba.

1. Tu programa solo funciona en windows, ya que se trata de un cajero, incluso en los bancos reales esto es aceptable. Porque le designan cierto tipo de hardware y software para todos los cajeros (al menos la mayoría del tiempo así es).

2. Tu código es relativamente largo y monolitico, solo tienes 3 funciones definidas de las cuales. 1 es el main y 2 tienen el prefijo menu que parece que hacen todo menos ser un menu. Ademas la función menú es bastante larga, posiblemente la puedas romper en funciones mas pequeñas.

3. Tienes valores hardcodeados, como opcion == 3, WTF!? es 3?, no seria mejor usar enums y que sea mas descriptivo? O string hardcodeados por todos lados, seria imposible de internacionalizar eso.

4. Tu estilo de indentación es ni siquiera es conciso, mucho menos elegante.

5. Manchas tu namespace importando todo lo de std.

6. Usas variables globales. Solo me falto encontrar un goto.