•
•
•
•
What is DaniWeb IT Discussion Community?
You're currently browsing the C++ section within the Software Development category of DaniWeb, a massive community of 391,785 software developers, web developers, Internet marketers, and tech gurus who are all enthusiastic about making contacts, networking, and learning from each other. In fact, there are 3,516 IT professionals currently interacting right now! Registration is free, only takes a minute and lets you enjoy all of the interactive features of the site.
Please support our C++ advertiser:
Views: 398 | Replies: 7
 |
•
•
Join Date: May 2008
Posts: 16
Reputation: 
Rep Power: 1
Solved Threads: 0
Newbie Poster
ok i got past that part i posted about before, now im having some more trouble. i can add 2 patients easily to the file. then i try to display them. it displays the first one just fine, but the second it tries to display the first value in the [1]th element of the array, i get an error message. that part of my code is marked with a /*error*/ just before it. easy to find plz help this is due monday.
heres my code
heres my code
c Syntax (Toggle Plain Text)
Last edited by Ancient Dragon : May 10th, 2008 at 8:01 pm. Reason: add line numbers for convenience
•
•
Join Date: Aug 2005
Location: near St Louis, Missouri, USA
Posts: 10,548
Reputation: 
Rep Power: 36
Solved Threads: 860
>>fflush(stdin);
fflush is not defined for input streams, only output, so the above may or may not work.
Is this supposed to be a C or a C++ program. Looks like C, but you have tossed in iostream for some unknown reason.
Whay did you write all that unnecessary pointer stuff ?? All it does is obfuscate your program.
Your use of gets_s() is incorrect. It's supposed to have two parameters, not one. The second parameter is the size of the input buffer.
The while loop starting on line 114 is incorrect. Here's how it should be coded: Also check out how the sizeof operator is coded in the line below.
fflush is not defined for input streams, only output, so the above may or may not work.
Is this supposed to be a C or a C++ program. Looks like C, but you have tossed in iostream for some unknown reason.
Whay did you write all that unnecessary pointer stuff ?? All it does is obfuscate your program.
Your use of gets_s() is incorrect. It's supposed to have two parameters, not one. The second parameter is the size of the input buffer.
The while loop starting on line 114 is incorrect. Here's how it should be coded: Also check out how the sizeof operator is coded in the line below.
while (fread(&patientstructpointer[*count],sizeof(patientstructpointer[0]),1,fp) > 0)
{
(*count)++;
} Last edited by Ancient Dragon : May 10th, 2008 at 8:17 pm.
I think it's about time we voted for senators with breasts. After all, we've been voting for boobs long enough. ~Clarie Sargent, Arizona senatorial candidate
Those who are too smart to engage in politics are punished by being governed by those who are dumber. ~Plato
Those who are too smart to engage in politics are punished by being governed by those who are dumber. ~Plato
•
•
Join Date: Aug 2005
Location: near St Louis, Missouri, USA
Posts: 10,548
Reputation:
Rep Power: 36
Solved Threads: 860
Here is how that whole function should have been coded to simplify all those pointers. I left in all those fflush(stdin) lines because I was too lazy to remove them. I also didn't change all those gets_s() function calls for the same reason.
c Syntax (Toggle Plain Text)