Problema C++ al enlazar

Iniciado por alanjo, 25 Marzo 2016, 00:22 AM

0 Miembros y 2 Visitantes están viendo este tema.

alanjo

Buenas gente, hace días vengo intentando resolver, un problema bastante simple que me dieron al decir verdad, el problema es que me pidieron que a dicho problemita, lo tenía que dividir en 3 ficheros, en el main, en la función y en el header, y luego los tenía que linkear, todo bien, compila perfecto, pero a la hora de linkear "sale todo bien", pero al ejecutar el programa, sólo sale lo del main.cpp y no se lleva a cabo el objetivo del programa el cual es "determinar promedios de alumnos", les dejo el código

funpromedio.h :
Citar#ifndef FUNPROMEDIO_H_INCLUDED
#define FUNPROMEDIO_H_INCLUDED

int funcpromedio(int nota1, int nota2, int nota3, float promedio);

#endif // FUNPROMEDIO_H_INCLUDED

funpromedio.cpp :
Citar#include <iostream>
#include "funpromedio.h"
using namespace std;
int funcpromedio(int nota1, int nota2, int nota3, float promedio)
{

    if(nota1>5)
        cout<<"Aprobado. 1er trimestre"<< endl;

        else if(nota1==5||nota1==4)
            cout<<"Regular, desaprobado, puede recuperar. 1er trimestre"<< endl;

        else
            cout<<"Aplazado. 1er trimestre"<< endl;

    if(nota2>5)
        cout<<"Aprobado. 2do trimestre"<< endl;

        else if(nota2==5||nota2==4)
            cout<<"Regular, desaprobado, puede recuperar. 2do trimestre"<< endl;

        else
            cout<<"Aplazado. 2do trimestre"<< endl;

    if(nota3>5)
        cout<<"Aprobado. 3er trimestre"<< endl;

        else if(nota3==5||nota3==4)
            cout<<"Regular, desaprobado, puede recuperar. 3er trimestre"<< endl;

        else
            cout<<"Aplazado. 3er trimestre"<< endl;

    promedio= (nota1 + nota2 + nota3) / 3;

    if(promedio>5)
        cout<<"Alumno aprobado"<<endl;
    else
        cout<<"Alumno desaprobado"<<endl;
        return 0;

}


main.cpp :
Citar#include <iostream>
#include "funpromedio.h"

using namespace std;


int main()
{
    int nota1;
    int nota2;
    int nota3;

    cout<< "Ingrese la nota del primer trimestre: "<<endl;
    cin>> nota1;
    cout<< "Ingrese la nota del segundo trimestre: "<< endl;
    cin>> nota2;
    cout<< "Ingrese la nota del tercer trimestre: "<< endl;
    cin>> nota3;

    return 0;
}

Gracias de antemano.

Kaxperday

Se ve que te lo has currao.

Sobretodo por la llamada a la función en el main XD.
Cuando el poder económico parasita al político ningún partido ni dictador podrá liberarnos de él. Se reserva el 99% ese poder.

alanjo

Cita de: Kaxperday en 25 Marzo 2016, 00:28 AM
Se ve que te lo has currao.

Sobretodo por la llamada a la función en el main XD.

Jajaja, lo que me dijeron es que la llamada a la función en el main, no era necesaria, porque está incluida en funpromedio.h

Kaxperday

Cita de: alanjo en 25 Marzo 2016, 00:34 AM
Jajaja, lo que me dijeron es que la llamada a la función en el main, no era necesaria, porque está incluida en funpromedio.h

Claro pero si en este caso quieres hacer el promedio deberás de llamarla en el main.
O llamar a otra función que la llame.

Se va a ejecutar lo que pongas en el main si no podes la llamada a la función allí no se ejecutará.

Solucion:

Código (cpp) [Seleccionar]

#include <iostream>
#include "funpromedio.h"

using namespace std;


int main()
{
    int nota1;
    int nota2;
    int nota3;

    cout<< "Ingrese la nota del primer trimestre: "<<endl;
    cin>> nota1;
    cout<< "Ingrese la nota del segundo trimestre: "<< endl;
    cin>> nota2;
    cout<< "Ingrese la nota del tercer trimestre: "<< endl;
    cin>> nota3;
    cout << "El promedio es: " << funcpromedio(nota1, nota2, nota3) << endl;

    return 0;
}
Cuando el poder económico parasita al político ningún partido ni dictador podrá liberarnos de él. Se reserva el 99% ese poder.

