Probleme mit dem C-Programm

Fussel94 schrieb:

Da stellt sich mir ja jetzt die Frage, wie fülle ich das Array richtig. Das muss ich ja prinzipiell auch über eine Funktion (Berechnung des BMI's aus Gewicht und Größe)und Schleife machen, oder?

Das beste ist, du füllst da Array dann, wenn du die Daten dafür hast.
Und das ist direkt in der Schleife, in der du die Daten einliest.

Fussel94 schrieb:

würde ich mir jetzt so vorstellen, ist aber bestimmt wieder völliger Quatsch:

Ja, ist es.
Du übergibst Werte für einen BMI (Länge und Masse) und berechnest daraus das ganze Feld.

Du kannst noch einen Paramter mit übergeben: Den Index des zu berechnenden Elements.
Aber das ist auch überflüssig, da du das auch als Zuweisung machen kannst.

ergebnis=bodymassindex(gewicht,groesse);
bmi[index] = ergebnis;
index++;

Fussel94 schrieb:

Prinzipiell stelle ich mir vor, dass meine Minimums-Funktion das Minimum mittels eines Vergleiches ermittelt. Also ich eine Variable für das Minimum habe und dieser Wert immer mit dem letzten errechneten BMI-Wert verglichen wird. Wenn dieser kleiner ist, als der von der Variablen, soll er gespeichert werden und mit dem nächsten verglichen. Nur dafür brauche ich anscheinend den Vektor, der bei mir leer ist und deshalbt macht der ausdruck 'ergebnis<min' keinen Sinn oder?

Für den Vergleich brauchst du erstmal zwei Werte. Du hast aber nur einen (das aktuelle Ergebnis)
Du musst deiner Funktion auch das bisherige Minimum mit übergeben.
Da du die Funktion dann aber auch in der Leseschleife aufrufst, brauchst du das Array nicht mehr.

Einlesen, Berechnung (evtl. ablegen im Array) und Bestimmung des Minimum kannst du alles in der Einleseschleife machen.

Möchtest du das Minimum erst später bestimmen, musst du das Array und die Anzahl der gültigen Einträge (im Array) an die Funktion mit übergeben.
In der Funktion ist dann die Schleife, die die Werte durchsucht.

Fussel94 schrieb:

DirkB schrieb:
float bmi (float, float);  // Diese Deklaration sollte vor main stehen
            ergebnis=bmi(gewicht,groesse);
}
Warum sollte das, jetzt vetreten für alle, die ähnlich sind, vor main stehen? Eigentlich ist es doch mein Funktionsaufruf und muss dieser nicht in der Funktion erfolgen?

Die Deklaration dient dazu, dem Compiler mitzuteilen, welche Paramter die Funktion erwartet.
Die Deklaration muss vor dem ersten Aufruf der Funktion gemacht werden.
Dies kann man auch erreichen, indem man die Funktion vorher definiert.

Bei printf, fopen und fscanf hast du ja vor deren Aufruf auch keine Deklaration geschrieben.
Die wird nämlich in stdio.h gemacht (und das steht auch vor main)

Soo, jetzt habe ich es denke ich mal erfolgreicht geschafft die BMI-Werte in ein Arry zu übergeben. Er zeigt sie mir auch korrekt an.
Das mit der Deklaration vor main leuchtet mir jetzt auch ein.

Nun nochmal eine Frage zu dem Minimum.

Funktion:

float minimum (float bodymassindex[], int MAX)
{
    int min=bodymassindex[0], i;
    for(i=1;i<size;i++)
    {
        if(bodymassindex[i]<min)
        {
            min=bodymassindex[i];
        }
    }return min;
}

Funktionsaufruf im main:

minimal=minimum(bodymassindex[MAX],index);
        printf("Minimum: %f",minimal);

Du hast ja gesagt, dass ich das Array und die Anzahl der gültigen Elemente in die Funktion übergeben muss.
Wie du wahrscheinlich selber erkennst, bekomme ich einen Fehler.
Ich bin mir zum einen nicht sicher, ob ich das Array in dieser Form an die Funktion übergeben kann und ob ich für die Anzahl 'MAX' oder den 'index' nehmen muss. Der 'index' Wert hat ja theoretisch "gezählt" wie oft der BMI berechnet wurde oder?

