¿Está bien hecho? (Primer programa)

[S3kh]

 Buenas noches. Intenté hacer un pequeeño e inútil programa, el cuál se le da un numero, un rango, un valor de incremento y un valor de decremento, y el dá un resultado. Por ejemplo:

Numero: 10
Rango: 7
Valor decremento: 0.7
Valor incremento: 3

Es decir, mostrará 7 valores anteriores descontando de a 0.7 y 7 valores posteriores incrementando de a 3:

9.3 8.6 7.9 7.2 6.5 5.8 5.1 ... Y por otro lado mostrará 13 16 19 22 25 28 31

El programa funciona, pero quisiera saber si cometí algún error, o algún detalles, alguna sugerencia o lo que fuere Muchas gracias de antemano!

Código (cpp) [Seleccionar] Expandir


#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
using namespace std;
//Prototipos
int Menu_Principal();
//MAIN
int main() {
    Menu_Principal();
    getch();
return 0;
}
//Principal
int Menu_Principal() {
  double Numero, decVal, incVal;
  int Rango, Contador, Opcion;
  Contador = 1;
    cout << "Numero: "; cin >> Numero;
    cout << "Rango: "; cin >> Rango;
    cout << "Decrementar en: "; cin >> decVal;
    cout << "Incrementar en: "; cin >> incVal;
    cout << endl;
        //////////////////////////
        //Anteriores             /
        //////////////////////////
        cout << "ANTERIORES A " << Numero << endl;
        while (Contador <= Rango) {
            cout << Numero - (Contador * decVal) << " ";
            Contador += 1;
        }
        //////////////////////////
        //Posteriores            /
        //////////////////////////
        cout << endl << endl << "POSTERIORES A " << Numero << endl;
        Contador = 1;
        while (Contador <= Rango) {
            cout << Numero + (Contador * incVal) << " ";
            Contador += 1;
        }
    cout << endl << endl;
    cout << "1- Salir" << endl;
    cout << "2- Otro numero" << endl;
    cin >> Opcion;
    if (Opcion == 1) exit(0);
    else Menu_Principal();
return 0;
}


[0xDani]

¿Por que incluyes conio.h?

[maxim_o]

Lo incluye por el getch()
Aunque esa biblioteca no es recomendable... Ya que no es estandar...
Usa mejor cin.get()

[0xDani]

Lo incluye por el getch()
Aunque esa biblioteca no es recomendable... Ya que no es estandar...
Usa mejor cin.get()

+1, eso era lo proximo que le iba a decir. No se porque la gente incluye librerias por costumbre, en vez de por necesidad. Ah, y por lo demas esta bastante bien.

[satu]

Hola

Por otro lado las bibliotecas de C++ son cstdlib y cstdio, (stdlib.h y stdio.h son de C )aunque utilizando cin y cout no se para qué quiere stdio

Además yo cambiaría la recursividad de Menu_Principal por un bucle

Saludos

[S3kh]

 No conocía ni cin.get, ni nada D: Y lo de la función Menu_Principal era porque tenía dudas con funciones y me quería sacar dudas xDD. No sabía eso de que las librerías de C++ eran otras ni nada xD Pero ya aprendí ahora D: Se nota que soy demasiado nuevo D: Gracias por eso de cin.get, y contarme lo de las .h