C++ Rückgabewert stimmt nicht

[Malvin (Gast)]

Hallo Zusammen,

kann mir jemand sagen warum der Rückgabewert der Funktion nicht stimmt? 
Was mache ich hier falsch?

#include "stdafx.h"
#include "stdlib.h"
#include "stdio.h"
#include "locale.h"


void eingabeMesswerte(int &anz, double &wert);
double ermittleMittelwert(int anz,int mW[]);

int _tmain(int argc, _TCHAR* argv[])
{
  int anzahl=0;
  int value[100];
  double wert;
  double mittelWert;


  char c = 'j';
  setlocale(LC_ALL,"German");
  

  do
  {
    system("cls");
    printf("Wenn Sie Messwerte eingeben möchten 'a' \n\n");
    printf("Wenn Sie Mittelwert erfahren möchten 'b' \n\n");
    printf("Wenn Sie das Programm beenden möchten 'n' \n\n");
    scanf("%c",&c);

    
  
    switch(c)
    {
      
    case('a'):
      {
        system("cls");
        printf("Bitte die Anzahl der Messwerte angeben: ");
        scanf("%i",&anzahl);
  
        eingabeMesswerte(anzahl,wert);
        
      }
    break;

    case('b'):
        
        mittelWert=ermittleMittelwert(anzahl,value);
        printf("\n\n\tMittelwert: %.2lf\n\n",mittelWert);
        system("pause");
        
    break;

    case('n'):
        return 0;
    break;
    }
    
  }
  
  while(c='j');
    
}

void eingabeMesswerte(int &anz, double &wert)
{

  int anzMw[100];
  int i;
  double messW = 0;

  for(i=0;i<100;i++)
  {
    anzMw[i]=i;
  }
  if(anz > 100)
  {
    printf("Sie können nur maximal 100 Messwerte aufnehmen.");
  }

  else
  {
    for(i=0;i<anz;i++)
    {
      printf("Geben Sie bitte den %i. Messwert ein: ",i+1);
      scanf("%lf",&messW);
      anzMw[i]=messW;

    }

    ermittleMittelwert(anz,anzMw);

  }

}

double ermittleMittelwert(int anz,int mW[])
{
  int i;
  
  double gesamt=0;
  double mittelWert;

  for(i=0;i<anz;i++)
  {
    gesamt = mW[i]+gesamt;
  }

  mittelWert=(gesamt/anz);
  return mittelWert;
  //printf("Der Mittelwert beträgt %.2lf\n\n", mittelWert);

}

Danke

Gruß
Malvin

[Peter II (Gast)]

Malvin schrieb:
> kann mir jemand sagen warum der Rückgabewert der Funktion nicht stimmt?

was kommt bei welchen eingaben raus und was erwartest du?

[Sebastian V. (sebi_s)]

Ich nehme mal an du meinst du die ermittleMittelwert Funktion. Die sieht 
an sich richtig aus. Irgendwie geht es aber mit deinen Variablen ein 
wenig drunter und drüber. Daher kommt wohl nicht das raus was du 
erwartest. Was z.B. soll int value[100] in deiner main Funktion sein? 
Das wird nur von ermittleMittelwert benutzt aber nirgendwo gefüllt. Dann 
gibt ermittleMittelwert natürlich nur quatsch aus. Es gibt aber auch 
noch einen zweiten Aufruf von ermittleMittelwert in deiner 
eingabeMesswerte Funktion. Dort füllst du int anzMw[100] mit double 
Werten die dann natürlich zu int konvertiert werden. Kommt daher 
eventuell was anderes als erwartet raus?

[rm-rf (Gast)]

Malvin schrieb:
> kann mir jemand sagen warum der Rückgabewert der Funktion nicht stimmt?

Die Funktion "eingabeMesswerte(int &anz, double &wert)" sieht nach C++ 
aus. Jedenfalls speichert sie die eingegebenen Werte in einer lokalen 
Variable, die geht bei Beendigung der Funktion verloren.

Der Parameter "wert" wird nicht verwendet - vermutlich soll auch 
"eingabeMesswerte" dort den Mittelwert retournieren?

[Malvin (Gast)]

Peter II schrieb:
> was kommt bei welchen eingaben raus und was erwartest du?

ich will "mittelWert" aus der Funktion in Main raus geben.

[Mitlesa (Gast)]

Malvin schrieb:
> ich will "mittelWert" aus der Funktion in Main raus geben.

Deklariere deine Variable so:

static double mittelWert;

sonst geht der Inhalt beim Rücksprung verloren.

[Malvin (Gast)]

Sebastian V. O. schrieb:
> Dort füllst du int anzMw[100] mit double
> Werten die dann natürlich zu int konvertiert werden.

wie so werden die Werte zu int konvertiert? Hier wird doch nur der 
Messwert an der Speicherstelle gespeichert oder nicht?