Habe den ersten Fehler darin gefunden.
Gibt aber jetzt einen zweiten. Er gibt mir den Minimal-Wert glatt aus, also normalerweise müsste da ein Wert in der Art 7.821 stehen, aber er schreibt nur 7.000

printf("Minimum: %.2f",minimum(bodymassindex,index));

float minimum (float *bodymassindex, int size)
{
    int min=bodymassindex[0], i;
    for(i=1;i<size;i++)
    {
        if(bodymassindex[i]<min)
        {
            min=bodymassindex[i];
        }
    }return min;
}

Fussel94 schrieb:

Habe den ersten Fehler darin gefunden.
Gibt aber jetzt einen zweiten. Er gibt mir den Minimal-Wert glatt aus, also normalerweise müsste da ein Wert in der Art 7.821 stehen, aber er schreibt nur 7.000

Erklär mal, was diese Zeile bedeutet:

int min=bodymassindex[0],i;

Na ich würde denke, dass ich damit das Erste Element des Vektor an eine lokale Variable übergebe, die somit meinen Startwert darstellt.
und i ist dann die Laufvariable mit der die Schleife durchlaufen wird, bis der kleinste Wert ermittelt werden konnte. oder??

ach mist ich mache ja dann aus meinem bodymassindex[] auch einen int vektor oder?

volkard

Fussel94 schrieb:

ach mist ich mache ja dann aus meinem bodymassindex[] auch einen int vektor oder?

Nein, hat nix mit vektor zu tun.

int min=bodymassindex[0],i;

wäre höflicher geschrieben

int min=bodymassindex[0];
int i;

Brauchbar wäre

float min=bodymassindex[0];
int i;

min ist ein int . Also können in min auch nur Ganzzahlen gespeichert werden.

Jaa, genau so habe ich es dann abgeändert, als du mir die Frage gestellt hattest :).

Dann hätte ich noch eine Frage. Desweiteren muss ich mit dem Programm bestimmen, wie viele Patieten der jeweiligen BMI-Klasse angehören.

Die Klassen zu den jeweiligen Werten habe ich ja bereits bestimmt:

char klassifikation (float ergebnis)
{

        if(ergebnis<19)
            printf("Untergewicht");
        if((ergebnis>=19)&&(ergebnis<=24.9))
            printf("Normalgewicht");
        if((ergebnis>=25)&&(ergebnis<=29.9))
            printf("Uebergewicht");
        if((ergebnis>=30)&&(ergebnis<=39.9))
            printf("Adipositas");
        if(ergebnis>=40)
            printf("mas. Adipositas");
        return 0;
}

und ich hoffe er hat sie hier in ein Array übergeben, ist ja eigentlich ähnlich den BMI-Werten, angezeigt werden sie auch richtig:

ergebnisklasse=klassifikation(ergebnis);
            k[aindex]=ergebnisklasse;
            aindex++;

            printf("%c\n", ergebnisklasse);

Um jetzt die Patietenanzahl für jede Klasse zu bestimmen, würde ich irgendwie in diesem char array eine Zeichenkette (entsprchend der Klassenbezeichnung) suchen lassen und immer wenn er eine gefunden hat, zählen.

Ich habe das ganze erst einmal versucht für die Klasse "Untergewicht" umzusetzen und wie so oft ist da wieder irgendetwas falsch und ich würde jetzt erst einmal vermuten, dass es an der Wertzuweisung der Variablen 'char zeichen1[]={"Untergewicht"}' liegt

Funktion:

int count_klasse(char *k, int size)
{
    int i=0;
    int z=0;
    char zeichen1[]={"Untergewicht"};

    for(i=0;i<size;i++)
    {
        if(k[i]==zeichen1[0])
            z++;
    }return z;
}

Wertausgabe in main:

printf("Untergewicht:%i",count_klasse(k,aindex));

Hat sich alles erledigt :).

Vielen Dank für die freundliche Hilfe