Printable View
I am unable to understand why the program doesnt compile?
[insert]Code:
#include <stdio.h>
#include <stdlib.h>
int main(int argc , char * argv[])
{
int n, i, j;
//int []a;
//n = atoi(argv[1]);
int a[10];
for(i = 0;i<10;i++)
{
int temp = random(10);
printf("\n%d", temp);
a[i] = temp;
}
for(i = 0; i<10 ; i++){
for(j = 0; j<10 ;j++){
if(a[j] > a[j+1]){
int t = a[j];
a[j] = a[j+1];
a[j+1] = t;
}
}
}
printf("\n Array in ascending order");
for(i = 0; i< 10;i++){
printf("\n %d", a[i]);
}
return 0;
}
roann - please post the error messages!
What is : random(10) ?
Sorry for that
The error message is
[insert]Code:1>sort.obj : error LNK2019: unresolved external symbol _random referenced in function _main
1>C:\Documents and Settings\ROHAN\My Documents\Visual Studio 2008\Projects\classc2\Debug\classc2.exe : fatal error LNK1120: 1 unresolved externals
1>Build log was saved at "file://c:\Documents and Settings\ROHAN\My Documents\Visual Studio 2008\Projects\classc2\classc2\Debug\BuildLog.htm"
1>classc2 - 2 error(s), 2 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
This is how i saw in one of the links to generate the random numbers.Quote:
Originally Posted by rdrast
rand(10)
There's nothing like random. It's rand() which produces random numbers. Your sorting function looks somewhat strange. Why you've used 'i' for loop if you're not using i as an index?Quote:
Originally Posted by roaan
Oh thanks i got the point but then this link told me to use it in the form of
random(10);
http://www.phanderson.com/C/random.html
When j=9, then the comparison a[9]>a[10] will be made and you know that it's out of bound.Code:for(i = 0; i<10 ; i++){
for(j = 0; j<10 ;j++){
if(a[j] > a[j+1]){
int t = a[j];
a[j] = a[j+1];
a[j+1] = t;
Oh yes i got the point for using the random. The thing is that its a bubble sort algorithm so i have to iterate two times.Quote:
Originally Posted by BEN10
Any time you see a webpage that says how TurboC does things, you shouldn't actually read any farther.Quote:
Originally Posted by roaan
(As an aside, *nix has a random(), but not a randomize(), as well.)
The prototype for rand() is:
rand(void);
Meaning you cannot provide a meaningful argument. rand() returns a number from 0 to RAND_MAX. So you can do this for 0-9:
or use "rand()%10" which will also work but probably is very slightly weighted against 9.Code:int maxfactor = RAND_MAX/10;
int val = rand()/maxfactor;
Okay i am using rand()%10 but then i realized you say its slightly weighted against 9. So what does that imply -> occurence of 9 would be less/ more as compared to other numbers.Quote:
Originally Posted by MK27
Well, say RAND_MAX is 17 (obviously it is much more than that, but it is still not a nice even number). Using rand()%10, we could get: 0, 1, 3, 4, 5, 6, 7, 8, 9, 0, 1, 2, 3, 4, 5, 6, 7. Ie, getting 8 or 9 is half as likely as 0-7.Quote:
Originally Posted by roaan
But since RAND_MAX is actually in the billions, the ratio will not be "half as likely". It will be more like 0-7 occur one hundred million and one times whereas 8 and 9 only occur one hundred million times.
So not much need to worry, and anyway I don't think there is any option which precludes this potential imbalance, it is just as applicable with the /maxfactor method, except that will only bias the last number. At least I think so, it's not totally clear to me without a paper and pen. But you get the point?
One way is to reduce the range, perhaps by discarding numbers at the end if they are generated, such that the number of integers in the range is perfectly divisble by 10.Quote:
Originally Posted by MK27