[SOLUCIONADO] Problema con CodeBlocks C++

[alanjo]

Muy buenos días, tardes, noches según la hora en la que lean. Paso a comentarles mi situación, estoy realizando un programa simple, el cual extraiga los promedios de ciertos alumnos, el profesor pidió que el programa esté dividido en un main.cpp y otro funcion.cpp donde contenga la resolución al problema de los "promedios". El tema es el siguiente, el programa sólo compilado en MAIN.cpp funciona a la perfección, ahora bien, no puedo compilar ambos .cpp porque me da un error de

C:\Users\XXXXX\Desktop\Programacion PRUEBAS\Promedios\Promedio\funpromedio.cpp|5|multi ple definition of `funcpromedio(int, int, int, int)'| obj\Debug\funpromedio.o:C:\Users\XXXXX\Desktop\Pro gramacion PRUEBAS\Promedios\Promedio\funpromedio.cpp|5|first defined here|
||error: ld returned 1 exit status|

El código del main.cpp es el siguiente:

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


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

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;
}

El código de la función "funpromedio.cpp":

#include "funcpromedio.h"
#include <iostream>
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;

}

El del header, que es más que obvio, pero por si las dudas lo pongo:

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

Si me pueden dar una mano, les agradecería. Muchas gracias gente.

[eferion]

Buenas (Si hablas por un medio en el que el tiempo es algo relativo es mejor dar un saludo atemporal).

Cuando dices que la cabecera es obvia cometes un par de errores:

• Asumes que la cabecera es obvia
• Das por hecho que es sencilla y que está bien hecha.

En C++ no hay cosas obvias y eso es algo que irás aprendiendo.

Un problema con tu cabecera es que no tiene guardas.

¿En qué afecta eso a tu problema? Si miramos el error:

multiple definition of `funcpromedio(int, int, int, int)'

Es decir, el programa está encontrando múltiples definiciones de la función... las guardas sirven precisamente para corregir ese problema.

El programa te da ese error porque al no tener guarda la cabecera ésta puede ser cargada varias veces desde distintas fuentes y eso puede confundir al compilador.

La guarda más conocida (por ser más estándar) es esta, aunque imagino que te sonará:

Código C++:

#ifdef ALGO
#define ALGO
 
// Contenido de la cabecera
 
#endif
 

Luego hay otra guarda que es

Código C++:

#pragma once
 
// Contenido de la cabecera
 

Pero en este caso, al no ser parte del estándar, puedes encontrarte con algun compilador que no la soporte.

Un saludo.

[alanjo]

Iniciado por eferion

Buenas (Si hablas por un medio en el que el tiempo es algo relativo es mejor dar un saludo atemporal).

Cuando dices que la cabecera es obvia cometes un par de errores:

• Asumes que la cabecera es obvia
• Das por hecho que es sencilla y que está bien hecha.

En C++ no hay cosas obvias y eso es algo que irás aprendiendo.

Un problema con tu cabecera es que no tiene guardas.

¿En qué afecta eso a tu problema? Si miramos el error:


Es decir, el programa está encontrando múltiples definiciones de la función... las guardas sirven precisamente para corregir ese problema.

El programa te da ese error porque al no tener guarda la cabecera ésta puede ser cargada varias veces desde distintas fuentes y eso puede confundir al compilador.

La guarda más conocida (por ser más estándar) es esta, aunque imagino que te sonará:

Código C++:

#ifdef ALGO
#define ALGO
 
// Contenido de la cabecera
 
#endif
 

Luego hay otra guarda que es

Código C++:

#pragma once
 
// Contenido de la cabecera
 

Pero en este caso, al no ser parte del estándar, puedes encontrarte con algun compilador que no la soporte.

Un saludo.

Muchísimas gracias por los consejos eferion, tanto por el gramático como por los de C++, la verdad es que soy relativamente nuevo en el tema. Respecto al problema; Cuando lo hago en linux, con el IDE Geany, no establezco las guardas y aún así el resultado es el mismo, ¿Problema? El profesor exige que sea en CodeBlocks o en su defecto VS 2015.
Al poner las guardas, de la siguiente manera:

#ifndef FUNCPROMEDIO_H_INCLUDED
#define FUNCPROMEDIO_H_INCLUDED

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

#endif // FUNCPROMEDIO_H_INCLUDED

Me sigue dando el mismo error a la hora de compilar, la verdad que es un programa sencillo, pero a la hora de compilar, es un problema, y no logró entender cuál. Gracias por tu respuesta. Tenga usted una buena jornada.

[eferion]

Código C++:

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

Para que nos orientemos, es parte de tu fichero main.cpp.

Si resulta que tienes un funpromedio.h que incluye una declaración de funcpromedio... ¿Por qué no usas dicha cabecera en main.cpp? Es una mala práctica incluir ficheros de código y su uso debería vigilarse ya que los resultados pueden no ser buenos como ahora entenderás.

Esto es lo que esperaría encontrarse cualquier programador de C++.

Código C++:

#include <iostream>
#include "funpromedio.h"
 
 using namespace std;
 
int main()
 

¿Cual es el problema? Básicamente que al incluir funpromedio.cpp estabas incluyendo también funpromedio.h (sí, ese include en funpromedio.cpp), y recordemos que funpromedio.h tiene una declaración de la función funcpromedio. Y después de tanto include añades una segunda declaración de la función funcpromedio(la que tienes en tu versión del main.cpp). ¿Resultado? El compilador encuentra dos declaraciones de la misma función y eso no le mola.

Un saludo.

[alanjo]

Muchas gracias por tu tiempo. El resultado sigue siendo el mismo, es decir, incluí el header de funpromedio.h quitando la declaración del main.cpp de funcpromedio, sigo incluyendo a "funpromedio.cpp" debido a que si no lo incluyo, ¿De dónde sacaré la función para resolver el problema? Me vengo partiendo la cabeza con un problema tan simple desde ayer, reitero, muchas gracias por tu tiempo y dedicación. Un saludo

[eferion]

Iniciado por alanjo

sigo incluyendo a "funpromedio.cpp" debido a que si no lo incluyo, ¿De dónde sacaré la función para resolver el problema?

Generar un programa a partir de código fuente, en C++, es un proceso que se genera en dos pasos.

En primer lugar actúa el compilador. Este programita coge tus ficheros de código uno a uno y genera lo que se denomina código objeto. El código objeto es una suerte de código compilado a medias ya que aun falta información, como el tamaño de los saltos. El compilador lo más que necesita es la declaración de funciones y tipos para poder preparar correctamente el código.

Una vez que el compilador ha hecho su trabajo pasa a actuar el linker. Este otro programita coge todos los ficheros de código objeto y los une en un ejecutable o librería. Como en este proceso ya se tiene toda la información se pueden calcular correctamente el valor de los saltos y otros ajustes finales.

Como ves, el compilador procesa unidades individuales de código y es tarea del linker coser dichas unidades para formar un código ejecutable.

Por cierto, normalmente compilador y linker vienen juntos en un único ejecutable. Se elige una u otra opción por línea de comandos.

Dicho con código. Para compilar tu programa se podrían lanzar las siguientes instrucciones

Código BASH:

gcc -c funpromedio.cpp
gcc -c main.cpp
 

Como ves cada fichero de código se compila por separado.

Una vez compilado todo el código se pasa el linker

Código BASH:

gcc -o programa.exe main.o funpromedio.o
 

Y fíjate que ahora sí se van a procesar los dos ficheros.

Moraleja: No necesitas incluir funpromedio.cpp en main.cpp. En el proceso de linkado se fusionarán el código de ambos ficheros y el programa funcionará correctamente.

Un saludo.

[alanjo]

Muchas gracias por tu orientación, ahora funciona perfectamente. Sinceramente, gracias. Marco el tema como solucionado.