Go Premium for a chance to win a PS4. Enter to Win

x
?
Solved

Urgent!!! Can't find the problem...

Posted on 2003-10-29
7
Medium Priority
?
197 Views
Last Modified: 2010-04-15
I wrote a program to find the bigger and the smaller number in a list but there's something wrong and don't know what it is becuse the program just gives me the biggest number while in the smaller number always appear 0.  Thanks for your help!!!
Here it is:

#include <stdio.h>

void main()
{
 int arr[10], n, dat, max=0, min=0;
 for(n=0;n<=9;n++)
 {
  printf("Number: ");
  scanf("%d",&dat);
  if(dat>max)max=dat;
 if(dat<min)min=dat;
 }
 printf("Bigger %d Smaller %d",max,min);
}
0
Comment
Question by:gothic130
  • 2
  • 2
  • 2
  • +1
7 Comments
 
LVL 18

Accepted Solution

by:
bobbit31 earned 600 total points
ID: 9645336
for(n=0;n<=9;n++)
 {
  printf("Number: ");
  scanf("%d",&dat);

  if (n == 0) min = dat;

  if(dat>max)max=dat;

  if(dat<min)min=dat;
 }
 printf("Bigger %d Smaller %d",max,min);
0
 

Author Comment

by:gothic130
ID: 9645408
>>if (n == 0) min = dat;

 Please could you explain me what this line does?
0
 
LVL 1

Expert Comment

by:guitardude101
ID: 9645745
You should initialize min to INT_MAX instead of 0
You should initialize max to INT_MIN instead of 0

They defined in <limits.h>

0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 18

Expert Comment

by:bobbit31
ID: 9645842
> Please could you explain me what this line does?

basically, your min starts out at 0... if all the numbers in the array are positive, min will always be 0.

what the line above does is it sets min to the first element in the array... if the first element is the min then it stays the min, otherwise it'll keep going through and update min accordingly (note that if (n==0) will only happen once)
0
 
LVL 46

Assisted Solution

by:Kent Olsen
Kent Olsen earned 400 total points
ID: 9645850

Using the code in the original question, a small change (similar to bobbit31's) should do just fine:

#include <stdio.h>

void main()
{
 int arr[10], n, dat, min, max;

 min = arr[0];                /*  Default min to the first value  */
 max = arr[0];               /*  Default max to the first value  */
 for(n=1;n<=9;n++)      /*  Search the rest of the list  */
 {
  printf("Number: ");
  scanf("%d",&dat);
  if(dat>max)max=dat;
 if(dat<min)min=dat;
 }
 printf("Bigger %d Smaller %d",max,min);
}


Kent
0
 
LVL 1

Expert Comment

by:guitardude101
ID: 9645854
gothic130 your algrythm will work if you just change the initialization as above.
It would also allow you to enter negative numbers.
0
 

Author Comment

by:gothic130
ID: 9645950
Thank you all:)
0

Featured Post

Concerto's Cloud Advisory Services

Want to avoid the missteps to gaining all the benefits of the cloud? Learn more about the different assessment options from our Cloud Advisory team.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

An Outlet in Cocoa is a persistent reference to a GUI control; it connects a property (a variable) to a control.  For example, it is common to create an Outlet for the text field GUI control and change the text that appears in this field via that Ou…
Examines three attack vectors, specifically, the different types of malware used in malicious attacks, web application attacks, and finally, network based attacks.  Concludes by examining the means of securing and protecting critical systems and inf…
The goal of this video is to provide viewers with basic examples to understand recursion in the C programming language.
Video by: Grant
The goal of this video is to provide viewers with basic examples to understand and use nested-loops in the C programming language.
Suggested Courses

885 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question