I know this is a dumb question but I cant figure this out

[chevon1920]

I am trying to do my assignment but I cant figure out how to get 8 data points per line to print to a file.
Here is the assignment

1. Program asks the user to enter an odd number as a BASE, using this Base it creates 48 multiples of the BASE and prints in a data file "data_3b.tx t", 8-data per line.

2. Program opens this data file, scans the contents of "data_3b.tx t" file and counts all the odd data, and prints them on the screen: 4 data per line.

3. Program computes the average of all the odd data. It prints the number of odd data points and their average on the screen in the given FORMAT.


I havent even gotten to reading the file, which I dont think I can do anyway. I'm still trying to figure out how to get 8 data points on one line, LOL! Here is my program so far. Sorry its not right justified. Can someone please help me. Also how am I supposed to tell the program to stop reading whenever I get to that part!
thanks for any help

[CODE=c]#include <stdio.h>
#include <math.h>
int main (void)
#define FILENAME "data_21.tx t"
#define CRT_SECURE_NO_D EPRECATE
{
double base;
int p, t, count, q, m, k;
FILE *data_2;

printf("enter an odd number as base");
scanf_s("%lf", &base);
data_2=fopen("d ata.txt","w");



for (p=9;p<=15;p++)
{
base=base*p;
for(k=1;k<=8;k+ +)
{
m=base*k;
fprintf(data_2" %20d",m);
}

printf("\n");
}
data_2=fopen("d ata.txt","r");
fscanf(data_2," %lf",&m);

return 0;
}[/CODE]

[Studlyami]

[CODE=c]
{
double base;
int p, t, count, q, m, k;

printf("enter an odd number as base");
scanf_s("%lf", &base);

for (p=9;p<=15;p++)
{
base=base*p;
for(k=1;k<=8;k+ +)
{
m=base*k;
fprintf(data_2" %20d",m);
}

printf("\n");
}
}[/CODE]

First why do you have so many variables for the first part? Variables named 'p' ,'q', 'm' ect. are bad names for variables so try to make them more descriptive. In your loops why do you have p= 9? <=15? what logic is behind that? Seeing how you need to print out 48 multiples i would use that as check condition for your loop. Then i would check to see if we have hit 8 items in a line. If so \n if not do nothing.

[CODE=c]
int Multiple = 0;
for(p = 1; p<=48; p++)
{
Multiple = base*p;
fprintf(data_2" %20d",Multiple) ;
///Here we check to see if we need a new line
if(p % 8 == 0)
printf("\n"); //print new line
}
[/CODE]

[chevon1920]

thanks for the reply. I know I have alot of variables defined, I was just randomly trying things so I had to create alot of variables. Not a good idea I suppose. I did realize that I can use a loop imbedded in another loop to get it to print 8 characters per line, but I cannot figure out how to print 4 characters per line to the screen. I could do the modulus of 2 and print, but I have no clue how to have it print 4 characters, it keeps printing one!

[Studlyami]

post your code and we will be glad to help (specifically the part on how you get the data from the file and then the part of where you try and display it on the screen). how is the data in the file separated? Seeing how it says print 4 data items per line each data item could vary in length correct (EX: Data value 1 = 333 Data value 2 = 3579)? so you will probably need to print more than just 4 characters to the screen.

Logically heres how i see it.

Grab Data Item
Is Data Item Odd?
no-> continue
yes-> Print Data Item on screen
Was the last printed item the fourth data item for that line?
yes--> New Line
no --> continue

[chevon1920]

here is my loop for the 8 data points per line calculation, this works beautifully, but I cant seem to grasp how to do it again for the 4 data points per line
for (p=1;p<=6;p++)
{
m=(base*p);
for(k=9;k<=16;k ++)
{
fprintf(data_2, "%5d",(m*k) );
fscanf("%lf",(m *k));

HERE this is what I have tried, but I dont really know what to count. Technically I know how many odd data points there are, but how do I tell the program to count them and print them 4 per line, because I dont know how high count should count
if ((m*k)%2!=0)
for (count=0;count< =48;count++)
printf("%7d\n", (m*k));

}

}
return 0;
}

Thanks so much for your response

[chevon1920]

First why do you have so many variables for the first part? Variables named 'p' ,'q', 'm' ect. are bad names for variables so try to make them more descriptive. In your loops why do you have p= 9? <=15? what logic is behind that? Seeing how you need to print out 48 multiples i would use that as check condition for your loop. Then i would check to see if we have hit 8 items in a line. If so \n if not do nothing.

[CODE=c]
int Multiple = 0;
for(p = 1; p<=48; p++)
{
Multiple = base*p;
fprintf(data_2" %20d",Multiple) ;
///Here we check to see if we need a new line
if(p % 8 == 0)
printf("\n"); //print new line
}
[/CODE]

OMG!!!!!!!!!!!! ! I just reread your post and that is the answer! Thank you so much, its so simple, to use the modulus to create a new line!!!!!

I LOVE YOU!!!!