Thread: output rand() in array

Hi everyone, this my first post. I'm wanting to output six unique integers, but the output is one integer. Any help please.

Code:

#include <stdio.h>
#include <time.h>

int main()
{
   int a, n;
   char m[6];
   char num[6] = {0};
   srand((unsigned)time (NULL));
   for(a = 0;a < 6;a++)
   num[a] = rand() % 40 + 1;{
   for(n = a + 1;n < 6; n++)
      m[n];
      {
       if(num[a] != m[n])
       printf("%d\n", num[a]);
       }
     }

   return 0;
}

Your program is actually equivalent to:

Code:

int main()
{
    int a, n;
    char m[6];
    char num[6] = {0};
    srand((unsigned)time (NULL));
    for(a = 0;a < 6;a++)
    {
        num[a] = rand() % 40 + 1;
    }

    {
        for(n = a + 1;n < 6; n++)
        {
            m[n];
        }

        {
            if(num[a] != m[n])
                printf("%d\n", num[a]);
        }
    }

    return 0;
}

In other words, your for loops have a single statement each as the body. You need to fix that by proper brace placement.

It's really a lot easier to read if you indent it nicely. At least make the opening and closing curly braces line up (or the closest equivalent if you like braces on the same line as for loops etc), and indent the code inside the braces.

What is this supposed to do?

Code:

m[n];

[edit] One way you could indent your code (4 spaces of indentation, K&R style braces):

Code:

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

int main() {
    int a, n;
    char m[6];
    char num[6] = {0};
    srand((unsigned)time (NULL));
    for(a = 0;a < 6;a++) {
        num[a] = rand() % 40 + 1;
   
        for(n = a + 1;n < 6; n++) {
            m[n];
        }
        if(num[a] != m[n])
            printf("%d\n", num[a]);
    }
    return 0;
}

Much easier to see what's going on, isn't it? If you wrote your code in this style out of habit, you wouldn't need laserlight's alignment skills to find bugs.
[/edit]