Thread: Something wrong with output

Start over.

1 - Figure out what your file should look like.
2 - Make a sample file.
3 - Make a program that loads that and displays it.
4 - Make a program that writes what your file should look like.
5 - Make a program that loads and writes however many you want.


Quzah.

Originally Posted by Adak

See my edit I just made in my last post for an example for scanf()

when you tried doing that in your compiler, did it work fine? In my case, it didn't. This is what I did in the add_record() (is it correct?):

Code:

void add_record() 
{

   record stud;
   char ch;
   FILE *fp;
    fp=fopen("std.dat","r+");
    if(fp==NULL)
	fp=fopen("std.dat","w");

    clrscr();
    gotoxy(35,1);printf("\n\nADD STUDENT RECORD\n\n");

    do
    {
        printf("\n\nStudent number (must be 9 digits) = ");
        scanf("%lld", &stud.snum);
        printf("\nPassword                          = ");
	scanf("%50s", &stud.password);
        printf("\nLast name                         = ");
	scanf("%31s", &stud.lname);
        printf("\nFirst name                        = ");
	scanf("%31s", &stud.fname);
        printf("\nMiddle initial                    = ");
	scanf("%3s", &stud.mi);
        printf("\nBirthdate (MMDDYY)                = ");
	scanf("%7s", &stud.bday);
        printf("\nCourse (ex.BSM)                   = ");
	scanf("%6s", &stud.course);

	fseek(fp,0,SEEK_END);
        fwrite(&stud,sizeof(stud),1,fp);

        printf("\n\nAdd another record? Y if yes\n\n");
        ch=toupper(getche());

    } while(ch=='Y');

    fclose(fp);
    getch();
}

May I see what how you did it? Maybe I didn't do it right.

If you are using fwrite, you should be pairing it with fread. You are writing the entire structure, not just the length of the string. Consider:

Code:

char array[ 100 ] = "Foo";

If I write that with fwrite, using sizeof to get the array size, as you are with your structure size, it's going to write out 100 characters. If I scan it in with fscanf, I'm only going to read four. The other 96 are still waiting to be read.


Quzah.

Originally Posted by preeengles

*SIGH* I'm getting really ........ed with this already. Just a while ago, the program worked fine. I thought the problem was finally done but when I tried rerunning the program, it crashed again.

I'm losing hope already. What's with all this random errors anyway? Sometimes, I have to exit the IDE or restart to computer just to have the code working right again. It's very inconsistent. Funny thing is, with the other codes I've made, this inconsistency never occurred. I'll try looking into other aspects. Maybe this wasn't just caused by the overflow. I asked someone to run my program (the one w/ the original long code) in his computer and he said he wasn't able to reproduce the bug. Weird...

I'm also considering quzah's suggestion about redoing my program. If you have some more ideas as to the cause of the bug, please do tell.

Thanks again.

Don't get frustrated! De-bugging is always harder than writing out the original code, particularly if it's not as clear as it should be.

But your program is quite clear, so let's carry on, here. You will *have* to develop your troubleshooting skills, and this is a great place to do it. Much better than sitting by yourself, looking cross-eyed at the monitor, without a clue.

I only added two records to the student data file, but the display worked over and over, without any errors (which it did *not* do before), but that depends on what data you entered.

I suspect your friend had no problems, because he just luckily entered data that was shorter than the buffer space, which left room for the end of string char.

Since the display works fine, we can say that the writing out of the data, and the reading in from the file, must also be working, with good (but not perfect), confidence.

One problem I've found in a few functions is that you open the file with a local FILE pointer. Then you exit the function, and leave the file still open. So the file is open, but you can't get to the file pointer, because it's no longer in scope, and has vanished.

Then in another function, the file is opened again by another local FILE pointer, and not closed before the function exits.

The del_SR function also looks like somebody was trying to build a "piano", there - lots of extra code.

It will take another day before I get the time to make up a half decent version to show you. Relax for a day, and then you can take the rough version i'll post up, and polish it off, from there, ok?

TTYTomorrow.

Originally Posted by Adak

What did you do in the program when it crashed? Had you been running it through modify record() ? I'll bet so.

Yeah, the display works fine, and only crashes when I modify one of the records inputted.

Originally Posted by Adak

I suspect your friend had no problems, because he just luckily entered data that was shorter than the buffer space, which left room for the end of string char.

Well he said he ran the program on Unix...I've never tried Unix before. Would running the program on Unix really make a difference?

Originally Posted by Adak

So the *big* problem remaining is the modify record function. Any other function that, by itself (without calling modify record), causes a crash or error?

Yes. I tried entering two records and then called delSR() which deletes one record.

delSR() is supposed to search the student first and display it on screen, then ask confirmation from the user if he truly wishes to delete the record displayed. It works fine when the student to be deleted is the 1st record added by the user. However, when the user prompts to delete the 2nd record, it would display weird characters like the ones shown on my first post.

It says something like this:

Code:

Student record found:

STUDENT #: 11111111
NAME      :  ░i╘↕§ever
BDAY       : 010101
COURSE  : 54

Are you sure you want to delete the above student record?

Originally Posted by Adak

Here's what I'd like you to do, delete the student.dat file, which is probably corrupted by the modify function, and add new student records. *DO NOT RUN MODIFY RECORD()*!

Now run the program through every function, in order, 5 times, *except for modify record*. One function 5 times, then the next function 5 times, and so on.

I've already tried doing those before. They all technically worked fine, except that the displayed characters were weird. Like I said in my first post, if only one record is added, everything works perfectly fine.

One problem I've found in a few functions is that you open the file with a local FILE pointer. Then you exit the function, and leave the file still open.

Hee...sorry about that. I thought it was ok.

The del_SR function also looks like somebody was trying to build a "piano", there - lots of extra code.

It's long because I had to copy the retained data to a new file and rename it as the original (that explains all the strcpys in there). del_SR's supposed to delete one record only. I couldn't think of any other method other than what I did. Forgive me for my noobness

It will take another day before I get the time to make up a half decent version to show you. Relax for a day, and then you can take the rough version i'll post up, and polish it off, from there, ok?

Thank you much for all the help. I should've constructed my question in a clearer way in my first post to avoid all this ambiguity. Sorry about that. I'm just no good with explaining.

you have not yet removed & in scanf calls that read strings?
also checking the return value of scanf and fwrite would be a good idea.

could you post smallest compilable sample illustration the problem?

Originally Posted by preeengles

quzah's asked me the same thing before (i.e. to post only a small sample)...but I don't even know exactly which part caused the problem.

so copy your workproject to the side and start removing part that are not essential to the bug. Compile and see that the bug still exists...

If removing something eliminates the bug - return it and examine closely.

If you still do not understand the issue - continue removing other parts till you get sample with the reasonable size that could be posted on the forum