Hallo,
komme leider nicht weiter. Problem ist mit Inkluden von Dateien

Fehlermeldung:

PHP-Code:

1>gui.obj : error LNK2005: "int ziffer1" (?ziffer1@@3HA) ist bereits in calc.obj definiert.
1>gui.obj : error LNK2005: "int a" (?a@@3HA) ist bereits in calc.obj definiert. 


Anbei die Dateien:

main.cpp:

PHP-Code:

#pragma once
#include "main.h"

int main()
{
    gui::einlesen();
    calc::calculate(a,b,c,d,operand);
    gui::printout(ziffer1,ziffer2,overflow_char);

} 


main.h:

PHP-Code:

#pragma once
#include <iostream>
#include <string>
#include "calc.h"
#include "gui.h"

using namespace std;
int a,b,c,d;
char operand; 


calc.cpp:

PHP-Code:

#include "main.h"

namespace calc
{
    void calculate(int a, int b, int c, int d, char operand)
    {
        
        if(operand=='+')
        {
            ziffer2=b+d;
            if(ziffer2>10)
            {
                a++;
                ziffer2=ziffer2%10;
            }

            ziffer1=a+c;
            if(ziffer1>10)
                overflow_char='x';
        }
        else
        {
            if(b<d)
            {
                b+=10;
                c++;
            }
            ziffer2=b-d;
            ziffer1=a-c;

        }

    }
} 


calc.h:

PHP-Code:

#pragma once

char overflow_char='\0';
int ziffer1,ziffer2;
namespace calc
{
    void calculate(int a, int b, int c, int d, char operand);
    
} 


gui.cpp:

PHP-Code:

#include "main.h"
namespace gui
{
    void einlesen(void)
    {
        cout<<"Bitte geben Sie die vier Ziffern ein: ";
        cin>>a>>b>>c>>d;
        cout<<"Bitte geben Sie '+' oder '-' ein: ";
        cin>>operand;
    }

    inline void printout(int a,int b, char overflow_char)
    {
        if(overflow_char=='x')
        {
            cout<<"OVERFLOW";
        }
        else
        {
            cout<<"Ergebnis: "<<ziffer1<<ziffer2<<endl;
        }
    }
} 


gui.h:

PHP-Code:

#pragma once

#include "main.h"

namespace gui
{
    void einlesen(void);
    inline void printout(int a,int b, char overflow_char);
} 


Ich danke euch im voraus

Probier mal das du ziffer eins als extern erstellst. Oder vll in einem anderen namespace und dann
MeinNameSpace::ziffer1 = 123;
Habe leuder kein Zeit gehabt den Code vollständig zu lesen werde morgen nochmal drüber gucken
Und bitte verbessert mich falls was falsch ist

Hi.

Man darf in Header Dateien keine Definitionen vornehmen. Definitionen gehören in genau eine Kompilationseinheit (also .cpp Datei).

Außerdem ist es keine gute Idee "using namespace" in Headerdateien zu notieren (Stichwort: namespace pollution).

Weiterhin solltest du dir aussagekräftigere Namen für Variablen überlegen. (Was soll a, b, c, d sein? Ziffern sind Symbole 0 bis 9 (Dezimalsystem))...?)

Und letztendlich sind globale Variablen kein wirklich guter Stil.

Code cpp:

1 2 3 4 5

// main.h   // (Vorwärts-) Deklaration extern int a, b, c, d; extern char operand;

Code cpp:

1 2 3 4 5

// main.cpp   // Definition int a, b, c, d; char operand;

Gruß

PS: Dasselbe gilt natürlich auch für die anderen Variablen die du in der calc.h definiert hast.

danke dir deepthroat. Deine Empfehlung hat mich gerettet.Das Programm läuft nun.

Man könnte auch bei Projektoptionen->Linker->Befehlszeile /FORCE hinzuschreiben.


cwriter

Zitat von cwriter

Man könnte auch bei Projektoptionen->Linker->Befehlszeile /FORCE hinzuschreiben.

Was allerdings nicht wirklich empfehlenswert und nur ein letzter Ausweg ist.

http://msdn.microsoft.com/en-us/library/70abkas3.aspx

Afile created with this option may not run as expected. The linker will not link incrementally when the /FORCE option is specified.

Gruß