Kleine Frage (wahrscheinlich nur ein kleiner Fehler)wichtig!

Crackheady · 13. November 2006, 17:43

Ich bin noch Schüler und habe erst seit 3 Wochen angefangen mim Coden...
Morgen muss ich für den Technik unterricht ein kleines Programm abgeben.
Hier der Code

Quellcode

#include <iostream.h>
#include <iomanip.h>
int main ()
{
float Stundenanzahl, Stundenlohn, Ueberstundengehalt;
cout.setf(ios::fixed);
cout.precision(2);
cout <<"Gearbeite Stunden: ";
cin >>Stundenanzahl;
cout <<"Stundenlohn: ";
cin >>Stundenlohn;
float Angestellt, Monatslohn;
cout <<"\nFuer Auszubildender die 0, Fuer Angestellter die 1 : "<<endl;
cin >>Angestellt;
if (Angestellt == 1)
{ float Fahrkostenpauschale;
Fahrkostenpauschale = 200;}
else
{ float Fahrkostenpauschale = 150; }
if (Stundenanzahl > 160)
{
float Ueberstunden = Stundenanzahl - 160;
float Ueberstundengehalt = Ueberstunden * Stundenlohn / 2;
float Monatslohn = Stundenlohn * 160 + Ueberstundengehalt;
}
else
{
float Monatslohn = Stundenlohn * Stundenanzahl;
float Ueberstundengehalt = 0;
}
float Endgeld = Monatslohn + Fahrkostenpauschale;
cout <<"Monatslohn"<<setw(24)<<"Ueberstundengehalt"<<setw(24)<<"Fahrkostenpauschale"<<endl;
cout <<Monatslohn<<setw(24)<<Ueberstundengehalt<<setw(24)<<Fahrkostenpauschale<<endl;
cout <<"**********************************************************"<<endl;
cout <<"Der Monatsverdiest beträgt "<<Endgeld<<" Euro";
cout <<"\n\n";
return 0;}

Alles anzeigen

Wenn ich compilen will kommt der error:
H:\non musik\TW\msvcpp\Arbeitslohn\main.cpp(47) : error C2065: 'Fahrkostenpauschale' : nichtdeklarierter Bezeichner

Also ein Problem mit der Variable, ich hab die halt nur in der If schleife definiert, aber anders gehts nicht.

Das Program soll den Monatsgehalt erechnen, aus der anzhal der Arbeitsstunden und dem Stundenlohn. Außerden sollen die Überstunden (ab 160 Stunden sinds Überstunden) mal der hälfte des Stundenlohns zum Monatsgehalt dazugerechnet werden. Außerdem sollen wir noch die Fahrkostenpauschale einbauen, für Azubis 160€ für Angestellte 200€.

wäre nett wenn mir jemand schnell sagen kann was genau ich falsch gemacht habe

edit:
benutzte MS Visual C++ 6.0 Pro

mfg Micha

Torben Brodt · 13. November 2006, 17:54

füge die Fahrkostenpauschale doch einfach zu Stundenanzahl, Stundenlohn und Ueberstundengehalt hinzu.

variablen deklariert man immer am anfang (beim guten, alten ANSI C war das noch pflicht)

cewbie · 13. November 2006, 17:55

warum sollte das nicht anders gehen? du musst die Variable oben einem Datentyp zuweisen...

Crackheady · 13. November 2006, 19:58

ok habs irgendwie hinbekommen...
habe alle Variablen am Anfang definiert.
Dann habe ich das float vor den andren Definitionen weggenommen (wusste garnet das das geht)
und dann gings

*freu*

damit hat sich die Sache erledigt.

thx!

Torben Brodt · 13. November 2006, 20:11

ohne mir deinen code im detail anzuschauen oder gar den compiler anzuwerfen, hab ich einfach mal alle floats nach oben gezogen

Quellcode

int main ()
{
float Stundenanzahl, Stundenlohn, Ueberstundengehalt, Fahrkostenpauschale, Ueberstunden, Ueberstundengehalt, Monatslohn, Angestellt, Endgeld;
cout.setf(ios::fixed);
cout.precision(2);
cout <<"Gearbeite Stunden: ";
cin >> Stundenanzahl;
cout <<"Stundenlohn: ";
cin >> Stundenlohn;
cout <<"\nFuer Auszubildender die 0, Fuer Angestellter die 1 : "<<endl;
cin >> Angestellt;
if (Angestellt == 1) {
Fahrkostenpauschale = 200;
} else {
Fahrkostenpauschale = 150;
}
if (Stundenanzahl > 160) {
Ueberstunden = Stundenanzahl - 160;
Ueberstundengehalt = Ueberstunden * Stundenlohn / 2;
Monatslohn = Stundenlohn * 160 + Ueberstundengehalt;
} else {
Monatslohn = Stundenlohn * Stundenanzahl;
Ueberstundengehalt = 0;
}
Endgeld = Monatslohn + Fahrkostenpauschale;
cout <<"Monatslohn"<<setw(24)<<"Ueberstundengehalt"<<setw(24)<<"Fahrkostenpauschale"<<endl;
cout << Monatslohn <<setw(24)<< Ueberstundengehalt<< setw(24)<< Fahrkostenpauschale<< endl;
cout <<"**********************************************************"<<endl;
cout <<"Der Monatsverdiest beträgt "<<Endgeld<<" Euro";
cout <<"\n\n";
return 0;
}