Sebastian V. O. schrieb:
> Was z.B. soll int value[100] in deiner main Funktion sein?

Die Funktion "ermittleMittelwert(variable, array)", damit ich den in 
Main aufrufen kann habe ich hier eine leere Array geladen.

Kann das der Fehler sein?

[Peter II (Gast)]

Mitlesa schrieb:
> Deklariere deine Variable so:
> static double mittelWert;
>
> sonst geht der Inhalt beim Rücksprung verloren.

nein, bitte nicht. Auch wenn das Programm damit eventuell funktioniert, 
ist diese Lösung nicht sinnvoll erweiterbar.

Der Ansatz vom Programm ist ja schon richtig, andere haben den Fehler ja 
schon gefunden, es wird einfach das falsche Array verwendet.

[Malvin (Gast)]

Mitlesa schrieb:
> Deklariere deine Variable so:
> static double mittelWert;
>
> sonst geht der Inhalt beim Rücksprung verloren.

Hab die Variable als static definiert, trotzdem ist der Rückgabewert
undefiniert--> -858993460.00

[Peter II (Gast)]

Malvin schrieb:
> Hab die Variable als static definiert, trotzdem ist der Rückgabewert
> undefiniert--> -858993460.00

lies dir die anderen Kommentare durch, du hast 2 Array und sie irgendwie 
vertauscht.

Gehe einfach schritt für Schritt im Debugger mal durch, und schau dir 
den Inhalt deiner Variablen an,

[casud (Gast)]

while(c='j');

... ist wahrscheinlich auch nicht das was du willst.

[rm-rf (Gast)]

Malvin schrieb:
> Hab die Variable als static definiert, trotzdem ist der Rückgabewert
> undefiniert--> -858993460.00

habs oben schon geschrieben, die eingegebenen messwerte sind weg sobald 
"eingabeMesswerte" zurückspringt.

[Malvin (Gast)]

casud schrieb:
> while(c='j');

ist eigentlich gewollt. Der soll so lange durch laufen, bis die anfrage 
mit nein bewertet wird.

[Peter II (Gast)]

Malvin schrieb:
> ist eigentlich gewollt. Der soll so lange durch laufen, bis die anfrage
> mit nein bewertet wird.

das steht dort aber nicht. Du solltest auch die Warnung vom Compiler 
beachten.

[Sebastian V. (sebi_s)]

Malvin schrieb:
> Sebastian V. O. schrieb:
>> Dort füllst du int anzMw[100] mit double
>> Werten die dann natürlich zu int konvertiert werden.
>
> wie so werden die Werte zu int konvertiert? Hier wird doch nur der
> Messwert an der Speicherstelle gespeichert oder nicht?

Nochmal der relevante Code:

int anzMw[100];
double messW = 0;
...
for(i=0;i<anz;i++)
{
  printf("Geben Sie bitte den %i. Messwert ein: ",i+1);
  scanf("%lf",&messW);
  anzMw[i]=messW;

}

Du lässt den Benutzer hier eine Zahl eintippen und speicherst diese in 
messW. messW ist ein double also kann man auch Zahlen mit 
Nachkommastellen eingeben. Wenn du diese aber mit anzMw[i]=messW in 
einem Array aus ints speicherst werden die Nachkommastellen einfach 
verworfen. Wenn du die double Werte behalten willst musst du schon ein 
double anzMw[100] Array erstellen. Gleiches für die ermittleMittelwert 
Funktion die auch ein Array von ints erwartet.

[Malvin (Gast)]

rm-rf schrieb:
> habs oben schon geschrieben, die eingegebenen messwerte sind weg sobald
> "eingabeMesswerte" zurückspringt.

Ja ist richtig, ich glaube es liegt daran dass ich hier 2 verschiedene 
Arrays verwende.

Ich probier es nochmal aus und gebe euch morgen bescheid.

DAnke euch allen.

Gruß
Malvin

[Sebastian V. (sebi_s)]

Ich vermute du wolltest eher sowas programmieren:

void eingabeMesswerte(int anz, double *wert)
{
  double messW;
  for(int i=0; i<anz; i++)
  {
    printf("Geben Sie bitte den %i. Messwert ein: ", i+1);
    scanf("%lf", &messW);
    wert[i] = messW;
  }
}

Diese Funktion ließt so viele Messwerte ein wie anz angibt und speichert 
die Werte in dem zweiten Parameter wert. Achte hier auf die Art wie die 
Parameter übergeben werden. Der erste Parameter ist ein einfacher int, 
keine Referenz auf ein int wie in deinem Beispiel. Eine Referenz auf ein 
int macht nur dann Sinn wenn die Funktion den originalen Wert verändern 
soll. Der zweite Parameter ist ein Pointer und zeigt auf das erste 
Element von einem Array. Darüber kannst du dann das Array was irgendwo 
in der main Funktion steht mit Daten füllen.