Thread: Pass values in functions and writing them to a txt file.

I made several comments on this code, hope this helps. This code is for fun, and have it doing a lot of what I want it to, just not all. I want random generated to write to a txt file. I tried to use an array but that failed. I wanted to use an array because i am only passing one value. Which makes sense since the random generated function is an int.

I made the fprintf as a comment but hopefully soon it will be able to send the values to the txt file. After that I will tackle the function.

Code:

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

#define HIGH 49
#define LOW 1

int random_generated()
{
    static int randValue = 1;
    static int randValue2= 0;
    static int numberout;
    
    int j=0,i=0;
    
    srand(time(NULL));
    
    while (i < 100) {
        
    for (j=0; j<6; j++) {       // loops 6 times resulting in the while loop producing 6 numbers
                                // that were produced at the sametime
        do{
            randValue= (rand() %(HIGH-LOW+1))+LOW;          //generates random values
            randValue2= (rand() %(HIGH-LOW+1))+LOW;         //generates random values
        }
        
        while (randValue!=randValue2);
        
        randValue = numberout;
            return numberout; // will call this in datafile() and print its output to a .txt
    }                           //sends one value but I want 6, do I need an Array?
                                // tried to use one but it failed, might have used it wrong
        i++;
  }
}

void datafile()
{
    char file_name[81];
    char *p;
    int write_to_file;
    int i;
    FILE *fp;
    
    printf("Enter a file name:");
    fgets(file_name,sizeof(file_name),stdin);
    p=strchr(file_name, '\n');
    if ((p)) {
        *p='\0';
    }
    
    fp =fopen(file_name,"r+");
    if(fp==NULL)
    {
        printf("The file was not successfully opened\n");
        printf("Please check if the file exists\n");
        exit(1);
    }
    
    write_to_file = random_generated();
    
    printf("%d",write_to_file);
    
    //fprintf(fp, write_to_file); // this will hopefully soon write to a txt file with the output from
    //random_generated...
    
    fclose(fp);
}

int main(int argc, const char * argv[])
{
    
    datafile();
    
    random_generated();
    
    return 0;
}

What IDE are using? I am using Xcode 4 and get non of the above...

OK!! Now we are getting somewhere. On lines 17, 36 and 43 I am getting warnings. Line 36 wants me to remove it, however I get warnings if I do and I making the function a void it resulting in an error because I am returning a value.

I have a plan for line 36, and line 17 was explained to me and should be ignored. I code use some objective-c to make it go way.

I think the do loop is generating random values twice and passing them to two different variables. The while loop keeps going while randValue is not equal to randValue2 and return an int. I want 6 ints returned so I might need an array. I was running this program differently and had random_generated print 6 number with in the function but they were not being passed to next function. Which is why I changed it to its current code.

What do these warning mean. The more fun projects I do the more I find out I do not know. @whiteflags, thank you for the advice. I didn't you could use return the way you did.

Why am I getting all these warnings when I simply call random_generator into data_files using an array. I want to fprintf the array to the txt file. I called it in main and the array was working fine. Something is not clicking over here.

Code:

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

#define HIGH 49
#define LOW 1

void number_generator(numbers[6]) //warning specifier missing, defaults to 'int'
{
    int i;
    srand(time(NULL));
    
    for (i=0; i<6; i++) {
        
numbers[i]= LOW + rand()%(HIGH - LOW); //this works
        
}
}

void datafile()
{
    char file_name[81];
    char *p;
    int receiveNumber[6];
    int i;
    FILE *fp;
    
    printf("Enter a file name:");
    fgets(file_name,sizeof(file_name),stdin);
    p=strchr(file_name, '\n');
    if ((p)) {
        *p='\0';
    }
    
    fp =fopen(file_name,"r+");
    if(fp==NULL)
    {
        printf("The file was not successfully opened\n");
        printf("Please check if the file exists\n");
        exit(1);
    }
    
   number_generator(receiveNumber);
    
    for (i=0; i<6; i++) {
    
    fprintf(fp,receiveNumber[i]); //format string is not a string literal (potentially insecure)
                                    //incompatable integer to pointer conversion passing 'int' to parameter of type of const char *
        
    }
    
    fclose(fp);
}

int main(int argc, const char * argv[])
{
    int receiveNumber[6];
    int i;
    
    datafile();
    
    number_generator(receiveNumber);
    
    for (i=0; i<6; i++) {
    
    printf("%d\n",receiveNumber[i]); //this is just confirm values are being passed
        
    }
    return 0;
}

Code:

void number_generator(numbers[6]) //warning specifier missing, defaults to 'int'

as the error says there is no type specifier
try

Code:

void number_generator(int numbers[6])

Code:

fprintf(fp,receiveNumber[i]);

Where is the format string ?
Kurt

fprintf, puts and gets both require a char. I just want to pass an int array. I have only passed variable to a data file with strings. I suppose that is my hang up.

OMG I feel like a retard!!!!!

Disregard, thank you for the help