Thema: C++ Code Nachfrage

Hallo


Wollt mal fragen was ihr an deisem Code verbessern würdet? Oder ist das so okay?

Code:

// Berechnung von Flächen!
//

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

using namespace std;

double umrechnen(double wert, int faktor)
{
    return wert * faktor;
}

void hektar(double v1, double v2)
{
    cout<<"\n\nWert in Quadratkilometer\tErgebnis in Hektar\n";
    while(v1<=v2)
    {
        cout << v1 << "\t\t\t\t" << umrechnen(v1,100)<<"\n" ;
        v1++;
    }
    return;
}

void ar(double v1, double v2)
{
    cout<<"\n\nWert in Quadratkilometer\tErgebnis in Ar\n";
    do
    {
        cout << v1 << "\t\t\t\t" << umrechnen(v1,
			10000)<<"\n" ;
        v1++;
    }
    while(v1<=v2);

    return;
}

void quadratmeter(double v1, double v2)
{
    cout<<"\n\nWert in Quadratkilometer\tErgebnis in Quadratmeter\n";
    for(v1;v1<=v2;v1++)
    {
        cout << v1 << "\t\t\t\t" << umrechnen(v1,1000000)<<"\n" ;
    }
}

int main()
{
    short menuvar;
    double val1, val2;
    cout<<"\n\n\t\t\t\tM E N U E\n\t\t\t\t=========\n\n\n";
    cout<<"\t1. Umrechnen in Hektar\n\t2. Umrechnen in Ar\n\t3. Umrechnen in Quadratmeter\n\t0. Programm beenden\n\n";
    cout<<"Bitte Waehlen Sie eine Option: ";
    cin >> menuvar;
    cout <<"\nGeben Sie bitte den Anfangswert ein: ";
    cin >> val1;
    cout <<"\nGeben Sie bitte den Endwert ein: ";
    cin >> val2;
    if(val2<val1)
    {
        cout<<"Der Endwert muss groesser als der Anfangswert sein !!!";
        return 0;
    }
    switch(menuvar)
    {
        case 1:
            hektar(val1,val2);
        break;

        case 2:
            ar(val1,val2);
        break;

        case 3:
            quadratmeter(val1,val2);
        break;

        case 0:
            return 0;
        break;

        default:
            cout<<"Ungueltige Eingabe !!!";
        break;
    }
	system("pause");
    return 0;
}

lg

Rein theoretisch koennte man alle Variablen der Uebersicht halber am Anfang deklarieren, aber so im grossen und ganzen sehe ich keinen Aenderungsbedarf !!
Welches Programm benutzt du?? CPP ?? Oder Microschrott Visual C++ ??

Gruesse
Cyanwasserstoff

Visual C++ 2010 Express

Cyanwasserstoff

Microschrott

o.o Ist ja wie "Windoof"^^

lg

die "return;"s in den void-methoden kannste eigentlich weglassen.