Bin ein absoluter c++ neuling und habe gerade mein erstes mini programm geschrieben (passend zum führerschein den ich gerade mache )

naja auf jeden fall habe ich mir den quelltext jetzt schon x-mal durchgelesen und komme einfach nicht auf den fehler...wäre nett wenn mir hier jemand kurz helfen könnte

#include <iostream>
#include <cstdlib>
#include <string>
#include <cstdio>
#include <conio.h>
#include <function>

using namespace std;

int main()
{
    int geschw, gefahr;
    string bw;

    cout << "Geben Sie eine Geschwindigkeit (km/h) ein: \n\n";
    cin >> geschw;
    cout << "Gefahrbremsung? (j/n)" << endl;
    cin >> gefahr;

    if ( gefahr == "n" || gefahr == "N")
    {
     int bw = (geschw / 10) * (geschw / 10); 
    }

    else
    {
    int bw = ((geschw / 10) * (geschw / 10)) / 2; 
    }

    int rw = (geschw / 10) * 3;
    int aw = rw + bw;

    cout << endl;
    cout << "Ihr Reaktionsweg betraegt: " << rw << " meter" <<endl;
    cout << "Ihr Bremsweg betraegt: " << bw << " meter" << endl;
    cout << "Ihr Anhalteweg betraegt: " << aw << " meter \n\n" << endl;
    getch();
    return 0;

}

was mir auf anhieb ins auge springt, ist, dass du mit ints rechnest. ich würde floats oder doubles nehmen, da vor allem bei den divisionen dir sehr viel genauigkeit verloren geht, da immer abgerunden wird.
wenn du zb als geschwindigkeit 9 eingibst, wird da gerechnet:

9/10=0
bw = 0*0 = 0

0/10 = 0
rw = 0*3 = 0

aw = 0+0 = 0

mit floats oder doubles:
9./10. = .9
bw = .9*.9 = .81

9./10. = .9
rw = .9*3 = 2.7

aw = 2.7 + .81 = 3.51

Warnungen:
1. Unnötig viele Headers inkludiert.
2. C++ C-Headers inkludiert. Man sollte wenn möglich nicht C und C++ mixen.
3. getch() ist nicht Platform unabhängig, genauso wie <conio.h>
4. Es wäre eine Überlegung wert, ob man nicht besser mit float oder double rechnen möchte.
5. Pass auf mit der Verwendung von using namespace std; .

Grüssli

In function int main()': invalid operands of types \float' and `const char[2]' to binary operator==' invalid operands of types \float' and `const char[2]' to binary `operator=='
no match for 'operator+' in 'rw + bw'

also ich habs jetzt mit float versucht aber hat nichts geändert...als ich dass ganze ohne if else funktion gemacht habe hats auch funktioniert

... Dev-C++ ...

3. Fehler: Du Verwendest Dev-C++
Nimm lieber Code::Blocks oder Microsoft Visual Studio 2008 Express Edition.
Dev-C++ wird nicht mehr weiterentwickelt und gilt als deprecated!

Grüssli

ach ja du kannst dir auch eine DVD mit dem kompletten VS 2008 EE bestellen bei microsoft

ok bin gerade ma laden...aber ist der codeblock sonst ok oder ist noch ein fehler drin?

Ich hoffe mal du hast meine Fehler 1 und 2 beachtet.
Ehm, doch habe gerade dank der Fehlermeldung noch einen Fehler gefunden:

int geschw, gefahr;
    string bw; // bw ist string

    cout << "Geben Sie eine Geschwindigkeit (km/h) ein: \n\n";
    cin >> geschw;
    cout << "Gefahrbremsung? (j/n)" << endl;
    cin >> gefahr;

    if ( gefahr == "n" || gefahr == "N")
    {
     int bw = (geschw / 10) * (geschw / 10); // bw ist int und wird gleich wieder verworfen
    }

    else
    {
    int bw = ((geschw / 10) * (geschw / 10)) / 2; // bw ist int und wird gleich wieder verworfen
    }

    int rw = (geschw / 10) * 3; // scheint ok zu sein
    int aw = rw + bw; // das geht nicht. bw ist hier ein leerer string.

Kann es sein, dass du oben bei string bw eigentlich int bw meintest und unten die ints vor dem bw weglassen?
Wenn das nicht klar ist, dann empfehle ich dringend nochmals die Grundlagen von C++ durch zu ackern!

Grüssli

#include <iostream>
#include <cstdlib>
#include <cstdio>

using namespace std;

int main()
{
    float geschw, gefahr;

    cout << "Geben Sie eine Geschwindigkeit (km/h) ein: \n\n";
    cin >> geschw;
    cout << "Gefahrbremsung? (j/n)" << endl;
    cin >> gefahr;

    if ( gefahr == "n" || gefahr == "N")
    {
     float bw = (geschw / 10) * (geschw / 10); 
    }

    else
    {
    float bw = ((geschw / 10) * (geschw / 10)) / 2; 
    }

    float rw = (geschw / 10) * 3;
    float aw = rw + bw;

    cout << endl;
    cout << "Ihr Reaktionsweg betraegt: " << rw << " meter" <<endl;
    cout << "Ihr Bremsweg betraegt: " << bw << " meter" << endl;
    cout << "Ihr Anhalteweg betraegt: " << aw << " meter \n\n" << endl;
    system("PAUSE");
    return 0;

}

Dravere schrieb:

Fehler:
1. Keine genaue Fehlerbeschreibung geliefert.
2. if ( gefahr == "n" || gefahr == "N")
"x" liefert ein char const*
'x' liefert ein char
if ( gefahr == 'n' || gefahr == 'N')

Warnungen:
1. Unnötig viele Headers inkludiert.
2. C++ C-Headers inkludiert. Man sollte wenn möglich nicht C und C++ mixen.
3. getch() ist nicht Platform unabhängig, genauso wie <conio.h>
4. Es wäre eine Überlegung wert, ob man nicht besser mit float oder double rechnen möchte.
5. Pass auf mit der Verwendung von using namespace std; .

Grüssli

Und du scheinst noch nachholbedarf in Sachen Scope zu haben! Naja, ich schreib dir nochmals ein paar Hinweise hin.

if ( gefahr == "n" || gefahr == "N") // "" gibt const char*, '' gibt char
    { 
     float bw = (geschw / 10) * (geschw / 10); // bw wird erstellt, Wert zugewiesen.
    }  // bw wird zerstört.

    else 
    { 
    float bw = ((geschw / 10) * (geschw / 10)) / 2; // bw wird erstellt, Wert zugewiesen.
    } // bw wird zerstört.

    float rw = (geschw / 10) * 3; 
    float aw = rw + bw; // bw ist gar nicht vorhanden!

Grüssli

PS: gibt es Code::Blocks auch auf deutsch?