Thread: I need help with this program to find Hi-low,med of three numbers.

I cannot get this program from having a continuous loop. I need to have a user-defined loop, but can't figure it out. Any help would be appreciated. Yes, this is an assignment for my 1st programming class.

Here is what I have so far:

Code:

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


int main(void)


{
 int a,b,c,high,med,low;

     printf("This programs sorts numbers from highest to lowest\n");

     printf("Enter 3 numbers seperated by a comma or 'Q' to quit\n");

      while(scanf("%d,%d,%d", &a,&b,&c) != 'Q')
{

   if (a < c && c < b)

  {
      high = b;
       med = c;
      low = a;
   }

      if (c < a && a < b)
    {
       low = c;
       med = a;
       high = b;
    }
      if (b < a && a < c)
   {
      high = c;
      low = b;
      med = a;
    }
    
    printf("Here are your numbers high to        low:%d,%d,%d\n",high,med,low);

     printf("Enter 3 numbers or 'Q' to quit:\n");
   }
     return 0;
   }

Look up the return value of scanf().
Kurt

Can you give me a hint? I am not sure what you are saying.

You're testing the return from scanf() in your while loop -- not the entered variable from the user. So the test will be SOMETHING that scanf() returns, which will (realistically), never equal to 'Q'.

It will be an integer, in a very low range of values.

So, are you guys saying I need to get rid of the while statement !='Q'? I tried that but it still won't give me the user-defined loop I desire. I might be doing something wrong, obviously, but I do appreciate all your help people.
Thanks again!

I'm saying you're not testing for the 'Q', like you believe you are, in that test by the while loop. scanf() returns an integer, and not a 'Q'.

What should you be testing for? Work it on out!

Thanks Adak, I was losing brain cells trying to figure this out. It works the way I want it to now. Thank all of you for your explanations.!