Thread: while loop

Originally Posted by andyouf

Thank you that is very helpful!! My only question then is why does this code work in not summing or averaging the 99999 for me when I use "[FONT=arial]if (number!=99999)" above sum equation.

You are always incrementing "loop_count". You need to place the loop_count increment statement inside the "if" code block along with the sum calculation:

Code:

if (number!=99999)
{   sum = sum + number;
    loop_count++;
}

Now you might possibly have a "loop_count" value of zero, if no valid values are entered. Zero is a valid value for "loop_count", since it represents the number of valid values entered (you don't want to count 99999).

Originally Posted by andyouf

Also, could I use one of your if stmt examples to make it so that if the user opens the program and enters no input (ie 99999) the program returns "no valid input entered."

You can now use "loop_count" in an "if else" statement after the loop has ended:

Code:

if(loop_count == 0) {

    /* print "no valid input entered." message */

} else {

    /* calculate average and print result, sum, etc */
}

-

Now if I use a function, say "get_smallest" to get smallest entered or "number" to get the integer entered will it automatically refer back to this loop in main that I'm building?

I think you sort of answered this when you said I'm always incrementing the loop_count throughout, thank you.

My professor wants getNumber to be a function that takes in an address parameter so that the integer entered in keyboard is picked up. I have that down but I don't see how to make this a separate function outside of main? He also wants me to do this with smallest integer entered. How do I declare that so that it refers to my loop?

Code:

/* Prompts user and gets integer numbers from keyboard, one number at a time. Ends program when 99999 entered. and displays various results.
        Written by:
        Date:       10/20/14
*/
#include <stdio.h>
#include <stdlib.h>


//function declarations


int main()
{
    int getNumber;
    int average;
    int sum = 0;
    int loop_count = 0;




    do {
        printf("Enter next integer:");
        loop_count++;


       scanf("%d", &getNumber);   /* read a single integer value input */
        if (getNumber!=99999)
        sum = sum + getNumber;
    } while (getNumber != 99999);


    average = sum / loop_count;


    printf("Number of Integers: %d\n", loop_count);
    printf("Average: %d\n", average);
    printf("The Sum of the Number Is: %d", sum);
}

getNumber shld just take one address parameter so that integer from keyboard is picked up...

Code:

#include <stdio.h>
#include <stdlib.h>


//function declarations


int main()
{
    int getNumber;
    int average;
    int sum = 0;
    int loop_count = 0;




    do {
        printf("Enter next integer:");
        loop_count++;
        scanf("%d", &getNumber);   /* read a single integer value input */
        if (getNumber!=99999)
        sum = sum + getNumber;
        if(getNumber == 99999 && loop_count == 1) {
        printf("No input was provided"\n);


}
    } while (getNumber != 99999);


    average = sum / loop_count;


    printf("Number of Integers: %d\n", loop_count);
    printf("Average: %d\n", average);
    printf("The Sum of the Number Is: %d", sum);
}

i get multiple errors when I try your if stmt. why do you initialize minimum to 99999?

Code:

{
    int getNumber;
    int average;
    int sum = 0;
    int loop_count = 0;
    int minimum=99999;




    do {
        printf("Enter next integer:");
        loop_count++;
        scanf("%d", &getNumber);   /* read a single integer value input */
        if (getNumber!=99999)
        sum = sum + getNumber;
        if(getNumber == 99999 && loop_count == 1) {
        printf("No input was provided"\n);
        if(getNumber < minimum)    // this tests new number against the current minimum
        minimum = getNumber;    // if new number is less, it becomes the new minimum
        printf("minimum number is %d\n"), minimum);
}
    } while (getNumber != 99999);


    average = sum / loop_count;


    printf("Number of Integers: %d\n", loop_count);
    printf("Average: %d\n", average);
    printf("The Sum of the Number Is: %d", sum);
}

Also, i already started answering your last question, so in case this helps any -



What does your main loop look like now?

In the versions so far, none of the individual values are being saved; so it's not possible to go back and test them for something like "smallest value". You would have to find that value within the loop, while the numbers were being input.

A simple "if" statement can handle this.
Writing the actual code is little different than explaining it, so:

Code:

int minimum = 99999;    // this is before main loop

if (number!=99999) {

    sum = sum + number;
    loop_count++;

    if(number < minimum)    // this tests new number against the current minimum
        minimum = number;    // if new number is less, it becomes the new minimum
}

-[/QUOTE]

actually if I change this printf("minimum number is %d\n"), minimum); to printf("minimum number is %d\n"), getNumber;

the program does not crash put also won't display minimum number stmt.

Code:

printf("minimum number is %d\n"), minimum);

Parentheses - you have one too many of these: ')'

When you are testing for a minimum, you start with the largest possible value. That way you are guaranteed to get any value smaller than that. If you started with the smallest possible value, you would never get a value smaller than that, and the value "minimum" would never be replaced with a new minimum value.

-

What does your print statement for the minimum value look like?

Or more important, what is the entire code that crashes?

-

That's a great tip Salem, you're right and it is confusing following both. Sry about that.

Originally Posted by megafiddle

What does your print statement for the minimum value look like?

Or more important, what is the entire code that crashes?

-

The code does not crash, but it outputs: "Enter next integer: minimum number is 99999"

Well yes, look at the order that you are doing things.

You are printing out "minimun" before ever reading in a value.

Untill the first value is read in, "getNumber" is meaningless, and "minimum" is meaningless.

-