ich habe testweise ein kleines Programm (Konsolenanwendung) geschrieben, welches diverse Mittelwerte erfasst. Das kompilieren funktioniert in Visual C++ Express 2005 ohne Fehler/Probleme.

Wenn ich das Programm starte und mir irgentetwas berechnen lassen will, bekomme ich den Error das das Programm auf geschützten Speicher zugreifen will. Ich habe kA woran das liegen kann. Wie gesagt, der Error kommt erst wenn eine andere Methode als fill ausgeführt wird.

/**
 * 
 *  Programm zur ermittlung diverser Mittelwerte,
 *  bei vorheriger Erfassung von Daten durch
 *	Eingabe von Benutzer.
 *  Von ---
 *
 */

#include "stdafx.h"
#include <cstdlib>
#include <iostream>
#include <string>
#include <math.h>

using namespace System;
using namespace std;

class cWerte {

      private:

              double werte[100];			// Array in dem später die Werte gespeichert werden
              string  eingabe;				// string um Eingabe zu überprüfen bevor diese als Wert hinzugefügt wird
			  size_t found;					// Wird benötigt um eine Position eines Teilstringes in einem String zu speichern
              bool   readingValues;			// Setzt fest ob weitere Werte ermittelt werden sollen
              int    lastIndex;				// Beinhaltet die Größe des Arrays werte

      public:

		     /**
			  * Klasse zum Einlesen von Werten
			  */
             void readValues() {

                  /** Alle Werte auf 0 setzen **/

                  int i = 0;
                  while( i != 101 ) {

                         werte[i] = 0;
                         i++;

                  }

                  /** Werte einlesen **/                  

                  cout << "  \n  Bitte geben Sie nun nach und nach ihre gew\x81nschten Zahlen ein.\n" <<
                          "  Verwenden Sie anstelle eines Kommas, einen Punkt, bzw ein n wenn\n" <<
                          "  Sie die Eingabe beenden m"<<"\x94"<<"chten\n\n";

                  readingValues = true;						// Zu Beginn wahr, da mindestens ein Wert bzw der Abbruchstring erfasst werden muss

                  int j = 0;        
                  while( j != 100 && readingValues ) {		// Maximal kann man 100 Werte eingeben

                         cout << "    - ";					// Der Form halber ;)
                         if( j < 9) cout << "0";			// Wenn die Zahl einstellig ist soll eine f\x81hrende Null ausgegeben werden
                         cout << j + 1 << ": ";				// Zeigt dann an bei dem wievieltem Wert man gerade ist

                         getline(cin,eingabe);				// Liest einen Wert ein (Wegen dem Variablentyp [string] nur mit getline möglich)
						 found = eingabe.find("n");			// Sucht nach einem n im eingegebenen Wert (Eigtl sollte man hier auf alle Buchstaben und Zeichen testen,
															// doch setzen wir eine korrekte Eingabe des Benutzers einfach mal als gegeben vorraus.

                         if( found == string::npos ) {				// Die position von n im string ist 0. In anderen Worten: Es ist nicht vorhanden :)
							 werte[j] = atof(eingabe.c_str());		// atof konvertiert den eingegebenen String in eine Zahl um diese in werte[j] abspeichern zu können.
						 }
                         else {
							readingValues = false;			// Der Benutzer wünscht keine weitere Eingabe von Zahlen
						 }

                         lastIndex++;						// Die zusätzliche Größeninformation über das Array wird inkrementiert.                                                                           
                         j++;

                  }

             };

             void getArethmetic() {

				 cout << "  \n"
				      << "  Das arethmetische Mittel betr\x84gt: ";

				 double aSumme = 0, bErgebnis = 0;
				 int i = 0;

				 while( i != lastIndex ) {
					 aSumme = aSumme + werte[i];
					 i++;
				 }

				 bErgebnis = aSumme / lastIndex;
				 cout << bErgebnis;

             };

/*             void getMedian() {

				 cout << "  \n"
			              << "  Der Median beträgt: ";

				 double aSumme = 0, bErgebnis = 0;
				 int i = 0;

				 while( i != lastIndex ) {
					 aSumme = aSumme + werte[i];
					 i++;
				 }

				 bErgebnis = aSumme / lastIndex;
				 cout << bErgebnis;

             };   */

	     void getHarmonic() {

				 cout << "  \n"
			          << "  Das harmonische Mittel betr\x84gt: ";

				 double aWert = 0, bErgebnis = 0;
				 int i = 0;

				 while( i != lastIndex ) {
					 aWert = aWert + ( 1 / werte[i] );
					 i++;
				 }

				 aWert = aWert / 2;
				 bErgebnis = 1 / aWert;
				 cout << bErgebnis;

             };

             void getGeometric() {

				 cout << "  \n"
			          << "  Das geometrische Mittel betr\x84gt: ";

				 double aProdukt = 0, bErgebnis = 0;
				 int i = 0;

				 while( i != lastIndex ) {
					 aProdukt = aProdukt * werte[i];
					 i++;
				 }

				 bErgebnis = sqrt(aProdukt);
				 cout << bErgebnis;

             };

			 void getMin() {

				 cout << "  \n"
			          << "  Der kleinste Wert ist: ";

				 int i = 0;
				 double min;

				 while( i != lastIndex ) {
					 if( i == 0 ) min = werte[i];
					 if( min > werte[i] ) min = werte[i];
					 i++;
				 }

				 cout << min;

			 }

