loops - Function that would get minimum, maximum and average in c

Question

Hello I wrote a function that continuously accepts integer inputs from the user until the user enters -999 (sentinel value). The function also identifies the highest and lowest value entered, the average of inputs. I already wrote a code for it but I can't seem to get the correct average value for it and need help revising it.

Here's my code:

#include <stdio.h>

int getMinMaxAve ( int *min, int *max, float *ave)
{
    int nVal;
    int nMin = 0;
    int nMax = 0;
    int nResult = 1;
    int nCount = 0;

    while (nVal != -999)
{
    printf ("Value:");
    scanf ("%d", &nVal);
    
    if (nVal !=-999) 
    {
        if (nVal > nMax)
            nMax = nVal;
        if (nVal < nMin)
            nMin = nVal;
        nResult += nVal;
        nCount += 1;
        
    }
}
        *min = nMin;
        *max = nMax;
        *ave = nResult / nCount;
}


int main ()
{
        int nMin;
        int nMax;
        int nResult;
        float fAve;

        nResult = getMinMaxAve (&nMin, &nMax, &fAve);

if (nMin == -999)
{
    printf ("Min: NA
");
    printf ("Max: NA
");
    printf ("Average: NA
");
}
else 
{
    printf ("Min: %d
", nMin);
    printf ("Max: %d
", nMax);
    printf ("Average: %f
", fAve);
}

return 0;
}

Answer 1

First the variable you keep the sum in nResult is initialized at 1. That must absolutely be zero.

Second *ave = nResult / nCount; does integer division as both nResult and nCount are integers.

A way out of this is to declare nResult as float. Then that same expression actually does what you expect it to do. And better rename that variable to totalSum, or something equally informative.

(But you better use double instead of float everywhere)

There are more problems with your code, but I limit myself to just answer your question about the average.