Solved

unable to read file propely

Posted on 2002-04-02
8
190 Views
Last Modified: 2010-04-15
i am suppose to read a file of munbers, ist row column is the time, 2nd column is the distance, program issuppose to print out the  number of rows, maximum distance and minimum distance with their respective times. i am suppost to use a sentinel of -1 . i am almost done with the codes, but it doesn't seem to be reading propely.



#include<stdio.h>
#include<stdlib.h>
#include<io.h>
#define Sentinel -1

void main()
{
     int i,rows;
     double time,min_time,max_time;
     double distance,min_dist, max_dist;

     FILE *rocket;
     FILE *data;

     rocket = fopen("rocket.txt","r");
     if(rocket==NULL)
     {
          printf("file not found\n");
     }
     else
     {
     fscanf(rocket,"%f",&rows);
     printf("Number of rows:%f ", rows);
     }
     for(i=0;i<50;++i)
     {
          fscanf(rocket,"%d%d",&time,&distance);
               if(distance>max_dist)
               {
                    max_dist=distance;
                    max_time=time;
               }
if(distance<min_dist)
               {
                    min_dist=distance;
                    min_time=time;
               }
     }
fclose(rocket);

data=fopen("data.txt","w");
fprintf(data,"Minimum Distance = %4.1d (Time = %d)",min_dist,min_time);
fprintf(data,"Maximum Distance = %4.1d (Time = %d)",max_dist,max_time);
     
     fclose(data);
}


0.0     0.0
30.0     5000.0
60.6     25000.0
90.0     60000.0
100.0     75000.0
110.0     30000.0
140.0     10000.0
170.0     2500.0
200.0     0.0
-1.0     -1.0
0
Comment
Question by:kalakutta
8 Comments
 
LVL 16

Expert Comment

by:imladris
ID: 6913152
Your data consists of two "columns" of numbers.

However, your program opens the file, then reads the first float number into rows;

else
{
    fscanf(rocket,"%f",&rows);
    printf("Number of rows:%f ", rows);
}

(I would expect it to pick up the first 0.0 here)

Then it goes into a loop to read pairs of time and distance:

for(i=0;i<50;++i)
{
    fscanf(rocket,"%d%d",&time,&distance);

time and distance are float variables, but they are being read with an integer format (%d).

In summary, I would think it should work a lot better, if you drop the first read for rows, then scan the pairs with:

    fscanf(rocket,"%f %f",&time,&distance);

0
 

Expert Comment

by:ComTech
ID: 6915505
listening...

CS Admin
0
 

Expert Comment

by:indira_shukla
ID: 6941690
I think following are the pobs in your program:
1) rows is declared integer but you are reading it as a floating point (although I know that it will type convert it).
2) when number of rows is in "rows" then why r u running the loop 50 times.. Is the file contain 50 rows?????

3)What is the initial value of distance,max_distance and min_distane?? so, the it will take the garbage and will not run properly.
4) you have taken max_distance and distance as floating point number and reading them as integers...This may be the source of improper working of the code..


 Just check these out...
Indira.
0
 
LVL 1

Expert Comment

by:tapasmondal
ID: 7000447
#include<stdio.h>
#include<stdlib.h>
#include<io.h>
#define Sentinel -1

void main()
{
    int i,rows;
    double time,min_time,max_time;
    double distance,min_dist, max_dist;

    FILE *rocket;
    FILE *data;

    rocket = fopen("rocket.txt","r");
    if(rocket==NULL)
    {
         printf("file not found\n");
    }
    else
    {
    fscanf(rocket,"%f",&rows);/* YOU HAVE DEFINED ROWS AS INT TYPE. SO U HAVE TO REPLACE %f BY %d */
    printf("Number of rows:%f ", rows);
    }
    for(i=0;i<50;++i)
    {
         fscanf(rocket,"%d%d",&time,&distance);

/* YOU HAVE DEFINED TIME AND DISTANCE AS DOUBLE TYPE. SO U HAVE TO REPLACE %d BY %f */
              if(distance>max_dist)
              {
                   max_dist=distance;
                   max_time=time;
              }
if(distance<min_dist)
              {
                   min_dist=distance;
                   min_time=time;
              }
    }
fclose(rocket);

data=fopen("data.txt","w");
fprintf(data,"Minimum Distance = %4.1d (Time = %d)",min_dist,min_time);
fprintf(data,"Maximum Distance = %4.1d (Time = %d)",max_dist,max_time);
   
    fclose(data);
}


0.0     0.0
30.0     5000.0
60.6     25000.0
90.0     60000.0
100.0     75000.0
110.0     30000.0
140.0     10000.0
170.0     2500.0
200.0     0.0
-1.0     -1.0



I THINK IT WILL WORK. IT WILL BE FINE IF U SUPPLY SOME DATA OF UR INPUT FILE.
0
What Is Threat Intelligence?

Threat intelligence is often discussed, but rarely understood. Starting with a precise definition, along with clear business goals, is essential.

 

Expert Comment

by:rodchamberlin
ID: 7123447
Actually,

since you have declared "rows", "time" and "disatance" as double you will need format specifier "%lf". "%f" would be for a "float" variable.

When you do the output (fprintf), you need to use "%f" or "%lf" (it doesn't actually matter which, since the 'C' language specification causes a float to be passed as a double when you are passing through a variable length argument list).

Hope this helps.
0
 

Expert Comment

by:cool_techie
ID: 7819753
are u sure u can write into file with "r" mode???
i guess problem is der.
see if is solved with r+
0
 

Expert Comment

by:cool_techie
ID: 7819762
i am sorry, i missed some code, there is no problem with file perms.
sorry again
0
 

Accepted Solution

by:
SpideyMod earned 0 total points
ID: 8279754
All,
I am unlocking this question in preparation for cleanup.  I will return in 7 days to finalize this question.  Please leave any recommendations for the final state of this question, I will take all recommendations into consideration.  Failing any feedback, I may decide in 7 days to delete or PAQ this question with no refund.  Thanks.

SpideyMod
Community Support Moderator @Experts Exchange
0

Featured Post

How to run any project with ease

Manage projects of all sizes how you want. Great for personal to-do lists, project milestones, team priorities and launch plans.
- Combine task lists, docs, spreadsheets, and chat in one
- View and edit from mobile/offline
- Cut down on emails

Join & Write a Comment

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…
This is a short and sweet, but (hopefully) to the point article. There seems to be some fundamental misunderstanding about the function prototype for the "main" function in C and C++, more specifically what type this function should return. I see so…
The goal of this video is to provide viewers with basic examples to understand and use structures in the C programming language.
The goal of this video is to provide viewers with basic examples to understand opening and writing to files in the C programming language.

743 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

Need Help in Real-Time?

Connect with top rated Experts

13 Experts available now in Live!

Get 1:1 Help Now