alanjo

Cita de: Kaxperday en 25 Marzo 2016, 00:41 AM
Claro pero si en este caso quieres hacer el promedio deberás de llamarla en el main.
O llamar a otra función que la llame.

Se va a ejecutar lo que pongas en el main si no podes la llamada a la función allí no se ejecutará.

Solucion:

Código (cpp) [Seleccionar]

#include <iostream>
#include "funpromedio.h"

using namespace std;


int main()
{
    int nota1;
    int nota2;
    int nota3;

    cout<< "Ingrese la nota del primer trimestre: "<<endl;
    cin>> nota1;
    cout<< "Ingrese la nota del segundo trimestre: "<< endl;
    cin>> nota2;
    cout<< "Ingrese la nota del tercer trimestre: "<< endl;
    cin>> nota3;
    cout << "El promedio es: " << funcpromedio(nota1, nota2, nota3) << endl;

    return 0;
}

Muchas gracias por la pronta respuesta Kaxperday, intenté realizar el llamado a la función que vos describes. Pero tengo el siguiente error:
CitarC:\Users\XXX\Desktop\Programacion PRUEBAS\Promedios\Promedio2\main.cpp|20|error: faltan argumentos para la función 'int funcpromedio(int, int, int, float)'|
C:\Users\XXX\Desktop\Programacion PRUEBAS\Promedios\Promedio2\funpromedio.h|4|nota: se declara aquí|
(Y me redirecciona a funpromedio.h). Muchas gracias

MAFUS

Has puesto como argumento a la función,  el float, una variable que en verdad usas como local. Los argumentos no se deben usar para crear variables locales. Mal, queda aplazada la asignatura que cursas de C.
La llamada a la función espera a que le pases todos los argumentos que espera, el float incluido.
Otro: si haces promedio un float no hay razón por la que la función deba devolver un int.
Más: si todos los argumentos de una división són enteros C la considera una división entera y así guardará el resultado, aunque lo metas en un float.

Kaxperday

Jajaja ostis se me olvido leer el código al completo.

Modifica la función promedio y dejala así:

Código (cpp) [Seleccionar]

float funcpromedio(int nota1, int nota2, int nota3);


Código (cpp) [Seleccionar]

float funcpromedio(int nota1, int nota2, int nota3)
{
   float promedio = 0;
   if(nota1>5)
       cout<<"Aprobado. 1er trimestre"<< endl;

       else if(nota1==5||nota1==4)
           cout<<"Regular, desaprobado, puede recuperar. 1er trimestre"<< endl;

       else
           cout<<"Aplazado. 1er trimestre"<< endl;

   if(nota2>5)
       cout<<"Aprobado. 2do trimestre"<< endl;

       else if(nota2==5||nota2==4)
           cout<<"Regular, desaprobado, puede recuperar. 2do trimestre"<< endl;

       else
           cout<<"Aplazado. 2do trimestre"<< endl;

   if(nota3>5)
       cout<<"Aprobado. 3er trimestre"<< endl;

       else if(nota3==5||nota3==4)
           cout<<"Regular, desaprobado, puede recuperar. 3er trimestre"<< endl;

       else
           cout<<"Aplazado. 3er trimestre"<< endl;

   promedio= (nota1 + nota2 + nota3) / 3;

   if(promedio>5)
       cout<<"Alumno aprobado"<<endl;
   else
       cout<<"Alumno desaprobado"<<endl;

       return promedio;

}


De esa manera cuando le pases la nota de los cuatrimestres te devolvera la media, y solo tienes que mostrarla por pantalla, suerte XD.
Cuando el poder económico parasita al político ningún partido ni dictador podrá liberarnos de él. Se reserva el 99% ese poder.

alanjo

Muchas gracias a ambos, a Mafus por la explicación teórica y a Kaxperday por su explicación práctica. Muchas gracias por su amabilidad, me funcionó. Es bueno saber que hay gente que dedica su tiempo a alguien, tengan ustedes una muy buena jornada.