Alles anzeigen

void · 13. November 2006, 20:16

Hi,
für Angestellt solltest du den Datentyp int anstelle von float nehmen.
Vergleiche mit Gleitkommazahlen sind,aufgrund ihrer internen Darstellung, problematisch.

Quellcode

if (Stundenanzahl > 160)
{
float Ueberstunden = Stundenanzahl - 160;
float Ueberstundengehalt = Ueberstunden * Stundenlohn / 2;
float Monatslohn = Stundenlohn * 160 + Ueberstundengehalt;
}
else
{
float Monatslohn = Stundenlohn * Stundenanzahl;
float Ueberstundengehalt = 0;
}

Alles anzeigen

Hier legst du innerhalb der Blöcke lokale Variablen Überstundengehalt und Monatslohn an.Diese überdecken die Sichtbarkeit der Variablen die du weiter oben deklariert hast.Beim Verlassen des Blocks sind die lokalen Variablen mitsamt deinen Berechnungsergebnissen Geschichte. Nimm die floats davor weg damit die Werte deiner Berechnung den richtigen Variablen zugewiesen werden.

Gruß void

void · 13. November 2006, 20:24

@d0nUt:
Ich setze auch meist die Variablendeklarationen an den Anfang von Funktionen.
Allerdings nur die,die ich auch wirklich in dem kompletten Scope brauche.
Alles was nur lokal benötigt wird,wird auch lokal deklariert.
Im aktuellen Fall z.B. Ueberstunden.
Es gibt keinen Grund warum die Lebensdauer von Ueberstunden sich über die ganze main ziehen sollte.Ist ja nur nen lokal benötigtes Zwischenergebnis.
Streng genommen bräuchte man Überstunden natürlich garnicht ist aber für die Lesbarkeit imho vorteilhaft.

Gruß void

Crackheady · 13. November 2006, 21:09

omg ...
mein system ist einfahc mal so abgestürtzt.. egal jetzt hab ichs fertig:

Quellcode

//**************************************************
//************** Programm: main.cpp ****************
//************ Aufgabe: Preisberechnung ************
//****** Autor, Datum: Micha, Welter 13.11.06 ******
//**************************************************
#include <iostream.h>
#include <iomanip.h>
int main ()
{
float Stundenanzahl, Stundenlohn, Endgeld, Ueberstundenlohn, Ueberstunden, Ueberstundengehalt, Fahrkostenpauschale, Monatslohn;
int Angestellt; //Definition aller Variablen
cout.setf(ios::fixed);
cout.precision(2); //2 Nachkommastellen anzeigen
cout <<"Gearbeite Stunden: ";
cin >>Stundenanzahl; //Eingabe der gearbeiteten Stunden
cout <<"Stundenlohn: ";
cin >>Stundenlohn; //Eingabe des Stundenlohns
cout <<"\nF\201r Auszubildender die 0, F\201r Angestellter die 1 : "<<endl;
cin >>Angestellt; //Abfrage, ob Angestellter oder Azubi
if (Angestellt == 1)
{ Fahrkostenpauschale = 200; }
else
{
if (Angestellt == 0)
{ Fahrkostenpauschale = 150; }
else
{ cout <<"Falsche Eingabe, starten Sie das Programm erneut.";
return 0; }
} //Deklaration des Fahrpreises
if (Stundenanzahl > 160)
{ Ueberstunden = Stundenanzahl - 160;
Ueberstundenlohn = Stundenlohn / 2;
Ueberstundengehalt = Ueberstunden * Ueberstundenlohn;
Monatslohn = Stundenlohn * 160 + Ueberstundengehalt; } //Berechnung des Monatslohns inklusive des Überstundengehalts
else
{ Monatslohn = Stundenlohn * Stundenanzahl;
Ueberstundengehalt = 0;
Ueberstundenlohn = 0; } //Berechnung des Monatslohns
Endgeld = Monatslohn + Fahrkostenpauschale; //Berechnung des Endgelds
cout <<"\n\nMonatslohn:"<<setw(28)<<Monatslohn<<" Euro"<<endl;
cout <<"Davon \232berstundengehalt:"<<setw(15)<<Ueberstundengehalt<<" Euro"<<endl;
cout <<"Fahrkostenpauschale:"<<setw(19)<<Fahrkostenpauschale<<" Euro"<<endl;
cout <<"********************************************"<<endl;
cout <<"Das Endgehalt betr\204gt"<<setw(18)<<Endgeld<<" Euro"; //Ausgabe der Beträge
cout <<"\n\n";
return 0;}

Alles anzeigen

vielen Dank für Eure Hilfe

MFG Micha

Kleine Frage (wahrscheinlich nur ein kleiner Fehler)wichtig!

Kleine Frage (wahrscheinlich nur ein kleiner Fehler)wichtig!

Quellcode

Quellcode

Quellcode

Quellcode

Teilen

Benutzer online 1

Tags