			 void getMax() {

				 cout << "  \n"
			          << "  Der gr\x94ßte Wert ist: ";

				 int i = 0;
				 double max;

				 while( i != lastIndex ) {
					 if( i == 0 ) max = werte[i];
					 if( max < werte[i] ) max = werte[i];
					 i++;
				 }

				 cout << max;

			 }

};

int main(int argc, char *argv[]) {

    bool running = true;		// Speichert ab, ob das Programm weiterlaufen soll
	cWerte werte;				// Erstellt ein Objekt der klasse cWerte
    char auswahl;				// Speichert die Eingabe des Benutzers später ab

    cout <<  "  \n" <<
             "               ############################\n" <<
             "               ##  Stochastik f\x81r Faule  ##\n" <<
             "               ############################\n" <<
             "  \n" <<
             "  Guten Tag. Mit diesem Tool k"<<"\x94"<<"nnen Sie in k\x81rzester Zeit\n" <<
             "  verschiedene Stochastische Angaben zu beliebig vielen\n" <<
             "  Werten automatisch berechnen lassen.\n" <<
             "  \n";

    werte.readValues();			// Das Programm läuft zum ersten mal, weshalb man Werte eingeben muss.

    while( running ) {			// Solange wie running wahr, das heist der Benutzer nicht beenden will, wird die Schleife ausgef\x81hrt

           cout << "  \n  Was m"<<"\x94"<<"chten Sie tun? Wählen Sie durch die Eingabe des Buchstaben.\n"
                << "        a: Neue Werte einlesen\n"
                << "        b: Das Arethmetische Mittel ausgeben lassen\n"
                << "        c: Das Harmonische Mittel ausgeben lassen\n"
                << "        d: Das Geometrische Mittel ausgeben lassen\n"
				<< "        e: Den kleinsten erfassten Wert ausgeben lassen\n"
				<< "        f: Den gr\x94\xE1ten erfassten Wert ausgeben lassen\n"
                << "        q: Das Programm beenden\n"
				<< "        ";

           cin  >> auswahl;		// Auswahl wird eingelesen und in auswahl gespeichert

                if ( auswahl == 'q' ) running = false;
           else if ( auswahl == 'a' ) werte.readValues();
           else if ( auswahl == 'b' ) werte.getArethmetic();
           else if ( auswahl == 'c' ) werte.getHarmonic();
           else if ( auswahl == 'd' ) werte.getGeometric();
		   else if ( auswahl == 'e' ) werte.getMin();
		   else if ( auswahl == 'f' ) werte.getMax();
           else                       cout << "  \nUnbekannte Operation.";

    }               

    return 0;
}

lastIndex wird ja beim Einlesen der Werte initialisiert.

Nein, lastIndex wird nicht initialisiert. Das könnte überhaupt nur in einem Konstruktor passieren, die Klasse hat aber keinen. Und auch sonst setzt du die Variable nirgends auf einen bestimmten Startwert, du zählst sie nur hoch -> Bumm.

MEETyA schrieb:

bzw was muss ich tun um das Problem zu lösen ohne den alten Wert zu überschreiben?

Warum setzt du lastIndex nicht einfach am Anfang von readValues auf einen passenden Startwert?

Problem gelöst

void readValues() 
{

    /** Alle Werte auf 0 setzen **/

    int i = 0;
    while( i != 101 ) /* 0 ... 100 */ {

        werte[i] = 0; /* bumm! werte[100] => Bereichsüberschreitung */
        i++;
    }
    // ...

double werte[100];

...
(1) Ein Array der Größe x hat Elemente 0 ... x - 1
(2) for-Schleifen sind wohl noch nicht behandelt wurden? Schmeiß das Tutorial, das du benutzt, weg, das hat zumindest eine sehr schlechte Strukturierung, wenn while-Schleifen und Arrays schon behandelt wurden, aber keine for-Schleifen!

void readValues() 
{
    std::fill(werte, werte + 100, 0.0);
    // ...

...

void getMin() {

                 cout << "  \n"
                      << "  Der kleinste Wert ist: ";

                 int i = 0;
                 double min;

                 while( i != lastIndex ) {
                     if( i == 0 ) min = werte[i];
                     if( min > werte[i] ) min = werte[i];
                     i++;
                 }

                 cout << min;

             }

Ich weiß gerade nicht, als was last_index definiert ist, aber gehen wir mal davon aus, das er korrekt belegt ist:
Warum legst du den Startwert von min nicht direkt auf werte[0] und beginnst bei i=1 mitm Durchlaufen? Dann brauchst du nicht jeden Schleifendurchlauf die if-Abfrage!

double get_min() const 
{
    if (lastIndex == 0) /* FEHLER! Keine Werte vorhanden! */

    double min(werte[0]);
    for (unsigned int i(1); i != lastIndex; ++i)
        if (werte[i] < min) min = werte[i];
}

und warum alles neu erfinden, wenn das eine Funktion schon macht?

double get_min() const
{ return std::min(werte, werte + lastIndex); }

Du kannst die gewünschte Lokale in den Stream "einbrennen":
cout.imbue(locale("de_DE"));

Dann musst Du Deine Benutzer nicht mit Dezimalpunkteingaben quälen.

Du kannst die gewünschte Lokale in den Stream "einbrennen":
cin.imbue(locale("de_DE"));

Dann musst Du Deine Benutzer nicht mit Dezimalpunkteingaben quälen.