0

Tengo un struct que me representa un numero fraccionario y la funcion crear:

typedef struct{
    int num;
    int deno;
}Racional;

Racional* crear(int, int);

Quiero mostrar dos numeros fraccionarios en consola, al asignarle los respectivos valores con la función crear el segundo numero fraccionario toma otros valores.

int main(){
    Racional *a, *b;
    int num, deno;

    cout<<"Programa que realiza operaciones basicas con racionales"<<endl<<endl;

    cout<<"Ingrese el valor del numerador del racional 1: ";
    cin>>num;
    cout<<"\nIngrese el valor denominador del racional 1: ";
    cin>>deno;
    a = crear(num, deno);
    cout<<"\nIngrese el valor del numerador del racional 2: ";
    cin>>num;
    cout<<"\nIngrese el valor denominador del racional 2: ";
    cin>>deno;
    b = crear(num, deno);

    cout<<"El racional 1 es: "<<a->num<<"/"<<a->deno<<endl; 
    cout<<"El racional 2 es: "<<b->num<<"/"<<b->deno<<endl;
}

Racional* crear(int num, int deno){
    Racional *a = (Racional*)malloc(sizeof(Racional));

    a->num = num;
    a->deno = deno;

    return a;
}

introducir la descripción de la imagen aquí

9
  • En C++ no debes usar malloc, en vez de ello use new: Racional *a = new Racional;
    – user22539
    Commented el 18 dic. 2019 a las 5:57
  • 1
    Esos resultados de donde salen? porque en tu código no está la impresión de esas líneas que muestras.
    – SuperG280
    Commented el 18 dic. 2019 a las 7:21
  • @eyllanesc Ya he probado usan el "new" y obtengo el mismo resultado.
    – Alexander
    Commented el 18 dic. 2019 a las 13:48
  • @SuperG280, ya he añadido la parte de la impresión pero no veo error alguno ahí.
    – Alexander
    Commented el 18 dic. 2019 a las 13:49
  • ¿Qué compilador estás usando?
    – eferion
    Commented el 18 dic. 2019 a las 13:51

1 respuesta 1

2

Tengo un struct que me representa un numero fraccionario y la funcion crear:

typedef struct{
    int num;
    int deno;
}Racional;

Racional* crear(int, int);

Mal comienzo. Dado que has etiquetado la pregunta como y estás usando utilidades de dicho lenguaje (como los flujos de entrada/salida de datos desde consola std::cin/std::cout) haz que tu código se ajuste a dicho lenguaje y no a su predecesor :

En C++:

  • Las estructuras son tipos de primer nivel, no necesitan una definición de tipo (typedef).
  • Las estructuras pueden tener constructores, no necesitan una función de creación. Pero como todos los miembros son públicos (comportamiento por defecto de las estructuras) podemos usar la inicialización de agregado.
  • Se evita el uso innecesario de punteros en crudo.

Por lo tanto, tu estructura Racional podría tener este aspecto:

struct Racional {
    int num{};
    int deno{};
};

Las llaves al final de las variables miembro hacen que el valor se inicialice a cero. Para crear un Racional cualquiera de estas opciones es válida:

Racional mitad{1, 2};         // ½
auto tercio = Racional{1, 3}; // ⅓

Como ves, no hay necesidad de una función crear y tanto mitad como tercio son instancias, no punteros.

Quiero mostrar dos numeros fraccionarios en consola.

En C++, la manera habitual de hacerlo es sobrecargando el operador de inyección en flujo de datos:

std::ostream &operator<<(std::ostream &o, const Racional &r) {
    return o << r.num << '/' << r.deno;
}

Con esta sobrecarga del operador, puedes mostrar un Racional directamente:

cout << "El racional 1 es: " << a << endl; 
cout << "El racional 2 es: " << b << endl;

Para leer desde consola, puedes sobrecargar el operador de extracción desde flujo de datos:

std::istream &operator>>(std::istream &i, Racional &r) {
    i >> r.num;
    i >> r.deno;
    return i;
}

Con estos cambios, tu código podría parecerse a:

int main()
{
    Racional a, b;

    std::cout << "Programa que realiza operaciones basicas con racionales\n\n";

    std::cout << "Ingrese el racional 1: ";
    std::cin >> a;

    std::cout << "\nIngrese el racional 2: ";
    std::cin >> b;

    std::cout
        << "El racional 1 es: " << a << '\n'
        << "El racional 2 es: " << b << '\n';

    return 0;
}

Puedes ver el código funcionando en Try it online!.


Esto no soluciona tu problema, porque el código que has mostrado no reproduce el problema que describes, deberías aportar un ejemplo mínimo verificable de tu problema, no un código que no reproduzca el problema que dices tener; hasta entonces tendrás que conformarte con esta respuesta.

Sin embargo, el código mostrado no tiene el problema que describes y hace que tener problemas como el que describes sea muy difícil.

5
  • No veo cómo esto corrige el error que tiene.. introduce 2 y 3 e imprime 4 y 4 ... vale que el error no aparece en la pregunta... pero esta respuesta dificilmente va a solucionar el problema... otro tema aparte es que tengas o no razón (que va a ser que si) en las cosas que comentas... pero no solucionan el problema
    – eferion
    Commented el 18 dic. 2019 a las 15:08
  • No lo corrige. Su código no se comporta como dice que se comporta. Commented el 18 dic. 2019 a las 15:20
  • 1
    entonces no hay error... si no hay error hay que cerrar la pregunta... no responderla 😜
    – eferion
    Commented el 18 dic. 2019 a las 15:21
  • Tienes razón :O pero no podía dejar que un código tan malo quedase sin su contraparte corregida >_< Commented el 18 dic. 2019 a las 15:23
  • ya, eso también lo comparto
    – eferion
    Commented el 18 dic. 2019 a las 15:25

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.